Tablica c++, blad przy podaniu wartosci max i min

Question

Mam program, który przy podaniu 0 kończy wpisywanie liczb do tablicy albo po jej zapełnieniu. Podaje max i min wartosci. Przy podaniu wiecej liczb to program, natomiast gdy podam z 3 liczby i podam 0, wykazuje dziwne wartosci w min i max

#include <iostream>
using namespace std;
const int MAX_N=10;
int main()
{
	int tab[MAX_N];
	int min,max;
	double srednia=0;
	
	for(int i=0;i<10;i++)
	{
		cin>>tab[i];
		if(tab[i]==0 ){
	    break;
			}
	}
	
	min=tab[0];
	for(int i=0;i<10;i++)
	{
		if(min>tab[i])
		min=tab[i];
		
	max=tab[0];	
	}
	for(int i=0;i<MAX_N;i++)
	{
		if(max<tab[i])
		max=tab[i];
	}
	
	cout<<min<<endl;
	cout<<max<<endl;
	
	
}

 

Answer 1

 Przy podaniu wiecej liczb to program

?

 Przy podaniu wiecej liczb to program, natomiast gdy podam z 3 liczby i podam 0, wykazuje dziwne wartosci w min i max

Nic w tym dziwnego. Na sztywno wymagasz, by tablica przechowywała 10 wartości, więc tyle też ich jest. Sęk w tym, że jeśli ich nie inicjujesz jakimiś wartościami na samym początku, to są one zupełnie losowe. Stąd jak przerwiesz wpisywanie w jakimś momencie, to najzwyczajniej w świecie w pozostałej części tablicy masz losowe wartości. A tu już nigdy nie wiesz, czy na danym miejscu masz 0, milion czy coś ujemnego. Cała magia. 

Ja polecam zamiast babrać się tablicą, to zająć tworem o nazwie vector. 

Comments

Przy podaniu wiecej liczb to program dziala,

Da się wpisując 0 zatrzymać wpisywanie liczb do tablicy z wejścia?

---

Pewnie, że się da. Na przykład zapamiętuj indeks miejsca, w które wpisałeś 0. I potem wyświetlaj wszystkie liczby do tego indeksu.

---

Zainicjować indeks=i, gdy będzie równy ==0, wtedy break?

---

Możesz ten indeks zadeklarować przed pętlą i używać jako licznika w pętli. W ten sposób zawsze otrzymasz poprawny indeks - niezależnie, czy pętla była przerwana (czyli wpisano 0), czy też nie. Gdyby tak po prostu przypisać go tylko wtedy, gdy wpisujesz 0, to ciężko byłoby iterować po tablicy od 0 do indeksu, bo zmienna trzymająca indeks byłaby 'podmieniana' tylko wtedy, gdy wpisano 0.

---

Trzy pętle są niepotrzebne - od razu w pierwszej pętli po wczytaniu liczby można sprawdzić, czy jest ona większa od dotyszczasowej największej, mniejsza od najmniejszej.

I problem liczenia tych liczba zniknie, tablica też okaże się niepotrzebna, bo w ogóle nie ma potrzeby pamiętania samych liczb, tylko wartości min i max.

---

Tu np. można użyć do...while

    int min=2147483647, max=0, number=0;
    
    do {
        cin >> number;
        if (min > number) min = number;
        if (max < number) max = number;
    } while (number != 0);
    
    cout << min << " : " << max;