search

Oraveczgergo / Webprog

0 stars 0 forks source link

DRY, KISS #18

Closed netdjw closed 3 years ago

netdjw commented 3 years ago

DRY, KISS desing principle oktatóanyag:

https://dzone.com/articles/software-design-principles-dry-and-kiss

Ez a két kódrész gyakorlatilag ugyanazt csinálja, egy funkcióba ki lehet szervezni:

https://github.com/Oraveczgergo/Webprog/blob/cafc2d9f6ab8740e7703cee28d483db3c6ebda4b/javascript.js#L42-L63

https://github.com/Oraveczgergo/Webprog/blob/cafc2d9f6ab8740e7703cee28d483db3c6ebda4b/javascript.js#L92-L113

Másrészt én a switch -et megszüntetném, mert túl sok kódot jelent. A színeket egy tömbbe, ha berakod, akkor a szám a tömb indexét fogja jelenteni. ....backgroundColor = colors[i]; valahogy így.

Oraveczgergo commented 3 years ago

A switch blokkok le lettek cserélve egyszerűbb kódra.