проблема с скриптом активации электронной почты

Question

Кто-нибудь хочет сделать снимок, почему эта активация электронной почты не работает? В принципе, я отправляю две переменные запроса url, которые хранят ключ активации и электронную почту как укусы. Я извлекаю их как list() и использую их как запросы db. Все объекты db работают нормально, когда тестируются, но когда я запускаю скрипт из электронной почты, он выводит эхо-комментарий: «учетная запись теперь активна ...», но когда я проверяю базу данных, она не обновляла эти вещи Я сказал это, например, keykey, status и userid.

Вот странная часть: когда у меня есть только один пользователь в базе данных, код выполняет так, как должен. Но как только я попробую добавить другого пользователя, он не обновляет новые записи в соответствии с инструкциями. Итак, мой вопрос: почему этот код работает только в том случае, если в базе данных есть один пользователь, а не когда есть еще один? Это не имеет никакого смысла. Было бы очень полезно помочь здесь.

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

я не знаю структуру вашей базы данных, но я предполагаю, что идентификатор является ключ автоматического увеличения. Я предполагаю, что проблема заключается в userid='1' этой части вашего кода, где вы всегда предполагаете, что идентификатор пользователя равен 1.

Вы также можете немного упростить код, просто запросив ключ активации непосредственно.

ie.

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

Кроме того, вы должны убедиться в том, что вы избегаете своего запросаString и электронной почты, чтобы предотвратить атаки впрыска базы данных. Вы можете использовать mysql_real_escape_string или альтернативу, основанную на вашем классе базы данных.