C: Problema assegnazione matrice

[Aletta]

Ciao a tutti,
ho un problema con l'assegazione dei valori ad una matrice creata dinamicamente. Appena provo ad essegnare i valori, mi dà errore di segmentazione.

void matrix_allocation(int **matrix, int N){
    int i, j;
    /* dynamic allocation for the matrix */
    /*allocation of the rows*/
    matrix = (int**)malloc(N * sizeof(int*));  
    if(matrix == NULL){
    printf("\n Error in memory allocation for the matrix \n");
    }
    /*allocation of the coloumns*/
    for(i=0; i<N; i++){   
    matrix[i]=(int*) malloc(N * sizeof(int));
    if (matrix[i] == NULL){
        printf("Cannot allocate enough memory\n");
        }
    }
}

void generate_matrix(int **matrix, int N){
    int i, j;
    for (i=0; i<N; i++){
        for (j=0; j<N; j++){
                    if(i==j){                   //the diagonal must be 0
                        matrix[i][j] = 0;
                    }
                    else{
            matrix[i][j] = rand()%10;
                        printf("%i\t", matrix[i][j]);
           
                    }
        }
                printf("\n");
    }
    printf("\nMatrix generated successfully!\n");
}

Appena arriva a matrix[i][j] = 0 (alla prima esecuzione), dà errore. Cosa sbaglio? Avete qualche idea? Grazie a tutti.

[Scara95]

matrix è sempre una variabile locale, perciò su allocation

void matrix_allocation(int ***matrix, int N){    int i;
    /* dynamic allocation for the matrix */
    /*allocation of the rows*/
    *matrix = (int**)malloc(N * sizeof(int*));  
    if((*matrix) == NULL){
    	printf("\n Error in memory allocation for the matrix \n");
    }
    /*allocation of the coloumns*/
    for(i=0; i<N; i++){   
    	(*matrix)[i]=(int*) malloc(N * sizeof(int));
    	if ((*matrix)[i] == NULL){
        	printf("Cannot allocate enough memory\n");
        }
    }
}

chiamata con

matrix_allocation(&nome_variabile, N);

Esempio self-contained riprendendo il tuo codice: http://ideone.com/0O3zZw

P.s. vorrei farti notare che non hai gestito correttamente gli errori: infatti nel caso non riuscissi ad allocare la memoria necessaria dovresti comunque liberare quella già allocata e restituire un codice di errore al main (che può essere impostare a NULL il puntatore, oppure cambiare il tipo della funzione e ritornare true/false per successo/fallimento).

[Aletta]

Non so come ringraziarti, era da un po' che ci sbattevo la testa!

[Scara95]

Non so come ringraziarti, era da un po' che ci sbattevo la testa!

Prego, considera però anche i problemi che ti ho esposto alla fine. Inoltre sarebbe meglio se usassi sempre la stessa convenzione per i nome:

allocate_matrix
generate_matrix
free_matrix

oppure

matrix_allocation
matrix_generation
matrix_freeing

A mio parere molto preferibile la prima...

[Aletta]

Si, lo terrò a mente per quanto riguarda la gestione degli errori, ho modificato il codice e ho messo come tipo di ritorno un int. Grazie di nuovo