[PHP]Sprawdzanie nr telefonu poprawienie (poprawnie kodu) Opcje

[szczabik]

Mam taki kod i nie chce działać wie ktoś czemu chce by użytkownik mógł wpisać w formularzu tylko 9 liczb ( nr komórki )

[PHP] pobierz, plaintext 
<?php
if ($nr == "" || !preg_match('/^[1-9][0-9]{,9}$/', $nr)) {
$error .= "<span style="color: #FF0000; text-align: center;">Musisz podać nr telefonu</span><br>";
}
?>
[PHP] pobierz, plaintext 

[.radex]

Ja bym po prostu sprawdził długość za pomocą strlen

[nitro18]

a oprócz tego nie możesz ustawić w polu tam gdzie ma user wpisać nr komórycoś typu: <input type="text" maxlength=9 name="nr_komory">

a w PHP sprawdź za pomocą strlen -> strlen($_POST[nr_komory])

[.radex]

nitro18 - jak podpowiadasz, to z głową

nie

<input type="text" maxlength=9 name="nr_komory">

tylko

<input type="text" maxlength="9" name="nr_komory">

nie

strlen($_POST[nr_komory])

tylko

strlen($_POST['nr_komory'])

[nitro18]

oj przepraszam radex, od 9 miesięcy nie miałem styczności z PHP i HTML (IMG:http://forum.php.pl/style_emoticons/default/sad.gif)

[szczabik]

Zrobiłem taki cos i działa ale teraz chce dodać by sprawdzać czy ciąg składa się tylko z cyfr

[PHP] pobierz, plaintext 
<?php
$nr =43133345600;
 
if (strlen($nr)>9) {
$error .= "<span style=\"color: #FF0000; text-align: center;\">Musisz podać nr telefonu</span><br>";
}
 
echo "$error";
?>
[PHP] pobierz, plaintext 

[nithajasz]

is_int()

[szczabik]

Zrobiłem tak i teraz wyskakuje błąd

[PHP] pobierz, plaintext 
<?php
$nr =43133345600aa;
 
if (is_int(strlen($nr))>9) {
$error .= "<span style=\"color: #FF0000; text-align: center;\">Musisz podać nr telefonu</span><br>";
}
 
echo "$error";
?>
[PHP] pobierz, plaintext 

Gdy zamiast is_int dam is_numeric też jest błąd

[nithajasz]

[PHP] pobierz, plaintext 
<?php
if (is_int(strlen($nr))>9) {
?>
[PHP] pobierz, plaintext 

Bo to zwraca TRUE or FALSE a nie długość ciągu (IMG:http://forum.php.pl/style_emoticons/default/smile.gif)

daj to tak :

[PHP] pobierz, plaintext 
<?php
if(strlen($nr)==9 && is_int($nr)) echo "ok"; else echo "blad";
?>
[PHP] pobierz, plaintext 

[szczabik]

I dalej jest błąd
chodzi to ze gdy użytkownik wpisze takie cos jako nr 13adaD839 to warunek zwróci mi prawdę że nr jest zły bo musi składać się z samych 9 cyfr np. 123873894 i jak warunek zwróci prawdę że nr jest zły to potem
Wyświetlam komunikat o błędzie ze nr jest zły
I mam taki kod cały

[PHP] pobierz, plaintext 
<?php
// Sprawdzam poprawność nr telefonu
 
if(strlen($nr)==9 && is_int($nr)) {
$error .= "<span style=\"color: #FF0000; text-align: center;\">Musisz podać nr telefonu</span><br>";
}
 
//Sprawdzam poprawność email
if ($email == "" || !preg_match("/^[-0-9A-Z_.]{1,50}@([-0-9A-Z_.]+.){1,50}([0-9A-Z]){2,4}$/i", $email)) {
$error .= "<span style=\"color: #FF0000; text-align: center;\">Musisz podać swój adres e-mail</span><br>";
}
 
//Jeśli nr telefonu jest dobry i emial dodaje to do bazy
if (!$error) {
 
if($tekst and $email and $przebieg) {
 
dbquery("INSERT INTO ".$db_prefix."intencja SET tekst='$tekst', email='$email', typ='$typ', user_id='".$userdata['user_id']."', modlitwa_datestamp='".time()."'");
 
}
 
echo "<center><strong>Twoja Wiadomość została wysłana!</strong></center>";
 
//Jeśli nr telefonu się nie zgadza lub emial wyświetlam komunikat 
} else {
 
echo "<center><br><b>Twoja Wiadomość nie została wysłana z następujących powodów:!<b><br>
<br>$error<br>Spróbuj ponownie.</center><br>"; 
 
}
?>
[PHP] pobierz, plaintext 

I dalej nie działa mi ten kod na sprawdzanie nr telefonu

[PHP] pobierz, plaintext 
<?php
if(strlen($nr)==9 && is_int($nr)) {
$error .= "<span style=\"color: #FF0000; text-align: center;\">Musisz podać nr telefonu</span><br>";
}
?>
[PHP] pobierz, plaintext 

[mike]

Moim zdaniem powinieneś filtrować tylko pozwalając na cyfry i znaki (, ), + i spację.
Możliwości wpisania numeru masz sporo a pomysłów użytkowników jeszcze więcej.

523489659
523 489 659
523 48 96 59
+48523489659
+48 523 489 659
+48 523 48 96 59

523489659
(52)3489659
(052)3489659
52 3489659
+48 52 3489659
052 3489659
(52) 3489659
(052) 3489659
+48 (052) 3489659

Który numer jest zły? Pisać wyrażenia na wszystkie możliwości? Zły pomysł.

Ten post edytował mike 13.06.2008, 14:59:06

[szczabik]

A jaka funkcaj odpowiada za sprawdzenie czy w ciągu występuje jakaś litera ?


Np. mam takie cos 43432a9 to funkcja to jaka funkcja zwróci mi prawdę ?

[hiszpanespaniol]

przecież w if'ie masz źle wpisane, a skrypt działa dobrze, powinno być tak:

[PHP] pobierz, plaintext 
<?php
if(strlen($nr)==9 && is_int($nr)) {
$error .= "<span style=\"color: #00ff00; text-align: center;\">To jest prawidłowy numer</span><br>";
}
else
	{
$error .= "<span style=\"color: #FF0000; text-align: center;\">Musisz podać nr telefonu</span><br>";
	}
?>
[PHP] pobierz, plaintext 

[szczabik]

Tak tylko musi mi zwracać prawdę gdy nr jest nie dobry a fasle gdy dobry

Czyli jak mam 3218a99321 to zwraca mi prawdę a jak mam 777321987 to false

[hiszpanespaniol]

w takim razie wystarczy krzyknąć na skrypt i działa poprawnie (IMG:http://forum.php.pl/style_emoticons/default/smile.gif)

[PHP] pobierz, plaintext 
<?php
if(strlen($nr)!==9 || !is_int($nr)) {
echo '<span style="color: #ff0000; text-align: center;">To nie jest prawidłowy numer</span><br>';
}
else
	{
echo '<span style="color: #00ff00; text-align: center;">numer jest 9 cyfrowy</span><br>';
	}
?>
[PHP] pobierz, plaintext 

edit:
wybacz, dałem echo, zamiast error, bo testowałem u siebie. no i zamiast && jest ||, ale to logiczne

Ten post edytował hiszpanespaniol 13.06.2008, 16:50:11

[pyro]

Ja odwołując się do postu mike'a -> numer telefonu można sprawdzić tak:

[PHP] pobierz, plaintext 
<?php
 
$nr = str_replace(' ', '', $nr);
// w nawiasach chyba nikt juz nie bedzie pisal numeru wiec pomine to...
// jeśli domowy to musi być z kierunkowym
if(ereg('^+?[0-9]{9,11}$', $nr))
{
// numer poprawny
}
else
{
// numer niepoprawny
}
 
?>
[PHP] pobierz, plaintext 

[szczabik]

No teraz wywala mi błąd jak zrobię tak

[PHP] pobierz, plaintext 
<?php
$nr =431333456aa;
 
if(strlen($nr)!==9 || !is_int($nr))  {
$error .= "<span style=\"color: #00ff00; text-align: center;\">To nie jest prawidłowy numer</span><br>";
}
 
echo "$error";
?>
[PHP] pobierz, plaintext 

I dalej jest błąd po za tym jeśli to ma działać z moim formularzem to musi mi if zwracać prawdę gdy nr jest nie poprawy bo inaczej nie wyświetli mi komunikatu

Zrobiłem tak nie działa

[PHP] pobierz, plaintext 
<?php
$nr =431333456aa;
 
$nr = str_replace(' ', '', $nr);
if(ereg('^+?[0-9]{9,11}$', $nr)){
$error .= "<span style=\"color: #00ff00; text-align: center;\">To nie jest prawidłowy numer</span><br>";
}
 
echo "$error";
?>
[PHP] pobierz, plaintext 

A to kod z moim formularzem

[PHP] pobierz, plaintext 
<?php
//Sprawdzam adres emial czy jest poprawny
if ($email == "" || !preg_match("/^[-0-9A-Z_.]{1,50}@([-0-9A-Z_.]+.){1,50}([0-9A-Z]){2,4}$/i", $email)) {
$error .= "<span style=\"color: #FF0000; text-align: center;\">Musisz podać swój adres e-mail</span><br>";
}
//Sprawdzam nr telefonu czy jest poprawny
 
$nr = str_replace(' ', '', $nr);
if(ereg('^+?[0-9]{9,11}$', $nr)){
$error .= "<span style=\"color: #00ff00; text-align: center;\">To nie jest prawidłowy numer</span><br>";
}
 
//Jeśli żadne z warunków nie zwrócił prawdy czyli ze zły nr to dodaje dane do bazy
if (!$error) {
 
if($tekst and $email and $przebieg) {
 
dbquery("INSERT INTO ".$db_prefix."intencja SET tekst='$tekst', email='$email', typ='$typ', user_id='".$userdata['user_id']."', modlitwa_datestamp='".time()."'");
 
}
 
echo "<center><strong>Twoja Wiadomość została wysłana!</strong></center>";
 
} else {
 
//Jeśli nr telefonu jest nie poprawny lub email wyświetlam błąd
 
echo "<center><br><b>Twoja Wiadomość nie została wysłana z następujących powodów:!<b><br>
<br>$error <br>Spróbuj ponownie.</center><br>"; 
//Zmienna $error odpowiedzialna za wyświetlenie błędów jeśli nr teflonu i emial są
 nie prawne wyświetli dwie informacje
}
?>
[PHP] pobierz, plaintext 

[pyro]

Zrobiłem tak nie działa

[PHP] pobierz, plaintext 
<?php
$nr =431333456aa;
 
$nr = str_replace(' ', '', $nr);
if(ereg(^+?[0-9]{9,11} , $nr)){
$error .= &#092;"<span style=\"color: #00ff00; text-align: center;\">To nie jest prawidłowy numer
&#092;";
}
 
echo &#092;"$error\";
?>
[PHP] pobierz, plaintext 

[b][b]

Nie przeczytałeś tego co dałem w komentarzach... Z tego co wnioskuje (co próbowałeś zrobić) to powinno wyglądać tak:

[PHP] pobierz, plaintext 
<?php
$nr =431333456aa;
 
$nr = str_replace(' ', '', $nr);
if(!ereg('^+?[0-9]{9,11}$', $nr)){
$error .= "<span style=\"color: #00ff00; text-align: center;\">To nie jest prawidłowy numer</span><br>";
}
 
echo "$error";
?>
[PHP] pobierz, plaintext 

Zobacz cyz teraz działa

Ten post edytował pyro 13.06.2008, 17:29:30

[szczabik]

Zobacz cyz teraz działa

Wywala błąd

Ten post edytował szczabik 13.06.2008, 18:01:58

[Athlan]

is_int dla stringa nigdy nie zwróci Wam true, int to typ danych. do takiego czegoś wykorzystujemy is_numeric(). Do is_numeric() wsadzamy stringa a nie wartość z strlen().