私のPHPスクリプトはmysqlから最初の行だけを読み込み、残りの行は一致するかどうかを確認しません

Question

私はユーザーがクラブに登録するためのスクリプトを書こうとしています。それが重複をチェックするはずの部分に達するまでうまく動作します。私は何がうまくいかないのか分かりません。助けてください！！！ は、スクリプトの中で意味がありません。かなりの数のものが1に対して複数のクエリがあります

<?php 
    mysql_connect ("sqlhost", "username", "password") or die(mysql_error()); 

    mysql_select_db ("databasename") or die(mysql_error()); 

    $errormsgdb = ""; 
    $errordb = "Sorry but that "; 
    $error1db = "Name"; 
    $error2db = "email"; 
    $error3db = "mobile number"; 
    $errordbe = " is already registered"; 
    $pass1db = "No Matching Name"; 
    $pass2db = "No Matching Email"; 
    $pass3db = "No Matching Mobile"; 
    $errorcount = 0; 
    $qResult = mysql_query ("SELECT * FROM table"); 
    $nRows = mysql_num_rows($qResult); 
    for ($i=1; $i< $nRows+1; $i++){ 

$result = mysql_query("SELECT id,fname,lname,dob,email,mobile,agree,code,joindate FROM table WHERE fname = '$ffname"); 
if ($result > 0) { 
$errorcount = $errorcount++; 
$passdb = 0; 
$errormsgdb = $error1db; 
echo "<div class=\"box red\">$errordb $errormsgdb 
} else { 
$pass = 1; 
$errormsgdb = $pass1db; 
echo "<div class=\"box green\">$errormsgdb</div><br />"; 
} 



    //--------------- Check if DB checks returned errors ------------------------------------> 
     if($errorcount <= 0){ 
     $dobp = $_REQUEST['day'].'/'.$_REQUEST['month'].'/'.$_REQUEST['year']; 
     $dob = $_REQUEST['year'].$_REQUEST['month'].$_REQUEST['day']; 
     //header('Location: thankyou.php?ffname='.$ffname.'&flname='.$flname.'&dob='.$dob.'&femail='.$femail.'&fmobile='.$fmobile.'&agree='.$agree.'&code='.$code.'&dobp='.$dobp); 
     echo "<div class='box green'>Form completed! Error Count = $errorcount</div>"; 
     } else { 
     echo "<div class='box red'>There was an Error! Error Count = $errorcount</div>"; 
      } 
    } 
    ?> 

Answer 1

、事前にありがとうございます！データを照会するだけではなく、一致するかどうかを確認するのはなぜですか？各ユーザーをループしてチェックするのではなく、 ...しかし、私は、あなたは動作しません

$errorcount = $errorcount++; 

これを行うあなたの主な問題は

$errorcount = " "; 

だと思う...あなたはそれが動作しませんインクリメント文字列として$errorcountを作成しました。あなたは数としてそれを初期化する必要があります

$errorcount = 0; 

が、その後インクリメントする：あなたは奇妙な方法でデータベースを使用している

$errorcount++; 

Example of it not working and then working here

Answer 2

。

最初は、テーブル内のすべてのデータを照会しています。

そしてループの中で、テーブル内のすべての行を再度照会していますが、この行は行です。これはすべて非常に非効率的です。

何をやるべきことはすべての重複

Answer 3

を求めるために、直接データベースに照会され、あなたの答えinvaluble特にManseUKすべてのあなたの助けのManseUKとヒキガエルをありがとうございました。私が解決した解決策は、だれかがより良い、あるいはより効率的な解決策を持っていれば腹を立てることです。

<?php 
mysql_connect ("mysqlhost", "username", "password") or die(mysql_error()); 

mysql_select_db ("database") or die(mysql_error()); 

$errormsgdb = ""; 
$errordb = "Sorry but that "; 
$error1db = "Name"; 
$error2db = "email"; 
$error3db = "mobile number"; 
$errordbe = " is already registered"; 
$pass1db = "No Matching Name"; 
$pass2db = "No Matching Email"; 
$pass3db = "No Matching Mobile"; 

// Formulate Name Query 
$queryname = sprintf("SELECT * FROM table 
WHERE fname='%s' AND mname='%s' AND lname='%s'", 
mysql_real_escape_string($ffname), 
mysql_real_escape_string($fmname), 
mysql_real_escape_string($flname)); 

// Perform Name Query 
$resultname = mysql_query($queryname); 

// Check result 
if (!$resultname) { 
    $message = 'Invalid query: ' . mysql_error() . "\n"; 
    $message .= 'Whole query: ' . $queryname; 
    die($message); 
} 

// Use result 
while ($row = mysql_fetch_assoc($resultname)) { 
    $dbfullname = strtoupper($row['fname'].$row['mname'].$row['lname']); 
    $fullname = strtoupper($ffname.$fmname.$flname); 
    if ($dbfullname == $fullname){ 
     $passdb = 0; 
     $errormsgdb = $error1db; 
     echo "<div class=\"box red\">$errordb $errormsgdb $errordbe</div><br />"; 
    } else { 
     $pass = 1; 
     $errormsgdb = $pass1db; 
     echo "<div class=\"box green\">$errormsgdb</div><br />"; 
    } 
} 

// Formulate Email Query 
$queryemail = sprintf("SELECT * FROM table 
WHERE email='%s'", 
mysql_real_escape_string($femail)); 

// Perform Email Query 
$resultemail = mysql_query($queryemail); 

// Check result 
if (!$resultemail) { 
    $message = 'Invalid query: ' . mysql_error() . "\n"; 
    $message .= 'Whole query: ' . $queryemail; 
    die($message); 
} 

// Use result 
while ($row = mysql_fetch_assoc($resultemail)) { 
    $cemail = strtoupper($femail); 
    $dbemail = strtoupper($row['email']); 
    if ($cemail != $dbemail) { 
     $passdb = 1; 
     $errormsgdb = $pass2db ; 
     echo "<div class=\"box green\">$errormsgdb</div><br />"; 
    } else { 
     $passdb = 0; 
     $errormsgdb = $error2db; 
     echo "<div class=\"box red\">$errordb $errormsgdb $errordbe</div>"; 
    } 
} 

// Formulate Mobile Query 
$querymobile = sprintf("SELECT * FROM table 
WHERE mobile='%s'", 
mysql_real_escape_string($fmobile)); 

// Perform Mobile Query 
$resultmobile = mysql_query($querymobile); 

// Check result 
if (!$resultmobile) { 
    $message = 'Invalid query: ' . mysql_error() . "\n"; 
    $message .= 'Whole query: ' . $queryemail; 
    die($message); 
} 

// Use result 
while ($row = mysql_fetch_assoc($resultmobile)) { 
$cmobile = ereg_replace("[^0-9]", "", $fmobile); 
    if ($cmobile != $row['mobile']) { 
     $passdb = 1; 
     $errormsgdb = $pass3; 
     echo "<div class=\"box green\">$errormsgdb</div><br />"; 
    } else { 
     $passdb = 0; 
     $errormsgdb = $error3db; 
     echo "<div class=\"box red\">$errordb $errormsgdb $errordbe</div>"; 
    } 
} 
?> 

は