Język C, brak zrozumienia napisanego "menu" w programie.

Question

Witam! Mam problem z poniższym napisanym przez siebie programem. Chciałem zrobić "menu" do wyboru, ale gdy odpalę program i chce wybrać przykładowo kwadrat muszę wpisać 2 razy dwójkę, a gdy trójkąt 3 razy trójkę. Nie wiem w czym tkwi problem. :(

#include <stdio.h>
#include <stdlib.h>



int main() {
	puts("Witaj! W tym porgramie mozesz wybrac czego pole chcesz liczyc,\naby to zrobic wybierz jedna z ponizszych cyfr");
	puts("Pole prostokata wybierz - 1\nPole kwadratu wybierz - 2\nPole rombu wybierz - 3\nPole trapezu wybierz - 4\nPole kola wybierz - 5\n");
	printf("Tu wpisz numer: ");
	
	int prostokat;
	scanf("%d", &prostokat);
	if(prostokat==1){
		
	int a, b;
	printf("Podaj a: ");
	scanf("%d", &a);
	printf("Podaj b: ");
	scanf("%d", &b);
	printf("Pole prostokata wynosi:%d", a*b);
}
	int kwadrat;
	scanf("%d", &kwadrat);
	if (kwadrat==2){
	int a,b;
	printf("Podaj a: ");
	scanf("%d", &a);
	printf("Podaj b: ");
	scanf("%d", &b);
	printf("Pole kwadratu wynosi:%d", a*b);
}
	int trojkat;
	scanf("%d", &trojkat);
	if(trojkat==3){
	
	int a,h;
	printf("Podaj a: ");
	scanf("%d", &a);
	printf("Podaj h: ");
	scanf("%d", &h);
	printf("Pole trojkata wynosi:%d", a*h*1/2);
	}
	
	return 0;
}

Answer 1

Powinieneś raz pobierać nr. menu od użytkownika a później w zależności od podanej przez niego wartości wykonać odpowiednią czynność:

int choose = 0;
scanf("%i", &choose);

if(choose == 1){
  // rysuj prostokąt
}else if(choose == 2){
   // rysuj kwadrat
}else if(choose == 3){
   // rysuj coś tam
}

 

Answer 2

To co opisałeś dzieje się dlatego, że po pierwszym pobraniu wartości program nie wie co to jest '2', którą wprowadzasz. Uwzględniłeś tylko co dzieje się, gdy użytkownik wprowadzi jedynkę. Gdy wpiszesz '2', to przy pierwszym if program nie ma żadnych instrukcji co zrobić z taką wartością więc nie robi nic. Kilka instrukcji później wartość jest pobierana ponownie i tutaj z kolei program zrobi cokolwiek tylko w przypadku, gdy wprowadzisz '2', bo nie ma poleceń co robić z innymi wartościami. Podobna sytuacja będzie przy trzecim pobraniu wartości, gdybyś w tym miejscu nagle zażyczył sobie obliczyć pole kwadratu to program nie zrobiłby nic bo ma instrukcje do wykonania tylko w sytuacji, gdy wartość jest równa '3'.

Co do poprawienia tego, to jak już ktoś wspomniał wcześniej, wprowadzona liczba powinna wpadać na if, który uwzględnia wszystkie możliwości, a później program na podstawie tej wartości powinien wskoczyć w odpowiedni blok instrukcji do wykonania.

Gdybyś zapisał to np. 

 

if(prostokat == 1)
{
 //instrukcje na pole prostotkata
}
else if(prostokat == 2)
{
//instrukcje na pole kwadratu
}
else if(prostokat == 3)
{
//instrukcje na pole trojkata
}

wtedy program odczyta wartość i sprawdzi pierwszy warunek, jeśli nie uda się go spełnić wtedy sprawdzi drugi itd. a wskoczy tylko tam, gdzie uda mu się uzyskać zgodność.

Zamiast instrukcji if można też użyć instrukcji switch:
 

switch(prostokat)
{
 case 1:
  //jakieś instrukcje
 break;
 case 2:
  // jakieś instrukcje
  break;
  itd.
}

efekt będzie taki sam.

Answer 3

Poeksperymentuj sobie z instrukcją switch :-) jest to instrukcja wyboru

int wybor;
printf("Wybierz opcję: \n");
printf("1. Dodawanie \n");
printf("2. Odejmowanie \n");
scanf("%d", &wybor);
int a, b, wynik;

switch(wybor)
{
    case 1:
         printf("Podaj a: \n");
         scanf("%d", &a);
         printf("Podaj b: \n");
         scanf("%d", &b);
         wynik = a + b;
         printf("Wynik: %d\n", wynik);
    break;
    case 2:
         printf("Podaj a: \n");
         scanf("%d", &a);
         printf("Podaj b: \n");
         scanf("%d", &b);
         wynik = a - b;
         printf("Wynik: %d\n", wynik);;
    break;

    default:
        printf("Brak opcji\n");
};