结构体中用字符串数组还是指针？代码正常运行却报C4047警告

嘿，我来帮你拆解这两个问题——首先是结构体里选char数组还是指针的抉择，然后是你代码触发警告的原因和修复方案：

一、结构体里用char数组还是指针？ #

这事得分情况看，完全取决于你的使用场景：

• 优先选char数组的场景：
  如果你能确定字符串的最大长度（比如元素名称最多20个字符，符号最多2个字符），那数组绝对是更省心的选择。它的内存直接嵌在结构体里，不用手动去malloc分配内存，也不用操心free释放，完全不会出现野指针或者内存泄漏的问题，用strcpy/strncpy赋值就行，安全性拉满。
  就像你现在定义的char name[20]; char symb[3];，刚好匹配元素名称和符号的固定长度需求，非常合适。
• 适合用char指针的场景：
  要是你需要处理变长字符串，或者不想浪费固定长度的内存（比如有些元素名称很短，但数组占了20字节空间），那可以用char *name;。但这里要注意坑：你必须手动给指针分配内存（比如用malloc），用完之后还要记得free，不然会内存泄漏；另外如果直接把字符串字面量赋值给指针，千万别去修改它——因为字面量是只读的，修改会触发未定义行为。

二、你的代码警告原因及修复 #

你遇到的Warning C4047本质是参数的指针类型不匹配，咱们一步步捋清楚问题出在哪：

原代码里的错误点： #

1. scanf的参数传递错误：
   你写的scanf("%s %s %lf", &name, &symbol, &e.wt);里，&name是char (*)[20]类型（指向整个char数组的指针），但scanf的%s格式符需要的是char*类型（指向数组首元素的指针）。其实数组名name本身就等价于&name[0]，是标准的char*类型，所以这里直接写name就行，多余的&反而导致类型不匹配。

2. strcpy的参数传递错误：
   同样的道理，strcpy(&e.name, &name);里，&e.name是char (*)[20]，而strcpy的第一个参数要求是char*；&name也是char (*)[20]，第二个参数要求是const char*。正确的写法是直接传数组名e.name和name——数组名会自动隐式转换为指向首元素的指针，刚好符合strcpy的参数要求。

修复后的完整代码： #

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

typedef struct element { 
    char name[20]; 
    char symb[3]; 
    double wt; 
} element;

int main() {
    element e; 
    char name[20]; 
    char symbol[3]; 
    // 去掉name和symbol前面的&
    scanf("%s %s %lf", name, symbol, &e.wt); 
    // 去掉e.name和name前面的&
    strcpy(e.name, name); 
    strcpy(e.symb, symbol);
    
    // 加个打印验证结果
    printf("元素名称: %s, 符号: %s, 权重: %.2lf\n", e.name, e.symb, e.wt);
    return 0;
}

额外的安全建议： #

为了避免输入的字符串超过数组长度导致缓冲区溢出（这是很常见的安全隐患），建议用更安全的函数替代：

• 如果是用微软的编译器，可以用scanf_s，它能限制输入长度：

  scanf_s("%19s %2s %lf", name, (unsigned)_countof(name), symbol, (unsigned)_countof(symbol), &e.wt);
  

• 用strncpy代替strcpy，并手动添加字符串结束符（因为strncpy不会自动加'\0'）：

  strncpy(e.name, name, sizeof(e.name)-1);
  e.name[sizeof(e.name)-1] = '\0'; // 确保字符串结束
  strncpy(e.symb, symbol, sizeof(e.symb)-1);
  e.symb[sizeof(e.symb)-1] = '\0';
  

内容的提问来源于stack exchange，提问作者Swajn