Minimum i maksimum - zadanko

Question

Siemka, postanowiłem zrobić sobie zadanko, które wczytuje od użytkownika pięć dowolnych liczb i ustala, która liczba jest najmniejsza, a która największa.Nie wiem czy dobrze je robię, ale gdy chcę skompilować to wyskakuje mi taki błąd: "reference to 'min'  is ambiguous" i to samo dotyczy max. Nie wiem jak ten błąd rozwiązać.

O to kod: 

#include <iostream>
#include <cstdlib>

using namespace std;

int liczba;
int min;
int max;
min=liczba;
max=liczba;
int main()
{
    cout << "Podaj piec liczb: " << endl;
    cout<<endl;

    for(int i=0;i<5;i++)
    {

        cout<<i+1<<" liczba: ";
        cin>>liczba;

        if(min>liczba)
        {
            min=liczba;
        }
        if(max<liczba)
        {
            max=liczba;
        }
    }
    cout<<"Najwieksza liczba to: "<<max;
    cout<<"Najmniejsza liczba to: "<<min;


    return 0;
}

 

Answer 1

W std są funkcje std::min i std::max - zmień nazwę zmiennych min i max albo usuń using namespace std;.

Answer 2

Problem tkwi najprawdopodobniej w tym, że inicjalizujesz zmienne min i max zmienną liczba która nie ma przypisanej wartości i nie wiadomo co się ma wydarzyć

Comments

czyli co mam zrobić?

Answer 3

Witaj.

Pamiętaj, że min i max to są słowa zarezerwowane przez C++. Zmień ich nazwy. Poza tym w linii 9 i 10 próbujesz przypisać min i max wartość zmiennej liczba, a w tej zmiennej nie ma jeszcze żadnej wartości bo nic jej nie przypisałeś. 

Pozdrawiam! 

Comments

Pamiętaj, że min i max to są słowa zarezerwowane przez C++.

A to od kiedy? Lista zarezerwowanych słów: klik  (2.11)

Tzn. oczywiście wiem o co ci chodziło :P, ale min i max nie są słowami zarezerwowanymi przez język. Faktycznie w bibliotece standardowej znajdują się funkcje o takich nazwach, ale też z tego powodu są w namespace-sie. Tutaj kolizja występuje tylko z powodu deklaracji using namespace std;

 

---

Dziękuję za rozjaśnienie, codziennie można się czegoś nauczyć :D

Pozdrawiam!

Answer 4

Nie mozesz nazwac zmiennych min i max bo to sa funkcje w c++, chociaz chyba w devie sie da.

http://cpp0x.pl/dokumentacja/standard-C++/max/466

http://cpp0x.pl/dokumentacja/standard-C++/min/465

Poczatkowo najlepiej przypisac 0, a jeszcze lepiej granice przedzialow int, czyli:

Przedzial inta to od -2147483648 do 2147483647, wiec

najmniejsza = 2147483647;

najwieksza = -2147483648;

lub przed petla wczytaj jedna liczbe i przypisz.

Answer 5

To jest bez sensu:

min=liczba;
max=liczba;

W dodatku zmienne w c++ nie mog sie nazywać min oraz max, bo to s słowa kluczowe, majce specjalne znaczenie w c++. Zmień ich nazwy

Pszypisujesz domyśln wartość (0) do zmiennych, które sa już domyślnie zainicjalizowane i maj taka sama wartość.

Po za tym, jeśli użytkownik poda same liczby większe niż 0 to program nie będzie działać poprawnie. Tak samo, jeśli poda same liczby większe niż 0. Bo zmienne min i max, jesli s globalne to domyślnie s inicjalizowane zerem.

Comments

min i max to nie słowa kluczowe, a jedynie funkcje, których pełne brzmienie jest z przestrzenią nazw std

Answer 6

Dziękuję Wam za pomoc. Problem rozwiązany :D