Zapełnienie tablicy losowi liczbami

Question

#include <stdio.h>
#include <stdlib.h>
#include <time.h>

int tab[], n, a, b;
void fill2(tab, n, a, b);
int main() {
	printf("Podaj liczbe n: ");
	scanf_s("%d", &n);
	
	
}
void fill2(tab, n, a, b) {
	for (int i = 0; i < n; i++) {
		srand(time(NULL));
		tab[i] = rand() % 19 + 2;
	}
	
}

Chce zapełnić tablice losowymi wartościami z przedziału. Co z tym kodem jest nie tak ? Liczę na pomoc.

Answer 1

void fill2(tab, n, a, b);

Tak nie wygląda deklaracja/definicja funkcji. Zgubiłeś typy.

int tab[], n, a, b;

Tablica nie ma rozmiaru. (i błędnie zrobiłeś zmienne globalnymi)

srand(time(NULL));

To inicjalizuje generator, więc jak za każdym razem go tak samo inicjalizujesz to będzie taka sama pierwsza "liczba losowa". srand() powinno się robić tylko raz, na samym początku programu, np na górze main()a.

Comments

Poprawiłem na coś takiego. Nie wiem dalej jak to poprawić. Dlaczego nie mogę napisać, że int tab[n] gdzie n będzie wpisywane z tablicy ? Czy nie powinienem użyć do tego tablicy dynamicznej ?

#include <stdio.h>
#include <stdlib.h>
#include <time.h>

int fill2(int tab[n], int n, int a, int b);

int main() {
	int tab[n], n, a, b;
	
	srand(time(NULL));
	printf("Podaj liczbe n: ");
	scanf_s("%d", &n);
	
	
}
int fill2(int tab[n],int n,int a,int b) {
	for (int i = 0; i < n; i++) {
		tab[i] = rand() % 19 + 2;
	}
	
}

 

---

Dlaczego nie mogę napisać, że int tab[n] gdzie n będzie wpisywane z tablicy ?

Dlatego, że w momencie tworzenia tablicy n nie istnieje. Dodatkowo podczas definiowania tablicy musisz podać jej rozmiar.

Czy nie powinienem użyć do tego tablicy dynamicznej ?

Dla małych tablic VLA wystarczą.

Nie dawaj w parametrach funkcji int tab[n] tylko int tab[].

---

VLA to zło.. :| Generują tak pogmatwany i obszerny kod, że żal patrzeć.

---

Generalnie tak, ale do takiego prostego zadania prawdopodobnie wystarczy.