使用函数内部的calloc，更改指针作为函数参数传递

Question

我不明白为什么第二个printf循环输出的数据不同于在函数作用域内完成的第一个printf循环。难道是指针在函数内部以某种方式改变，所以当它返回时它会返回一个不同的值？

输出：

第一个printf内部功能：
零件TMP | 01245
零件X | 40001
零件Y | 98760

第二个printf以外的功能，在主：
它返回jiberish并且与在函数内打印时不一样。 我试着fprintf，这样我就可以在这里快速粘贴结果，但后来我收到了一个无法解释的调用堆栈错误。

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

void ProtocolParse_Start(int *numParts,char **parts, char *str, const char* partsDelim) 
{ 
    int partCount = strChrCount(str,'~'); 
    *numParts = partCount; 

    parts = (char**)calloc(partCount,sizeof(char)); 
    char *tempPart; 


    tempPart = strtok (str,partsDelim); 
    parts[0] = (char*)calloc(strlen(tempPart),sizeof(char)); 
    strcpy(parts[0],tempPart); 


    int i =1; 
    for(; i < partCount; i++) 
    { 
     tempPart = strtok (NULL, partsDelim); 
     parts[i] = (char*)calloc(strlen(tempPart),sizeof(char)); 
     strcpy(parts[i],tempPart); 
    } 

    i =0; 
    for(; i < partCount; i++) 
    { 
     printf ("%Parts %s\n",parts[i]); 
    } 

} 

void ProtocolParse_End(int numParts,char **parts) 
{ 
    int i = 0; 
    for (; i < numParts; i++) 
     free (parts[i]); 

    free (parts); 
} 


int main() 
{ 
    char proto[32] = "TMP|01245~X|40001~Y|98760~"; 

    char **parts; 
    int numParts; 
    ProtocolParse_Start(&numParts, parts,proto,"~"); 


    int i =0; 
    for(; i < numParts; i++) 
    { 
      printf ("%Parts %s\n",parts[i]); 
    } 

    ProtocolParse_End(numParts,parts); 

    return 0; 
} 

任何人都可以请说明我的问题。因为我不确定我在做什么错误？

Answer 1

该函数内parts的赋值对的char **parts没有影响。为了修改它，你需要传递一个指向parts的指针，并添加一个额外的间接级别（是的，你会得到三个星号）。

将数据分割为字符串的代码也不正确：您需要分配一组字符指针，然后将每个令牌分别复制到该数组中。

void ProtocolParse_Start(int *numParts, char ***parts, char *str, const char* partsDelim) 
{ 
    int partCount = strChrCount(str,'~'); 
    *numParts = partCount; 

    *parts = malloc(partCount * sizeof(char*)); 

    char *tempPart; 
    tempPart = strtok (str,partsDelim); 
    (*parts)[0] = malloc(strlen(tempPart)+1); 
    strcpy((*parts)[0], tempPart); 

    int i =1; 
    for(; i < partCount; i++) 
    { 
     tempPart = strtok (NULL, partsDelim); 
     (*parts)[i] = malloc(strlen(tempPart)+1); 
     strcpy((*parts)[i],tempPart); 
    } 

    i =0; 
    for(; i < partCount; i++) { 
     printf ("%Parts %s\n", (*parts)[i]); 
    } 

} 

我进行了三次修改了代码：

• 替换calloc与malloc：初始化每个元素无论如何，所以没有理由去填零块
• 在前面去掉石膏malloc - 这不是必需的C
• 增加一个到strlen(tempPart) - 你需要这个空字符串。

Answer 2

有不同的错误：

当您将参数传递给函数它总是被复制。 你给了char **parts，它被复制。

函数内部用覆盖指针calloc ed的新地址覆盖复制的输入。

考虑一个简单的例子：

void doSomething(int a){ 
    a=5; 
} 
///... 
int b = 6; 
doSomething(b); 

当你调用doSomething(b)，你的B没有改变。 如果您想要更改它，您必须将指针传递给b。

void doSomething(int* a){ 
    *a=5; 
} 
///... 
int b = 6; 
doSomething(&b); 

这与你的char*阵列一样。

你在你的主中有char** parts，你想要设置为分配数组。 所以你必须通过它的指针。并将获得的地址写入解除引用的指针。

另一个大错误是，你歪歪斜撞到第一个calloc。它应该是sizeof(char*)。

你的例程应该像这样开头：

void ProtocolParse_Start(int *numParts,char ***parts, char *str, const char* partsDelim) 
{ 
    int partCount = strChrCount(str,'~'); 
    *numParts = partCount; 

    *parts = (char**)calloc(partCount,sizeof(char*)); 
    char *tempPart; 
    //... 

（所有进一步访问的功能部件已提领部分）

和呼叫必须看起来像：

ProtocolParse_Start(&numParts, &parts,proto,"~");