[MySQL][PHP]walidacja kodu posztowego- co niegra ... Opcje

[mattii]

mam taki kod, cos w nim jest zle ?

[PHP] pobierz, plaintext 
 
$kodpocztowy = strip_tags($_POST['kodpocztowy']);
 
$kodpocztowy = $DBcon->real_escape_string($kodpocztowy);
 
 
function Sprawdz_Kod($kodpocztowy)
{
 
return preg_match('^[0-9]{2}[-][0-9]{3}$', $kodpocztowy);
$dobrykod=Sprawdz_Kod($kodpocztowy);
}
 
 if (
 
 $dobrykod==0
 
 )
 
 {
 echo "Błędny kod pocztowy.";
 
 
 
 
 
 
 }
 
 elseif ($dobrykod==1)
 
 {
 
reszta instrukcji, ktora się nigdy nie wykonuje - bo niezaleznie co wpisze mam echo "Błędny kod pocztowy.";
 
}
[PHP] pobierz, plaintext 

[kapslokk]

Wyrażenie regularne musi zaczynać i kończyć się tym samym znakiem.

[PHP] pobierz, plaintext 
return preg_match('@^[0-9]{2}[-][0-9]{3}$@', $kodpocztowy);
[PHP] pobierz, plaintext 

http://php.net/manual/en/regexp.reference.delimiters.php

Poza tym sprawdzasz czy kod jest dobry w funkcji do sprawdzania czy kod jest dobry, po returnie - czyli tak na prawde nigdzie.

Ten post edytował kapslokk 29.09.2016, 18:19:55

[mattii]

to teraz mam tak:

[PHP] pobierz, plaintext 
function Sprawdz_Kod($kodpocztowy)
{
 
return preg_match('@^[0-9]{2}[-][0-9]{3}$@', $kodpocztowy);
 
}
 
 $dobrykod=Sprawdz_Kod($kodpocztowy);
 
 if (
 
 $dobrykod==0
 
 )
[PHP] pobierz, plaintext 

i tytuacja się odwróciła, tzn. nawet jak podam zły kod pocztowy to nie wyrzuci informacji o błędzie..

[viking]

Teraz jeszcze poczytaj jak się korzysta z if. Bo na pewno nie w sposób jaki tu przedstawiasz.

[mattii]

cały if wygląda tak:

[PHP] pobierz, plaintext 
 
if ($dobrykod==0)
 
 {
	echo " Prawidłowy format kodu to 11-111 ";
 
}
 
 elseif ($dobrykod==1)
 
 {
 
echo "dobry kod";
 
}
 
[PHP] pobierz, plaintext 

[Rysh]

[PHP] pobierz, plaintext 
if(Sprawdz_Kod($kodpocztowy)){
//zły kod
} else {
//dobry kod
}
[PHP] pobierz, plaintext 

[szydlak]

Ja na twoim miejscu zrobiłbym to tak:
<input type="text" name="kodpocztowy" pattern="[0-9]{2}\-[0-9]{3}" >

Ta część od słowa pattern załatwia Ci walidację kodu pocztowego z poziomu html. Czyli nie puści dalej jak kod pocztowy nie będzie w formacie 00-000.

[Rysh]

Ja na twoim miejscu zrobiłbym to tak:
<input type="text" name="kodpocztowy" pattern="[0-9]{2}\-[0-9]{3}" >

Ta część od słowa pattern załatwia Ci walidację kodu pocztowego z poziomu html. Czyli nie puści dalej jak kod pocztowy nie będzie w formacie 00-000.

To po stronie przeglądarki, a po stronie serwera i tak to trzeba zwalidować.

[szydlak]

To po stronie przeglądarki, a po stronie serwera i tak to trzeba zwalidować.

W sumie racja. Bo jak zbadałem element input i usunąłem atrybut pattern to puściło formularz.

[mattii]

[PHP] pobierz, plaintext 
if(Sprawdz_Kod($kodpocztowy)){
//zły kod
} else {
//dobry kod
}
[PHP] pobierz, plaintext 

u mnie zadziałało odwrotnie tzn.:

[PHP] pobierz, plaintext 
 
if(!(Sprawdz_Kod($kodpocztowy)){
//zły kod
} else {
//dobry kod
}
 
[PHP] pobierz, plaintext 

bo rozumiem,ze funkcja Sprawdz_Kod($kodpocztowy) jesli dobry kod zwraca 1, a jeśli zły 0 ?


co do pattern , znalazłem takie rozwiązanie...mam firefoxa, ale po dodaniu atrybutu nic mi się nie wyświetlało...ponadto podobno na starszych przegladarkach i safari nie działa? takie info znalazłem na innej stronie.

no i fakt walidacji przy wpisie do bazy powoduje,ze szukałem rozwiązania w php.

[viking]

Bo Rysh odwrotnie napisał ale powinieneś sam do tego dojść. preg_match zwraca 1,0 albo false. Walidacja w przeglądarce powinna być tylko opcjonalna. Nigdy nie polegaj na tym.

Dodatkowo
$kodpocztowy = $DBcon->real_escape_string($kodpocztowy);

Najpierw rób walidację a dopiero później operacje na bazie.

[Rysh]

u mnie zadziałało odwrotnie tzn.:

Bo Rysh odwrotnie napisał ale powinieneś sam do tego dojść. preg_match zwraca 1,0 albo false.

Pomyliłem się (IMG:style_emoticons/default/facepalmxd.gif)

bo rozumiem,ze funkcja Sprawdz_Kod($kodpocztowy) jesli dobry kod zwraca 1, a jeśli zły 0 ?

jeśli nie wiesz co zwraca funkcja napisz sobie:

[PHP] pobierz, plaintext 
var_dump(Sprawdz_Kod('zly-kod'));
var_dump(Sprawdz_Kod('22-222'));
var_dump(Sprawdz_Kod('12-XX2'));
[PHP] pobierz, plaintext 

I zobacz co zwraca w zależności od danych wejściowych.

[mattii]

dziekuje Panowie za pomoc, pomału wiem coraz wiecej (IMG:style_emoticons/default/wink.gif)

mam jeszcze pytanie -

jak wprowadze błędny kod pocztowy i nacisne submita to dostaje informacje, ze kod jest bledny itp..., a wszystkie wypełnione dane w formularzu zostaja skasowane

co musze zrobić aby te dane się nie kasowały ? ew skasował się tylko ten kod pocztowy jak jest błędny ?

[kpt_lucek]

dziekuje Panowie za pomoc, pomału wiem coraz wiecej (IMG:style_emoticons/default/wink.gif)

mam jeszcze pytanie -

jak wprowadze błędny kod pocztowy i nacisne submita to dostaje informacje, ze kod jest bledny itp..., a wszystkie wypełnione dane w formularzu zostaja skasowane

co musze zrobić aby te dane się nie kasowały ? ew skasował się tylko ten kod pocztowy jak jest błędny ?

Ustawić je w formularzu

[mattii]

moje pola w formularzu wygladaja tak:

[PHP] pobierz, plaintext 
<input class="form-control" type="text" required="" placeholder="Nazwisko" maxlength="20" name="nazwisko">
[PHP] pobierz, plaintext 

wiec tutaj ustawione jest nazwisko do wyświetlania aby uzytkownik wiedział co ma wpisać w dane pole

[viking]

To jeszcze musisz ustawić value.