Структура C++ с потерей значения char* при возврате? ⇐ C++

[Anonymous]

Я новичок в C++ и не могу понять, почему мой код теряет значение имени пользователя и пароля.

Код: Выделить всё

Client *client = new Client();

char *authBuffer = new char[MAX_BUFFER_SIZE]; // buffer contains 234567|john|pwd4john

bufferToClient(authBuffer, client);

printf("id=%i username=%s password=%s\n", client->id, client->username, client->password);

// output is: id=234567 username= password=

// function

void bufferToClient(char *buffer, Client *client) {

memset(client, 0x00, CLIENT_STRUCT_SIZE);

char *token = strtok(buffer, PIPE); // pipe |

client->id = atoi(token);
client->username = strtok(0, PIPE);
client->password = strtok(0, PIPE);

printf("%i\n", client->id);
printf("%s\n", client->username);
printf("%s\n", client->password);

// everything is ok and prints out the values as expected

// 234567
// john
// pwd4john

delete token;
}

// struct

struct Client {

unsigned int id;
const char *username;
const char *password;

virtual ~Client() {

delete username;
delete password;
}
};

static const int CLIENT_STRUCT_SIZE = sizeof(Client);

Я уверен, что это должно быть что-то простое, но я просто не могу понять, почему оно продолжает отбрасывать значения имени пользователя и пароля. В какой-то момент имя пользователя вернулось нормально, но пароль был перепутан.
Одна из вещей, которые я пробовал, была:

Код: Выделить всё

client->username = std::string(token).c_str();
client->password = std::string(token).c_str();

вывод: client id=234567 username=john пароль=    ²
Но, как вы видите, имя пользователя правильное, но пароль перепутан?
вывод: client id=234567 username=john pass=    ²
Но, как вы можете видеть, имя пользователя правильное, но пароль перепутан?
p>
Обновление... чтобы справиться с висячими указателями, я применил этот подход?

Код: Выделить всё

void bufferToClient(char *buffer, Client *client) {

memset(client, 0x00, CLIENT_STRUCT_SIZE);

char *token, *tokenPtr;

tokenPtr = token = new char[strlen(buffer) + 1];

token = strtok(buffer, PIPE);

client->id = atoi(token);
client->username = strtok(0, PIPE);
client->password = strtok(0, PIPE);

delete[] tokenPtr;
}

Обновление: похоже, теперь оно работает. Честно говоря, я не уверен, что заставило это начать работать, и я не хочу возвращаться, чтобы это выяснить. Что я сделал, так это удалил преобразования std::strings().c_str()... затем я добавил указатель на исходный указатель (чтобы я мог сделать то, что, как я надеюсь, является правильным удалением []), поскольку strtok перемещается указатель вместе, как кто-то объяснил в комментариях. И я обнаружил ошибку, когда я печатал переменную, но забыл поместить переменную в оператор печати, что меня сбило с толку, потому что было напечатано значение, похожее на то, что я ожидал, но неверное. Я думаю, что это были преобразования std::strings().c_str(), хотя, как только я избавился от них, имя пользователя и пароль возвращались правильно.
К вашему сведению, этому 15+ лет код, в который я вношу несколько улучшений... поэтому это не последний стиль C++. Для старого кода это невероятно быстро по сравнению с тем же кодом в Java21. Это как сравнивать Bugatti с Honda Civic 1980-х годов… хаха. Я думаю, "чем ниже ты спускаешься, тем быстрее ты идешь".

Подробнее здесь: https://stackoverflow.com/questions/786 ... n-returned