Probleme beim Erzeugen eines Arrays der keine doppelten Elemente haben soll

[mannimedia]

Hallo zusammen,

ich möchte einen Array erstellen, der 3 Zahlen enthält. Jede Zahl darf aber nur einmal vorkommen, wobei die erste Zahl feststeht und die beiden anderen per Zufall generiert werden. Bei mir klappt es meistens, aber es kann vorkommen, dass die 2. und 3, Zahl gleich sind.

Hier mal mein Skript. Wahrscheinlich geht es viel schöner, ich bin also über jede Anregung dankbar.

PHP-Code:

maxSpalt = 7;
drei = new Array();
drei.push(maxSpalt);    
erzeugeArray();


function erzeugeArray() {
    zahl = Math.round((Math.random()*9)+6);
    if(drei.length < 3){
        for(i=0;i<=drei.length;i++){
            if(zahl == drei[i]) {
                erzeugeArray();
                break;
            } else {
                drei.push(zahl);
            }
        }
    } else {
        trace(drei[0]);
        trace(drei[1]);
        trace(drei[2]);
    } 
} 


Danke und Grüße
Manni

[Der Frager]

Guckst du:

PHP-Code:

maxSpalt = 7;
drei = new Array();
drei.push(maxSpalt);
function erzeugeArray() {
    while (drei.length != 3) {
        zahl != drei[0] && zahl != drei[1] ? drei.push(zahl) : zahl=Math.round((Math.random()*9)+6);
    }
    trace(drei[0]);
    trace(drei[1]);
    trace(drei[2]);
}
erzeugeArray(); 


[mannimedia]

Klappt einwandfrei. Vielen Dank.

Nur was genau macht diese Zeile?

PHP-Code:

zahl != drei[0] && zahl != drei[1] ? drei.push(zahl) : zahl=Math.round((Math.random()*9)+6); 


Verstehe die Schreibweise nicht ganz.

[readme.txt]

Das ist eine vereinfachte Schreibweise für eine if-Abfrage.

[Xeef]

nicht ganz aber fast

?: conditional operator
expression1 ? expression2 : expression3

Instructs Flash to evaluate expression1, and if the value of expression1 is true, it returns the value of expression2; otherwise it returns the value of expression3.

grosster unterschied ist das du ein ruckgabe wert hast
und das es immer ein "if else" ist



und der beispiel ist ein beispiel fur unuberschaubaren code sonders gleichen !

[Xeef]

neben bei solte der code nicht mall functioniren !

Wen
nun Wen MM "&&" so umgestezt hete wie in der hilfe beschriben !

&& logical AND operator
expression1 && expression2

Performs a Boolean operation on the values of both expressions. If expression1 and expression2 are both true, then true is returned; otherwise, false is returned.

[mannimedia]

Hallo.

dank schonmal für die Antworten. Bedeutet die obere Schreibweise jetzt das

PHP-Code:

while (drei.length != 3) {
        if (zahl != drei[0] && zahl != drei[1]){
            drei.push(zahl);
        } else {
            zahl=Math.round((Math.random()*9)+6);
        }
} 


Oder wie würde man das in der ausführlichen Schreibweise schreiben?

[Der Frager]

Ja. So sähe es dann wohl mit if/else aus.

Zitat von Xeef

...und der beispiel ist ein beispiel fur unuberschaubaren code sonders gleichen !

Pffft...

[mannimedia]

Danke.

Muss mich erstmal an die Schreibweise gewöhnen. Bei der einfachereren blicke ich mehr durch. Ist wahrscheilich ein Gewohnheitsding.

LG
Manni

[Der Frager]

"Eigentlich" sollte man das auch so machen, wie du es jetzt hast.

[mannimedia]

Das hört man gerne

Merci fürs helfen und erklären.

[hazy fantazy]

function erzeugeZufallsliste( laenge, startWert )
{
     var ergebnis = new Array();
     ergebnis.push( startWert );
 
     while( ergebnis.length < laenge ){
         var zufallszahl = Math.floor( Math.random() * 100);
         if( ! finde( ergebnis, zufallszahl ))
         {
               ergebnis.push( zufallszahl );
         }
     }
     return ergebnis;
}
 
function finde( liste, wert )
{
    for( var i in liste) if( liste[i] == wert) return true;
    return false;    
}
 
var ergebnis = erzeugeZufallsliste( 3, 7 );
trace( ergebnis ); 

Ungetestet, aber so grob sollte es klappen
mfg. h

[michael]

PHP-Code:

function getThreeRandomsFrom(){
    return arguments.sort(function(){return random(3)-1;}).slice(0,3);
}

threeRandoms = getThreeRandomsFrom(7,8,9,10,11,12,13,14,15);
trace(threeRandoms); 


[Xeef]

woll schon auch was netes posten von wegen

aber denkste !

hazy fantazy's scheint die optimale losung zu sein !

function erzeugeZufallsliste(laenge, startWert) {
    var ergebnis = new Array();
    ergebnis.push(startWert);
    while (ergebnis.length<laenge) {
        var zufallszahl = Math.floor(Math.random()*10000);
        if (!finde(ergebnis, zufallszahl)) {
            ergebnis.push(zufallszahl);
        }
    }
    return ergebnis;
}
function finde(liste, wert) {
    for (var i in liste) {
        if (liste[i] == wert) {
            return true;
        }
    }
    return false;
}
T = getTimer();
var ergebnis = erzeugeZufallsliste(1000, 7);
trace(getTimer()-T);
///
///
T = getTimer();
A = [];
for (a=0; a<10000; a++) {
    A[a] = a;
}
trace(getTimer()-T);
T = getTimer();
A.sort(function () {
    return random(3)-1;
}).slice(0, 1000);
//
trace(getTimer()-T); 

auch mit "splice" ist es nicht besser !

alerdings hat hazy fantazy's
hier ein problem (scheint aber immer noch acceptabel zu sein was mich ver wundert !)

100 zahlen random aus 0-101