[www]Listy informacyjne, ocena pod kątem rozmowy kwalfikacyjnej Opcje

[ssstrz]

Witam
Prosił bym was szanownych programistów którzy mają to szczęście i już są czynnie w zawodzie o ocenę strony http://codecats.byethost14.com nie pod kątem pomysłu (bo wg mnie nie ma tu nic fajnego) ale raczej z punktu widzenia pracodawcy. Czy w/w stronka może być punktem wyjścia podczas rozmowy dla kogoś kto szuka pracownika z podstawową znajomością Html+css i dobra znajomością PHP? Kod także podlega ocenie: http://codecats.byethost14.com/inc - ukryłem tylko model gdyż tam jest hasło do bazy danych.
- wszelkie testy pod kątem bezpieczeństwa mile widziane
- jeśli komuś będzie się chciało to proszę o dodanie jakichś danych do strony itp.
- konta testowe w stopce

Ten post edytował ssstrz 29.04.2013, 20:57:22

[ssstrz]

Dziękuje wam za wartościowe opinie, ale mam jeszcze kilka próśb o rozwinięcie myśli:

- brakuje dokumentacji
- metody są niezrozumiałe

-myśle że niezrozumiałe metody wynikają właśnie z tego powyżej. Postaram się pomyśleć o jakiejś dokumentacji przy następnym projekcie.

- to że zmieniłeś doctype na html5, nie znaczy że jest to od razu strona pisana w tym standardzie

Wiem o tym, że nie wystarczy zmienić doctype. Możesz pokazać mi dokłądnie gdzie odbiegam od standardu, gdyż korzystałem tylko z dokumntacji w3c i http://validator.w3.org/check?uri=http%3A%...ethost14.com%2F

- zainteresuj się w PHP standardem PSR jeśli chodzi o nazewnictwo np. namespace pl\news co to jest to pl?

- ok poczytam o PSR, no to 'pl' to nawyk który przyjąłem w pisaniu na Androida/Jave - tam się w ten sposób nazywa przestrzenie nazw, tak samo jak metody nazywam: jakasMetoda() - źle to jest widziane w PHP?

1.1 - tworzysz pełno pustych znaczników, nie wiadomo po co
1.2 - niepotrzebnie rozbiłeś sekcje CSS na osobne pliki

ad 1.1 - zgadzam się tutaj zawaliłem - zwrócę na to uwagę przy następnym "projekcie"
ad 1.2 - obawiałem się że styl css na ponad 1000 linijek kodu bedzie źle widziany i mało czytelny, ale jeśli tak się nie robi to postaram się to wyeliminować.

Zmień wygląd strony, usuń placeholdery ("Logo umieść tutaj" ?questionmark.gif), podmień testową szachownicę na coś innego, bo teraz absolutnie nie nadaje się do pokazania.

- Czemu placeholdery wywalić?
Co do grafiki: "Logo umieść tutaj" szachownicy, obrazków facebooka to już coś do czego nie przywiązywałem uwagi, gdyż grafika to nie moja działka, chociaż z drugiej strony jeśli chce coś pokazać podczas rozmowy kwalfikacyjnej to pasuje żeby strona z każdego aspektu była dopracowana

Zmiany:

[PHP] pobierz, plaintext 
Warning: htmlspecialchars() expects parameter 1 to be string, array given in /home/vol10/byethost14.com/b14_12903337/htdocs/inc/controller/Validator.inc on line 8
[PHP] pobierz, plaintext 

- Dziękuję ślicznie (IMG:style_emoticons/default/smile.gif) poprawione

[PHP] pobierz, plaintext 
$menuView=$this->navigationMenu(null,null,null,'all',null,null,null);
[PHP] pobierz, plaintext 

- wyrwałem chwasta teraz argumentem jest po prostu tablica

http://codecats.byethost14.com/inc/view/captucha.php
kapucha?

- poprawione

Podsumowanie:
Potrzebowałem opini ludzi którzy na PHPie zjadają zęby, żeby zobaczyć w którym miejscu ja jestem. Moje wnioski to: narazie wstrzymam się z szukaniem pracy, poczytam o "PSR", "ACL" zrobie kolejny projekt używając np: "Twitter Bootstrap" - jeśli uznam że błędów z bieżącego projektu już nie popełniłniam to wrzucę tutaj do oceny.

Jeszcze jedno pytanie: Czy np. gdybym się zajął nauką Frameworka czy to mogło by wyeliminować błędy z w/w projektu i narzucić mi przyjęte zasady w phpie? Czy narazie zrobić wszystko na czystym PHPie tak jak napisałem wyżej i zapomnieć o Frameworku i pracy na jakiś czas?