• Najnowsze pytania
  • Bez odpowiedzi
  • Zadaj pytanie
  • Kategorie
  • Tagi
  • Zdobyte punkty
  • Ekipa ninja
  • IRC
  • FAQ
  • Regulamin
  • Książki warte uwagi

Ocena kodu javascript - todo app

+2 głosów
115 wizyt
pytanie zadane 6 lutego w JavaScript, jQuery, AJAX przez Marchiew Mądrala (6,300 p.)
Cześć,
Stworzyłem w ramach ćwiczeń standardową już stronę todo. Zacząłem od działania strony, a nie wyglądu i na działaniu mi zależy, żeby został oceniony przez kogoś.
Byłbym bardzo wdzięczny za wszelkie uwagi odnoście szybkości, czytelności, łatwości rozbudowy kodu oraz co zrobiłem źle i jakbyście Wy to zrobili inaczej.
Z góry dzięki.

Link do kodu: https://github.com/Krzaczor/todo-list

1 odpowiedź

–1 głos
odpowiedź 6 lutego przez afqedart Gaduła (4,350 p.)
Jak dla mnie dużo nie potrzebnego kodu. Chyba, że nie rozumiem do końca działania programu. To jest po prostu przeklikiwanie tekstu do innego statusu? Jeśli tak to można to zrobić o wiele proście, ale jak się dopiero uczysz to nie mam nic przeciw.
komentarz 6 lutego przez Marchiew Mądrala (6,300 p.)
Zrobiłem abstrakcję dla localStorage w którym są tablice rzeczy do zrobienia i zrobionych.
Zrobiłem abstrakcję dla pojedynczego zadania które się dodaje, usuwa i przenosi do listy rzeczy zrobionych i usuwa z rzeczy zrobionych.
Nie operuję czysto na localStorage tylko po sparsowaniu przenoszę zawartośc do tablic jsowych i na nich dokonuję operacje.
Zadania to nie "li.textContext" tylko typ tablicy, nazwa operacji i index zadania są zapina e w przyciskach. Nie zmieniam na surowo DOMu poprzez "li.innerHTML = """ czy nie wiadomo co jeszcze.
Starałem siię też żeby metody dawały wybór w ich wykorzystaniu i były na tyle elastyczne żeby jak wrócę po jakimś czasie i coś będę chciał zmienić to zmienię tylko w części poniżej tego długieko komentarza gdzie jest w sumie ten właściwy kod, wyżej są klasy których chce unikać zmiany.
Ufff, mam nadzieję, że to co napisałem jest chodź trochę zrozumiałe

Podobne pytania

0 głosów
1 odpowiedź 143 wizyt
+5 głosów
2 odpowiedzi 166 wizyt
0 głosów
2 odpowiedzi 123 wizyt
pytanie zadane 17 stycznia 2018 w Nasze projekty przez Gambr Mądrala (6,870 p.)
Porady nie od parady
Zadając pytanie postaraj się o odpowiedni tytuł, kategorię oraz tagi.Tagi

60,251 zapytań

105,938 odpowiedzi

220,099 komentarzy

32,460 pasjonatów

Przeglądających: 285
Pasjonatów: 13 Gości: 272

Motyw:

Akcja Pajacyk

Pajacyk od wielu lat dożywia dzieci. Pomóż klikając w zielony brzuszek na stronie. Dziękujemy! ♡

Oto dwie polecane książki warte uwagi. Pełną listę znajdziesz tutaj.

...