Проблема с проверкой SQL

почему проверка не работает ? $uid=$intfi; $inbox_udate=$eml['data']['udate']; $inbox_from=$conn->real_escape_string($eml['data'][0]['from']['name'].' '.$eml['data'][0]['from']['address']); $subject_text=$conn->real_escape_string($eml['data'][0]['subject']); $body_text=$conn->real_escape_string(strip_all_html($eml['data'][0]['message'])); $company_email_id=8; $date_create=$formatted;//$parser->getHeader('date'); $filename_eml=$item; $check_sql = " SELECT id FROM archive_mail WHERE filename_eml = '$filename_eml' AND uid = $uid LIMIT 1 "; $result = $conn->query($check_sql); if ($result && $result->num_rows > 0) { //echo "⚠️ Пропущен (уже в базе): $filename_eml\n"; //continue; }else{ $sql = " INSERT INTO archive_mail ( uid, inbox_udate, inbox_from, subject_text, body_text, company_email_id, date_create, filename_eml ) VALUES ( '$uid', '$inbox_udate', '$inbox_from', '$subject_text', '$body_text', '$company_email_id', '$date_create', '$filename_eml' ) "; if ($conn->query($sql)) { //$info_0p[] = "✅ Импортирован: $filename_eml\n"; } else { $info_0p[] = "❌ Ошибка при импорте $filename_eml: " . $conn->error . "\n"; file_put_contents('../_!!!_DEBUG-arhiveMAIL.txt',$info_0p,1,FILE_APPEND); } $intfi++; }

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

🔍 1. Типы данных и кавычки в SQL

В запросе:

sql
SELECT id FROM archive_mail
WHERE filename_eml = '$filename_eml' 
  AND uid = $uid
LIMIT 1

$filename_eml — строка, и вы обернули её в кавычки — ОК
$uid — число, если это действительно число, кавычки не нужны — ОК

🔧 Но: если $uid содержит строку или даже пробелы (например, '123 '), сравнение не сработает.

✅ Проверка:

php
var_dump($filename_eml);
var_dump($uid);

🧼 2. Необработанные пробелы / регистр

Если в базе filename_eml хранится, например, 'file1.eml', а в переменной пришло ' file1.eml ' или 'FILE1.EML' — сравнение не сработает.

✅ Решение:

php
$filename_eml = trim(strtolower($filename_eml));

И аналогично в SQL можно использовать LOWER(filename_eml) = '$filename_eml' — но лучше нормализовать до запроса.

✅ 3. Действительно ли в базе уже есть запись?

Проверь вручную через SQL:

sql
SELECT * FROM archive_mail WHERE filename_eml = 'значение' AND uid = значение

Если не находит — возможно, данные действительно уникальны или вы сравниваете с неверными значениями.

🛡 4. SQL-инъекция / ошибка запроса

Если $filename_eml содержит ', \, или другие символы, и они не экранированы — это может нарушать SQL.

✅ Вы экранируете $filename_eml при вставке (real_escape_string) — но не экранируете при SELECT.

Добавьте:

php
$filename_eml = $conn->real_escape_string($filename_eml);

🧪 5. Промежуточный debug

Перед $conn->query($check_sql) выведите SQL:

php
echo "DEBUG: $check_sql\n";

Попробуйте этот SQL прямо в phpMyAdmin или консоли — сработает ли он там?

🔁 6. Возможный дубликат, но `uid` отличается

Может быть, файл filename_eml совпадает, а uid — разный (например, при повторной загрузке).

✅ Решение: Если уникальность по filename_eml, проверьте только его:

php
WHERE filename_eml = '$filename_eml'

Или, если уникальность по обоим полям, удостоверьтесь, что uid тоже совпадает.

💡 Заключение

Добавьте в код отладку:

php
echo "Проверка: SELECT id FROM archive_mail WHERE filename_eml = '$filename_eml' AND uid = $uid\n";

И после $result = $conn->query($check_sql) добавьте:

php
if (!$result) {
    echo "❌ SQL Error: " . $conn->error . "\n";
}

Если хочешь — можешь прислать конкретное значение $filename_eml, $uid и результат ручного запроса к БД — помогу точнее.

переделать на prepared statements

Отлично! Использование prepared statements (подготовленных выражений) повысит безопасность и стабильность кода, особенно для защиты от SQL-инъекций.

