Prośba o sprawdzenie kodu

0

Napisałem z nudów takie coś https://github.com/Taraj2/Simple-Chat/tree/master/NodejsWebApp1 niby wszystko działa tak jak zaplanowałem i jest ok ale pisząc to natknąłem się na parę problemów np. tworząc elementy lepiej użyć createElement() czy po prostu innerHTML w rezultacie otrzymujemy identyczny wynik więc która opcja jest lepsza. Jeżeli ktoś ma jakieś rady co zrobić żeby to było bardziej "poprawnie" napisane chętnie je usłyszę xD

0

Jak dla mnie:

  1. Za duże pliki js (w sumie to plik). Pliki 100 do 150 linijek czyta się o wiele lepiej.
  2. Podobnie funkcje (ale widziałem znacznie gorsze).
  3. If'ologia - za dużo używasz tam if..else, a potrójne zagłębienie to już przesada.
  4. Pliki less - jak dla mnie też za dużo zagłębień. źle się to czyta.
  5. Zewnętrzne pliki/biblioteki trzymaj w osobnym folderze/folderach niż twoje pliki.
  6. innerHTML - bardziej wydajny, do dodawania mniejszych elementów, jak chcesz dodać jakiś większy element to raczej lepiej createElement()

1 użytkowników online, w tym zalogowanych: 0, gości: 1