Új hozzászólás Aktív témák

• #5943 Sk8erPeter nagyúr PumpkinSeed #5929

  Új Válasz 2016-02-29 10:52:02 #5943

  Új hozzászólás

  Összes hozzászólása itt Válaszok az összes hozzászólására itt Válaszok erre a hozzászólásra

  Privát üzenet küldése

  

  Sk8erPeter

  nagyúr

  válasz PumpkinSeed #5929 üzenetére

  Ez így elég csúnya, picit zsongott az agyam a kód olvasása közben.
  Itt ráadásul a for...in ciklusnak nincs is haszna, sőt.
  A getStartTime metódusnévből ráadásul nem érthető, hogy most igazából a szabad órák kezdőidejét szeretnéd lekérni.
  Érdemes egyébként néha a sok-sok indexelés több helyen történő használata helyett inkább a ciklusmag elején eltárolni változóba az aktuális értéket, vagy akkor belül is for...in-t használni.
  És ezenkívül szebb lenne, ha objektumként passzolnád át tömb helyett, hogy mondjuk egy startTime és nextTime lekérdezhető legyen így attribútumnév szerint (és nem kellene agyonindexelgetni a tömböket).

  Számomra ez jóval olvashatóbb, persze még ezen is lehetne szépíteni, most 5 percből ennyire futotta:
  https://jsfiddle.net/d1jntk9a/1/

  Szerk.: ja, most nézem, a freehours-nál is asszem többszörösen egymásba ágyazott tömböt szerettél volna, ennek a további indexelése lemaradt, mindegy, a lényeg végül is érthető.

Új hozzászólás Aktív témák