C++ nie pokazuje funkcji

Question

Cześć, mam problem, po uruchomieniu programu nie pokazuje się wartość minimalna ani wartość maksymalna. Pokazuje jedynie liczby posortowane. Nie mam pojęcia, co zrobić, by pokazywało wszystko.

#include <iostream>
#include <algorithm>

using namespace std;

int wartoscMaksymalna(int tab[], int n)
{
	int max=tab[0];
	for (int i=0; i<n; i++)
	{
		if (max<tab[i])
			max=tab[i];
	}
	return max;
}

int wartoscMinimalna(int tab[], int n)
{
	int min=tab[0];
	for (int i=0; i<n; i++)
	{
		if (min>tab[i])
			min=tab[i];
	}
	return min;
}

void Sortuj(int tab[], int n)
{
	for (int i=0; i<n-1; i++)
	{
		for (int j=0; j<n-1; j++)
		{
			if (tab[j]>tab[j+1])
				swap(tab[j] , tab[j+1]);
		}
	}
}

void PokazStan(int tab[], int n)
{
	for (int i=0; i<n; i++)
	{
		cout<<i+1<<". element: "<<tab[i]<<endl;
	}
}

int main()
{
	int n;
	cout<<"Podaj ilosc liczb: ";
	cin>>n;
	int tab[n];
	for (int i=0; i<n; i++)
	{
		cout<<"Podaj "<<i+1<<" liczbe: ";
		cin>>tab[i];
	}
	cout<<"Wartosc maksymalna wynosi: "<<wartoscMaksymalna(tab, n);
	cout<<"Wartosc minimalna wynosi: "<<wartoscMinimalna(tab, n);
	Sortuj(tab, n);
	PokazStan(tab, n);
	return 0;
}

 

Comments

napisać program samodzielnie, bo jeśli nie potrafisz wypisać tego co chcesz, po spojrzeniu na kod, twierdze że nie jesteś jego autorem.

---

Właśnie nie wiem czemu za pierwszym razem nie zadziałało, ale usunąłem i dodałem jeszcze raz i działa. A program napisałem sam, ale musiało się coś nie załadować

---

@Bartek7630, AFAIK do tworzenia dynamicznych tablic, powinieneś użyć zapisu np.

    int n;
    cout<<"Podaj ilosc liczb: ";
    cin>>n;

    int *tab = new int[n];
   
    ...
 
    delete[] tab;
    return 0;

lub użyć np. std::vector

---

Dzięki za wskazówkę 

Answer 1

Wynika z tego, że wartość maksymalna i minimalna są wyświetlane tylko jako wartości, a nie jako część tablicy posortowanych. Aby wyświetlić wartość maksymalną i minimalną razem z tablicą posortowaną, należy umieścić je wewnątrz funkcji PokazStan.

Przykład:

void PokazStan(int tab[], int n)
{
cout<<"Wartosc maksymalna wynosi: "<<wartoscMaksymalna(tab, n)<<endl;
cout<<"Wartosc minimalna wynosi: "<<wartoscMinimalna(tab, n)<<endl;
for (int i=0; i<n; i++)
{
cout<<i+1<<". element: "<<tab[i]<<endl;
}
}

 

Comments

Po posortowaniu zawartości tablicy wyszukiwanie maksymalnej i minimalnej wartości jest bez sensu - to po prostu pierwsza i ostatnia wartość w tablicy.