największa liczba z tablicy

Question

Moje zadanie to wprowadzić ręcznie liczby do tablicy, a potem za pomocą funkcji znaleźć największą liczbę z tablicy. Nie działa do końca poprawnie bo pokazuje cały czas pierwszy element tablicy, a program wyświetla w dodatku komunikat że przy wywoływaniu funkcji używam zmiennej max, która nie była inicjowana (a przecież jest w funkcji).

Proszę o pomoc

#include <iostream>

using namespace std;

float max_value(float* tab, unsigned int ilosc)
{
	float max;
	float x = tab[0];
	for (int l=0; l<ilosc; l++)
	{
		if(x>max)
			max=tab[l];
		else 
		    max=x;
	}
	return max;
}

int main()
{
	
unsigned int ilosc;

cout<<"Podaj ile liczb chcesz wprowadzic: ";
cin>>ilosc;
cout<<endl;

float* tab = new float[ilosc];
int i;

for (i=0; i<ilosc; i++)
{
	cout<<"Podaj "<<i+1<<" liczbe: ";
	cin>>tab[i];
}

cout<<"Najwieksza liczba sposrod podanych to: "<<max_value(tab,ilosc)<<endl;

delete [] tab; 
  
  system ("PAUSE");
  return 0;
}

 

Comments

float max_value(float* tab, unsigned int ilosc)
{
    float max = tab[0];
    for (unsigned i=0; i<ilosc; i++)
    {
        if(tab[i]>max)
            max=tab[i];
    }
    return max;
}

 

Answer 1

Przeanalizuj dokładnie co robi funkcja max_value. Najlepiej rozrysuj to sobie na kartce dla trzech liczb: 1,2,3.

Zmienna max nie jest zainicjowana. Może tam trafić zupełnie losowa wartość. Używasz tej zmiennej w porównaniu.

float max_value(float* tab, unsigned int ilosc)
{
    float max;
    float x = tab[0]; // po co ci ta zmienna?
    for (int l=0; l<ilosc; l++)
    {
        if(x>max)
            max=tab[l];
        else // dlaczego?
            max=x;
    }
    return max;
}

 

Comments

Właśnie chyba mi się udało to zamienić. Zobaczyłem jak wielki błąd napisałem. Takie coś chyba działa.

float max_value(float* tab, unsigned int ilosc)
{
	float max=0;
	for (int l=0; l<ilosc; l++)
	{
		float x=tab[l];
		if(x>max)
			max=tab[l];
	}
	return max;
}

 

---

Przetestuj dla liczb -1, -2, -3

---

Też to zobaczyłem że dla ujemnych będzie kaplica. Ale to chyba rozwiąże się w ten sposób

float max_value(float* tab, unsigned int ilosc)
{
    float max=tab[0];
    for (int l=0; l<ilosc; l++)
    {
        float x=tab[l];
        if(x>max)
            max=tab[l];
    }
    return max;
}

 

---

Prawda?

---

Brawo.
Jedno małe ale: zmienna x jest tak naprawdę niepotrzebna. Jeżeli chcesz jej użyć dla czytelności to użyj jej też w linii nr 8.