Discussion :

[zephyroth777]

Bonjour @ tous !

Je ne savais pas trop où poster vu qu'il y a pas mal de rubriques!

Voilà, j'ai un soucis :

J'aimerais remplir un tableau de 50 cases de manière aléatoire avec des nombres entiers relatifs aléatoires !

Voici le code qui me pose problème :

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
 
for(i=0; i<50; i++)
      //Génération d'un indice aléatoire compris entre 0 et 50
	tmp = (int)(Math.random()*50);
      //Si la case d'indice généré plus tôt est vide
	if(tab[tmp] == 0)
	{
      //On la remplit avec un entier positif aléatoire
		tab[tmp] = (int)(Math.random()*100);
	}
      //Sinon le nombre d'itérations de la boucle augmente de 1
	else
	{
		i--;
	}
      // De meme pour les entiers négatifs
	tmp = (int)(Math.random()*50);
	if(tab[tmp] == 0)
	{
		tab[tmp] = (int)(Math.random()*-100);
	}
	else
	{
		i--;
	}

Problème ce code fonctionne mal, je pense que je rate une erreur grosse comme une maison mais là je sèche ...
Un peu d'aide serait la bienvenue !

[vahid]

Salut
Essai ceci pour générer un entier entre 0 et 50 inclus

Code :

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

1
2
3
 
Random r = new Random(Math.random()*10000);
int number = r.nextInt(51);

[zephyroth777]

Je ne veux pas que 50 soit inclus puisque la derniere case du tableau a l'indice 49.

Le code que j'ai mis ne remplit pas toutes les cases du tableau et je ne comprends pas pourquoi !

[vahid]

Peut etre parce que Math.random()*100 ca peut donner 0 si tu le converti en int donc a ce moment la t as l impression que ton tableau ne s'est pas remplit alors qu'il a été remplit avec 0.

De plus, pour remplir ta case [49], ton Math.random doit tomber sur 0,98 ou plus, ce qui est une probabilité assez rare. Tu devrais vraiment utiliser l'objet random...

[zephyroth777]

Bon j'ai fais les modifications :

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
 
int indice=0;
int i=0;
Random r = new Random();
for(i=0; i<50; i++)
	//Génération d'un indice aléatoire compris entre 0 et 50
	indice = r.nextInt(50);
	//Si la case d'indice généré plus tôt est vide
	if(tab[indice] == 0)
	{
		//On la remplit avec un entier positif aléatoire
		tab[indice] = (int)(Math.random()*72);
	}
	//Sinon le nombre d'itérations de la boucle augmente de 1
	else
	{
		i--;
	}
	// De meme pour les entiers négatifs
	indice = r.nextInt(50);
	if(tab[indice] == 0)
	{
		tab[indice] = (int)(Math.random()*-72);
	}
	else
	{
		i--;
	}

Aucun changement, seules 2 ou 3 cases sont différentes de 0 ...

[®om]

Je ne comprends pas ton code... Tu es dans une boucle en i++ et tu fais un i-- dans le corps... o_O

En gros, tu veux un tableau de 50 cases rempli avec des chiffres de -50 à 49?

Si oui:

Code :

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

1
2
3
4
5
final Random random = new Random();
int tab[] = new int[50];
for (int i = 0; i < tab.length; i++) {
	tab[i] = random.nextInt(100) - 50;
}

[zephyroth777]

Pour être plus clair, je souhaite que chaque case soit choisie au hasard (entre 0 et 49) pour y mettre des entiers choisis eux-aussi au hasard (entre -99 et 99) !

Je sais j'me complique la vie mais c'est juste un jeu !

[louac]

Salut, dans tes codes, j'ai l'impression que ta boucle "for" tire successivement 50 int dans le vent avant d'affectuer un random à une seule valeur... ne te manquerait-il pas une bonne paire d'accolades ?

[vahid]

Code :

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

tab[indice] = (int)(Math.random()*72);

Pas bien

Ca c mieux :

Code :

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

tab[indice] = r.nextInt(199)-99;

Ce qui donne avec le code de rom (pourquoi remplir les cases dans le desordre ? )

Code :

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

1
2
3
4
5
6
 
final Random random = new Random(); 
int tab[] = new int[50]; 
for (int i = 0; i < tab.length; i++) {  
   tab[i] = random.nextInt(199) - 99; 
}

[nono_31]

Salut,
En effet cela ne présentes pas d'intérêt de remplir les cases en désordre.
De plus tu vérifies qu'une case est vide si sa valeur est égale à 0 or tu génères des nombres entre -99 et 99 donc la case n'est pas forcément vide.

Si tu veux absolument remplir dans un ordre aléatoire ton tableau, tu n'as qu'à créer une liste avec les 50 indices en en sélectionner un au hasard à chaque itération puis supprimer l'indice sélectionné de la liste.

Code :

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

1
2
3
4
5
6
7
8
9
 
final Random rnd=new Random();
//construction tableau indices
ArrayList<Integer> indices=new ArrayList<Integer>();
for (int i = 0; i < 50; i++) {indices.add(i);}
//sélection aléatoire des indices
while(!indices.isEmpty()) {
	System.out.println(indices.remove(rnd.nextInt(indices.size())));
}