Хотя вы уже обнаружили, что cin >> str;
не требуется, поскольку вы просто снова пишете в str
с помощью cin.getline (str, sizeof str)
, существует ряд дополнительных проблем, на которые следует обратить внимание:
1. Если ваш компилятор не древний, вам следует #include <string>
, а не C-заголовок string.h
;
2. Не использовать magic-цифры в вашем коде.Если вам нужна константа, например, для максимального числа символов в str
, #define
константа или используйте глобальный enum
, чтобы сделать то же самое, например,
#define MAXC 200 /* if you need a constant, #define one (or more) */
...
char str[MAXC]; /* don't use 'magic-number', use a constant */
Таким образом, когда иесли в будущем вы измените количество символов в str
, вам не придется выбирать весь ваш код и менять каждое вхождение magic-number , например, cin.get(str,200);
.
3. Проверка КАЖДОГО пользовательского ввода.В противном случае неудачный ввод может установить бит ошибки в вашем входном потоке, а дополнительные попытки чтения из потока с установленным битом ошибки могут привести к неопределенному поведению.Вы можете сделать:
if (!getline(cin,str1)) { /* VALIDATE every input */
cerr << "error: input failure - str1.\n";
return 1;
}
и
if (cin.get (str, sizeof str))
cout << str << endl;
( примечание: больше нет попыток чтения после cin.get (str, sizeof str)
, поэтому вы защитите использование str
достаточно)
4. Всегда выводите символ новой строки после вывода последней строки, чтобы убедиться, что ваша программа POSIX-совместима.В противном случае во многих операционных системах вы запутаете запрос пользователя при записи в stdout
или создадите выходной файл, не совместимый с POSIX, если перенаправите вывод в файл, например,
my cat has none01:22 wizard:~/dev/src-cpp/tmp/debug>
Поместите его в целом,Вы можете сделать что-то вроде:
#include <iostream>
#include <string> /* depending on your compiler */
#define MAXC 200 /* if you need a constant, #define one (or more) */
using namespace std;
int main (void) {
char str[MAXC]; /* don't use 'magic-number', use a constant */
string str1;
cout << "enter a string: ";
if (!getline(cin,str1)) { /* VALIDATE every input */
cerr << "error: input failure - str1.\n";
return 1;
}
cout << str1 << endl;
cout << "enter second string: ";
// cin >> str; /* not needed */
if (cin.get (str, sizeof str))
cout << str << endl;
}
Пример использования / Вывод
$ ./bin/cin.get_getline
enter a string: my dog has fleas
my dog has fleas
enter second string: my cat has none
my cat has none