[H.....s]

Czy mógłby mi ktoś wyjaśnić co zrobiłem źle w tym programie? ćwiczę używanie funkcji do...while. Gdy wpisuję wybór inny jak '1', '2', to wtedy wyświetla się:
wybór:
wybor:
Wie ktoś może dlaczego to się dzieje?

void dodawanie(void);
void odejmowanie(void);
int main()
{
char wybor;
printf("1.Dodawanie\n2.Odejmowanie\n");
do
{
printf("Wybór:\n");
wybor=getchar();
}while(wybor!='1'&&wybor!='2');
if(wybor=='1')dodawanie();
if(wybor=='2')odejmowanie();

#codereview #programowanie #naukaprogramowania

[ravpawlisz]

@Hertesus: ale wklej cały kod...

[a.....m]

@Hertesus: nie korzystaj z getchar - gdy potwierdzasz znak enterem wprowadzasz do bufora 2 znaki i oba znaki są wykonywane w pętli.

[diemsi]

@Hertesus: to dość proste, ale będę wredny i nie podam Ci odpowiedzi ;-) za to proponuję wyświetlać wynik funkcji getchar() (zm. wybor) i zastanowić się, co on właściwie oznacza. Miłej zabawy! :)

[GrS11]

@Hertesus: Ja myślę, że dlatego, że tak napisałeś pętle?
W do while warunek jest na końcu i jeżeli nie jest on spełniony, to polecenie z do jest wykonywane ponownie.

EDIT: Chyba zamiast && rozsądniej byłoby dać || (?)

[diemsi]

@GrS11: nie, bo wtedy warunek będzie prawdziwy zawsze = nieskonczona pętla

[jkrsmkr]

@diemsi: Polecam tego Wykopowicza.

[dinokoder]

@Hertesus: po prostu walnij tego printfa "wybór" przed pętlę, to nie będzie Cię tak drażniło jak wybierzesz nie to co trzeba.

[H.....s]

@anonimowy_anonim: @dinokoder: @jkrsmkr: @diemsi: @GrS11: W pewnym sensie rozwiązałem problem.
diemsi: Wyjawisz mi tę tajemnicę?

[a.....m]

@Hertesus: wypisuj sobie kod ASCII wprowadzanego klawisza - to dostaniesz odpowiedź - to co pisałem wyżej :)

[v0ldenet]

@Hertesus: znak nowej linii to też znak, dodatkowo terminale blokują się aż dostaną ten znak.

Użyj getline, termios.h, ignoruj niepoprawne znaki (10, 13 i EOF) albo użyj nowoczesnego języka programowania.