C++. "Wypadają" dwie opcje tekstowe zamiast jednego.

Question

#include <iostream>

using namespace std;

int main()
{
    int wiek1, wiek2, wiek3;
    wiek3=wiek1&&wiek2;

    cout<<"Podaj jeden wiek tutaj: ";
    cin>>wiek1;
    
    cout<<"Podaj drugi wiek tutaj: ";
    cin>>wiek2;
    

    if (wiek3<=100)
    {
        cout<<"Gratulacje! Macie oboje 100 albo i ponad 100 lat, a mianowicie: "<<wiek1<<" i "<<wiek2<<" lat. Gratuluje!\n";
    }

    else ((wiek1>>100)&&(wiek2>>100));
    {
       cout<<"Macie oboje "<<wiek1<<" i "<<wiek2<<" lat. ";
    }

    return 0;
}

 

Więc tak: chciałem stworzyć mały program (gdyż jestem dopiero początkujący) który prosi o podanie wieku dwóch osób i jeżeli te osoby mają po 100 albo ponad 100 lat to program miał się zachować wyjątkowo, a gdyby osoby miały mniej niż 100 lat to wyświetlać komunikat jak wyżej. Problem w tym, że po wpisaniu wieków wyświetla mi od razu dwa komunikaty. Czy ktoś mógłby mi pomóc? Dziękuję z góry.

Comments

To ci się kompiluje w ogóle?

---

Zapewne jest to głupi błąd, ale dopiero co zacząłem przygodę z programowaniem i nie wiem co tutaj zrobić. No chyba, że cały kod jest źle napisany :O

---

No chyba, że cały kod jest źle napisany :O

W zasadzie tak :D Zobacz moją odpowiedź 

Answer 1

wiek3=wiek1&&wiek2;

Co to wg ciebie robi? W tym miejscu wiek1 oraz wiek2 są niezainicjalizowane, więc cokolwiek by to robiło - nie ma sensu. Wyjaśniając jednak operator &&: to jest operator logiczny więc wiek1 ora wiek2 zostają zrzutowane na bool (konwersja na bool: 0->false, [wartość różna od 0]->true). Następnie operator && daje true jeśli oba argumenty są true, lub daje false w innym przypadku. Następnie wynik operatora jest przypisywany do wiek3 i rzutowany na int na zasadzie false->0, true->1.

    if (wiek3<=100)
    {
        cout<<"Gratulacje! Macie oboje 100 albo i ponad 100 lat, a mianowicie: "<<wiek1<<" i "<<wiek2<<" lat. Gratuluje!\n";
    }

 Dlaczego w takim razie stosujesz operator mniejszy-lub-równy?

    else ((wiek1>>100)&&(wiek2>>100));
    {
       cout<<"Macie oboje "<<wiek1<<" i "<<wiek2<<" lat. ";
    }

Prawdopodobnie chodziło ci o else if. Co to obecnie robi: jeśli nie (wiek3<=100), to wykonuje ((wiek1>>100)&&(wiek2>>100)) (czyli w zasadzie nic nie robi, bo wynik nie jest nigdzie zapisywany);, a cout w linii 24 wykonywany jest bezwarunkowo.

Comments

Ajajajajaj. No, ale cóż, dzięki za odpowiedź.

---

A może mi ktoś do kija wyjaśnić co tam robi operator strumienia?

---

A, tego nie zauważyłem nawet XD Obstawiam, że zamiast >> miało być > lub =>.

@KayonPL, generalnie operatory >> oraz << dla intów to są bitshifty (tj. przesunięcia bitowe, doczytaj). wiek1>>100 oznacza "przesuń bitowo wartość wiek1 o 100 w prawo" co wg standardu jest UB (Undefined Behaviour, doczytaj) bo 100 jest większe od długości bitowej int (typ zmiennej wiek1), ale w praktyce możesz być pewien, że po prostu dostaniesz 0.

---

E.. No tak.. Dwuznaczność operatorów. Dzięki.