Program nie wykonuje działań w CODE::BLOCKS

Question

Mój problem polega na tym, że po wpisaniu pierwsze i drugiej liczby wyświetla się komunikat "Nie ma takiej opcji w menu". Używam Linuxa/Ubuntu.14.04.

PS. Przepraszam ale nastąpiła pomyłka, ten kod poniżej jest mój a ten wcześniejszy to był kod z tutorialu.

 

#include <iostream>
#include <stdio.h>
#include <cstdlib>
#include <ncurses.h>
using namespace std;

float x,y;
char wybor;

int main()
{

for(;;)
{

    cout << "Podaj 1 liczbe: ";
    cin >> x;
    cout << "Podaj 2 liczbe: ";
    cin >> y;

    cout<<endl;
    cout << "MENU GLOWNE" << endl;
    cout << "-----------------" << endl;
    cout << "1. Dodawanie" << endl;
    cout << "2. Odejmowanie" << endl;
    cout << "3. Mnozenie" << endl;
    cout << "4. Dzielenie" << endl;
    cout << "5. Koniec programu" << endl;


    cout << endl;
    wybor=getch();

    switch(wybor)
    {
    case '1':
            cout<<"Suma = "<<x+y;
    break;

    case '2':
            cout<<"Roznica = "<<x-y;
    break;

    case '3':
            cout<<"Iloczyn = "<<x*y;
    break;

    case '4':
            if (y==0) cout << "Nie dzielimy przez zero!";
            else cout<<"Iloraz = "<<x/y;
    break;

    case '5':
            exit(0);
    break;

    default: cout<<"Nie ma takiej opcji w menu!";
    }
    getchar();getchar();
    system("clear");
}
    return 0;
}

 

Answer 1

Trochę poza tematem, ale...

Używam Linuxa/Ubuntu.14.04.

Skąd wziąłeś <conio.h>? Z tego co wiem on jest specyficzny dla Windowsa.

Comments

Już poprawione:)

Answer 2

U mnie też nie działa . To jest chyba wina funkcji getch()

bo po getch(); dałem cout

Answer 3

Switch służy do porównywania liczb całkowitych, niczego poza tym.

Comments

Ale to jak jest char to sobie zamienia na ASCII ;)

---

OK, kod ASCII to liczba, ale on go w przykładzie nie używa, a słowo "liczba całkowita" nie oznacza tylko zmiennej typu int czy enuma.

---

Czyli w jaki sposób mam zedytować kod?

---

Po prostu użyj ifów, ale jeśli bardzo chcesz użyć tego switcha, to sprawdź kody tych znaków w tablicy ASCII.

---

Nic nie ruszaj, używanie switcha na charach w ten sposób jest jak najbardziej poprawne.

---

No nie, nie używa, a przynajmniej jawnie, bo kompilator chyba sam sobie to przekonwertowuje do int-a (ASCII). No, na filmikach Pana Mirosława takie rozwiązanie działa http://youtube.com/watch?v=ib7LXBNfZ7g To jest ten odcinek, o ile się nie mylę.

A któż to ci dał łapkę w dół, chłopczyku? ;)

---

Ten kod powyżej jest mój, tamten był z kursu. Mój błąd.

---

Ja dałem tej odpowiedzi, bo zdanie ani nie jest prawdziwe, ani nie ma związku z problemem.

bo kompilator chyba sam sobie to przekonwertowuje do int-a (ASCII)

Zgadza się. Zacytuję standard: 

character literal (...) has type char, with value equal to the numerical value of the encoding of the c-char in the execution character set <tu: można założyć że chodzi o ASCII>.

---

Czyli jak to ASCII mam podstawić?

---

Nic nie rób, ze switchem wszystko jest ok. Zgadzam się z tą drugą odpowiedzią, że prawdopodobnie chodzi o coś z działaniem getch().

Spróbuj cout'ować "wybor". (cout << (int)wybor;) Strzelam że wpadł tam znak entera (nowej linii).

---

Też tak coś myślałem z tym enterem, ale kod jest (chyba) taki sam jak w kursie Pana Zet, więc nic o tym nie mówiłem.

---

Dobra, mój strzał że to z curses jest rzecz. On normalnie jest stworzony do pełnych interfejsów konsolowych i z tego co widzę powinno się go używać dopiero po wywołaniu initscr();. I faktycznie, wtedy łapanie inputu działa, ale z kolei przestaje działać poprawnie u mnie endl... coś, curses pewnie nie lubi gdy cokolwiek innego niż on próbuje pisać na ekran.

Sugestia moja: zignoruj getch(). To nie jest częścią standardu C++ i zawsze był bolesnym punktem zarówno na Windowsie jak i na Linuxie, a wszystko tylko dlatego że komuś się nie chce wcisnąć głupiego entera.

---

To miejsce:

 

cout << endl;
    wybor=getch();

 

Ma wyglądać tak?

 

    cout << endl;
    wybor=getch();
    cout << (int)wybor;

 

---

Tak; pewnie dostaniesz -1, jak ja. Patrz moja najnowsza odpowiedź wyżej.