Wykrywanie błędu w instrukcji warunkowej if.

Question

Witam, po długim czasie siedzenia nad poprawnym rozwiązaniem i niemożnością znalezienia poprawnej odpowiedzi zwracam się do Państwa z prośbą o sugestię.

Problem najprawdopodobniej tkwi gdzieś w instrukcji warunkowej "if". Program się poprawnie kompiluje, jednak zawsze wykonuje instrukcje, które powinny być wykonane przy spełnieniu warunku pierwszego.

#include <iostream>
#include <math.h>
using namespace std;

int main()
{
    string wybor;
    int i, promien;
    cout<<"OBLICZANIE POLA POWIERZCHNI LUB OBJETOSCI KULI"<<endl;
    cout<<"Podaj wartosc dlugosci promienia: "<<endl;
    cin>>promien;
    cout<<"Co chcesz policzyc? Jesli Pole - wcisnij 1 lub p, jesli objetosc - wcisnij 2 lub v. Odpowiedz zatwierdz naciskajac 'Enter'."<<endl;
   

    for (i=1; i<2; i++)

        {   cin>>wybor;

            if (wybor == "p" || 1)
                {
                    cout<<"Pole powierzchni kuli o promieniu "<<promien<<" jest rowne: "<<(4 * M_PI * (promien*promien))<<".";
                    i++;
                }
            else if (wybor == "v" || 2)
                {
                    cout<<"Objetosc kuli o promieniu "<<promien<<" jest rowne: "<<(4/3 * M_PI * (promien*promien*promien))<<".";
                   i++;
                }
            else
                {
                    cout<<"Bledny wybor! Sprobuj ponownie."<<endl;
                }
        }


}

 

Answer 1

if (wybor == "p" || 1) to tak jakbyś napisał if (wybor == "p" || true) czyli if(true) -> stąd zawsze pierwszy warunek ci się wykonuje.

Comments

Dziękuję bardzo za pomoc!

Czyli w jaki sposób zmienić 1 aby nie oznaczało już wartości logicznej?
"1" również nie działa

---

if (wybor == "p" || wybor=="1")

---

ja bym usunął pętle:

#include <iostream>
#include <math.h>
using namespace std;
 
int main()
{
    string wybor;
    int i, promien;
    cout<<"OBLICZANIE POLA POWIERZCHNI LUB OBJETOSCI KULI"<<endl;
    cout<<"Podaj wartosc dlugosci promienia: "<<endl;
    cin>>promien;
    cout<<"Co chcesz policzyc? Jesli Pole - wcisnij 1 lub p, jesli objetosc - wcisnij 2 lub v. Odpowiedz zatwierdz naciskajac 'Enter'."<<endl;
    
   cin>>wybor;
 
            if (wybor == "p")
                {
                    cout<<"Pole powierzchni kuli o promieniu "<<promien<<" jest rowne: "<<(4 * M_PI * (promien*promien))<<".";
                    i++;
                }
            else if (wybor == "v")
                {
                    cout<<"Objetosc kuli o promieniu "<<promien<<" jest rowne: "<<(4/3 * M_PI * (promien*promien*promien))<<".";
                   i++;
                }
            else
                {
                    cout<<"Bledny wybor! Sprobuj ponownie."<<endl;
                }
        
 
 
}

Dopasuj ten kod do swoich potrzeb.

---

@VirtualMember,
Dobra mam to, zamieniłem po prostu na:

  else if (wybor == "v" || wybor=="2")

i działa. Jeszcze raz wielkie dzięki za tak szybką i dokładną odpowiedź o tej dość specyficznej porze.

Oczywiście oznaczam jako "najlepsza odpowiedź".
Jeszcze raz dzięki!

xZenit

---

@Inn
Również dzięki!

---

Usuniecie pętli nie wchodzi w grę, ponieważ program wysypał by się po wybraniu błędnej opcji. (po wykonaniu instrukcji pod "else" nastąpiło by zakończenie działania, zamiast powrót do "cin'a".

Problem już rozwiązany.

Dzięki wszystkim za pomoc!

Zamykam.
xZ.

---

Właśnie zauważyłem drobny, dodatkowy błąd, który sprawiał, że pętla nie działała w sposób zamierzony.
Zamiast:

for (i=1; i<2; i++)

Musi być:

 for (i=1; i<2; i)

Teraz pętla nie zrywa się po jednokrotnym wykonaniu, a dopiero po spełnieniu odpowiednich warunków (określonych później w "if" oraz "else if").

Wersja finalna kodu znajduje się w dodatkowym komentarzu pod głównym pytaniem.

Answer 2

Finalna, poprawnie kompilująca się i działająca wersja kodu:

#include <iostream>
#include <math.h>
using namespace std;

int main()
{
    string wybor;
    int i, promien;
    cout<<"OBLICZANIE POLA POWIERZCHNI LUB OBJETOSCI KULI"<<endl;
    cout<<"Podaj wartosc dlugosci promienia: "<<endl;
    cin>>promien;
    cout<<"Co chcesz policzyc? Jesli Pole - wcisnij 1 lub p, jesli objetosc - wcisnij 2 lub v. Odpowiedz zatwierdz naciskajac 'Enter'."<<endl;


    for (i=1; i<2; i)

        {  cin>>wybor;
            if (wybor == "p" || wybor=="1" )
                {
                    cout<<"Pole powierzchni kuli o promieniu "<<promien<<" jest rowne: "<<(4 * M_PI * (promien*promien))<<".";
                    i++;
                }
            else if (wybor == "v" || wybor=="2")
                {
                    cout<<"Objetosc kuli o promieniu "<<promien<<" jest rowne: "<<(4/3 * M_PI * (promien*promien*promien))<<".";
                   i++;
                }
            else
                {
                    cout<<"Bledny wybor! Sprobuj ponownie."<<endl;
                }
        }


}