[C] Wartość max, min oraz suma losowych elementów tablicy

[Assassin-Girl]

Cześć. Mam do zrobienia program, który znajduje maksymalną oraz minimalną wartość w tablicy wypełnionej losowymi liczbami z wybranego przez nas zakresu i sumuje ich wartości. Oto mój kod:

Kod: Zaznacz cały

#include <stdio.h>
#include <time.h>
#include <stdlib.h>
#define size 10

int losuj(int n)
{
	int i;
	static int raz=1;
	if (raz)		
	{				
		srand(time(0));
		raz=0;
	}
	i = rand()%n;
	return i;
}

int main()
{
	int i, n, min, max, suma=0, t[size];
	printf("Podaj zakres losowych liczb:	");
	scanf("%d",&n);
	printf("
");
	for(i=0;i<size;i++)
	{
		t[i]=losuj(n);
		printf("%d ",losuj(n));
		suma +=t[i];
	}
	min=max=t[0];
	for(i=0;i<size;i++)
	{	
		if(t[i]>max) max=t[i];
		if(t[i]<min) min=t[i];
	}
	printf("
SUMA: %d
MIN: %d
MAX: %d",suma,min,max);
	return 0;
}

Nie mam pojęcia, dlaczego nie chce działać. Dla nielosowych elementów (od 1 do 100) działa jak złoto.

Za wszelkie wskazówki z góry dzięki.

[Dasio11]

Kod: Zaznacz cały

...
	for(i=0;i<size;i++)
	{
		t[i]=losuj(n);
		printf("%d ",losuj(n));
		suma +=t[i];
	}
...

Wszystko jest dobrze, tylko wyświetlane są nowe losowe liczby, a nie elementy tablicy t. Wystarczy zamienić:

printf("%d ",losuj(n)); \(\displaystyle{ \to}\) printf("%d ", t[i]);.

[Assassin-Girl]

I wszystko jasne.

Dzięki wielkie.

[Althorion]

Ja się jeszcze przyczepię do funkcji losuj:

Kod: Zaznacz cały

static int raz=1;
if (raz)      
{            
   srand(time(0));
   raz=0;
}

Że co, że jak, że po co?

[Assassin-Girl]

Dla mnie ta funkcja też jest tak potrzebna jak rybie ręcznik, ale dodam na swoją obronę, że jest wymysłem prowadzącej, nie moim.

[Ser Cubus]

zamień

Kod: Zaznacz cały

#define size 10

na

Kod: Zaznacz cały

const size 10

stosowanie define nie jest błędem, ale powinno się tego unikać

[Althorion]

Dla mnie ta funkcja też jest tak potrzebna jak rybie ręcznik, ale dodam na swoją obronę, że jest wymysłem prowadzącej, nie moim.

OK. Przyczepiłem się dlatego, że choć wydzielanie małych funkcji z programu jest ogólnie rzeczą dobrą, to w tym przypadku jest to nadmiarowe: funkcja losuj(int n) ma tę samą funkcjonalność, co rand() % n po jednorazowej inicjalizacji RNG. Można uważać, że Twoje rozwiązanie ma przewagę w postaci połączenia tej inicjalizacji z jej wykorzystaniem, ale uważam że jego wady przeważają nad tą zaletą: funkcja ta ma efekt uboczny (właśnie tę inicjalizację), jest też wg mnie nadmiernie skomplikowana (większość jej ciała stanowi jednorazowa instrukcja).