電子メールアクティベーションスクリプトの問題

Question

誰かがこの電子メールのアクティベーションが機能しない理由を知りたいですか？基本的には、アクティベーションキーと電子メールをスティールとして格納する2つのURLクエリ変数を送信しています。私はそれらをlist()として抽出し、それらをdbクエリとして使用しています。すべてのdbオブジェクトは正常に動作していますが、電子メールからスクリプトを実行すると、「アカウントは現在アクティブです...」というエコー文が表示されますが、データベースをチェックすると、私はそれをすなわちアクティベーションキー、ステータス、およびユーザーIDに伝えました。

奇妙な部分は次のとおりです。データベースに1人のユーザーしかいない場合、コードは必要なだけ実行されます。しかし、別のユーザーを追加しようとすると、指示どおりに新しいレコードは更新されません。だから私の質問：なぜこのコードは、データベースに1人のユーザーがいて、それ以上の人がいるときには機能しないのですか？それは意味をなさない。ここで本当に助けに感謝します。

include $_SERVER['DOCUMENT_ROOT']. '/video_dating/includes/Database.php'; 
list($queryString,$email) = explode('&', $_SERVER['QUERY_STRING']); 

print_r($email); 
$dbname = new Database(); 
$dbname->Query('select activationkey, id, email from users'); 
$userinfo = $dbname->Getdata(); 
foreach ($userinfo as $users) 
{ 

    if ($queryString == $users['activationkey']) 
    { 
    $dbname->Query("update users set activationkey='', status='activated', userid='1' where email='$email'"); 
    echo "Thank you for registering. Your account is now active. Please login any time."; 

    } 
    else 
     echo "Sorry, your account was not activated."; 
} 

Answer 1

フィールドuseridはキーとして定義できますか？はい、それは一意である必要がありますが、あなたは常に1

に設定している場合は、UPDATEからその部分を削除してください：

, userid='1' 

Answer 2

私はあなたのデータベースの構造を知らないが、私は、ユーザーIDがあると仮定します自動インクリメントキー。私は問題があると推測していますuserid='1'あなたのコードのこの部分は、常にあなたのIDが1であると仮定します。

また、アクティベーションキーを直接照会するだけでコードをかなり単純化することもできます。

ie。

$dbname->Query('select activationkey, id, email from users WHERE activationkey="'.$queryString.'" and email = "'.$email.'"'); 

さらに、あなたは本当にあなたがデータベースインジェクション攻撃を防ぐために、あなたのqueryStringと電子メールをエスケープされていることを確認しなければなりません。 mysql_real_escape_stringまたはデータベースクラスに基づく代替を使用することができます。