Почему в char (C++) записывается лишняя информация?

Question

[iRumba]

Итак, имеется структура

struct userInfo{
  char uCardNum[16];
  char uPIN[4];
  float uBalance;
};

Имеется класс, который управляет коллекцией таких структур. В нем есть процедура add

void add(char CN[16],char PIN[4],float Balance){
    userInfo ui;
    strcpy(ui.uCardNum,CN);
    strcpy(ui.uPIN,PIN);
    ui.uBalance=Balance;
    users.push_back(ui);
    printf(ui.uCardNum);
  };

Все просто, создаем структуру, наполняем ее данными, кладем в vector users
Но в uCardNum записывается не только номер карты, но и дописывается пин код. То есть
printf(ui.uCardNum);
выводит номер карты с пинкодом (например отправляю номер карты 1111111111111111 и пинкод 2222, на выходе будет 11111111111111112222) ((

Answer 1

[SHVV]

В зыбыли место под терминальный ноль. Прибавьте по одному символу каждой строке.

Comments

[iRumba]

Я не знаю как задать этот вопрос гуглу, поэтому спрошу у вас. В какое место нужно вставлять этот самый \0?
В вызов процедуры? Так?
users.add("1111111111111111\0","1111\0",500);

[SHVV]

Нет, главное место под него подготовить. У вас же все массивы ровно под размер строки, но ноль не помещается. То есть, надо увеличить размеры всех массивов на 1. Остальное компилятор сам всё сделает, так как строковые литералы и функции работы со строками автоматически вставляют ноль.

[SHVV]

MoreBeauty: И вообще, если пишете на С++, переходите на std::string что-ли.

[iRumba]

SHVV: string? а как быть с функциями, которые работают с char*?
Да я не то что бы пишу на С++, просто контрольные делаю... забавно, что сделал клиент-сервер и споткнулся на таком пустяке )))

[SHVV]

MoreBeauty: Во-первых, string может отдать и char*, это не проблема.
Во-вторых, у string есть куча своих методов и операторов для работы.
В-третьих, использовать char* - это С, а не С++.
Но я не знаю, какие у вас требования, может вас заставляют работать с char*, чтобы вы лучше понимали, как всё устроено, так как string внутри тот же char*, только в более "умной" обёртке.

[iRumba]

SHVV: ясно. Спасибо за помощь. В будущем использую string, сейчас уже не буду все под него переписывать.

[GavriKos]

SHVV: немного неверно. string отдает const char*, поэтому некоторые функции, которые модифицируют строку, работать не смогут. Но в целом да - переходите на std::string :-)

[mayorovp]

GavriKos: и опять не вполне верно. Если дать таким функциям &s[0], то они работать будут. Только надо внимательно за длиной следить...

Хотя это уже костыль получается.

Answer 2

[maaGames]

На самом деле, функции "strcpy(ui.uCardNum,CN); strcpy(ui.uPIN,PIN);" у тебя работают не правильно и могут приводить к краху программы из-за порчи стэка. В дебаге даже ругнуться компилятор на это может. Эти строки копируют символы из одной строки в другую, пока не встретится нуль-терминант в строке источнике, а у тебя нуль-терминанта нигде нет. Теоретически, в этих местах могут хоть гигабайты мусора скопироваться...
Как уже сказали, нужно либо нуль-терминант в конце строк делать (но это увеличит размер каждой строки на один символ), либо выводить эти строки не как строку, а через форматную строку, посимвольно.