Состояние MySQLi не работает должным образом

Когда пользователь пытается зарегистрироваться, я проверяю первое табличное устройство, если есть какой-либо доступный device_id, я хочу предоставить пользовательскую пробную версию = 0, и если нет никакого device_id, я хочу вставить device_id в эту таблицу и хочу, чтобы пользовательская пробная версия = 1, но в настоящее время она всегда установите trial = 1 и вставьте device_id в таблицу device. Мой текущий код, как показано ниже

$serial = $POST["serial"];
    $fcm = $POST["fcm"];
    $trial  = 0;
    $trial_sql = "SELECT FROM device WHERE device_id = $serial";
        $trial_result = mysqli_query($conn, $trial_sql);
        if (mysqli_num_rows($trial_result) == 0) {
         $device_sql = "INSERT INTO device(device_id) VALUES('$serial')";
         if ($conn->query($device_sql)){
             $trial = 1;
            }   
    }
    $sql = "INSERT INTO user(name, email, password, device_id, trial, fcm) VALUES('$name', '$email', '$password', '$serial', $trial, '$fcm')";
    if ($conn->query($sql)) {
        $response["code"] = 1;
    }
    return json_encode($response);

Дайте мне знать, если кто-нибудь сможет исправить мою ошибку. Спасибо

Ваш код уязвим для атак SQL-инъекция. Вы должны использовать подготовленные операторы с привязанными параметрами через mysqli или PDO. У Эта почта есть несколько хороших примеров.

Alex Howansky 12.04.2018 16:25

@AlexHowansky Спасибо за ваш комментарий и предложение. Я сделаю это до того, как начну жить. Дайте мне знать, если вы можете помочь мне решить проблему с моим местонахождением. Спасибо

Khushi Patel 12.04.2018 16:27

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

Alex Howansky 12.04.2018 16:28

Если device_id является строкой, вам понадобятся кавычки вокруг значения при выборе (или используйте подготовленные операторы)

Nigel Ren 12.04.2018 16:36

@NigelRen Вы решили мою проблему. Спасибо

Khushi Patel 12.04.2018 16:47
Стоит ли изучать PHP в 2026-2027 годах?
Стоит ли изучать PHP в 2026-2027 годах?
Привет всем, сегодня я хочу высказать свои соображения по поводу вопроса, который я уже много раз получал в своем сообществе: "Стоит ли изучать PHP в...
Symfony Station Communiqué - 7 июля 2023 г
Symfony Station Communiqué - 7 июля 2023 г
Это коммюнике первоначально появилось на Symfony Station .
Оживление вашего приложения Laravel: Понимание режима обслуживания
Оживление вашего приложения Laravel: Понимание режима обслуживания
Здравствуйте, разработчики! В сегодняшней статье мы рассмотрим важный аспект управления приложениями, который часто упускается из виду в суете...
Установка и настройка Nginx и PHP на Ubuntu-сервере
Установка и настройка Nginx и PHP на Ubuntu-сервере
В этот раз я сделаю руководство по установке и настройке nginx и php на Ubuntu OS.
Коллекции в Laravel более простым способом
Коллекции в Laravel более простым способом
Привет, читатели, сегодня мы узнаем о коллекциях. В Laravel коллекции - это способ манипулировать массивами и играть с массивами данных. Благодаря...
Как установить PHP на Mac
Как установить PHP на Mac
PHP - это популярный язык программирования, который используется для разработки веб-приложений. Если вы используете Mac и хотите разрабатывать...
0
5
24
2
Перейти к ответу Данный вопрос помечен как решенный

Ответы 2

SELECT * FROM device 

Или же

SELECT columnA, columnB FROM device

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

Привет! Это все та же проблема.

Khushi Patel 12.04.2018 16:33

Вам нужно сделать "SELECT device_id FROM ..." в этом первом, затем

Nerdi.org 12.04.2018 16:34

Я давно не использовал неподготовленные операторы. Я придерживаюсь подготовки, связывания параметров, связывания результата и выполнения. Это позволяет очень легко увидеть, где все работает и работает. wbr.bz/QueryPro - хорошее место для начала

Nerdi.org 12.04.2018 16:36
Ответ принят как подходящий

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

$trial_sql = "SELECT * FROM device WHERE device_id = '$serial'";

Цитаты не были бы проблемой, если бы вы использовали заранее подготовленные операторы.

Переломить ситуацию против обучения / пропаганды небрежных и опасных методов кодирования. Если вы отправите ответ без подготовленных заявлений вы можете подумать об этом перед публикацией. Дополнительно более ценный ответ исходит из демонстрации OP правильного метода.

Jay Blanchard 12.04.2018 16:55

Другие вопросы по теме