C - ошибка сегментации при доступе указателя в файле заголовка

Question

Я программирования на C, и есть два файла, func2.h:

#define NN 20 

void network_construction(int **veins, int *num_veins){ 

     int i, j; 


     for(i=0;i<NN;i++){ 
      num_veins[i] = NN/2; 
     } 


     veins = malloc(NN * sizeof(*veins)); 
     for (i = 0; i < NN; i++) { veins[i] = malloc(num_veins[i] * sizeof(*(veins[i]))); } 
     for (i = 0; i < NN; i++) { for (j = 0; j<num_veins[i];j++) { veins[i][j] = -1; } } 


    return; 
    } 

и main.c:

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

void main(){ 

    int num_veins[NN]; 
    int **veins; 


    network_construction(veins, num_veins); 

    printf("\n%d\n", num_veins[19]); 
    printf("\n%d\n", veins[2][2]); 

return; 
} 

num_veins [] дает правильный номер 10, но при попытке доступа к любым элементам вен они дают ошибку сегментации. Как я могу использовать в main.c значения вены, которые заполнены func2.h? Что я делаю неправильно и почему это происходит?

EDIT:

Эти коды представляют собой упрощенную версию более крупные. В ответах есть способ решить проблему, но это не совсем то, что я хочу. Мне нужно сделать динамическое распределение в func2.h, поэтому я не хочу никаких вычислений с участием вен [] [] в main.c перед вызовом network_construction().

Answer 1

Код ниже добивается того, чего я хотел:

void network_construction(int ***veins, int num_veins[]){ 
    int i,j; 

    for(i=0;i<NN;i++){ 
    num_veins[i] = NN/2; 
    } 

    (*veins) = malloc((NN) * sizeof(**veins)); 

    for(i=0;i<NN;i++){ 
    (*veins)[i] = malloc(num_veins[i]*sizeof(*(*veins)[i])); 
    } 

    for (i = 0; i < NN; i++) { 
    for (j = 0; j < num_veins[i]; j++) { 
     (*veins)[i][j] = -1; 
    } 
    } 
    return;} 

Для вызова в основной():

network_construction(&veins,num_veins); 

Answer 2

veins = malloc(NN * sizeof(*veins)); есть ваша проблема. Вам действительно нужен int***, чтобы сделать то, что вы хотите. Внутри network_construction аргумент вены на самом деле является копией переменной veins в вашем main. Поэтому, когда вы malloc его, изменяется только копия, но не основная переменная.

Более простой пример той же задачи будет следующим образом:

void foo(int a){ 
    a=5; 
} 
void foo2(int* a){ 
    *a=5; 
} 
int main(){ 
    int a=0; 
    foo(a); 
    printf("%d\n",a); 
    foo2(&a); 
    printf("%d\n",a); 
} 

который печатает 05. Ваш случай то же самое, но вместо int и int*, думать о int** и int***

Answer 3

Ваш вызов функции должен выглядеть следующим образом:

network_construction (& вены, num_veins);

Тогда в улове вены функции network_construction следующим образом:

void network_construction(int ***veins, int *num_veins){ 

     int i, j; 


     for(i=0;i<NN;i++){ 
      num_veins[i] = NN/2; 
     } 


     *veins = malloc(NN * sizeof(**veins)); 
     for (i = 0; i < NN; i++) { *veins[i] = malloc(num_veins[i] * sizeof(**(veins[i]))); } 
     for (i = 0; i < NN; i++) { for (j = 0; j<num_veins[i];j++) { *veins[i][j] = -1; } } 


    return; 
    } 

Answer 4

Ваш таНос в network_construction() функция является проблемой. Легко для того чтобы преодолеть эту вещь, чтобы изменить свои функции следующим образом:

Ваш func.c должен выглядеть следующим образом:

#define NN 20 

    void network_construction(int **veins, int *num_veins){ 

      int i, j; 

      for (i = 0; i < NN; i++) { for (j = 0; j<num_veins[i];j++) { veins[i][j] = -1; } } 


     return; 
     } 

Ваша главная функция будет выглядеть следующим образом:

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

void main(){ 

    int num_veins[NN]; 
    int **veins; 
int i; 


     for(i=0;i<NN;i++){ 
      num_veins[i] = NN/2; 
     } 


     veins = malloc(NN * sizeof(*veins)); 
     for (i = 0; i < NN; i++) { veins[i] = malloc(num_veins[i] * sizeof(*(veins[i]))); } 

    network_construction(veins, num_veins); 

    printf("\n%d\n", num_veins[19]); 
    printf("\n%d\n", veins[2][2]); 

return; 
}