Из-за чего может происходить два запроса к БД вместо одного?

Question

[Torento20345]

Из-за чего пхп может 2 раза сделать запрос к БД?
Так получается, что 1 из 1000 раз может случайно произойти так, что два раза добавится одна и та же запись в БД.
Как найти проблему и избежать подобного?

В начале скрипта стоит проверка на существующую запись по такому же имени.
Значит получается так, что делается два запроса к БД, а не два срабатывания скрипта повторно.

Comments

[Slava Rozhnev]

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

[Torento20345]

Slava Rozhnev, если я добавлю UNIQUE INDEX, то без проверки в пхп к sql не буду добавляться повторно значения?
или всё же проверка должна быть?

[Slava Rozhnev]

https://phpize.online/?phpses=21815e840ccb4456a97a...

Answer 1

[Алексей Уколов]

Не получается. Если в вашем скрипте делается только один запрос на вставку, то никакими волшебными силами электричества вторая запись в нём появиться не может - кто-то запускает скрипт одновременно второй раз и проверка в нём успевает пройти до того, как запись появится в первом.

Comments

[Torento20345]

function sql_check($col, $table, $where) {

    global $mysqli;

    $query = "SELECT $col FROM $table $where";
    if($sql = $mysqli->prepare($query)) {
        $sql->execute();
    } else {
        $error = $mysqli->errno . ' ' . $mysqli->error;
        echo $error;
    }
    
    $sqlResult = $sql->get_result();

    if($sqlResult->num_rows) {
        echo "false";
        die();
    }

}

sql_check('name', 'products', "WHERE url=\"$productURL\"");

А sql с добавлением уже ниже идёт.

[Алексей Уколов]

И что?

[FanatPHP]

а можно спросить, за чем вот это вот всё - prepare, execute?
не проще написать query()?

[Torento20345]

FanatPHP, читал в книге про подготовленные запросы, что надо писать так из-за надежности, что другие запросы в инпутах не сработают, если запрос подготовленный.

[Torento20345]

Алексей Уколов, так, а шо делать?
Я пример кода привел, как вдруг я что-то не так написал.

[FanatPHP]

Torento20345, prepare - это не волшебная палочка, которая вдруг каким-то образом сделает "другие запросы из инпутов" нерабочими.
из-за "надёжности" запрос надо писать совсем по-другому, заменяя все переменные в нем на знаки вопроса, и потом привязывая их через bind_param. Только в этом случае запрос будет надёжным и безопасным.

а то, что у вас сейчас, вообще никакой безопасности не обеспечивает и по факту ничем не отличается от вызова query()

[Алексей Уколов]

Torento20345, в таблицу уникальный индекс добавьте или мьютекс на весь свой код.