code kürzen

[thorben.schmitt]

hi, wie kürze ich am besten diese if-abfrage?
dass nicht die ganzen oder-unterscheidungen drinn sind? da noch mind. 6 unterscheidungen dazukommen...
(array?!?!)
oder gehts nimmer kürzer?

tilePos == this.num-1 || tilePos == this.num-(numRows+1) || tilePos == this.num+(numRows-1) ? this._alpha=alpha : this._alpha=100; 

[Scrawer]

naja wenn das alles bzw eines davon Bedingung sein muss... kannst es nicht kürzen wie ich.....

Flash arbeitet die Abfrage ja von links nach rechts ab... sobald eines true ist...
bricht er ab.. und der rest kommt gar nicht mehr an die reihe in deinem fall da es ja ein 'or' ist
daher solltest es halt sinnvoll sortieren....
sprich jenes das am öftesten eintritt ganz nach links setzen.....

wegen Array.... sicher du kannst das ganze in eine Funktion auslagern... wo du per array argument die werte übergibst...
und in einer for schleife die Bedingungen so lange prüfst bis eine eintritt und dann ein break u. return true setzt...
jedoch macht das den code nur übersichtlicher... aber sicher nicht schneller... eher langsamer

[sebastian]

kA wie man das vereinfacht, hab auch nichts passendes in der referenz gefunden

aber schonmal so viel:

this._alpha = (tilePos == this.num-1 || tilePos == this.num-(numRows+1) || tilePos == this.num+(numRows-1)) ? alpha : 100; 

Sebastian

[thorben.schmitt]

ok, danke euch, werd wohl alles in eine zeile schreiben, da die preformance sowie so schon nichtmehr die beste ist

[bokel]

Du könntest num rauskürzen:

var tp = tilePos - this.num;
this._alpha = (tp == - 1 || tp == - (numRows + 1) || tp == (numRows - 1)) ? alpha : 100; 

Alles in eine Zeile zu schreiben bringt übrigens performancemäßig nichts. Das wäre ein bißchen zu einfach
Du kannst die Bedingungen nach ihrer Häufigkeit sortieren, also die Bedingung, die am meisten vorkommt zuerst abfragen, das könnte etwas Performance bringen.

mfg r.