Pierwsze kroczki z C++

Question

#include <iostream>
#include <windows.h>

using namespace std;
string pogoda;
int main()
{


    cout << "Jaka jest pogoda?(Dobra/Zla) :";
    cin >> pogoda;
{


    if(pogoda=="Dobra");
{


        Sleep( 1000 );



        for (int i=15; i>=1; i--)
                    cout << i <<endl;



                    cout <<"START"<<endl;
}

    else(pogoda=="Zla");

{
    cout <<"Odmowa lotu" << endl;
}



    return 0;
}

Chcę wykonać "programik", który zadaje pytanie "Jaka jest pogoda?" Jeśli pogoda jest dobra, to program uruchamia odliczanie od 15 w dół i wyświetla komunikat "START". W przeciwnym wypadku (jeśli pogoda jest zła) program wyświetla komunikat "Odmowa lotu". Siedzę nad tym, męczę się, i cały czas coś, zawsze jakiś błąd...

Comments

Przestudiuj odcinek tutorialu Mirka Zelenta o pętlach i myślę że szybko się uporasz z problemem.

Albo w sumie trochę Ci pomogę abyś zrozumiał lepiej. Spójrz na konstrukcję pętli a zrozumiesz jak działają. Przy okazji polecam taki sposób formatowania aby nie pogubić się z tymi klamrami. Ale to już jak kto woli.

 
    if(pogoda=="Dobra") {
         for (int i=15; i>=1; i--) {
             cout << i <<endl;
             Sleep(1000);
         }
         cout <<"START"<<endl;
    }

    else if(pogoda=="Zla") {
        cout <<"Odmowa lotu" << endl;
    }

---

Właśnie jestem w trakcie kursu p. Mirka, ale za Chiny ludowe nie mogę znaleźć tego (tych) drobnego błędu(ów). Nie wiem czy to ja mam pecha, ale po wpisaniu powyższego kodu, CodeBlocks pokazuje mi błąd na... ostatnim nawiasie klamrowym znajdującym się na samym końcu (pod return)

---

Jak pokazuje błąd na nawiasie to prawdopodobnie liczba nawiasów zamykających i otwierających nie jest równa, albo brakuje średnika 

Masz tutaj cały kod i sprawdź czym się różni mój od Twojego. 

#include <iostream>
#include <windows.h>

using namespace std;

int main() {
    string pogoda;
    cout << "Jaka jest pogoda?(Dobra/Zla) :";
    cin >> pogoda;

    if(pogoda=="Dobra") {
         for (int i=15; i>=1; i--) {
             cout << i <<endl;
             Sleep(1000);
         }
         cout <<"START"<<endl;
    }

    else if(pogoda=="Zla") {
        cout <<"Odmowa lotu" << endl;
    }

    return 0;
}

 

---

    if(pogoda=="Dobra");
{
 
 
        Sleep( 1000 );
 
 
 
        for (int i=15; i>=1; i--)
                    cout << i <<endl;
 
 
 
                    cout <<"START"<<endl;
}

Po if dałeś średnik, więc czy warunek będzie prawdą czy nie, to i tak wykona się to co jest między klamrą otwierającą a zamykającą. Bo to co jest między klamrami już nie dotyczy warunku if. Średnik po if oznacza, że to pusty warunek. Tak samo zrobiłeś z else. if to nie funkcja, tylko keyword, więc się średnika nie umieszcza!

Answer 1

Niepotrzebne średniki po if i else.