Arrays und If-Abfrage

[Real2k]

Hallo, ich hab folgende variablen:
runde, gesamt1, gesamt2, gesamt3, gesamt4, aktspieler

und folgende funktion:

PHP-Code:

function save() {
    playerarray = new Array("1", "2", "3", "4");
    for (i=0; i<playerarray.length; i++) {
        if (_root.aktspieler == [playerarray[i]]) {
            _root["gesamt"+playerarray[i]] = _root["gesamt"+playerarray[i]]+_root.runde;
        }
        _root.runde = 0;
    }
    naechster();
} 


Was passieren soll, wenn funktion save ausgeführt wird:
Wenn spieler 1 an der reihe ist, also

PHP-Code:

_root.aktspieler=1 


dann soll

PHP-Code:

_root.gesamt1=_root.gesamt1+_root.runde 


passieren.
Wenn spieler 2 an der reihe ist, also

PHP-Code:

_root.aktspieler=2 


dann soll

PHP-Code:

_root.gesamt2=_root.gesamt2+_root.runde 


passieren.
und so weiter.
aber es tut sich absolut nix
ich nehme an es hat mit dem Array zu tun?
Bin für jede Hilfe dankbar!

gruß
real2k

[hgseib]

== [playerarray[i]]

warum packst du das in eckige klammern?
mit eckigen klammern erstellt man ein array.
du testest, ob _root.aktspieler ein array ist.

und deine for-schleife macht wenig sinn!?

for (i=0; i<playerarray.length; i++) { 
  if (_root.aktspieler == [playerarray[i]]) { 
    _root["gesamt"+playerarray[i]] = _root["gesamt"+playerarray[i]]+_root.runde; 
  } 
  _root.runde = 0; 
}

kannst du ersetzen mit:
_root["gesamt"+ _root.aktspieler] += _root.runde;
_root.runde = 0;

[Real2k]

hmm joa. nun geht es, aber nur für spieler 1
das verwirrt ja erstrecht..

edit: die variable "runde" zählt für alle spieler einen bestimmten erreichten wert für diese eine runde aus. also das ist keine variable die immer um +1 nach oben geht.

[hgseib]

playerarray = new Array("1", "2", "3", "4");

anführungszeichen ist text und keine zahl.

edit: ist mir persönlich wurscht. du addierst "runde" nur zu einem spieler, zum aktspieler.

[Real2k]

hier mal was ich machen will:
funktioniert einwandfrei, ist aber halt viel zu lange.

PHP-Code:

function save() {
    if(_root.aktspieler==1){
        _root.gesamt1=_root.gesamt1+_root.runde;
    }
    if(_root.aktspieler==2){
        _root.gesamt2=_root.gesamt2+_root.runde;
    }
    if(_root.aktspieler==3){
        _root.gesamt3=_root.gesamt3+_root.runde;
    }
    if(_root.aktspieler==4){
        _root.gesamt4=_root.gesamt4+_root.runde;
    }
    naechster();
} 


[hgseib]

du hast eine for-schleife für alle
und mit deinem if-test filterst du einen einzigen davon heraus.

das ist wiedersprüchlich!

entweder es soll für alle sein, dann lass deinen if-test weg
oder es ist nur für einen, dann lass deine for-schleife weg

function save() {
  for (i=0; i<playerarray.length; i++) { 
    _root["gesamt"+i] += _root.runde; 
  }
  _root.runde = 0; 
  naechster(); 
}

[Real2k]

PHP-Code:

function save() {
  for (i=0; i<5; i++) { 
    _root["gesamt"+i] += _root.runde; 
  }
  _root.runde = 0; 
  naechster(); 
} 


dann doch gleich so, oder?

[hgseib]

jain.
absolute werte sind immer schlecht!
zum einen, was bedeutet die 5 und dann, was ist wenn sich die 5 mal ändert? (im quellcode kommt die 5 vielleicht hundert mal vor, welche davon wären dann zu ändern?)


besser wäre z.b.:
spieleranzahl=5;
bzw. ginge auch:
spieleranzahl=playerarray.length;
...
for (i=0; i< spieleranzahl; i++) {

nach der bezeichnung spieleranzahl kann man gut suchen, jeder versteht sofort um was es geht und man kanns einfach ändern.


ach so:
und programmierer beginnen immer mit 0 zu zählen ;-)
also
runde, gesamt0, gesamt1, gesamt2, gesamt3, aktspieler
weil man so die laufvariablen direkt als indexzähler benützen kann.