Problem mit Funktionen C++

[f.o.g.]

servus, hab ein problem bei einer funktion in c++
ich wollte eigentlich "nur" eine eingabe darauf prüfen ob ein integerwert eingegeben wurde und wenn es so ist diesen zurück liefern.
folgendes hatte ich mir ausgedacht.

int getint(void) 
{
int i;
i=0;
cin >> i;
if ( cin.fail()) i=-1;

return i;
}

die funktion befindet sich in einer schleife welche solange ausgeführt wird wie -1 zurückgeliefert wird.
das ganze funzt nur leider nicht so wie es soll.
folgendes passiert:
ich gebe z.b. ein g ein die funktion liefert folgerichtig -1 zurück.
jetzt kommt der erneute aufruf und ich kann nichts mehr eingeben.
wie kann ich dieses problem lösen. hab schon gegooglet.
ich verwende visual c++ 2008.
bin für jede hilfe sehr dankbar.
greetz der f.o.g.

 

[Skyte]

die funktion befindet sich in einer schleife

Nein, tut sie nicht.

jetzt kommt der erneute aufruf und ich kann nichts mehr eingeben.

Bei mir nicht.
Es funktioniert bei mir so: Wenn ich einen int eingebe, wird dieser zurueckgegeben, wenn nicht, dann -1.
Sonst nix weiter.


Allerdings verwende ich den g++ Compiler von Cygwin.
Kannst ja vllt mal das hier probieren:

int getint(void) {
int i;
if (!(cin >> i)) {
cin.clear();
i = -1;
}
return i; 
}

PS: Wie kommst du darauf ein C++ Problem bei "PHP, MySQL und Perl" reinzustellen? o0

 

[f.o.g.]

zum ersten. hast mich vllt falsch verstanden bzw ich mich falsch ausgedrückt ich hab im hauptprogramm ne schleife welche solange ausgeführt wird wie -1 zurückgegeben wird.
der erste aufruf erfolgt völlig korrekt es wird -1 zurückgegeben und der aufruf der funktion erneut gestartet. allerdings kann ich jetzt nichts mehr eingeben.
bin mal mit dem debugger drüber und der ist einfach über den cin >>i befehl drübergegangen ohne das ich was eingeben konnte.
bin c++ anfänger habs aber meinem kollegen auf arbeit gezeigt und er meinte es sollte so gehn.
ps: admin ist bereits wegen falschem forum informiert.

 

[Skyte]

Aso jo, stimmt, falsch verstanden, sorry.

Aber wie waers, wenn du gleich ne Funkiton machst, die diese Schleife beinhaltet?

int getint(void) {
int i = -1;
int err = 1;

while (err) {
if (!(cin >> i)) {
cin.clear();
cin.ignore(std::numeric_limits::max(), '\n');
err = 1;
}
else err = 0;
}

return i;
}

 

[f.o.g.]

würde ja gern den ganzen code posten komm aber grad net ran liegt auf arbeit. abgesehn davon ist der ganze code mitlerweile um die 800 zeilen code
ich hab im hauptprog ne fußgesteuerte schleife welche 2 abbruchbedingungen hat ich hab dann die zweite bedngung testweise mal rausgenommen. desshalb wollte ich die schleife eigenlich nicht in meiner funktion haben aber ich probiers morgen mal aus.

 

[Skyte]

Aso ok, naja hab jetzt mal auch ne Schleife rumgebaut.

Also der schleifenfreie Funktionscode sieht fast genauso aus wie mit Schleife.

Also das hier funzt:

int getint(void) {
int i;
if (!(cin >> i)) {
cin.clear();
cin.ignore(std::numeric_limits::max(), '\n');
i = -1;
}
return i; 
}

 

[Broken Sword]

Flasches Forum => Moved und außerdem habe ich den Titel angepasst.

Gruß
Broken Sword

 

[f.o.g.]

vielen dank ich werde den code morgen mal probieren und hoffe es geht dann auch. hast du mal eine eingabe wie 1g probiert?

ps: danke Broken Sword

 

[Skyte]

hast du mal eine eingabe wie 1g probiert?

"1g" wird als "1" erkannt. Weiss aber nicht wie man dem auf einfache Weise entgegentreten soll, da auch cin.fail() in dem Fall false ist.

PS: Allerdings wird (bei der schleifenfreien Variante) auch immer das Problem bleiben, dass man keine "-1" eingeben kann, weil -1 ja der Fehlerwert ist.

 

[mawhonic]

Wenn der Fehler auch bei einer Eingabe wie "1g" auftreten soll, muss man wahrscheinlich selbst Hand anlegen. Soll heißen: Man liest die Eingabe als String ein und überprüft dann Zeichen für Zeichen, ob es erlaubt ist:

bool isInteger(String in)
{
if(in.at(0) != 0x2D && (in.at(0) < 0x30 || in.at(0) > 0x39))
return false;

for(int i = 1; i < in.length(); ++i)
{
if(in.at(i) < 0x30 || in.at(i) > 0x39)
return false;
}

return true;
}

Hier ein Beispiel von mir, ich konnte es allerdings bisher nicht testen! Des Weiteren müsste zur Nutzung der eingegebenen Zahl diese natürlich anschließend noch (unter Verwendung von Stringstreams) in eine echte Zahl konvertiert werden.

 

[Skyte]

Oder eine andere Variante:
Ich merk mir den String, mach Integer draus, mach wieder String draus und vergleiche.
Wenn sie nicht gleich sind, dann hing da noch was dran.

Also das hier funzt jetzt auch mit "1g" und solchen Sachen:

#include 
#include 
#include 

using namespace std;

int getint(void) {
int i;
string tmp;
stringstream s1;

cin >> tmp;
s1  if (!(s1 >> i)) {
cin.clear();
cin.ignore(numeric_limits::max(),'\n');
i = -1;
}
else {
s1.clear();
s1.str("");
s1  if (s1.str() != tmp) i = -1;
} 

return i;
}

 

[f.o.g.]

Vielen Dank an Skyte der mir sehr weiter geholfen hat vor allem mit seinem letzten Beitrag.
zwar streikt der compiler bei dieser zeile cin.ignore(numeric_limits::max(),'\n'); ich hab aber dann den ersten parameter durch eine sehr hohe zahl ersetzt.
kann dann auch zu
greetz der f.o.g.