Discussion :

[Battosaiii]

Tout d'abord randint gener uniformement une valeur entiere dans l'intervalle 0 8619 qui marche et qui fait partie d'une librairie que j'ai prise.
J'aimerais creer dans le tableau tab[1200] des valeurs uniques. Pour cela j'ai creer une fonction recursive unicity pour verifier l'unicite de chaque valeur. Malheureusement Je sais plus loin dans le code que je n'obtiens pas 1200 valeurs unique dans le tableau. Comment faire pour y remedier ?

Code :

Sélectionner tout - Visualiser dans une fenêtre à part

1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
17
18
19
20
21
22
23
24
25
26
27
28
29
30
31
void Unicity(int tab[1200],int i)
{
	int j,x;
	x=randint(0,8619);
	for (j=0;j<i;j++)
	{
		if(x==tab[j]) 
		{
			Unicity(tab,i);
 
		}
 
 
	}
	tab[i]=x;
 
}
 
 
 
void GenerateVoiceRandomly(int tab[1200])
{
int i;
	for(i=0;i<1200;i++)
		{
 
 
		Unicity(tab,i);
 
		}
}

[fearyourself]

Ton algorithme est faux. Ce n'est pas vraiment une question de C.... En plus, faire une récursive comme tu le fais n'est pas une bonne idée...

L'algorithme correct serait plutôt:

Code :

Sélectionner tout - Visualiser dans une fenêtre à part

1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
Pour i allant de 0 à 1199
   pasencore = vrai 
 
   Tant que pasencore
       pasencore = faux
       irand = randint(0,8619);
       Pour j allant de 0 à i - 1
          Si irand==tab[j]
              pasencore = vrai
          FinSi
       FinPour
    FinTantQue
 
  tab[i] = irand;
FinPour

Avec les optimisations possibles biensûr... Remarque bien les différences avec ce que tu fais...

[Battosaiii]

En appliquant ta solution le programme reste a l'infini dans la boucle pour i=828. Je suppose que ca vient de randint mais si quealu'un a des idees/solution en regardant le programme ...

Code :

Sélectionner tout - Visualiser dans une fenêtre à part

1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
17
18
19
20
21
22
23
24
25
26
27
void GenerateVoiceRandomly(int tab[1200])
{
	int i,irand,j;
BOOL not_yet;
for(i=0;i<1200;i++)
	{
	not_yet=TRUE;
	while(not_yet)
		{
		not_yet=FALSE;
		irand=randint(0,8619);
		printf("pour i=%d irand=%d\n",i,irand);
 
			for(j=0;j<i;j++)
				{
				if(tab[i]==tab[j])
					{
					not_yet=TRUE;
					}
 
				}
			}
 
 
	tab[i] = irand;
 
	}

[fearyourself]

Dans la boucle interne, la condition du Si était fausse, désolé... Petite erreur de placement du tab[i]=irand..

Je l'ai corrigé dans le post de base...

Jc

[Battosaiii]

yes ca marche fearyourself thanks !