To-Do List - pierwszy program HTML/CSS/JS - prośba o ocenę kodu

Question

Witam,
Prosiłbym o ocenę mojego pierwszego programu w HTML/CSS/JS. Jest to prosty To-Do-List.

Kod poniżej:
https://github.com/szymcio32/to-do-list

Będę wdzięczny za wszelkie uwagi i sugestie.

Pozdrawiam
Szymon

Comments

Rekord liczba antywzorców per najkrótszy kod w kategorii Frontend.

Niestety, ale nie ma co oceniać, bo do przerobienia jest wszystko.

---

Wybacz,ale pierwsze zdanie nic mi nie mówi.

Jak pisałem wyżej, jest to mój pierwszy program i zdaję sobie sprawę, że jest tam mnóstwo rzeczy do poprawienia.

Answer 1

Do rzeczy:

1. onclick="add()"

   Po co to? Czemu raz używasz archaicznego onclick a raz addEventListener?

2. <ol id="list">
   </ol>

   To również możesz wygenerować w JS

3. Nie styluj po id

4. Rozdziel dużą funkcję add() na mniejsze np.: addElement(), addRemoveButton() itd.

5. element.setAttribute("id", "element"); 

 Nie trzeba tak, można po prostu

element.id = `element` 

Źródło

 

Comments

Dziękuje bardzo.

Poprawiłem kod według powyższych wskazówek.