Discussion :

[orphen]

Bonjour a tous ,

J'ai coder une mini fonction qui génère un nombre aléatoire et prend en argument le multiplicateur (le nombre que ne doit pas dépasser le random).

Il peut arriver que ma fonction renvoie deux ou trois fois de suite le même chiffre.

Code :

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

1
2
3
4
 
var randomNb = function(nb){
		return Math.floor(Math.random() * (nb + 1));
	};

Pouvez vous m'aider svp?

[orphen]

j'ai tester quelque chose comme ça

Code :

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

1
2
3
4
5
6
7
8
9
10
11
 
var randomNb = function(nb){
		var newNumber=0;
		while(previousRandomNumber == newNumber){
			newNumber=Math.floor(Math.random() * (nb + 1)); 
		}
 
		previousRandomNumber = newNumber;
 
		return newNumber;
	};

previousRandomNumber étant une variable globale initialisée a 0.
Mais cela ne fonctionne pas .

Je commence a sécher.

[SpaceFrog]

je ne pige pas trop le paramètre que tu passes ?

un random est un random.... après tu peux les stockers dans un array et tester si tu as déja retourné le meêm chiffre.

Mais le principe du random c'est que tu peux avoir dexu fois de suite le même retour

sinon tu peux faire un array avec le nombre et tu fais un shuffle avant de faire un pop du coup il ne te retourne qu'une fois chaque element de l'array jusqu'à épuisement du stock ...

[Archimède]

Oui, ok pour le stockage des random dans un tableau déclaré en dehors de la fonction et une comparaison du résultat avec le balayage du tableau... mais pour une petite valeur de nb, tu vas vite épuiser les différentes solutions (vu le floor...) et ta fonction ne te renverra plus rien. et si tu mets un do while, ça va mouliner sans rien sortir.
Curieux comme demande...

[orphen]

Je ne comprend pas trop ce qui est curieux dans le fait d ene pas vouloir avoir deux fois de suite le meme nombre.
exemple 1987586254 et pas 1987558625.

J'ai finalement trouver pourquoi lorsque je stoquait dans une variable global cela ne fonctionnait pas.
En fait la fonction est appeler deux fois au même moment pour gérer deux truc différents du coup ma variable global se met a jour deux fois de suite et peux retomber sur le même nombre.

J'ai ajouter un booleen en parametre qui me permet de prendre en compte ou non la variable global.
Ce n'est surement pas la meilleure solution mais a default d'en avoir une autre pour le moment au moins cela fonctionne.

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
 
        //Global random nb
        var uniqueRandoms = [];
 
	var randomNb = function(nb, checkTwin) {
		// refill the array if needed
		if(checkTwin){
			if (!uniqueRandoms.length) {
				for (var i = 0; i < nb; i++) {
					uniqueRandoms.push(i);
				}
			}
			var index = Math.floor(Math.random() * uniqueRandoms.length);
			var val = uniqueRandoms[index];
 
			// now remove that value from the array
			uniqueRandoms.splice(index, 1);
			return val;
		}
		return Math.floor(Math.random() * (nb+1));
	}

[orphen]

Oui, ok pour le stockage des random dans un tableau déclaré en dehors de la fonction et une comparaison du résultat avec le balayage du tableau... mais pour une petite valeur de nb, tu vas vite épuiser les différentes solutions (vu le floor...) et ta fonction ne te renverra plus rien. et si tu mets un do while, ça va mouliner sans rien sortir.
Curieux comme demande...

Le but de mon random n'est pas de passer dans les nombre qu'une seule fois mais d’éviter qu'il ne se repettent directement genre 13465547.

Orphen

[Archimède]

Ok, je pensais que tu voulais toujours des valeurs différentes sans jamais de répétiton. En fait, tu désirais seulement que deux valeurs successives ne se répètent pas...C'est en effet différent.