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

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.