expected ';' before '}' token

Question

#include <iostream>
using namespace std;

int main () 

{
	int i, a, n, max;
	cout << "ILE ELEMENTOW? \n";
	cin >> n;
	for(i = 0; i < n; i++)
	{
		cout << "PODAJ LICZBE:";
		cin >> a;
		if (i == 0)
		max = a;
		else
		if (a > max)
		max = a;
	}
cout << "LICZBA NAJWIEKSZA TO: " <<max;
cout << "\n";
cout << "PROGRAM NAPISAL BARTOSZ LUCZAK";
cout << "\n";
system ("pause");
return 0	
}

 

Comments

Brakuje średnika po return 0

---

#include <iostream>
#include <climits>
using namespace std;

void pause()
{
    cout << "\n\nPROGRAM NAPISAL © BARTOSZ LUCZAK";
    cout << "\nNacisnij ENTER - aby zakonczyc.";
    cin.ignore();
    cin.get();    
}
 
int main()
{
    int a, n, max = INT_MIN;
    cout << "ILE ELEMENTOW? "; cin >> n;
    for (int i = 0; i < n; ++i)
    {
        cout << "PODAJ LICZBE [ " << i + 1 << " ] : "; cin >> a;
        if (a > max) max = a;
    }
    
    cout << "\nLICZBA NAJWIEKSZA TO: " << max;
    
    pause();
    return 0;   
}

 

---

To ja też zaproponuję małe poprawki

int main()
{
    cout << "ILE ELEMENTOW? ";
    int n;
    cin >> n;
    if (cin.fail() || n <= 0) {
        cerr << "Przekazano nieprawidłowe dane" << endl;
        return 1;
    }
    int max = INT_MIN;
    for (int i = 0; i < n; ++i)
    {
        cout << "PODAJ LICZBE [ " << i + 1 << " ] : ";
        int a;
        cin >> a;
        if (cin.bad()) {
            cerr << "Przekazano nieprawidłowe dane" << endl;
            return 1;
        }
        if (a > max)
            max = a;
    }
     
    cout << "\nLICZBA NAJWIEKSZA TO: " << max;
     
    pause();
    return 0;   
}

Albo nawet

int main()
{
    auto wczytaj_liczbe = [](const std::string &wiadomosc) {
        int liczba;
        cin >> liczba;
        if (cin.fail())
            throw runtime_error("Nieprawidlowy format danych");
        return liczba;
    }
    try {
        auto n = wczytaj_liczbe("ILE ELEMENTOW?");
        if (n <= 0)
            throw runtime_error("Ilosc elementow musi byc wieksza od zera");
        auto max = wczytaj_liczbe("PODAJ LICZBE 1");
        for (int i=1; i<n; i++) {
            auto a = wczytaj_liczbe("PODAJ LICZBE "+to_string(i+1));
            if (a > max)
                max = a;
        }
         cout << "\nLICZBA NAJWIEKSZA TO: " << max;
    }
    catch (std::exception &ex) {
        cerr << ex.what() << endl;
        return 1;
    }
   
    pause();
    return 0;
}

 

Answer 1

Brak średnika za return 0. W dodatku niezainicjalizowane zmienne.

Answer 2

Tak jak przedmówcy mówią, średnik po return 0 :)