ElectroYou

Ciao a tutti!

Ho creato scritto questo semplice programma in C che acquisisce una serie di numeri e li stampa in matrice:

Codice: Seleziona tutto

#include <stdio.h>
//Definisco dimensione matrice
#define DIM1 3
#define DIM2 3
int main () {
    int matrice[DIM1][DIM2];
    int i=0,j=0;
   
    //Inserisco i numeri che compongono la matrice   
    printf("Inserisci i numeri della matrice:\n");
    for(i=0; i<DIM1; i++) {
        for(j=0; j<DIM2; j++) {
            printf("==> ");
            scanf("%d", &matrice[i][j]);
            }
    }

    //Stampa matrice
    for(i=0; i<DIM1; i++) {
        for(j=0; j<DIM2; j++) {
            printf("%d ",matrice[i][j]);
        }
        printf("\n");
    }
    return 0;
}

Le dimensioni della matrice sono definite attraverso la funzione

Codice: Seleziona tutto

#define DIM

Fin qui tutto a posto, il programma funziona correttamente. Allora ho pensato di modificarlo facendo decidere all'utente di quante righe e quante colonne è formata la matrice. Con uno

Codice: Seleziona tutto

scanf

l'utente decide il numero di righe e di colonne, inserisce poi tutti i numeri, ma a questo punto il programma al posto di stamparmi la matrice corretta, mi stampa solamente gli ultimi due valori che ho immesso.

Codice: Seleziona tutto

#include <stdio.h>
int main () {
    int  righe=0, colonne=0;
    int matrice[righe][colonne];
    int i=0,j=0;
   
    printf("Numero di righe: ");
    scanf("%d", &righe);
    printf("Numero di colonne: ");
    scanf("%d", &colonne);
       
    printf("Inserisci i numeri della matrice:\n");
    for(i=0; i<righe; i++) {
        for(j=0; j<colonne; j++) {
            printf("==> ");
            scanf("%d", &matrice[i][j]);
            }
    }

    //Stampa matrice
    for(i=0; i<righe; i++) {
        for(j=0; j<colonne; j++) {
            printf("%d",matrice[i][j]);
        }
        printf("\n");
    }
    return 0;
}

Non riesco a capire dove sbaglio, il codice "in teoria" dovrebbe essere identico, solo che ho aggiunto la possibilità di decidere le righe e le colonne!

Grazie
Ciaoo :)

Una domanda.

Con quale compilatore e su quale sistema operativo lavori?

Ad ogni modo se non ricordo male non mi sembra corretto dichiarare una matrice di 0 righe e 0 colonne e successivamente fare un po' quello che vuoi.

Dovresti allocare la memoria.

Altro problema a cui potresti venire incontro è il buffer di input "sporco".

Guarda che hai dichiarato una matrice di zero righe e zero colonne!
Anche assegnando un valore diverso alle due variabili non succede niente, sempre zero per zero restano.
Inoltre le matrici non si possono allocare (deciderne la dimensione, per intenderci) durante l' esecuzione del programma
Dovresti invece allocare delle variabili dinamicamente con le funzioni malloc o calloc.

P.S.: le matrici si dichiarano (non si creano), i programmi si scrivono (non si creano). Siamo in un forum tecnico ed utilizziamo i termini corretti.

Oltre alle funzioni malloc e calloc, e perché no aggiungiamo anche la realloc, bisogna lavorare con i puntatatori fin da subito e non usare subito gli array.

Ciao!

Grazie per le risposte! Provo con i puntatori e poi vi faccio sapere in effetti mi sono dimenticato di utilizzare le funzioni malloc!

Grazie
Ciaoo!

simo85 ha scritto:Oltre alle funzioni malloc e calloc, e perché no aggiungiamo anche la realloc ...

A questo punto, per non far torto a nessuno, mettiamoci anche la free.

Ciao!

Ho provato a rifare l'esercizio con la funzione malloc ma non ci sono riuscito .

Io devo prima dichiarare il puntatore e poi con la malloc dichiarare la grandezza del vettore? In questo caso non riesco a capire come far definire la grandezza della matrice direttamente all'utente.

Grazie
Ciaoo!

Ciao tecfil, visto che sto scrivendo un programma in C in cui la funzione main prende in input dei valori dalla linea di comando, ho pensato al tuo esercizio in cui volevi dare un valore al numero di righe e colonne subito all'esecuzione del programma, così ho scritto un piccolo esempio illustrativo che spero possa esserti utile.

Codice: Seleziona tutto

#include <stdio.h>
#include <stdlib.h>
#include <libgen.h>

int main(int argc, char *argv[])
{
   if(argc != 3)
   {
      printf("\nusage: %s ROWS COLS\n\n", basename(argv[0]));
      exit(1);
   }
   else
   {
      int rows = atoi(argv[1]) + 1, cols = atoi(argv[2]) + 1;
      int matrix[rows][cols];
      int ir = 0, ic = 0;

      for(ir = 0; ir < rows; ir++)
      {
         for(ic = 0; ic < cols; ic++)
            matrix[ir][ic] = ir * ic;
      }

      for(ir = 1; ir < rows; ir++)
      {
         for(ic = 1; ic < cols; ic++)
            printf("[%.2d]", matrix[ir][ic]);
      
         printf("\n");
      }
      
      exit(0);
   }
}


PS: vedo che mi hai anticipato. Posta il codice..

tecfil ha scritto:Io devo prima dichiarare il puntatore e poi con la malloc dichiarare la grandezza del vettore?

Si. Dichiari il puntatore e successivamente con la malloc decidi tu quanti byte di memoria allocare.
Visto che hai difficoltà con la malloc ti posto un semplice esempio in cui vengono allocati 7 byte di memoria.

Codice: Seleziona tutto

#include <stdio.h>
#include <stdlib.h>

void main(void)
{
   char *pointer = NULL;
   pointer = (char*) malloc (sizeof(char) * 7);

   free(pointer);
   exit(0);
}

Se c'è qualcosa che non capisci chiedi pure.