Tableau dynamique 2D non initialisé - argument à fonction C

Question

J'ai une question pour vous!

I ont un programme de C dans lequel i ont un tableau 2D dynamique dans ma fonction principale:

int **neighbours; 

qui i initialiser basé sur des arguments de ligne de commande. Le problème est que je veux nettoyer le code un peu. Pour ce faire, je veux créer une fonction dans un fichier d'en-tête qui prend le tableau non initialisé comme argument de référence et l'initialiser là.

Voilà comment j'appelle la fonction:

init(&nodeNum, &neighbours, argv[1], &nodes); 

c'est comment je déclare la fonction dans le fichier d'en-tête:

int init(int *nodeNum, int ***neighbours, char *arg, struct node **nodes) 

et voici comment essayer d'allouer la mémoire:

neighbours=malloc(*nodeNum*sizeof(int *)); 
     for(i=0;i<*nodeNum;i++){ 
      neighbours[i] = malloc(*nodeNum*sizeof(int *)); 
     } 

bien évidemment quelque chose ne va pas bien et il se bloque! des conseils sur ce que je peux faire mal?

grâce à l'aide de gens sympathiques, le malloc fonctionne maintenant, mais le remplissage des nœuds échoue!

est ici tout le code de la fonction:

int init(int *nodeNum, int ***neighbours, char *arg, struct node **nodes){ 
FILE *file; 
char buffer[4]; 
int i,y,distance; 

if(strcmp(arg,"-l")==0){ 
     file = fopen("C:\\Users\\trendkiller\\thesis\\matrix.txt","r"); 
     printf("parsing adjacency matrix from file\n\n"); 
     //file = fopen ("C:\\Users\\trendkiller\\thesis\\matrix.txt", "r") ; 
     *nodeNum = atoi(fgets(buffer,4,file)); 
     *neighbours=malloc(*nodeNum*sizeof(int *)); 
     for(i=0;i<*nodeNum;i++){ 
      (*neighbours)[i] = malloc(*nodeNum*sizeof(int *)); 
     } 
     printf("this prints\n"); 
     printf("number of nodes: %s", buffer); 
     for(i=0;i<*nodeNum;i++){ 
      for(y=0;y<*nodeNum;y++){ 
       fgets(buffer,4,file); 
       (*neighbours)[i][y]=atoi(buffer); 
          } 
     } 


    } 
    else{ 
     *nodeNum = atoi(arg); 

     *neighbours=malloc(*nodeNum*sizeof(int *)); 
     printf("this prints\n"); 
     for(i=0;i<*nodeNum;i++){ 
      (*neighbours)[i] = malloc(*nodeNum*sizeof(int *)); 
     } 

     *nodes=malloc(*nodeNum*sizeof(struct node)); 

     for(i=0;i<30;i++){ 
      (*nodes)[i].x=rand()%100; 
      (*nodes)[i].y=rand()%100; 
     } 
     printf("this prints\n"); 

     printf("creating new adjacency matrix\n\n"); 
     for(i=0; i<*nodeNum; i++){ 
      for(y=0; y<*nodeNum; y++){ 
       distance=sqrt((((*nodes)[y].x-(*nodes)[i].x)*((*nodes)[y].x-(*nodes)[i].x))+(((*nodes)[y].y-(*nodes)[i].y)*((*nodes)[y].y-(*nodes)[i].y))); 
       if(i==y){ 
        (*neighbours)[i][y]=-1; 
       } 
       else if(distance<=20){ 
        (*neighbours)[i][y]=1; 
       } 
       else { 
        (*neighbours)[i][y]=0; 
       } 
      } 

     } 

     file = fopen("C:\\Users\\trendkiller\\thesis\\matrix.txt","a+"); 
     fprintf(file,"%d\n",*nodeNum); 
     for(i=0;i<*nodeNum;i++){ 
      for(y=0;y<*nodeNum;y++){ 
       fprintf(file,"%d\n", (*neighbours)[i][y]); 
      } 
     } 
    } 

return 0;

}

Merci d'avance pour votre aide!

Answer 1

Le problème est que neighbours est passé par la valeur, donc les modifications à l'intérieur de la fonction n'ont aucun effet. Vous devez passer par pointeur, et le modifier indirectement:

int init(int *nodeNum, int ***neighbours_ptr, char *arg, struct node *nodes) { 
    *neighbours_ptr = malloc(*nodeNum*sizeof(int *)); 
    for(i=0;i<*nodeNum;i++){ 
     (*neighbours_ptr)[i] = malloc(*nodeNum*sizeof(int *)); 
    } 
} 

Vous devez également passer &neighbours au lieu de neighbours comme second argument de init.

Answer 2

Dans le code exemple que vous donnez une erreur est ici:

neighbours[i] = malloc(*nodeNum*sizeof(int *)); 

où il devrait être

neighbours[i] = malloc(*nodeNum*sizeof(int)); 

Pourquoi utilisez-vous une émulation d'un tableau 2D au lieu d'un vrai? Cela fonctionne depuis C99:

int (*neighbours)[nodeNum] = malloc(sizeof(int[nodeNum][nodeNum])); 

Vous ne voudriez pas avoir besoin d'un init comme le vôtre qui attribue simplement la mémoire. Pourquoi vous passez un pointeur sur nodeNum et pas directement sur la valeur? Si vous avez toujours besoin d'une fonction init qui effectue une initialisation et pas seulement une allocation.L'interface ressemblerait à quelque chose comme

void init(size_t nodeNum, int neighbours[nodeNum][nodeNum]);