• 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
129 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,520 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ź 156 wizyt
+5 głosów
2 odpowiedzi 174 wizyt
0 głosów
2 odpowiedzi 124 wizyt
pytanie zadane 17 stycznia 2018 w Nasze projekty przez Gambr Mądrala (7,100 p.)
Porady nie od parady
Nie wiesz jak poprawnie zredagować pytanie lub pragniesz poznać którąś z funkcji forum? Odwiedź podstronę Pomoc (FAQ) dostępną w menu pod ikoną apteczki.FAQ

62,468 zapytań

108,621 odpowiedzi

226,795 komentarzy

36,303 pasjonatów

Przeglądających: 66
Pasjonatów: 0 Gości: 66

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.

...