Forum PHP.pl

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 Zbieram też antywzorce. Gdzież indziej mógłbym się o to zapytać niż nie tutaj

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ę

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

Jeśli znacie jakieś piszcie proszę

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

głupota i lenistwo.... dwa najczęściej spotykane na forum złe nawyki...


A z bardziej przyziemnych rzeczy to np.
zapytania w pętli, gdzie główna pętla idzie z zapytania a w niej znowu pętla z zapytania
A już szczyt szczytów zapytania w rekurencji....

@aras785 Twój kod nadal jest dla mnie nie zrozumiały (nie wiem po co ten SELECT na początku) - ale przeanalizuję to

@!*! formatowanie (stricte) nie jest błędem a raczej niedbalstwem. Chyba, że ktoś nie lubi używać {} przy pętlach i instrukcjach a potem coś do nich dopisuje i się dziwi że zawsze działają ;P

[PHP] pobierz, plaintext 
 
if($costam === true)
    zrob_cos();
    zrob_cos_jeszcze();
 
// zamiast:
 
if($costam === true) {
    zrob_cos();
    zrob_cos_jeszcze();
}
 
// itp...
 
[PHP] pobierz, plaintext 

Temat z najczęściej pojawiającymi błędami jest mi znany - myślałem że wyłapie coś jeszcze osobno Pomysł z przeglądaniem działu "oceny" jest jednak dobry - dzięki tam na pewno coś znajdę

@nospor

Na głupotę nie ma rady ;P ale nie mogę zakładać że każdy na nią cierpi - sam robię masę błędów a potem mi wstyd ;P ale nie uważam się za totalnego głupola Co do lenistwa... true, true... :/

Zapytania w pętli to faktycznie częsta przypadłość Dużo ludzi na początku nie wie co to JOIN, nie ma pojęcia co to relacyjność :/ no i przede wszystkim jak już napisali jakieś zapytanie i im działa, to już go nie ruszają i piszą nowe. Każdy z nas się chyba zgodzi, że na początku nauki termin "wydajność" jest nam obcy

Ale zaintrygowałeś mnie tą rekurencją z zapytaniami!! Jak gdzie w życiu tego nie widziałem - nie wiem nawet jak to napisać masz jakiś przykład?

Ok przyznam szczerze... z tym drzewkiem... że bym na to nie wpadł co prawda od jakiegoś czasu sporo siedzę w SQL'u i obczajam jego wszelkie tajniki (poza administracją może bo to mi na nic) i może przez to wydaje mi się to dziwne, choć kiedyś kto wie - może i sam tak pisałem

Ten drugi błąd to też bardziej pod formatowanie I kurcze coś w tym jest - trzeba pisać "ładnie", estetycznie, zachowywać te wszystkie (jak to się na początku wydaje) głupie zasady ja osobiście nie przestrzegam tylko jednej. Czytałem parę propozycji jak pisać i formatować kod czy o nazewnictwie zmiennych itd. (także dla różnych frameworków itp.) więc mi się to utrwaliło i z tego wszystkiego mam swoją jedną wizję łączącą po kawałku ze wszystkiego ale nie przestrzegam zasady o tym, żeby linia miała <=80 znaków i nie znam osoby (osobiście) która tego przestrzega (oczywiście nie pisze ciurkiem ale jak coś mi wyskoczy poza 80 znaków to tak zostawiam po prostu).

Najgorszy błąd jaki mi się kiedyś przydarzył i naprawdę nie wiedziałem co z tym robić (pomogło dopiero przepisanie kodu od nowa ) wyglądał tak:

(może nie błąd, ale przeoczenie)

[PHP] pobierz, plaintext 
for($i = 0; $i < $jakasWartosc; $i++); {
     // wykonaj jakiś kod
}
[PHP] pobierz, plaintext 

