C++ Erkennen ENTER-Taste durch den Benutzer gedrückt

Ich habe eine Schleife, wo ich den Benutzer auffordern, einen Namen einzugeben. Ich muss stoppen, wenn der Benutzer die EINGABETASTE drückt..... oder wenn 20 Namen eingetragen wurden. Aber meine Methode nicht zu stoppen, wenn der Benutzer die EINGABETASTE drückt

//loop until ENTER key is entered or 20 elements have been added
bool stop = false;
int ind = 0;
while( !stop || ind >= 20 ){

    cout << "Enter name #" << (ind+1) << ":";
    string temp;
    getline(cin, temp);
    int enterKey = atoi(temp.c_str());        

    if(enterKey == '\n'){
        stop = true;            
    }
    else{
        names[ind] = temp;
    }

    ind++;


}

InformationsquelleAutor user69514 | 2010-05-10

c++getline user-input

4

Wandeln Sie das Lesen von string in integer mit atoi:
```
int enterKey = atoi(temp.c_str());        
```
Wenn temp ist ein string wie "1234" dies enterKey zu 1234. Dann vergleichen Sie enterKey zu den ASCII-Wert \n. Dies ist den meisten wahrscheinlich nicht tun, etwas sinnvolles.

Auch std::getline nur Lesen, werden die Zeichen bis zum, aber nicht einschließlich, der nächste '\n'. Wenn ein Benutzer die EINGABETASTE drückt, ohne die Eingabe von anderen Zeichen, std::getline wird ein leerer string zurückgegeben. Wenn ein string leer ist, kann leicht geprüft werden, mit seinen empty() Methode:
```
getline(cin, temp);
if (temp.empty()) {
  stop = true;
}
```
- youre right. dies funktioniert
InformationsquelleAutor sth
2

getline Essen Ihre Trennzeichen, das '\n', so dass Sie wahrscheinlich wollen zu überprüfen, die für einen leeren string. Tun Sie es, bevor der Aufruf von atoi.

InformationsquelleAutor LukeN
2

versuchen:
```
while( !stop && ind < 20 )
```
oder:
```
using namespace std;
vector <string> names; //edited.
for (int ind = 0; ind < 20; ++ind)
{
    cout << "Enter name #" << (ind+1) << ":"; 
    string temp;
    getline(cin, temp); 
    if (temp.empty())
        break;
    names.push_back(temp);
}
```
- Eigentlich ist es ein bisschen ein Durcheinander. Es sind ein paar Fehler...
- Ja, ich Stimme Ihnen vollkommen zu, ein for Schleife ist der Weg zu gehen. Dein code hat einen Fehler, obwohl. Der resultierende Vektor wird mit 20 leeren Saiten, gefolgt von einigen Reihe von Antworten.
- Sie verließ die atoi Schritt. Benutzt du einen vector<int> nicht auf vector<string>
- Er sollte überprüfen werden, sowohl die Länge der Eingabe (zu erkennen in die leere Zeile) und er sollte die überprüfung der stream-Zustand, weil nur gute Codierung.
- OP nicht legen Sie die int in den Behälter, die int war nur zu erkennen, dass eine neue Zeile.
- Danke @Geoff bearbeitet, um korrekt zu sein. Verändert Geist auf halbem Weg durch, indem Sie es aus benutzen konnte, Namen[ind] so user69514 nicht zu verwirrt, aber ging mit best-practice-statt...
InformationsquelleAutor Charles Beattie
1

Versuchen stop = temp.empty() statt. getline enthält keine neue-Zeile-Zeichen. Eine leere Zeile sollte im Ergebnis eine leere Zeichenfolge.

Auch Charles ist es korrekt, deine while-Bedingung falsch ist, verwenden Sie while( !stop && ind < 20). Die Art und Weise haben Sie es geschrieben, muss der Benutzer geben Sie 20 Werte, und eine leere Zeile. Charles' ändern sagt zu unterbrechen, wenn eine der beiden Bedingungen erfüllt ist (wenn nicht beide).

Der Vollständigkeit halber, hier ist der neue vorgeschlagene code:
```
bool stop = false;
int ind = 0;
while( !stop && ind < 20 ){

    cout << "Enter name #" << (ind+1) << ":";
    string temp;
    getline(cin, temp);
    if(temp.empty()) {
        stop = true;
    } else {
        names[ind] = temp;
    }

    ind++;    
}
```
Persönlich würde ich den code schreiben, wie folgt:
```
vector<string> names;
for(int ind = 0; ind < 20; ind++) {
  cout << "Enter name #" << (ind + 1) << " (blank to stop): ";
  string name;
  getline(cin, name);
  if(name.empty() || cin.eof()) {
     break;
  }
  names.push_back(name);
}

cout << "Read " << names.length() << " names before empty line detected." << endl;
```
- Warum nicht einfach brechen, wenn temp.empty()? Dann brauchen Sie nicht die else-Struktur
- ah. ja. sollte <. Ich kopierte und verpasste den Fehler dort.
- Sie auch vergessen zu flip ind >= 20 Bedingung.
- Ich Stimme zu, es sollte eine for-Schleife mit einem break. Wenn wir schreiben, alles, was der OP Probleme hatte ursprünglich nicht klar.
- Ich habe meine persönliche Vorliebe, die ich denke, ist in-line mit dir @jmucchiello
InformationsquelleAutor Geoff
0

Den Sie verwenden möchten cin.get(); cin >> temp; ich glaube.
- cin >> temp wird nur ein token. Es ist möglich OP will Aussagen, dass keine Leerzeichen enthalten.
InformationsquelleAutor Puppy

Schreibe einen Kommentar

Du musst angemeldet sein, um einen Kommentar abzugeben.