C++ Gets Identifier Not Found

Question

In my old C++ project I use to use the gets() command. I've done my research and noticed that it is not reliable anymore and my project won't run with using it.

I use this bit of code right here: Load(gets(new char[50]));

How would I now get this line of code to work properly? And if you could provide an explanation.

You can find some alternatives to `gets` in this question, but @ThomasMatthews is correct, use modern C++ replacements rather than trying to patch up old code. https://stackoverflow.com/questions/1694036/why-is-the-gets-function-so-dangerous-that-it-should-not-be-used — Retired Ninja, Sep 15 '17 at 02:11
@SirRyan98 you don't need to use `new char` with those. You can convert `std::string` to a chararcter array with `.c_str()`. — Saustin, Sep 15 '17 at 02:13
@SirRyan98: Apparently, you haven't done enough research on [`std::istream::getline`](http://en.cppreference.com/w/cpp/io/basic_istream/getline). — Thomas Matthews, Sep 15 '17 at 02:13
Please review the example here: http://www.cplusplus.com/reference/string/string/getline/ — Saustin, Sep 15 '17 at 02:15

score 1 · Accepted Answer · answered Sep 15 '17 at 02:17

1

Here's a simple solution:

std::string text;
std::cout << "Enter some text to load: ";
std::getline(cin, text);
Load(text.c_str());

If you must use character arrays, here's a code fragment:

const size_t ARRAY_CAPACITY = 64U;
char text[ARRAY_CAPACITY];
std::cout << "Enter some text to load: ";
cin.getline(&text[0], ARRAY_CAPACITY);
Load(text);

answered Sep 15 '17 at 02:17

Thomas Matthews

56,849
17
98
154

I get an error of text that states "argument of type "const char ^" is incompatible with perimeter of type "char ^" – SirRyan98 Sep 15 '17 at 02:22
For which line in which example? BTW, you never posted the declaration for `Load`. – Thomas Matthews Sep 15 '17 at 02:23
Always post error messages verbatim. There are no perimeters in C++, only with closed shapes. :-) – Thomas Matthews Sep 15 '17 at 02:25

C++ Gets Identifier Not Found

1 Answers1