Génération d'entiers relatifs

Version imprimable

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:

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 !

09/08/2007, 18h23
vahid
Salut
Essai ceci pour générer un entier entre 0 et 50 inclus
Code:

1 2 3 Random r = new Random(Math.random()*10000); int number = r.nextInt(51);
09/08/2007, 18h30
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 !
09/08/2007, 18h36
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...

Bon j'ai fais les modifications :

Code:

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 ...

09/08/2007, 19h21
®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:

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; }
09/08/2007, 19h30
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 !
09/08/2007, 19h50
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 ?
09/08/2007, 20h09
vahid
Code:

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

Pas bien

Ca c mieux :

Code:

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

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

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; }

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:

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()))); }