Ocena strony [To do list]

Question

Witam wszystkich.

Proszę was o ocenę mojego ostatniego projektu.

Link do repozytorium: https://github.com/Danroxx/To-do-list

Link do strony: https://danroxx.github.io/To-do-list/

Za rady jak i poświęcony czas bardzo dziękuje :)

Pozdrawia

Comments

Ja na twoim miejscu wyczyściłbym global scope'a obejmując cały ten kod w IIFE. Dodatkowo fajnie jakby lista się zapisywała w localstorage żeby była tam po odświeżeniu stronki.

Answer 1

Według mnie powinieneś zrobić walidację aby nie można było dodawać pustych zadań

Answer 2

Tak jak kolega @BT101 zauważył, obecnie twój kod jest narażony wycieki informacji do global scope. Zainteresuj się tutaj modułami, pomogą one rozwiązać Ci ten problem. Inna sprawa: po co zwracasz coś w funkcjach, skoro nic potem nie robisz z tymi wartościami? Na koniec mam dla Ciebie radę. Obecnie twój program jest napisany bardzo funkcyjnie. W ramach treningu możesz przepisać go na obiektowo, wykorzystując mvc lub programowanie reaktywne (np. RxJS). Myślę, że możesz sporo z tego wyciągnąć.

Comments

Wielkie dzięki za porady. Ostatnio właśnie zainteresowałem się programowaniem funkcyjnym i dlatego wykorzystywałem funkcje. Wiem, że dla mojego skryptu daleko do programowania funkcyjnego, ale chciałem spróbować. Według Ciebie co jest bardzie korzystne pisanie programu funkcyjnie, obiektowo czy reaktywnie ?

---

Nie ma jedynego słusznego sposobu. Staraj się dobierać sposób myślenia pod konkretne problemy.

---

Nie jest napisany funkcyjne :P

---

@zgrybus  Racja, ale nie przychodziło mi lepsze słowo do głowy. Może bardziej dokładne jest z użyciem funkcji albo pseudo-funkcyjne? A co ty proponujesz?

---

Chyba programowanie proceduralne?

---

Dobra nie było tematu. Zapomniałem, że takie coś istnieje.

Answer 3

w node js do napisania w 3 dni :D ... na tej podstawie słabo, ale  przynajmniej się rozwijasz :)