为结构中的 char* 赋值

Question

我的代码有问题，我无法将字符串值分配给结构中的 char*。谁能告诉我我的代码有什么问题以及为什么？

#include <iostream>
using namespace std;

typedef struct{
char* name;
char* city;
int age;
} person;
void main()
{
person * user;
user = (person*)malloc(sizeof(person*));

cout << "Please fill in the user info.." << endl << "Name: ";
cin >> user->name;
cout << "Age: ";
cin >> user->age;
cout << "City";
cin >> user->city;

cout << "The user info is:" << endl << "Name: " << user->name << endl << "Age: " << user->age << endl << "City: " << user->city << endl;
system("pause");
}

非常感谢。

Answer 1

您的代码是 C 和 C++ 风格的可怕混合，正如 Mike 的评论所说，选择一种语言并正确使用它。

#include <iostream>
#include <string>
using namespace std;

struct person {
  string name;
  string city;
  int age;
};

int main()
{
  person user;

  cout << "Please fill in the user info.." << endl << "Name: ";
  cin >> user.name;
  cout << "Age: ";
  cin >> user.age;
  cout << "City";
  cin >> user.city;

  cout << "The user info is:\n" << "Name: " << user.name << "\nAge: " << user.age << "\nCity: " << user.city << endl;
  system("pause");
}

不要在不需要时动态分配（这样就不会有分配错误内存量的风险，也不会有不为字符串分配内存的风险，这些都是您在原始程序中犯的错误） .

main 必须返回 int 而不是 void

typedef struct {...} x; 在 C++ 中不是必须的，直接说struct x {...};

Don't overuse endl

Answer 2

您已经用两种不同的语言标记了这个问题，并且似乎是在以两种语言的可怕混合进行编码。你应该选择一种语言并坚持下去。

如果这是 C++，则使用标准库：

#include <string>
#include <iostream>

struct person {
    std::string name;
    std::string city;
    int age;
};

int main() {    // not void
    person user;
    // ...
    std::cin >> user.name;
    // ...
}

如果它是 C，那么你需要为字符串分配内存。从堆中分配它们：

person user;
user.name = malloc(MAX_NAME_SIZE);
user.city = malloc(MAX_CITY_SIZE);

或将它们嵌入到结构中：

typedef struct {
    char name[MAX_NAME_SIZE];
    char city[MAX_NAME_SIZE];
    int name;
} person;

当您从输入中读取数据时，请注意不要溢出这些固定大小的缓冲区。

如果出于某种原因你真的想为结构本身使用malloc，那么为结构分配足够的空间，而不仅仅是一个指针：

user = malloc(sizeof(person));

Answer 3

malloc(sizeof(person));

这将为保存 2 个 char* 和 1 个 int 创建内存。

你还需要为你错过的 struct 中的 char*s 分配内存。

Answer 4

我不知道c++，但如果您选择坚持使用c，您的user->name 不会分配内存，user->city 也不会。您需要使用具有适当大小的malloc() 为指针变量分配内存，然后使用它们来存储值。

还有，

user = (person*)malloc(sizeof(person*));

应该是

user = malloc(sizeof(*user));

那么，void main()要改成int main()，需要加上return 0。最好将退出状态发送回 shell。

---

注意：通过查看

的用法

#include <iostream>
using namespace std;

它看起来更像是 c++ 的东西，但 IMO，逻辑应该是一样的。