Cześć chciałem zrobić kalkulator w c++ ale wyskoczył mi błąd i nie wiem jak to naprawić

Question

Robiłem wybór działania jakie bedzie uzyte ale nie zadzialalo i nie wiem co zrobić pomozcie
 

#include <iostream>

 using namespace std;

 int main()
  {
      int wybor;
      int wybor2;
      int liczba1;
      int liczba2;
      int liczba3;
      int liczba4;
      int liczba5;
      
      cout<<"KALKULATOR"<<endl<<"Ile liczb chcesz uzyc(max 5): ";
      cin>>wybor;
    cout<<"Wpisz 1 liczbe: ";
      cin>>liczba1;
   cout << "Wybierz dzialanie:"<<endl<<"1. Dodawanie"<<endl<<"2. Odejmowanie"<<endl<<"3. Mnozenie"<<endl<<"4. Dzielenie"<<endl<<"Wybor: ";
   cin>>wybor2;
   
   if(wybor=2)
   cout<<"Wpisz 2 liczbe: "; cin>>liczba2;
   
   
   if(wybor=2)&(wybor2=1);
   {
   cout<<"Suma wynosi: "<<liczba1+liczba2;
   }
  else if (wybor=2)&(wybor2=2);
  {
   cout<<"Roznica wynosi: "<<liczba1-liczba2;
  }
    else if (wybor=2)&(wybor2=3);
 {
  cout<<"Iloczyn wynosi: "<<liczba1*liczba2;
 }
    else if (wybor=2)&(wybor2=4);
    {
       cout<<"Iloraz wynosi: "<<liczba1/liczba2;
    }
   

   return 0;
  }

  [Error] 'else' without a previous 'if'

jakby co to oczywiscie nie jest dokonczony kod tylko chcialem sprawdzic czy zadziala ta część

Comments

Prosimy wstawiać kod na forum w przeznaczone do tego bloczki. Ponadto należy zawsze dokładnie opisać swój problem - czyli np. podać błąd jaki się wyświetla, opisać co się robi i co się dzieje. Samo "nie zadziałało, pomóżcie" to nie jest właściwy opis sytuacji.

---

Samo "nie zadziałało, pomóżcie" to nie jest właściwy opis sytuacji.

w pewnym sensie OP, podał treść komunikatu błędu

 [Error] 'else' without a previous 'if'

---

Racja, zwracam honor, za szybko spojrzałem i musiało mi się zlać z kodem. Choć można byłoby się pokusić o bardziej szczegółowy błąd np. wskazujący na linię, ale tu kod jest niewielki a błąd raczej prosty, więc można uznać, że jest ok.

---

@Arkadiusz Waluk, masz w 100% rację

Samo "nie zadziałało, pomóżcie" to nie jest właściwy opis sytuacji.

ale "bardzo początkujący programiści" mają spore trudności z opisaniem sensownie swojego problemu, dla nich po prostu coś "nie działa" .

---

Oczywiście, należy tu uwzględnić, że każdy kiedyś zaczynał, miał podobne pytania i podobnie był zupełnie zdezorientowany w temacie. Dlatego też staramy się na to zwracać uwagę np. nad formularzem zadania pytania, w FAQ czy później w komentarzach, aby taka osoba na przyszłość już wiedziała jak zadać to pytanie lepiej, czyli podać więcej szczegółów ;)

Answer 1

• usuń średnik z końca linii z if-ami
  if(wybor=2)&(wybor2=1);
• logiczne and to &&
  if (wybor=2) && (wybor2=1)
• dla if-a użyj zapisu if (...) i dopiero grupuj wartości logiczne
  if ((wybor=2) && (wybor2=1))
• porównywanie wartości logicznych ==
  if ((wybor==2) && (wybor2==1))

[ on-line ]

 

 

P.S. jeżeli chcesz dać możliwość w programie podania więcej niż dwóch liczb lepszym rozwiązaniem od

int liczba1, liczba2, liczba3, liczba4, liczba5;

IMHO, będzie użycie tablicy, np.

int liczby[5]={0};

 

[ C++. Podstawy w 4h ]

Comments

dzieki

Answer 2

Wywal te średniki na końców linijek z if i else. Składnia polecenia if:

if (warunek) polecenie [ else polecenie ] ;

Po warunku nie ma średnika.