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

• PumpkinSeed addikt

  #5944

  #5943 Sk8erPeter

  2016-02-29 11:27:54

  Új Válasz

  Ú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

  

  PumpkinSeed

  addikt

  #5943 Sk8erPeter

  #5943 Sk8erPeter

  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ő.

  Köszönöm, az biztos, hogy van mit javítani a JS tudásomon, és elég erősen rajta is vagyok, amikor időm engedi. A változónevek meg csak ideiglenesek, refaktoráltam is, csak az a jsFiddle-n nem látszódik.

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