Problem z komendą If . Yes/No

Question

Cześć! Mam problem z tą komendą. Mógłby ją ktoś naprawić?

.......

 case '3':
    system("cls");
    cout << "Are you sure that you want Restart your computer?" << endl;
    cout << "(YES/NO)" << endl;
    cin >> odp;

    if ((odp=='yes'&&(odp=='YES'))
    {
    cout << "LOL";
    }
    else if ((odp=='no')&&(odp=='NO'))
    {
        system("cls");
        goto FirstMenu;
    }
.....

 

Comments

Jakiego typu jest odp?

---

int

---

Proszę umieszczać kod źródłowy w bloczku.

---

W takim razie co próbujesz osiągnąć? Próbujesz porównać liczbę całkowitą z napisem?

Answer 1

Na pierwszy rzut oka widzę np. 'yes'.

W języku C++ napisy (łańcuchy znaków) umieszcza się w podwójnych cudzysłowiach, pojedyncze są zarezerwowane dla pojedynczych znaków.

Poza tym (choć to już raczej nie ma wpływu na poprawność działania, ale dotyczy dobrych praktyk) staraj się nie używać funkcji system(), ani instrukcji goto.

Do tego tutaj połknąłeś chyba jeden nawias:

 if ((odp=='yes'&&(odp=='YES'))

No i tak jak kolega mówi, powinieneś poprawić operator logiczny z AND na OR - bo nigdy tekst nie będzie jednocześnie równy dwóm tekstom. 

Comments

Dziękuje

Answer 2

Zmien &&  na ||

Comments

Dziękuje