PHP 함수의 값을 올바르게 반환하지 않습니까?

Question

나는 폼의 각 부분을 검증하는 함수를 사용하고있다. 이 특정 양식의 경우 전화 번호는 선택 항목입니다. 그래서 나는 다음과 같은 기능을 함께했다 :

if(isset($_POST['phone'])) { $phone = $_POST['phone']; } 

//Call phone validation function 
$phone = validPhone($phone); 
if (strlen($phone) == 0) { 
    $mistakes[] = 'Your phone number must contain only NUMERIC characters.'; 
} 

    function validPhone($phone) { 
     if ($phone != '') { 
      $phone = trim($phone); 
      if (!ctype_digit(str_replace(' ', '', $phone))) { 
       $phone = ''; 
      } else { 
       //accept phone entry and sanitize it 
       $phone = mysql_real_escape_string(stripslashes($phone)); 
      } 
     } else { 
      $phone = 'not specified'; 
     } 
     return $phone; 
    } 

내 문제는 $ 전화를 위해,이 비트를 뭔가를 입력하지 않는 경우이다 :

else { 
    $phone = 'not specified'; 
} 

아무것도 없기 때문에, 생략 될 것으로 보인다 데이터베이스에 입력됩니다.

다음은 적절한 데이터베이스 PHP입니다 : 아무것도 사용자가 입력되지 때마다 데이터베이스에 항목이 '지정되지 않은'것에 대한

//Insert Into Database 
$sql="INSERT INTO signups (phone) 
VALUES 
(''".$phone."')"; 

if (!mysql_query($sql)) { 
    die('Error: ' . mysql_error()); 
} 

내 의도입니다. 내가 뭔가 잘못하고 있니? 이 모든

더 중요한 것은, 당신이 테스트를 구동하고 조사해야 캐치 될 수 있도록

Answer 1

느슨한 비교기 == 때문에 문제가 발생합니다.

$phone = null (값이 게시되지 않은 경우처럼) $phone == ''은 true로 평가됩니다. ''은 (는) null과 비슷합니다. 당신은 엄격한 비교 ===을 사용하지 않거나 empty()을 사용하는 것이 더 좋습니다. 입력 값을 살균을 포함하여 변수를 (초기화 할 때 응용 프로그램에 더 안전하지 않은 데이터를 전달하는 것은 의미가 없기 때문에 나는 개인적으로, 앞까지 이러한 모든 물건을 처리 여기에 내가 제안 할 수있는 것입니다 :. 또한

$phone = ''; 
if (!empty($_POST['phone'])) { 
    $phone = mysql_real_escape_string(stripslashes(trim($_POST['phone']))); 
} 

$phone = validPhone($phone); 
if ($phone === '') { 
    $mistakes[] = 'Your phone number must contain only NUMERIC characters.'; 
} 

function validPhone($phone) { 
    if ($phone === '') { 
     return 'not specified'; 
    } else if (!ctype_digit(str_replace(' ', '', $phone))) { 
     return ''; 
    } else { 
     return $phone; 
    } 
} 

당신이해야 코드에서 사용되지 않는 mysql_* 함수를 mysqli 또는 PDO 함수로 대체하는 방법을 찾으십시오. $phone에 '지정되지 않음'이라는 반환 값을 사용하면 해당 조건을 트리거하지 않으므로 $mistake을 채우려면 코드를 변경해야합니다.

Answer 2

변경 if ($phone != '') {if (trim($phone) != '') {

Answer 3

if (!empty(trim($phone)) 

이 일치합니다하려면, 널 (null) "$ false로 전화 = 0"이다 개발은 개발 속도를 향상시키고 오류를 줄여줍니다. Checkout PHPUnit - https://github.com/sebastianbergmann/phpunit/