字符指针

Question

这里有一个片段：

void addproductInterface(Tsklep **head){ 
    char* name = (char*)malloc(sizeof(char)*100); 
    double price; 
    do{ 
     printf("Name: "); 
     scanf("%s[^\n]", name); 
     fflush(stdin); 
     printf("\nPrice: "); 
     scanf("%lf", &price); 
     fflush(stdin); 
     addProduct(&(*head), name, price); 
    } while(prompt("Do you want to add another one?")); 

它的工作原理，但在我添加另一个产品，它改变了以前的一个（和以前的）这个名字。 看来，我每次都通过相同的指针，我只是改变一个数组（当我添加另一个产品时）它指向的数组。 我理解正确吗？ 你有什么想法如何解决它？

Answer 1

将name指向的内容移到循环中。

Answer 2

这听起来像你所描述的，是的。很难确定没有看到addProduct()的代码，但那将是分配新内存的地方。

你应该使用一个临时的，自动（栈），缓冲区的输入，然后做永久的分配，当你存储的记录，在addProduct()：

do{ 
    char name[64]; 
    double price; 

    printf("Name: "); 
    scanf("%63s", name); 
    fflush(stdin); 
    printf("\nPrice: "); 
    scanf("%lf", &price); 
    fflush(stdin); 
    addProduct(&(*head), name, price); 
} while(prompt("Do you want to add another one?")); 

你也应该错误检查scanf()呼叫，如果发生意外输入，它们可能会失败。

另外，don't cast the return value of malloc() in C。

Answer 3

您只在函数的开头分配一次名称，以便覆盖每个循环执行的内容。在do-while循环中移动分配。

Answer 4

招行char* name = (char*)malloc(sizeof(char)*100);的do while内循环如下，

do{ 
    char* name = (char*)malloc(sizeof(char)*100); 
    printf("Name: "); 
    scanf("%s[^\n]", name); 
    fflush(stdin); 
    printf("\nPrice: "); 
    scanf("%lf", &price); 
    fflush(stdin); 
    addProduct(&(*head), name, price); 
} while(prompt("Do you want to add another one?")); 

Answer 5

你应该将内循环的分配。

（顺便说一下，你的电话的scanf容易溢出name，因为你没有限制的大小）。