Wyświetla mi się dodatkowa linijka podczas kompilacji programu

Question

Cześć.  Od jakiegoś czasu borykam się z zadaniem i nie wiem gdzie leży błąd. Proszę o wskazanie i ewentualne wytłumaczenie czemu akurat tak. Cały czas pojawia mi się pierwsza linijka podczas niektórych kompilacji. Gdzie jest błąd oraz czy ten kod jest dobrze napisany, czy można go napisać lepiej. Dzięki za odpowiedź.

#include <iostream>
#include<string>
using namespace std;
string wiek1;
string wiek2;


int main()

{
 {
    
        cout<<"A teraz zrobisz zadanie gdzie program polazuje ktora osoba jest starsza,jesli obie maja wiecej jak 100 lat program sie zachowa w odpowiedni sposob  "<<endl<<"\n";
        cout<<"Podaj wiek pierwszej osoby"<<endl;
        getline (cin,wiek1,'\n');
        cout<<"Podaj wiek drugiej osoby"<<endl;
        getline(cin,wiek2,'\n');
   
        if (wiek1 > "100" && wiek2 > "100" )
 
        {
            cout<<"osoba pierwsza i osoba druga maja wiecej jak 100 lat - gratulacje!"<<endl;
        }
else if (wiek1 == "100" && wiek2 == "100" )
         {
           cout<<"Osoba pierwsza i osoba druga maja po 100 lat! doczekac takiego wieku to wielkie szczescie :)"<<endl;
         }
        if (wiek1 == wiek2)
         {
             cout<<"obie osoby sa w tym samym wieku "<<endl;
         }
   else if (wiek1>wiek2)
     {
         cout<<"Osoba pierwsza jest starsza od osoby drugiej "<<endl;
     }
     else
     {
         cout<<"osoba pierwsza jest mlodsza od osoby drugiej "<<endl;
     }
   }  
     
return 0;
}

 

Comments

Jeżeli nie ma wyraźnej potrzeby to staraj się deklarować zmienne w main()

Scope of Variables in C++

 

#include <iostream>

using namespace std; 
 
int main()
{
    // "Musisz porównywać liczby" -  @adrian17
    int wiek1=0, wiek2=0;

    // ... reszta kodu
      
    return 0;
}

lub (ponieważ wiek nie jest liczbą ujemną)

#include <iostream>

using namespace std; 
 
int main()
{
    // "Musisz porównywać liczby" - @adrian17
    unsigned int wiek1=0, wiek2=0;

    // ... reszta kodu
      
    return 0;
}

 

długi tekst dla lepszej "czytelności" możesz zapisać np. tak

#include <iostream>

using namespace std; 
 
int main()
{
    // "Musisz porównywać liczby" - @adrian17
    unsigned int wiek1=0, wiek2=0;
    
    cout << "A teraz zrobisz zadanie gdzie program polazuje ktora osoba jest starsza,\n"
            "jesli obie maja wiecej jak 100 lat program sie zachowa w odpowiedni sposob."
            "\n\n";
         
    cout << "Podaj wiek pierwszej osoby\n";
     cin >> wiek1;
     
    cout << "Podaj wiek drugiej osoby\n";
     cin >> wiek2;
     
    if (wiek1 > 100 && wiek2 > 100)
    {
        cout << "osoba pierwsza i osoba druga maja wiecej jak 100 lat - gratulacje!" << endl;
    }
    else if (wiek1 == 100 && wiek2 == 100)
    {

    // ... reszta kodu
      
    return 0;
}

 

---

main jest otwarty dwoma klamrami

---

@maciko84; to oznacza (komentarz:  // reszta kodu)  

    else if (wiek1 == 100 && wiek2 == 100)
    {
 
    // ... reszta kodu
       
    return 0;

że tu trzeba dopisać kod zaprezentowany przez OP-a, czyli

        cout << "Osoba pierwsza i osoba druga maja po 100 lat! doczekac takiego wieku to wielkie szczescie :)" << endl;
    } 
    else if (wiek1 == wiek2)
    {
        cout << "obie osoby sa w tym samym wieku" << endl;
    }
    else if (wiek1 > wiek2)
    {
        cout << "Osoba pierwsza jest starsza od osoby drugiej" << endl;
    }
    else
    {
        cout << "osoba pierwsza jest mlodsza od osoby drugiej" << endl;
    }

 

całość  [ on-line ]

#include <iostream>
 
using namespace std; 
  
int main()
{
    // "Musisz porównywać liczby" - @adrian17
    unsigned int wiek1=0, wiek2=0;
     
    cout << "A teraz zrobisz zadanie gdzie program polazuje ktora osoba jest starsza,\n"
            "jesli obie maja wiecej jak 100 lat program sie zachowa w odpowiedni sposob."
            "\n\n";
          
    cout << "Podaj wiek pierwszej osoby\n";
     cin >> wiek1;
      
    cout << "Podaj wiek drugiej osoby\n";
     cin >> wiek2;
      
    if (wiek1 > 100 && wiek2 > 100)
    {
        cout << "osoba pierwsza i osoba druga maja wiecej jak 100 lat - gratulacje!" << endl;
    }
    else if (wiek1 == 100 && wiek2 == 100)
    {
        cout << "Osoba pierwsza i osoba druga maja po 100 lat! doczekac takiego wieku to wielkie szczescie :)" << endl;
    } 
    else if (wiek1 == wiek2)
    {
        cout << "obie osoby sa w tym samym wieku" << endl;
    }
    else if (wiek1 > wiek2)
    {
        cout << "Osoba pierwsza jest starsza od osoby drugiej" << endl;
    }
    else
    {
        cout << "osoba pierwsza jest mlodsza od osoby drugiej" << endl;
    }

       
    return 0;
}

 

Answer 1

podczas niektórych kompilacji

kompilacja to proces tworzenia programu z kodu .cpp. Odpalanie programu to już... odpalanie, wykonywanie :)

W każdym razie...

if (wiek1 > "100" && wiek2 > "100" )

Porównujesz tutaj tekst, nie liczby. W sortowaniu tekstu (jak np w książce telefonicznej lub indeksie na końcu słownika), tekst "50" byłby po tekście "100", więc "50">"100". Musisz porównywać liczby, nie tekst.

Comments

Faktycznie zadziałało teraz poprawnie wystarczyło przenieść zmienna do sekcji Maine i usunąć liczbę z znaków niealfabetyczmych . Dodatkowo zapytam w jakich sytuacjach mogę deklarować nie w sekcji w Maine tylko jak zrobiłem ja to ?

---

Dodatkowo zapytam w jakich sytuacjach mogę deklarować nie w sekcji w Maine tylko jak zrobiłem ja to 

Chyba nie zrozumiałem. Nie ma "sekcji Main", jest funkcja o nazwie main().

Zmienne powinno się praktycznie zawsze definiować w funkcji, a nie globalnie (tak jak to zrobiłeś w oryginalnym pytaniu).

---

Ok, rozumiem dzięki !!