Oczywiście ta pętla odwalała trochę roboty i to nie łatwej i nigdy nie chciała się wyświetlić :/ co bym nie robił Głupi średnik (który swoją drogą sam nie wiem jak się tam dostał zmarnował mi kupę czasu bo zdebugowałem cały skrypt praktycznie

EDIT: Te dwa ostatnie powalają nospor ale fakt nawet, że chyba na forum raz z czymś takim "pomogłem"

Ten post edytował Sephirus 14.01.2013, 14:10:59

Tyle jest błędów i nieprawidłowych konstrukcji popełnianych przez ludzi, że ten cały wątek chyba nie ma celu, bo po pierwsze chyba miałby z 500 stron, a po drugie całe programowanie sprowadza się do poprawnego pisania algorytmów, więc ten temat jest tak jakby jak całe to forum

@pyro Masz całkowitą rację niestety - to jest "rzeka", można by wymieniać w nieskończoność mimo to uważam, że dobrze jest ciągle wałkować ten temat dlaczego?

Jestem sobie programistą, uważam że pozjadałem wszystkie rozumy wchodzę na taki wątek i czytam, tu się uśmiechnę, tam się zaśmieje ale może, w pewnym momencie naglę przystanę i stwierdzę - o kurde... nie wiedziałem, że tak nie można

No i poza tym chciałbym zebrać nieco materiałów na ten temat by się nimi podzielić z "nowymi" w pracy i ich na to uczulić a sam wszystkiego nie pamiętam

@nospor Co do operatorów to fakt, ale bardzo podobała mi się odpowiedź Erixa w temacie o najczęstrzych błędach, że "if($costam = 123)" to nie bug - tylko feature To mi nasunęło pewną myśl: rzadko spotykam się z tym żeby ktoś to stosował :/

[PHP] pobierz, plaintext 
 
// w zamyśle jakas_funkcja zwraca albo null albo ciąg znaków
 
// zamiast (co jest dla mnie najlepsze)
if($wynik = jakas_funkcja()) {
    echo 'Wynik prawidlowy i równy ' . $wynik;
}
 
// jest
$wynik = jakas_funckja();
if($wynik) {
    echo 'Wynik prawidlowy i równy ' . $wynik;
}
 
// no nie wspominając już o błędnym podejściu
if(jakas_funkcja()) {
    echo 'Wynik prawidlowy i równy ' . jakas_funkcja();
}
[PHP] pobierz, plaintext 

Czym może to być spowodowane że rzadko to ludzie stosują? Niewiedza? Gorsza przejrzystość kodu?

własnie w dziale przedszkole pojawił się problem z mojego posta wczesniej:
if ($zm = 'b')

Bład tak czesty, ze nawet podczas pisania posta pojawiają się takie tematy

@nospor, bzdury Pan pleciesz! To nie jest żaden błąd, tylko konstrukcja, która choć nie zawsze stosowana z rozwagą, w wielu przypadkach usprawiedliwiona.

Wg. mnie chodzi o przejrzystość. Sam tego nie stosuję choć dobrze wiem jak to działa Wolę mieć przejrzysty kod niż super hiper featured

@pyro to
if ($zm = 'b')
te tez nie jest błąd, tylko konstrukcja, rzadko stosowana z rozwagą

Pomijam przypadki, gdy wyłączenie E_NOTICE jest usprawiedliwione. Mówię o sytujacji, gdy to w żaden sposób nie jest usprawiedliwione, np. gdy zaczynasz projekt od 0 i lecisz już z taką dyrektywą. Wówczas to jest fatalny błąd!

NIe chce mi się powtarzać, więc podam tylko linka
http://nospor.pl/notice-wyswietlac-czy-nie.html

No to nie dopowiedziałeś i można było bardzo łatwo to źle zrozumieć. Trzeba było od razu dać link do artykułu, bo z tego co napisałeś można zrozumieć, że stosowanie tego jest zawsze złe.

Znajomość różnicy między && i AND, a także || i OR może się czasem przydać.

[PHP] pobierz, plaintext 
$result1 = true AND false; // true
$result2 = true && false; // false
$result3 = false OR true; // false
$result4 = false || true; // true
[PHP] pobierz, plaintext 

Dobrze, przepraszam. Niepotrzebnie przyjąłem, że każdy zrozumie o co mi chodzi

@Tuminure o właśnie, ludzie często nie wiedzą do czego służą nawiasy oraz co to ważność operatora i często nie kumają czemu
2+3*4 nie jest równe 20
Oczywiście mówię tu w odniesieniu do mysql i php

Co do errorów to prawda... ale to jest celowe jakby nie patrzeć ;P zapominają o cudzysłowach itp. a potem jak im wyskakuje wiele notice'ów a mogą je ukryć (bo przecież wszystko działa) to sobie je olewają i ukrywają

U mnie w firmie jak raz włączyliśmy na głównym (starym) portalu logowanie do pliku wszystkich błędów z noticami włącznie to przez 30 minut plik z logiem miał 10GB

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

No właśnie, i weź potem w takiej ilości błędów znajdź człowieku jakąś literówkę.... nie ma szans.... i dlatego zawsze od początku trzeba pisać poprawnie. Gdy się tego nie zrobiło to potem już pozostaje jedynie na stałe wyłączenie błędów NOTICE i szukać przez 5 godzin jakis banalnych błędów, które z NOTICE znalazłoby się w dwie sekundy