чтение из stdin

[McZim]

Всем привет. Я начинаю сходить с ума, никак не могу понять в чем причина. в общем имеется некая функция некоторого класса, ее задачей является прочитать из stdin ввод имени пользователя/пароля/экземпляра базы. Организовал я это вот так:

Код:

void класс::функция()
{
    char* instance,*user,*passwd;

    cout << "Enter Instance: ";
    fgets(instance, 4096, stdin);

    cout << "Enter Username: ";
    fgets(user, 4096, stdin);

    cout << "Enter Password: ";
    fgets(passwd, 4096, stdin);
...
...
...

проблема заключается в том, что указатель user и passwd имеют одинаковый адрес памяти, соответственно когда я читаю из stdin в user то автоматом получаю тоже значение и в passwd. Смотрите вложение там скрин.

З.Ы.: качество картинки плохонькое, что бы весило поменьше 94,5Кб

[Джон]

Как работает fgets? Она сама выделяет память? Если да, то такого быть не должно.
Выдели сам память под эти переменные.

[McZim]

Вот описание fgets. Она входит в stdio.h

char * fgets ( char * str, int num, FILE * stream );

Get string from stream

Reads characters from stream and stores them as a C string into str until (num-1) characters have been read or either a newline or a the End-of-File is reached, whichever comes first.
A newline character makes fgets stop reading, but it is considered a valid character and therefore it is included in the string copied to str.
A null character is automatically appended in str after the characters read to signal the end of the C string.

Parameters

str
    Pointer to an array of chars where the string read is stored.
num
    Maximum number of characters to be read (including the final null-character). Usually, the length of the array passed as str is used.
stream
    Pointer to a FILE object that identifies the stream where characters are read from.
    To read from the standard input, stdin can be used for this parameter.

Return Value
On success, the function returns the same str parameter.
If the End-of-File is encountered and no characters have been read, the contents of str remain unchanged and a null pointer is returned.
If an error occurs, a null pointer is returned.
Use either ferror or feof to check whether an error happened or the End-of-File was reached.

[McZim]

Разобрался. Нужно всеже инициализировать память для указателей, вот только не понятно почему? Ушел учить мат.часть.

Код:

    char* instance = new char[4096];
    char* user = new char[4096];
    char* passwd = new char[4096];

[RXL]

McZim, см. функцию getline - она сама выделяет память, если переданный указатель равен NULL.

[McZim]

Есть небольшой минус у функции fgets она добавляет символ "\n", за этом нужно следить!!! Если не очень хочется можно воспользоваться функцией scanf.

Например:

Код:

char* username = new char[1024];

count << "Enter User Name: "
scanf("%s", username);

[McZim]

RXL, функция getline работает со строками (string& str), в моем случае char*, лишняя работа по преобразованию мне ни к чему.

[RXL]

McZim, ошибаешься: man getline
Это функция никакого отношения к iostreams и C++ не имеет. Входит в glibc.

[McZim]

RXL, хм странно А вот это что?

http://www.cplusplus.com/reference/string/

[RXL]

McZim, надо думать, что это совсем другая функция.

Кстати, не понятно, почему пользуясь iostreams ты работаешь с char*. Логично использовать родное.

[McZim]

RXL, потому что мне нужно передавать дальше в функцию именно char*

[RXL]

Макс, надо придерживаться чего одного. Иначе, по мере роста проекта, будут множественные труднонаходимые ошибки.

Не далее как вчера ловил багу в модуле, который без проблем работал не меньше 3 лет (его написали еще до меня). Взял модуль и осыпался - до полной неработоспособности. Проектировать надо и программить руками, а не из чурочек подпорки стругать... Это я о прошлых разработчиках.

[McZim]

RXL, согласен на 100%. Но у меня не очень большой опыт в проектировании проектов.

[RXL]

McZim, никогда не вредно учиться. Стоит читать коды больших GPL-проектов (если форматирование кода плохое - можно не читать - 90%, что проект с ошибками), разбираться в приемах.