Ocena serwisu

Witam

Chciałbym, żebyście ocenili moją stronę internetową: www.europilka.pl . Zrobiłem ją sam, nikt nie pomagał. Główne założenia:

a) Strona poświęcona piłce nożnej;
b) Aktualności oraz wyniki;
c) Społeczność:

• Możliwość zakładania kont,
• Możliwość komentowania,
• Możliwość "zbierania" znajomych,
• Oceniania komentarzy, newsów,
• Forum tematyczne,
• Tworzenie własnej galerii zdjęć,
• Obstawianie wyników,
• Możliwość wysyłania prywatnych wiadomości,
• Otrzymywanie różnych powiadomień ( w tym na temat aktywności naszych znajomych).

Całość wykonana w C# + MVC + AJAX + Jquery/Javascript. Chciałbym, żeby strona była pokazem moich możliwości i swoistym portfolio do znalezienia pracy w branży. Od siebie tylko dodam, że skopany jest plik CSS, gdybym miał na nowo pisać stronę, to zupełnie inaczej by on wyglądać. Tak samo zrobiłbym tą stronę responsywną.

Wielkie dzięki za wszelkie sugestie!

Na wstępie: uwag mam sporo, napiszę część tych grubszych rzeczy, na które zwracamy uwagę na rozmowach rekrutacyjnych u nas w firmie. Zaznaczam od razu, że nie jest to strona, którą warto się pochwalić w obecnej formie. Jeśli wykażesz chęci, to może coś z tego będzie - obecnie trochę blado to wypada.

Ogólne wrażenia:

• strona działa bardzo wolno - odpowiedź serwera po 4-5s to słaby wynik, celuj znacznie poniżej sekundy dla pojedynczego requesta i sygnalizuj dłuższe (tj. 200ms+) ładowanie danych, np. spinnerem - poczytaj trochę o podstawowej optymalizacji
• nieatrakcyjny wygląd, wszystkie jest takie surowe i bezpłciowe (wybacz, ale od razu mi się to nasuwa na myśl) - ostre kanty, brak cieni, głębi, zły kolor główny, brak relacji z akcentującym, itd. jeśli design nie leży ci, to może warto skorzystać z bazowych szablonów, albo np. Bootstrapa i w oparciu o to robić coś, a nie szukać własnego rozwiązania - ludzie kupują oczami
• niekonsekwencje w paddingach, marginesach już na pierwszy rzut oka, podobnie z rozmiarami czcionek, ogółem typografia trochę leży, Arial to nie jest jedyna słuszna czcionka, wiesz?
• polski język w kodzie to przekleństwo i ogromny minus na starcie, chyba że aplikujesz do firmy robiącej kolejną aplikację pokroju tej dla PKW

CSS:

• brak jakiejkolwiek metodyki (zainteresuj się chociażby BEM), modularności, kod jest nieczytelny i brakuje mu organizacji
• kiepskie nazewnictwo i brak spójności w tym
• nadużywasz position tam gdzie jest niepotrzebne i odnoszę wrażenie, że nie do końca rozumiesz to - rodzi to sporo bugów
• ID-ki w CSS to raczej zła praktyka, podobnie jak !important
• dużo powielonego kodu
• mnóstwo liczb, które nic nie mówią, nie ma między nimi relacji, przypadkowe wartości dobierane na oko

JS:

• globalny scope
• chociażby jakiś wzorzec modułu można zastosować
• i znów nazewnictwo (srsly, zmienKolorPrzycisk2?)
• funkcje jak blokujKlawisze czy PolozenieRamki to jakieś totalne nieporozumienie - czytając ten kod nie jestem w stanie stwierdzić co autor miał na myśli, w dodaku setInterval to brudny hack w tym miejscu włącznie z easter eggiem w postaci evala
• nie widzę sensu użycia jQuery
• brak średników w kilku miejscach

Jeśli chcesz, żebym coś rozwinął, to zapraszam na priv.

@Sand24 ID w CSS zła praktyka? Od kiedy i dlaczego?
Dopóki służą nam do rozróżniania głównych bloków strony - to jak najbardziej poprawna.
Nie mówiąc o tym, że daje (wiadomo, milisekundy, ale jednak) kopa przy stosowaniu tych regułek.

@shagohad:
Nie mam dużo czasu więc:

1. Fatalnie długa odpowiedź serwera - http://i.imgur.com/8iglCas.png
2. Zmiana stylu podświetlenia poprzez javascript? W dodatku przez event podpięty inline?

A w sumie u góry masz sporą odpowiedź i na razie Ci wystarczy ;)

@dzek69

Z powodu specificity. Praktyka pokazała, że zrozumienie tego zagadnienia nie wystarczy w życiu codziennym, bo CSS (i UX, ale to dyskusja nie na ten wątek) jest traktowany czasem jako piąte koło u wozu. Sporo reguł się przesłania, albo niepotrzebnie zwiększa złożoność kodu z powodu m.in. ID-ków. Z klasami nie ma tego problemu, co więcej jak stosujesz jakąś przejrzystą metodykę, to ID popsuje ci kompletnie hierarchię (i czytelność) w kodzie, a w dużych projektach to ma znaczenie. Może niewielkie dla stronek, ale aplikacji dla milionów użytkowników. ID nie jest też reużywalne, a reguły powinny być pisane w myśl zasady DRY i KISS.

Poza tym za często spotykałem się z sytuacją, że ktoś ID przesłania dodając nową klasę do selektora (albo co gorsza kolejne ID), potem po paru miesiącach jest problem z przesłonieniem czegoś i stosuje się important, potem po jakimś czasie znów trzeba coś zmienić i robi się syf.

W google'u znajdziesz sporo artykułów na ten temat (chociażby wpisując don't use id in css). A tak przy okazji tematu gorąco polecam prezentację Harry'ego Robertsa "Normalising designs for better quality CSS" - daje do myślenia, szczególnie jeśli masz do czynienia z podobnymi problemami z designerami i CSS-em.