[HalEmmerich]

Porozmawiajmy o superglobalach. Używać czy nie? Dlaczego? Wiem, że zmienne globalne zawsze i wszędzie uznawane są za zło, ale czy jest jakiś powód do unikania odwołań do $_POST i innych w kodzie? Mając przed sobą dwie opcje - utworzenie wrappera na GETa, oraz po prostu odczytanie jego zmiennych, ostatnie wydaje mi się łatwiejsze i szybsze.
#php #webdev #filozofiaprogramowania

[Lipathor]

@HalEmmerich: sanitize data ( ͡° ͜ʖ ͡°)

[Larin]

Szczerze to chyba zależy od tego, co chcesz zrobić. Robiąc zlecenie na 1-2 wieczory robię nieraz takie spaghetti, że nie chciałbym nikomu tego pokazywać, ale tu nie o to chodzi :)

Natomiast biorąc pod uwagę, że nawet microframeworki (myślę np. o sileksie, lumenie, slimie) mają jakiś wrapper na odbierane dane - to świadczy o tym, że większość postrzega to jako rozsądniejsze, w konkretne powody się nie zagłębiam, a co powiedział @Lipathor to

[q.....u]

@HalEmmerich: Ja używam wrapperów, bo jak się odnoszę bezpośrednio do _GET itp to mi netbeans podkreśla na czerwono ;)

[I.....n]

@qwertyu: no tak, najważniejszy powód.. :D

[G.....n]

@HalEmmerich: Nie używaj, bo nie masz potrzeby. Jeśli masz, to Twój kod jest zły, bo zależy ściśle od środowiska uruchomieniowego i nie jest reusable.

[HalEmmerich]

@Ginden: Niby tak, ale zastanówmy się - czy będę to odpalał na czymś innym niż apache + modphp?
@Lipathor: Egzorcyzmy na danych mogę robić odwołując się bezpośrednio do superglobali... chyba.

[G.....n]

@HalEmmerich: Pewnie tak. Np. może się okazać, że zmieniasz metodę z GET na POST. Albo chcesz wykorzystać ten sam kawałek kodu w GET i POST. Jeśli bezpośrednio odwolujesz się do zawartości $_GET to musisz skopiować kod.
Co więcej - nie masz gwarancji, że np. obie metody wywołały taką samą walidację danych wejściowych.
Możliwe, że np. wykonujesz kod w konsoli w ramach jakiegoś unit testu. Nie masz tam wypełnionych $GET i

[HalEmmerich]

@Ginden: Jeśli zmienię z GET na POST, to używając wrapperów nie będę musiał zmienić w kodzie tyle samo? Zawsze mogę zrobić na wstępie:

$data=$_GET; if(isset($_POST)) $data=$_POST;

O unit testach może nie będę się wypowiadał, bo głupio mi się przyznawać... ʕ•ᴥ•ʔ

[Larin]

Hm... tylko myśląc w ten sposób to jeszcze prościej $_REQUEST. Ale potem to już tylko modlitwa, najlepiej do dwóch bogów naraz ;)

[HalEmmerich]

@Larin: Jeśli odprawię egzorcyzmy na danych, nie jest mi obojętne czy użytkownik wysłał posta, geta czy oba po części? Strona działa, a i żadnego większego zagrożenia bezpieczeństwa chyba nie ma. Czy rozchodzi się tylko o przenośność kodu?

[Larin]

Uważam, że jeżeli dobrze to spreparujesz, to jest ok - kwestia właśnie przenośności kodu o której mówił @Ginden, i też po prostu tego jak ten kod wygląda.
Zgadzam się, że fajnie jeśli PHP jest "the right way", ale jak mówiłem od samego początku - zależy co robisz. Jeżeli tego nawet nie unitujesz... to dobierzmy narzędzia do zadania ;)

[J.....Q]

@HalEmmerich: Odczytywanie superglobali powinno odbywać się poprzez http://php.net/manual/en/function.filter-input.php
Wtedy nie trzeba odwoływać się bezpośrednio do tych zmiennych. (właściwie to w ogóle nie występuje do nich odwołanie)

[MacDada]

#filozofiaprogramowania

@HalEmmerich: Raczej #naukaprogramowania

1. Globale utrudniają śledzenie kodu – szukaj później czy ktoś Ci się nagle do nich w losowym momencie nie odwołuje – albo tym bardziej czy gdzieś Ci ich po cichu nie modyfikuje.
2. Globale uniemożliwiają robienie podrequestów – takie Symfony potrafi np symulować żądanie do jakiejś strony, żeby uprościć flow aplikacji.
3. Globale utrudniają/uniemożliwiają automatyczne testowanie kodu (TDD)
4. Jest gazylion gotowego kodu, którego użycie eliminuje powyższe

[HalEmmerich]

@MacDada: Ad. 1. Jeśli piszę sam, to raczej wiem co i w którym miejscu robię.
Ad. 2. To akurat brzmi ciekawie.
Ad. 4. Jest dużo gotowców, które każdy może używać, ale najpierw wolałbym wiedzieć jak i dlaczego działają tak, a nie inaczej.
#filozofiaprogramowania a nie #naukaprogramowania, bo pośrednio wpis był pytaniem "czy małe, nieprzenośne strony robione przez jedną osobę, wykorzystywane w jednym miejscu są złe".

[Jurigag]

@MacDada: phalcon micro > silex @HalEmmerich: to ktoś jeszcze dzisiaj korzysta z apache i modphp ? myślałem że nginx + php-fpm to standard obecnie.....

[HalEmmerich]

@Jurigag: Postawione i skonfigurowane wiele lat temu, okazjonalne aktualizacje niczego nie zepsuły. No i łatwo się uruchamia na windowsie. Ale na serwerze mam nginx i fpm.
No i modrewrite bez grzebania w configu.

[MacDada]

Ad. 1. Jeśli piszę sam, to raczej wiem co i w którym miejscu robię.

@HalEmmerich: Pogadamy jak Twój kod będzie miał więcej niż 100kloc lub jak będziesz chciał wrócić do tego kodu za rok. Jak już nic nie będziesz pamiętał ;-)

Ad. 2. To akurat brzmi ciekawie.

automatyczne testowanie kodu (TDD) – to jest ciekawsze, serio, profesjonalizm wymaga, żeby móc powiedzieć „tak ten kod działa”, a nie tylko „wczoraj przeklikałem się

[Jurigag]

@MacDada: wiadomo że warto, ale lepiej korzystać z lepszych rozwiązań i szybszych :P

[MacDada]

@Jurigag:

1. Make it work
2. Make it right
3. Make it fast

Spoko jak już wiesz jak dobrze pisać kod i wtedy bierzesz się za optymalizację. Nie wcześniej :)

Premature optimization is the root of all evil -- DonaldKnuth.