Jeden Wielki OffTopick, jeśli masz pisać na temat, nie wchodź! Opcje

[SmokAnalog]

Spojrz na to z tej strony: autor miedzy innymi ma sprawdzac, czy zmienne tekstowe (email, imie, nazwisko), nie są puste. Uzywajac twojej metody sprawdzi tylko to. Uzywajac empty, dojdzie mu jeszcze filtr na 0 (IMG:style_emoticons/default/biggrin.gif) Same plusy.

Nie mam złego dnia, tylko denerwuje mnie ignorancja w temacie walidacji (IMG:style_emoticons/default/tongue.gif) Nie widzę żadnego argumentu za tym, żeby zakładać filtr na liczbę 0. Dla jednego to czepianie się, dla innego to poważne podejście do tematu i sam tak do tego podchodzę. Takie bezsensowne zależności to furtka dla błędów.

[nospor]

Robie walidacje na niepustosc, potem robie walidacje na poprawnosc imienia. Gdzie tu furtka?
Dzieki empty, jak ktos wpisze 0, to nie musze meczyc serwera dodatkową walidacją na imie, bo wylapie mi to empty. Robie to z pelną swiadomością.

W przypadku z tego tematu, rowniez nie ma mowy o zadnej furtce.

Oczywiscie, jak ktos zacznie uzywac empty dla sprawdzania pol, gdzie sie wpisuje liczby i 0 jest dopuszczalne,wowczas bedzie furtka dla bledu. Dlatego pisze: jak ktos wie jak dziala empty to wszystko wporzadku.

Podobny problem jest z isset - jak wiesz jak sie zachowuje i mimo wszystko nadal tego uzywasz, to wszystko wporzadku. Gorzej jakbys uzywal i nie wiedzial jak dokladnie sie zachowuje

[SmokAnalog]

Oczywiście, Twoje rozwiązanie zadziała, ale z punktu widzenia myślącego programisty jest złe. Sprawdzanie wartości pod kątem liczby powinno być osobną operacją, w innym przypadku prowadzi do trudnych do wyłapania błędów. Jesteś doświadczonym programistą, więc wiesz jak wielki wpływ na jakość kodu ma jego zdolność do ponownego użycia (jak zgrabnie przetłumaczyć reusability? (IMG:style_emoticons/default/biggrin.gif) ).

Z empty, Twój algorytm przybiera na przykład taką postać:

Kod

Jeśli imię jest pustym łańcuchem znaków lub zerem:
    Wyświetl komunikat, że wartość nie została wpisana
Jeśli imię jest liczbą:
    Wyświetl komunikat, że imię nie może być liczbą

Pierwsza instrukcja warunkowa powinna być bytem niezależnym od drugiej, bo sprawdzamy zupełnie inną właściwość. Bycie zerem jest dla stringa równoznaczne z byciem jakąkolwiek inną liczbą, a nie ma kompletnie żadnego związku z byciem pustym ciągiem znaków.

Ty wiesz jak działa empty, ja wiem jak działa empty. Nie ukrywam, że jestem zaskoczony Twoim stanowiskiem w tym temacie.

[Qss]

spoko, koledze który potrzebuje pomocy wystarczy. Uczy się. Nie pisze on "aplikacji rządowej w PHP" wiec cały wasz wywód jest trochę niepotrzebny

[nospor]

Nie ukrywam, że jestem zaskoczony Twoim stanowiskiem w tym temacie.

No coz.... wg Ciebie jestem przeciez niemyslacym programistą wiec nie wiem skad te zaskoczenie (IMG:style_emoticons/default/smile.gif)

Jesli przyjac, ze ma sie tak zachowywac skrypt jak napisales:

Jeśli imię jest pustym łańcuchem znaków :
Wyświetl komunikat, że wartość nie została wpisana
Jeśli imię jest liczbą:
Wyświetl komunikat, że imię nie może być liczbą

To faktycznie, uzycie empty da zly wynik dla pewnych wartosci. Ja się z tym z Tobą zgadzam.
W tym temacie jednak nie o to chodzilo.

Zaczalem dyskusje z Tobą gdyż napisales:

ale morał jest taki, że empty to idiotyzm.

Nie, empty nie jest idiotyzmem, jest to naprawde bardzo fajna funkcja i uzywam jej w bardzo wielu miejscach, nie tylko podczas walidacji. Być może pisząc to zdanie miales na mysli pewien kontekst w jakim to pisales. Byc moze, ale z tego zdania to nie wynikalo wiec włączyłem się w dyskusje (IMG:style_emoticons/default/smile.gif)

[SmokAnalog]

Nie, empty nie jest idiotyzmem, jest to naprawde bardzo fajna funkcja

To nawet nie jest funkcja, tylko potwór (IMG:style_emoticons/default/tongue.gif)

Nazywając ją idiotyczną, miałem na myśli jej zachowanie na łańcuchach znaków.

• Tablice OK.
• Liczby, jeśli wiadomo, że to liczba sensu stricte, to OK.
• Boolean trochę dziwne, no ale dobra.
• Łańcuchy znaków to padaka.

[buliq]

Od czego są zamki w drzwiach (IMG:style_emoticons/default/biggrin.gif)

