gdzie jest błąd

Question

Witam, wykonuje zadanie z książki(Poproś użytkownika o podanie wieku dwóch osób i wskaż, która z nich jest starsza.
Jeśli obie osoby mają powyżej 100 lat, program powinien zachować się w szczególny sposób).

Mój kod:

#include <iostream>
#include <string>
using namespace std;
int main ()
{
int uzytk1,uzytk2;
cout << "Podaj wiek uzytkownika1: " << "\n";
cin >> uzytk1;
cout << "Podaj wiek uzytkownika2: " << "\n";
cin >> uzytk2;
    if (uzytk1 > uzytk2)
        {
            cout << "uzytk1 jest starszy";
        }
    else if((uzytk1 > 100 && uzytk2 > 100))
        {
            cout << "za starzy" << "\n";
        }
    else
        {
            cout<< "uzytk2 jest starszy"<<"\n";
            return 0;
        }
}

Problem w tym, że je nie widzę tego ostatniego nawiasu klamrowego, wcześniej miałem problem bo gdy był tak jak w kodzie, wyświetlało mi się, że jest za dużo, w obecnej chwili kopiując z code::block tutaj kopiuje mi się z } kończącym funkcję main, w kodzie u siebie tego nie widzę, wytłumaczy mi ktoś dlaczego?

Answer 1

Ciało każdej funkcji powinno się zaczynać i kończyć nawiasami klamrowymi. Może masz coś nie tak z autouzupełnianiem w code::blocks.

Comments

Problem w tym, że po raz pierwsz się z takim czymś spotkałem, niczego ukrytego nie ma, przynajmniej tak mi się wydaję, a gdy najeżdżam na nawias { po funkcji main nawiasu zamykającego nie ma nigdzie podkreślonego..

---

Program wydaje się być ok, bo nie kompilowałem. dorzuć return 0;
Pytania możesz zapisać tak:

cout << "Pytanie? \n";

Koniunkcję (&&) zamień na alternatywę: || 

---

śmieszna sprawa bo usunąłem puste pole i wstawiłem jeszcze raz nawias zamykający funkcję main, i dział, tak czy inaczej dzięki za zainteresowanie i pomoc :)

---

A skoro myślisz o programowaniu poważnie to przerzuć się na Visual Studio 11. Trochę kobyła (11GB), ale pisze się świetnie.

---

Nie ma sprawy  

---

Visual Studio 11 jest darmowe? Narazie idę do 3 klasy technikum informatycznego i strony www oraz programowanie najbardziej przypadły mi do gustu, zamierzam trochę przerobić w temacie, jeśli się utrzymam to możliwe, że mogę o tym myśleć poważniej(piszę utrzymam, ponieważ zdaję sobie sprawę, że programowanie to rzecz nie łatwa).

---

Nie jestem pewny ale niekomercyjnie raczej tak. Pisz pisz i jeszcze raz pisz. 

Answer 2

nawiasy sa ok - dlaczego u Ciebie nie ma } - musisz poszukać sam

ale co do kodu :

1. warunek >100 powinien być najpierw sprawdzony, bo np. dla 104 i 102 będziesz miał uzyt1 jest starszy

2. return 0 tak dla porządku ( nie ma to znaczenia w tym przypadku)

   - musi byc jako ostatnia instrukcja a nie w sekcji else

Comments

Rzeczywiście nie zauważyłem, dzięki :)