Najwieksza wartosc w tablicy nie działa

Question

Witam, zacząłem robic ostatnio proste zadania z programowaniaz tablic i wyszukaniu najmniejszej i najwiekszej wartosci w niej. Jak działa mi znalezienie najmniejszej wartosci, tak nie wiem co robie zle w stosunku do tej najwiekszej:

#include <iostream>
#include <cstdlib>
using namespace std;
int main ()
{
int liczby[100], n;
int mini; int maxi;
mini=liczby[0];
maxi=liczby[0];
cin>>n;
for (int i=0; i<n; i++)
{
    cin>>liczby[i];
    if (liczby[i]>maxi)
        maxi=liczby[i];
    if (liczby[i]<mini)
        mini=liczby[i];

}
cout<<endl;
cout<<mini<<endl;
cout<<maxi<<endl;

    return 0;
}

Z góry dzieki za pomoc

Answer 1

Pierwszy element tablicy jest niezainicjowany. Możesz rozwiązać ten problem na 2 sposoby:

1. Najpierw wczytać dane, a dopiero później szukać największej wartości:

   cin >> n;
   for (int i = 0; i < n; i++)
   {
   	cin >> liczby[i];
   }
   
   int mini = liczby[0];
   int maxi = liczby[0];
   for (int i = 0; i < n; i++)
   {
   	if (liczby[i] > maxi)
   		maxi = liczby[i];
   	if (liczby[i] < mini)
   		mini = liczby[i];
   
   }
   

2. Możesz zainicjować mini i maxi najmniejszą i najmniejszą wartością jaką może przyjąć int:

   #include <iostream>
   #include <limits> // koniecznie dla std::numeric_limits
   
   using namespace std;
   int main()
   {
   	int liczby[100], n;
   	int mini = numeric_limits<int>::max();
   	int maxi = numeric_limits<int>::min();
   	cin >> n;
   	for (int i = 0; i<n; i++)
   	{
   		cin >> liczby[i];
   		if (liczby[i]>maxi)
   			maxi = liczby[i];
   		if (liczby[i]<mini)
   			mini = liczby[i];
   
   	}
   	cout << endl;
   	cout << mini << endl;
   	cout << maxi << endl;
   
   	return 0;
   }
   

Comments

okej, dzieki wielkie :D

Answer 2

mini=liczby[0];
maxi=liczby[0];

Iniciujesz te wartości śmieciami, co oznacza, że w maxi może znaleźć się maksymalna wartosc inta jeszcze przed wczytaniem pierwszej liczby. 

Zainiciuj te zmienne pierwszą wczytaną liczbą, a jeśli bardzo nie chcesz to mini maksymalnym intem a maxi minimalnym intem.

Answer 3

Trochę z innej beczki, proponuję zapoznać się z biblioteką <algorithm> tam masz funkcję:  min_element, max_element lub minmax_element.

Answer 4

Na początku maxi i mini są null, bo tablica nie ma jeszcze liczb, przez co obydwa if'y się nie sprawdzają, albo wywołują error. Musiałbyś linie 8 i 9 dać za 13.

EDIT:

Zamiast zer w indeksach daj zmienną i.

Pozdrawiam.

Comments

Na początku maxi i mini są null, bo tablica nie ma jeszcze liczb

Są losowymi wartościami np 5495831 a nie nullami. Int nie może być nullem.

przez co obydwa if'y się nie sprawdzają, albo wywołują error

 Wywołują się tylko porównują śmieci (losowe wartości)

Zamiast zer w indeksach daj zmienną i.

tą, która jest inicjowana kilka linijek niżej w pętli?

---

Na początku maxi i mini są null

Co to jest null? Niezainicjalizowane zmienne lokalne przyjmują takie wartości jakie były w pamięci na której zostały zaalokowane. 

przez co obydwa if'y się nie sprawdzają, albo wywołują error

Jaki error?

Musiałbyś linie 8 i 9 dać za 13.

 Zamiast zer w indeksach daj zmienną i.

W taki sposób zawsze wynikiem będzie mini==liczby[n-1], maxi==liczby[n-1]

Answer 5

Po pierwsze: w ogóle nie potrzebujesz tablicy.

Po drugie: bez sensu na początku do maxi oraz mini przypisujesz liczby[0] skoro wartości liczb podajesz w pętli. W tym przypadku najlepiej jakbyś na początku do mini oraz maxi przypisał odpowiednio największą oraz najmniejszą wartość reprezentowalną przez int. Tzn: INT_MAX oraz INT_MIN lub std::numeric_limits<int>::max() oraz std::numeric_limits<int>::min().