Albo w gabinecie na drzwiach , jak pracujesz i dzieci wchodzą to niech nie zamykają drzwi za sobą, jak ich nie ma to zamknięte, jak nie pracujesz to gabinet zamknięty. I wszyscy szczęśliwi! (IMG:style_emoticons/default/biggrin.gif)

[Turson]

spoko, koledze który potrzebuje pomocy wystarczy. Uczy się. Nie pisze on "aplikacji rządowej w PHP" wiec cały wasz wywód jest trochę niepotrzebny

Wręcz przeciwnie. Nie musi to służyć autorowi wątku, ale wszystkim czytającym.


SmokAnalog, przecież i tak po sprawdzeniu czy input nie jest pusty powinno się w tym przypadku sprawdzić preg_replace czy prawidłowe dane wpisano, więc czemu aż tak się czepiesz empty?

[kamil4u]

1) Nie mam garazu
2) Jak bede mial to myslisz ze nie zajrzą tam? Co ty, dzieci nie znasz? wink.gif

Dzieci, jak dzieci - przekonaj żonę (IMG:style_emoticons/default/smile.gif)

[nospor]

Jaki potwor....
Ja uzywam empty do sprawdzania takich rzeczy: null, '', 0, FALSE. W niektorych miejscach nie ma dla mnie znaczenia ktora z tych wartosci wystapi, gdyz dla mnie zawsze to jest empty. To wlasnie robi ta funkcja. Zas gdy bedzie 1, 2,3 ...... 100000, TRUE, 'smok analog przesadza' to wowczas to nie jest empty i tak wlasnie dziala ta funkcja.

Mozesz sie nie zgadzac z tym, twoje prawo, ale nie pisz, ze to idiotyzm czy potwor, jesli nie rozumiesz po co ona powstala.

Nazywając ją idiotyczną, miałem na myśli jej zachowanie na łańcuchach znaków.

No i ok. Tylko ze jak z bazy dostaję ciag '0' to akurat dla mnie w moich przypadkach to jest wlasnie empty (IMG:style_emoticons/default/smile.gif) Wiec zalezy od czasu i miejsca (IMG:style_emoticons/default/wink.gif)

I Qss ma racje, zrobil sie tu lekki offtopic wiec wydzielam posty.

[Turson]

Albo w gabinecie na drzwiach , jak pracujesz i dzieci wchodzą to niech nie zamykają drzwi za sobą, jak ich nie ma to zamknięte, jak nie pracujesz to gabinet zamknięty. I wszyscy szczęśliwi! (IMG:style_emoticons/default/biggrin.gif)

To jest dopiero pomysł! Popieram (IMG:style_emoticons/default/smile.gif)

[nospor]

Dzieci, jak dzieci - przekonaj żonę

A co to zona cyckow nie widziala? (IMG:style_emoticons/default/tongue.gif)

Albo w gabinecie na drzwiach , jak pracujesz i dzieci wchodzą to niech nie zamykają drzwi za sobą, jak ich nie ma to zamknięte, jak nie pracujesz to gabinet zamknięty. I wszyscy szczęśliwi!

Moj gabinet to jednoczesnie plac zabaw dzieci (IMG:style_emoticons/default/smile.gif)

[SmokAnalog]

ale nie pisz, ze to idiotyzm czy potwor, jesli nie rozumiesz po co ona powstala

Potwór, bo nie zachowuje się jak funkcja.

[PHP] pobierz, plaintext 
empty('test');
[PHP] pobierz, plaintext 

Rozumiem po co powstała, uwierz mi.

[nospor]

I co w podanym kodzie jest znowu zle?

Rozumiem po co powstała, uwierz mi.

A zachowujesz sie jakbys nie rozumial.... Podalem ci przyklady, gdzie uzycie tej funkcji jest jak najbardziej poprawne.

[SmokAnalog]

I co w podanym kodzie jest znowu zle?

Odpal go, nauczysz się nowej rzeczy (IMG:style_emoticons/default/wink.gif)

[nospor]

A wystarczy zajrzec do manuala
http://pl1.php.net/manual/en/function.empty.php

Variable to be checked

Odpalasz nieprawidlowo a potem znowu sie dziwisz (IMG:style_emoticons/default/tongue.gif) Jedyne co sie nauczylem, to to, ze nie umiesz czytac manuala (IMG:style_emoticons/default/wink.gif)

Powód edycji: [nospor]:

[pedro84]

@SmokAnalog - z całym szacunkiem, ale bredzisz. Albo tkwisz w złym przekonaniu, albo starasz się być kontrowersyjny. To i to jest słabe.

Zobacz sobie np. jak wygląda walidator NotBlank z Symfony 2: https://github.com/symfony/symfony/blob/mas...nkValidator.php.

[SmokAnalog]

Czekam aż ponownie nazwiesz tego potwora funkcją.

[nospor]

Tak, to jest funkcja, ktora jako parametr wymaga podania zmiennej...

[SmokAnalog]

I kto tu nie umie czytać manuala:

Kod

Note: Because this is a language construct and not a function, it cannot be called using variable functions.

@Nospor, sam pytałeś co w tym kodzie jest nie tak, czyli tego nie wiedziałeś. Dlaczego zamiast się przyznać do błędu próbujesz mnie ośmieszyć? Bawi Cię to?