私のPHP登録スクリプトに問題がある

Question

このスクリプトは、メールとパスワードの長さをチェックする一連の "if"ステートメントに当たるとハングアップするようです。これらのステートメントを削除すると、データがdbに正しく挿入されます。

<?php 

ob_start(); 
session_start(); 

if (!empty($_POST['email']) && !empty($_POST['password']) && !empty($_POST['confirmpassword'])) { 

$email = strip_tags($_POST['email']); 
$password = md5(strip_tags($_POST['password'])); 
$confirmpassword = md5(strip_tags($_POST['confirmpassword'])); 

$errors = array(); 

if (strlen($email) < 6) { 
    $errors[] = "Email too short."; 
} 
if (strlen($email) > 25) { 
    $errors[] = "Email too long."; 
} 
if (strlen($password) < 2) { 
    $errors[] = "Password too short."; 
} 
if (strlen($password) > 25) { 
    $errors[] = "Password too short."; 
} 
if ($password !== $confirmpassword) { 
    $errors[] = "Passwords do not match."; 
} 

if (count($errors) == 0) { 
    // Include database config file then connect to database 
    require('db_config.php'); 
    $connection = mysql_connect(DB_HOST,DB_USERNAME,DB_PASSWORD) or die("Database Connection Error"); 
    $database = mysql_select_db(DB_NAME) or die("No Database"); 

    // Create query 
    $query = "INSERT INTO bah_register VALUES ('','$email','$password')"; 

    // Query database and 
    mysql_query($query); 

    // Success message 
    echo "Thanks for signing up!"; 

} else { 
    foreach ($errors as $error) { 
     echo $error . "<br />"; 
    } 
} 
} 

?> 

Answer 1

あなたの問題は、長さをチェックする前にパスワードをmd5 INGのしていることです。これにより、パスワードが32文字になります。これは制限よりも大きく、エラーが発生します。

Answer 2

私はあなたのコードに問題があるかわからないが、あなたの電子メールのためにあなたは、このようなものを使用して検討するかもしれない：

if(!preg_match('/^[^@]+@[a-zA-Z0-9._-]+\.[a-zA-Z]+$/', $email)){ 
    $errors[] = "Email is not valid."; 
} 

多くの電子メールは、より長いです25文字。

Answer 3

あなたはstrlen($password) > 25をチェックすると、パスワードは長い25個のシンボルよりもmd5 hashsumです。おそらく

Answer 4

エラー配列とforeachのがイースリーコード以下
適切な電子メールの検証がfilter_var機能で行うことができます echo implode('<br />', $errors);

strip_tagsの機能に置き換えることができ、パスワードの元の値をチェックしたいですおそらくパスワードの一部が削除されます。 «<私の>スーパーパスワード»