PHP / MySQL не будет вставлять и отображать как следует? - PullRequest
0 голосов
/ 15 марта 2011

Что делает (или должен делать) следующий скрипт:

  • Подключается к БД
  • Включает функцию для создания 5-значного кода
  • Пользователь вводит своиадрес электронной почты
  • Проверяет, является ли это действительный адрес электронной почты
  • Вставляет электронное письмо в столбец 'email'
  • Проверяет, существует ли электронная почта, если да, сообщите пользователю об этом и прервитеscript
  • Запускает функцию для создания 5-значного кода
  • Проверяет столбец «уникальный_код», если он уже существует, если это так, цикл из функции создания 5-значного кода
  • Есливсе верно, скрыть форму и отобразить (ajax из отдельного JS) спасибо div
  • Показать уникальный код для пользователя

Все работает, однако unique_code невставлен в БД и не отображается, когда отображается «Спасибо!».

Что я делаю не так и что нужно изменить?

Спасибо!

Код

    <?php

    require "includes/connect.php";

    function generateCode($length = 5) {

    $characters = 'bcdfghjkmnpqrstvwxyz';

    $string = '';
    for ($i = 0; $i < $length; $i++) {
        $string .= $characters[rand(0, strlen($characters) - 1)];
    }

    return $string;

}


$msg = '';

if($_POST['email']){

    // Requested with AJAX:
    $ajax = ($_SERVER['HTTP_X_REQUESTED_WITH']  == 'XMLHttpRequest');

    try{
        //validate email
        if(!filter_input(INPUT_POST,'email',FILTER_VALIDATE_EMAIL)){
            throw new Exception('Invalid Email!');
        }

        //insert email
        $mysqli->query("INSERT INTO coming_soon_emails
                        SET email='".$mysqli->real_escape_string($_POST['email'])."'");

        //if already exists in email column
        if($mysqli->affected_rows != 1){
            throw new Exception('You are already on the notification list.');
        }

        if($ajax){
            die('{"status":1}');
        }

        //start creating unique 5 digit code
        $unique_code = "";
        $inserted = false;

        // Keep looping until we've inserted a record
        while(!$inserted) {

        // Generate a code
        $unique_code = generateCode();

        // Check if it exists
        if ($result = $mysqli->query("SELECT unique_code FROM coming_soon_emails WHERE unique_code = '$unique_code'")) {

        // Check no record exists
        if ($result->num_rows == 0) {

            // Create new record
            $mysqli->query("INSERT INTO coming_soon_emails (email,unique_code) VALUES ('" . $mysqli->real_escape_string($_POST['email']) . "','$unique_code')");

            // Set inserted to true to ext loop
            $inserted = true;

            // Close the result object
            $result->close();

        }
        } else {

        // Quit if we can't check the database
        die('Something went wrong with select');
    }   
}

    }

    catch (Exception $e){

        if($ajax){
            die(json_encode(array('error'=>$e->getMessage())));
        }

        $msg = $e->getMessage();        
    }
}
?>


<!DOCTYPE html>
<html>
<head>
<meta http-equiv="Content-Type" content="text/html; charset=utf-8" />
<title>example</title>

<link rel="stylesheet" type="text/css" href="css/styles.css" />

</head>

<body>

<div id="container">

    <form id="form" method="post" action="">
        <input type="text" id="email" name="email" value="<?php echo $msg?>" />
        <input type="submit" value="Submit" id="submitButton" />
    </form>

    <div id="thankyou">
    Thank you! <?php echo $unique_code;?></p>
    </div>


</div>
<script src="http://ajax.googleapis.com/ajax/libs/jquery/1.4.3/jquery.min.js"></script>
<script src="js/script.js"></script>
</body>
</html>

Ответы [ 2 ]

2 голосов
/ 15 марта 2011

Кажется, вы используете ключевое слово private вне определения класса, что недопустимо.

0 голосов
/ 15 марта 2011

Есть ли у вас первичный ключ для 'email' в таблице coming_soon_emails?Поскольку вы уже вставили одну запись для данного адреса электронной почты, это не позволит вам вставить вторую запись с уникальным значением.

Почему бы не выполнить ОБНОВЛЕНИЕ вместо ВСТАВКИ, если вы определили уникальный ключ

Добро пожаловать на сайт PullRequest, где вы можете задавать вопросы и получать ответы от других членов сообщества.
...