Вот переписанный участок твоего кода с использованием prepared statements для проверки и вставки в базу:

✅ Переписанный код с `prepare`:

php
$uid = $intfi;
$inbox_udate = $eml['data']['udate'];
$inbox_from = $eml['data'][0]['from']['name'] . ' ' . $eml['data'][0]['from']['address'];
$subject_text = $eml['data'][0]['subject'];
$body_text = strip_all_html($eml['data'][0]['message']);
$company_email_id = 8;
$date_create = $formatted;
$filename_eml = $item;

// Подготовка запроса для проверки
$check_sql = "
    SELECT id FROM archive_mail
    WHERE filename_eml = ? AND uid = ?
    LIMIT 1
";
$check_stmt = $conn->prepare($check_sql);
$check_stmt->bind_param("si", $filename_eml, $uid);
$check_stmt->execute();
$check_stmt->store_result();

if ($check_stmt->num_rows > 0) {
    // Уже в базе
    // echo "⚠️ Пропущен (уже в базе): $filename_eml\n";
    // continue;
} else {
    // Подготовка запроса на вставку
    $insert_sql = "
        INSERT INTO archive_mail (
            uid, inbox_udate, inbox_from, subject_text, body_text, company_email_id, date_create, filename_eml
        ) VALUES (?, ?, ?, ?, ?, ?, ?, ?)
    ";
    $insert_stmt = $conn->prepare($insert_sql);
    $insert_stmt->bind_param(
        "iissssis",
        $uid,
        $inbox_udate,
        $inbox_from,
        $subject_text,
        $body_text,
        $company_email_id,
        $date_create,
        $filename_eml
    );

    if ($insert_stmt->execute()) {
        // Успешно импортирован
        // $info_0p[] = "✅ Импортирован: $filename_eml\n";
    } else {
        $info_0p[] = "❌ Ошибка при импорте $filename_eml: " . $conn->error . "\n";
        file_put_contents('../_!!!_DEBUG-arhiveMAIL.txt', $info_0p, FILE_APPEND);
    }

    $insert_stmt->close();
}

$check_stmt->close();
$intfi++;

🛠 Примечания:

Используются ? в SQL вместо переменных, а данные передаются через bind_param().
Типы для bind_param():
- i — integer (uid, inbox_udate, company_email_id)
- s — string (все остальные)
store_result() используется, чтобы можно было проверить количество строк через $check_stmt->num_rows.

Если нужно, могу также обернуть это в функцию или класс — скажи.

что это "iissssis",

Отличный вопрос!

Строка "iissssis" в методе:

php
$insert_stmt->bind_param("iissssis", ...);

— это форматная строка типов данных, которая указывает bind_param(), какие типы данных будут переданы в SQL-запрос.

Каждая буква соответствует одному параметру в списке, и строго соответствует порядку аргументов. Вот расшифровка:

🔠 Форматные символы:

Символ	Тип данных PHP	Описание
`i`	integer	Целое число
`s`	string	Строка
`d`	double (float)	Число с плавающей точкой
`b`	blob	Двоичные данные (blob/файл и т.п.)

📦 Пример `"iissssis"` в твоём случае:

php
bind_param(
  "iissssis",
  $uid,              // i — integer
  $inbox_udate,      // i — integer (timestamp)
  $inbox_from,       // s — string
  $subject_text,     // s — string
  $body_text,        // s — string
  $company_email_id, // s — ?? но по смыслу должно быть i!
  $date_create,      // s — string (дата)
  $filename_eml      // s — string
);

⚠️ Важно: `company_email_id` — это ID, значит должно быть `i`, не `s`.

✅ Итог:

Правильная строка должна быть:

php
"iiisssis"

Или более понятно с пояснением:

php
bind_param(
  "iiisssis",         // формат типов
  $uid,               // i
  $inbox_udate,       // i
  $inbox_from,        // s
  $subject_text,      // s
  $body_text,         // s
  $company_email_id,  // i ← поправлено!
  $date_create,       // s
  $filename_eml       // s
);

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

Если ты хочешь, я могу проверить и другие части на соответствие типам.

а другая запись синтаксиса есть ?

Да, в mysqli синтаксис bind_param("types", ...) — обязательный при использовании подготовленных выражений, но есть альтернативы с другими расширениями или подходами, если ты хочешь избежать этой громоздкой строки "iiisssis".

