Menu w C, zabezpieczenie przed podaniem błędnej opcji

Question

Witam tworzę prosty program w C który musi się opierać na menu wyboru. Mniejsza z tym co kryje się pod funkcją Wykonaj() gdyż to teraz nie ma znaczenia. Program wyświetla proste menu i oczekuje od użytkownika jakiegoś wyboru, użytkownik podaje jakiego dokonuje wyboru z klawiatury.Problem w tym tkwi że chciałbym się zabezpieczyć przed tym gdyby użytkownik podał jakąś błędną opcję w menu. Gdy użytkownik poda np. "5" lub "6" wszystko jest okej, jednak podczas podania "a" lub "0,2" program się ciągle zapętla nie można nawet na spokojnie wpisać niczego. Czy ktoś wie jak rozwiązać ten problem ? Byłbym wdzięczny za pomoc. Pozdrawiam, życzę miłej niedzieli.

void Menu()
{
printf("1.Wybierz 1\n2.Wybierz 2\n3.Zakoncz\n");

int wybor;
int wymiar;
scanf("%d",&wybor);

if (wybor == 1)
{
    Wykonaj(1);
}
if (wybor == 2)
{
    Wykonaj(3);
}
if (wybor == 3)
{
    printf("Program zostanie zaraz zamkniety");
    exit(0);
}
if ((wybor!=1) || (wybor!=2)|| (wybor!=3))
{
    printf("Podales zly numer - nie ma takiej opcji ! Podaj numer jeszcze raz !\n\n");
    Menu();
}
}

int main()
{
Menu();
}

 

Comments

while (1)
{
if (wybor == 1)
{
    Wykonaj(1); break;
}
if (wybor == 2)
{
    Wykonaj(3); break;
}
if (wybor == 3) break;
{
    printf("Program zostanie zaraz zamkniety");
    exit(0);
}
if ((wybor!=1) || (wybor!=2)|| (wybor!=3))
{
    printf("Podales zly numer - nie ma takiej opcji ! Podaj numer jeszcze raz !\n\n");
    Menu(); 
continue;
}
}

tylko nie wiem czy w c jest continue; while ale mi się wydaje że powinien być.

---

@boniek, Jeżeli switch nie działa, możesz użyć if else. Chociaż u mnie switch działa bez zarzutu. Kod to przeanalizowania:

(zwróc uwagę na getchar())

void Menu()
{
printf("1.Wybierz 1\n2.Wybierz 2\n3.Zakoncz\n");
char wybor;
int wymiar;
scanf("%c", &wybor);

switch (wybor)
{
        case '1': Wykonaj(1); break;
        case '2': Wykonaj(2); break;
        case '3': printf("Program zostanie zaraz zamkniety"); exit(0); break;
        default: printf("Podales zly numer - nie ma takiej opcji ! Podaj numer jeszcze raz !\n\n"); getchar(); Menu(); break;
}
}
int main()
{
Menu();
return 0;
}

 

Answer 1

Zamiast if-ów daj switch i jako opcje domyślną daj wywołanie funkcji menu wtedy jeżeli nie będzie opcja 1 czy dwa to wyświetli się jeszcze raz menu.

Comments

W sumie słuszny pomysł. Dzięki, tak też spróbuję ;) !

---

Tylko czy w C jest switch?

---

Jest, jest. Akurat w tym menu to rozwiąże mój problem ale bardzo ciekawi mnie czy da się jakość zabezpieczyć by użytkownik nie mógł do zmiennej zadeklarowanej jako int wpisywać liter czy też innych specjalnych znaków, a tylko same cyfry.

---

@12david12, Sprawdziłem i przy switchu program zachowuje sie tak samo :(

---

#include <stdio.h>


void Wykonaj(int a) {
	printf("%d\n", a);
}

void Menu()
{
    printf("1.Wybierz 1\n2.Wybierz 2\n3.Zakoncz\n");
    char wybor;
    int wymiar;
    scanf("%c",&wybor);
    switch(wybor){
        case '1':
            Wykonaj(1);
            break;
        case '2':
            Wykonaj(2);
            break;
        case '3':
            Wykonaj(3);
            break;
        default:
            printf("Podales zly numer - nie ma takiej opcji ! Podaj numer jeszcze raz !\n\n");
            Menu();
    }
}
int main()
{
    Menu();
}

 

---

@12david12 Proszę przeczytaj opis problemu.

---

I u mnie się nie zapętla przy wprowadzaniu liter.

Answer 2

Spróbuj użyć switch'a:

#include <stdio.h>

void Menu()
{
    printf("1.Wybierz 1\n2.Wybierz 2\n3.Zakoncz\n");
    int wybor;
    int wymiar;
    scanf("%d",&wybor);
    switch(wybor){
        case 1:
            Wykonaj(1);
            break;
        case 2:
            Wykonaj(2);
            break;
        case 3:
            Wykonaj(3);
            break;
        default:
            printf("Podales zly numer - nie ma takiej opcji ! Podaj numer jeszcze raz !\n\n");
            Menu();
    }
}
int main()
{
    Menu();
}

 

Comments

@DeBos123 Proszę przeczytaj opis problemu.

Answer 3

switch(wybor){
		case 1: Wykonaj(1); break;		
		case 2: Wykonaj(3); break;
		case 3: printf("Program zostanie zaraz zamkniety"); exit(0);
		
		default:
 				printf("Podales zly numer - nie ma takiej opcji ! Podaj numer jeszcze raz !\n\n");
				
				cin.clear();
				cin.ignore(100, '\n'); // Czysci bufor oraz zapobiega przed wykonaniem sie petli n-razy gdy wciścniesz n-liter
				//system("CLS"); <- Wyczyści ekran
				
				Menu();
	}

 

Comments

To nie jest C