Najczęstsze błędy w PHP i antywzorce, Kto zna jakieś fajne :) Opcje

[Sephirus]

Witam,

Ostatnio zastanawiałem się, jakie są często spotykane błędy, a raczej złe nawyki bądź po prostu bardzo nie wydajne lub nieoptymalne kawałki kodu jakie można znaleźć/napisać w PHP (IMG:style_emoticons/default/smile.gif) Zbieram też antywzorce. Gdzież indziej mógłbym się o to zapytać niż nie tutaj (IMG:style_emoticons/default/smile.gif)

Przykład wszystkim pewnie dobrze znany:

[PHP] pobierz, plaintext 
$tablica; // powiędzmy 1000 elementów, jakichś obiektów, sporych
 
for($i = 0; $i < count($tablica); $i++) {
    // jakieś operacje
}
[PHP] pobierz, plaintext 

Poprawna forma:

[PHP] pobierz, plaintext 
$tablica; // powiędzmy 1000 elementów, jakichś obiektów, sporych
 
$elementsCount = count($tablica);
for($i = 0; $i < $elementsCount; $i++) {
    // jakieś operacje
}
[PHP] pobierz, plaintext 

@aras785 - masz rację dodałem poprawną formę (IMG:style_emoticons/default/smile.gif)

Szukam podobnych konstrukcji, antywzorców programowania itp. Zależy mi na tym by były one proste (IMG:style_emoticons/default/smile.gif) Nie szukam jakichś skomplikowanych 100 linijkowych kodów na dowód tego, że są źle napisane - chodzi mi o same zamysły (IMG:style_emoticons/default/smile.gif)

Jeśli znacie jakieś piszcie proszę (IMG:style_emoticons/default/wink.gif)

Ten post edytował Sephirus 14.01.2013, 13:41:14

[Pilsener]

Najgorsze są dla mnie błędy związane z wydajnością i bezpieczeństwem, typu:

• niczym nie zabezpieczona rekurencja (tym gorsza, że uzależniona od parametrów z zewnątrz)
• echowanie czego popadnie prosto z requesta lub tablicy $_SERVER, konstrukcje typu ... where id = $_GET['id'] i tak dalej
• pobieranie wszystkiego z bazy jak leci albo wrzucanie do sesji jak leci - no bo wygodnie mieć wszystko w sesji, najlepiej całą bazę danych od razu wrzucić sobie do sesji, "by mieć taki kesz"
• "dziś miejsce na dysku nie jest już problemem" - słyszałem to już x razy po czym po chwili dysk pada bo logujemy do plików pierdów bez liku

A jedne z najbardziej irytujących:

• jednolita konfiguracja dla każdego środowiska, przez co regularnie commituje ustawienia lokalne albo po update wściekam się, że coś nie działa
• brak jakichkolwiek komentarzy czy dokumentacji, najczęściej im paskudniejszy kod tym komentarzy jak na złość mniej
• brak sprawdzania, czy coś się wykonało, założenie, że jak funkcja coś zwraca to z pewnością jest to spam, potem coś po prostu nie działa i sobie możesz szukać kilka dni no bo przecież "funkcja powinna wygenerować warninga chociaż jak jej kiepawo idzie"
• używanie gdzie popadnie różnych fantastycznych wynalazków cudownego dziecka skryptu typu eval, goto, dynamiczne generowanie zmiennych - ekstrakcja to mały pikuś przy tym
• obiektowość rozumiana tak, że np. co każde 100 linijek walniemy klasę a co 20 metodę, tworzenie jakiś dziwnych fabryk czy raczej młockarni obiektów które tworzą takie plewy, że człowiek tęskni za dynamicznym generowaniem zmiennych (IMG:style_emoticons/default/smile.gif)