Вот краткий обзор альтернатив:

✅ 1. `mysqli` с `bind_param` — текущий стандарт (нет другой формы)

php
$stmt = $conn->prepare("INSERT INTO table (a, b) VALUES (?, ?)");
$stmt->bind_param("is", $a, $b);
$stmt->execute();

⚠️ Здесь обязательно указывать "is" — других вариантов в mysqli нет. Но есть обходные пути 👇

✅ 2. Использовать `PDO` вместо `mysqli`

Если хочешь избавиться от "iiisssis" — перейди на PDO. Там ты можешь привязывать параметры по имени, и PHP сам определит типы данных:

php
$pdo = new PDO('mysql:host=localhost;dbname=test', 'user', 'pass');

$stmt = $pdo->prepare("
    INSERT INTO archive_mail (
        uid, inbox_udate, inbox_from, subject_text, body_text, company_email_id, date_create, filename_eml
    ) VALUES (
        :uid, :inbox_udate, :inbox_from, :subject_text, :body_text, :company_email_id, :date_create, :filename_eml
    )
");

$stmt->execute([
    ':uid' => $uid,
    ':inbox_udate' => $inbox_udate,
    ':inbox_from' => $inbox_from,
    ':subject_text' => $subject_text,
    ':body_text' => $body_text,
    ':company_email_id' => $company_email_id,
    ':date_create' => $date_create,
    ':filename_eml' => $filename_eml
]);

🔹 Плюсы:

Без "iiisssis" — типы автоматически определяются.
Более читаемо (по именам).
Удобно масштабируется.

✅ 3. Автоматическая генерация `bind_param()` через вспомогательную функцию

Если остаёшься на mysqli, можно сделать универсальную функцию, которая сама определяет "isss" и вызывает bind_param():

php
function bind_params_dynamic($stmt, $params) {
    $types = '';
    foreach ($params as $param) {
        if (is_int($param)) $types .= 'i';
        elseif (is_float($param)) $types .= 'd';
        elseif (is_string($param)) $types .= 's';
        else $types .= 'b';
    }
    $stmt->bind_param($types, ...$params);
}

Использование:

php
$stmt = $conn->prepare("INSERT INTO table (a, b) VALUES (?, ?)");
bind_params_dynamic($stmt, [$a, $b]);
$stmt->execute();

💡 Вывод

Метод	Требует `"iiisssis"`	Поддержка именованных параметров	Автоопределение типов
`mysqli` + `bind_param`	✅ Да	❌ Нет	❌ Нет
`PDO`	❌ Нет	✅ Да	✅ Да
`mysqli` + helper	✅ Да (авто)	❌ Нет	✅ Да (через PHP)

Если интересует — могу помочь переписать твой код на PDO.

ChatGPT может допускать ошибки. Рекомендуем проверять важную информацию.

Вы сказали:

ChatGPT сказал:

🔍 1. Типы данных и кавычки в SQL

🧼 2. Необработанные пробелы / регистр

✅ 3. Действительно ли в базе уже есть запись?

🛡 4. SQL-инъекция / ошибка запроса

🧪 5. Промежуточный debug

🔁 6. Возможный дубликат, но uid отличается

💡 Заключение

Вы сказали:

ChatGPT сказал:

✅ Переписанный код с prepare:

🛠 Примечания:

Вы сказали:

ChatGPT сказал:

🔠 Форматные символы:

📦 Пример "iissssis" в твоём случае:

⚠️ Важно: company_email_id — это ID, значит должно быть i, не s.

✅ Итог:

Вы сказали:

ChatGPT сказал:

✅ 1. mysqli с bind_param — текущий стандарт (нет другой формы)

✅ 2. Использовать PDO вместо mysqli

✅ 3. Автоматическая генерация bind_param() через вспомогательную функцию

💡 Вывод

🔁 6. Возможный дубликат, но `uid` отличается

✅ Переписанный код с `prepare`:

📦 Пример `"iissssis"` в твоём случае:

⚠️ Важно: `company_email_id` — это ID, значит должно быть `i`, не `s`.

✅ 1. `mysqli` с `bind_param` — текущий стандарт (нет другой формы)

✅ 2. Использовать `PDO` вместо `mysqli`

✅ 3. Автоматическая генерация `bind_param()` через вспомогательную функцию