Question

我有一个代码:

void Read(int T[], int len) {
    int i;
    T = (int*) malloc(len * sizeof *T);
    for (i=0;i<len;i++) {
        scanf("%d", &T[i]);
    }
}

我用这种方式:

int *T;
Read(T,len);

接下来,我想写我的桌子:

void Write(int T[], int len) {
    int i;
    for(i=0;i<len;i++) {
        printf("%d, ", T[i]);
    }
    printf("
");
    return;
}

使用它:

Write(T,len);

它给了我错误的结果。我几乎可以肯定这个问题与“ 强/ 强/ 强” 有关, 但我无法解决这个问题。

提前感谢

Answer 1

其中一个问题可能是您在 Read 函数中重新修改本地变量 T :

int *T;
Read(T, len);
// ...

void Read(int T[], int len) {
    int i;
    T = (int*) malloc(len * sizeof *T);
    // ...
}

T 在 Read 中的 是真实 T 变量的复制件。您为副本指定了新值, 但原件保持不变。



要实际修改外部 T , 请将一个指针切换到外部 :

int *T;
Read(&T, len);
// ...

void Read(int **T, int len) {
    int i;
    *T = (int*) malloc(len * sizeof *T);
    for (i=0;i<len;i++) {
        scanf("%d", &(*T)[i]);
    }
}

Answer 2

尝试以下功能( 函数 < code> write 未改变) :

void Read(int T[], int len) {
    int i;
    for (i=0;i<len;i++) {
        scanf("%d", &T[i]);
    }
}

T = (int*) malloc(len * sizeof *T);
Read(T,len);

Write(T,len);

<强度 > UPDATE : 但如果您想要 read 来分配内存, 它必须接收指针的指针 :

void Read(int *T[], int len) {
    int i;
    T = (int**) malloc(len * sizeof **T)

    for (i=0;i<len;i++) {
        scanf("%d", &(*T)[i]); /* not the easiest expression in the language */
    }        
}

Read(&T, len);
Write(T, len);

实际上,修改 Read 直接返回指针比较容易,但也许这不是你想要的: int *Read(int)len {{__/code>。

Answer 3

内特的解决方案是好的和花花公子但我发现它有点隐约。

C本身是相当模糊的因为它是,我不认为有什么理由使它这么多两倍。我建议你使用以下:

将您的 lese read(int T[], int len) 的签名更改为 int* read(int T[], int len) 。

这样做有多重优势。它不仅使您的代码更容易读取, 而且还消除了下流读数所带有的“ 隐藏” 副作用( 部分), 并明确指出了您的意图, 这样您就不会那么容易忘记拨打 < code> free () , 仅仅因为您看到一个指针被退回, 从而提示函数中可能存在一种隐蔽的动态分配。另外, 您需要担心的间接因素更少。

然后,您可以做以下工作:

int* Read(int T[], int len) {
    int i;
    T = (int*) malloc(len * sizeof(T));
    for (i=0; i<len; ++i) {
        scanf("%d", &T[i]);
    }
    return T; // Basically the only change!
}

现在,在调用您的 Read () 的函数中:

int main() {

const int size = 5;
int *T = 0; // It s not needed to do a prior declaration and initialization
            // but it does silence a warning which is a good thing.
T = Read(T, size);
Write(T, size);

// Do more stuff and free() whenever needed

return 0;
}

友情链接