Super je ideja a bogami i realizacija, bas izgleda profi! :)
Evo par konstruktivnih kritika u vezi koda:
- Lepo je negovati nas jezik ali gledaj da sve sto radis i pises u kodu bude na engleskom. Nikad ne znas kad ces na nekom projektu raditi sa ljudima koji ne znaju srpski, a engleski je defakto postao esperanto IT sveta. Sve rezervisane reci su takodje na engleskom pa ovaj mix jezika deluje ruzno i neprofesionalno.
- NIKAD nemoj koristiti globalne promenljive (razloga imas na netu)
- Bilo bi lepo da grupises fajlove po folderima (js/, css/, img/,...)
- Koristi neki lint kojim ces sebi zadati set pravila i koji ce ti ukazati na nepravilnosti i odstupanja u kodu od tih pravila. Preporuka
https://eslint.org/
- Vidim da se svi kanali nalaze u "kanali[0]" promenljivoj. Nije mi jasno zasto imas "kanali" niz koji ima samo 1 elemenat (koji je niz svih YT klipova). Znaci imas niz koji ima samo 1 element - niz koji onda ima YT klipova u svojim elementima. Ne vidim zasto onda nisi jednostavno stavio sve YT klipove u niz "kanali".
- Bilo bi dobro da se parametri za podesavanje (YT klipovi recimo) cuvaju u posebnom fajlu, bilo bi cak i bolje da to bude JSON. Razlog je da i ne-programeri mogu da lako izmene neka podesavanja i uspesno pokrenu program. Drugi razlog je zbog toga sto je mnogo lakse drzati podesavanja na jednom mestu. Treci razlog - JSON jako puno ljudi zna, JS ne bas...
Ideje za poboljsanje:
- Uz pomoc localStorage-a bi mogao da sacuvas trenutna podesavanja koje korisnik gleda (jacinu zvuka, izabran kanal,...). Tako ce korisnik koji se vrati na sajt imati sve isto kao kad je otisao sa njega.
- Pogledaj ES6 sintaksu
[Ovu poruku je menjao brux002 dana 26.02.2018. u 14:53 GMT+1]
Violence is the last refuge of the incompetent.