Ist das in C++ nicht korrekt?

Question

#include <iostream> 

using namespace std; 

int main() 
{ 
    int szam,tart; 
    cout << "Num="; 
    cin >> szam; 
    while(szam!=tart){ 
     tart=szam; 
     cout << "Now insert a number which is not " << tart << "\n Your number is="; 
     cin >> szam; 
     if(szam==tart) 
      cout << "And you failed."; 
    } 
    return 0; 
} 

Ist dieser Code nicht korrekt? Mein Lehrer sagt immer wieder, dass es nicht korrekt ist, weil ich einen Wert für "scharf" Variable hätte nennen sollen, wie "cin >> tart".

Bitte antworten Sie so bald wie möglich.

Answer 1

Da tart noch nicht zugewiesen wurde, bevor es verwendet wird (im Vergleich), ist das Ergebnis des anfänglichen Vergleichs ein undefiniertes Verhalten.

Answer 2

Es ist richtig, aber Sie sollten tart einen Wert geben. Andernfalls wird es mit einem zufälligen initialisiert.

Answer 3

#include <iostream> 

using namespace std; 

int main() 
{ 
    int szam; 
    int tart=0;//need to initialize or need to call cin >> tart; 
    cout << "Num="; 
    cin >> szam; 
    while(szam!=tart){ 
     tart=szam; 
     cout << "Now insert a number which is not " << tart << "\n Your number is="; 
     cin >> szam; 
     if(szam==tart) 
      cout << "And you failed."; 
    } 
    return 0; 
} 

Answer 4

Es ist schlecht ausgebildet gcc erhalten Sie auch eine Diagnose geben:

main.cpp:13:15: warning: 'tart' may be used uninitialized in this function [-Wmaybe-uninitialized] 
    while(szam!=tart){ 
      ~~~~^~~~~~ 

mit nicht initialisierten Variablen ist nicht definiertes Verhalten.

Answer 5

"int herb;" erstellt eine Variable namens tart des Typs int, der Wert ist jedoch nicht festgelegt (undefiniert), was bedeutet, wenn Sie ihn verwenden, kann er und (meistens) Fehler verursachen.