Написал код чтобы он сверял с бд. Если файл не существует в бд он удаляется с хостинга?

Question

[maluky]

Здравствуйте. Помогите с написанием кода. Написал код чтобы он сверял с бд, если файл не существует в бд он удаляется с хостинга. А он удаляет все файлы с папки. Помогите поправить. Только учусь.

// Параметры подключения к базе данных
$servername = "localhost";
$username = "username";
$password = "password";
$dbname = "username";
$conn = new mysqli($servername, $username, $password, $dbname);

// Проверка подключения к базе данных
if ($conn->connect_error) {
    die("Ошибка подключения к базе данных: " . $conn->connect_error);
}

// Выполнение запроса к базе данных для получения списка всех файлов
$sql = "SELECT img_images FROM img_ads";
$result = $conn->query($sql);

// Получение списка файлов в папке
$imagesFolder = "media/images/";
$files = glob($imagesFolder . "*.webp");

// Проход по каждому файлу в папке
foreach ($files as $file) {
    // Проверка, существует ли файл в базе данных
    $filename = basename($file);
    $fileInDb = false;

    if ($result->num_rows > 0) {
        while ($row = $result->fetch_assoc()) {
            if ($row['img_images'] == $filename) {
                $fileInDb = true;
                break;
            }
        }
    }

    // Удаление файла с хостинга, если он не существует в базе данных
    if (!$fileInDb) {
        if (unlink($file)) {
            echo "Файл удален: " . $file . "<br>";
            // Запись результатов в файл log.txt
            file_put_contents("log/log.txt", "Удален файл: " . $file . PHP_EOL, FILE_APPEND);
        } else {
            echo "Не удалось удалить файл: " . $file . "<br>";
            // Запись результатов в файл log.txt
            file_put_contents("log/log.txt", "Не удалось удалить файл: " . $file . PHP_EOL, FILE_APPEND);
        }
    }

    // Сброс указателя результатов запроса перед новой итерацией
    $result->data_seek(0);
}

// Закрытие подключения к базе данных
$conn->close();

Answer 1

[DooX]

// Сброс указателя результатов запроса перед новой итерацией
    $result->data_seek(0);

Привет. При каждой итерации цикла foreach он сбрасывает указатель результатов запроса $result->data_seek(0), что приводит к повторной обработке всех строк результата запроса. Это приводит к удалению всех файлов, так как они поочередно не находятся в базе данных на каждой итерации.

Попробуй по другому.

// Параметры подключения к базе данных
$servername = "localhost";
$username = "username";
$password = "password";
$dbname = "username";
$conn = new mysqli($servername, $username, $password, $dbname);

// Проверка подключения к базе данных
if ($conn->connect_error) {
    die("Ошибка подключения к базе данных: " . $conn->connect_error);
}

// Выполнение запроса к базе данных для получения списка всех файлов
$sql = "SELECT img_images FROM img_ads";
$result = $conn->query($sql);

// Получение списка файлов из базы данных
$filesInDatabase = array();
if ($result->num_rows > 0) {
    while ($row = $result->fetch_assoc()) {
        $filesInDatabase[] = $row['img_images'];
    }
}

// Получение списка файлов в папке
$imagesFolder = "media/images/";
$files = glob($imagesFolder . "*.webp");

// Проход по каждому файлу в папке
foreach ($files as $file) {
    // Проверка, существует ли файл в базе данных
    $filename = basename($file);

    if (!in_array($filename, $filesInDatabase)) {
        if (unlink($file)) {
            echo "Файл удален: " . $file . "<br>";
            // Запись результатов в файл log.txt
            file_put_contents("log/log.txt", "Удален файл: " . $file . PHP_EOL, FILE_APPEND);
        } else {
            echo "Не удалось удалить файл: " . $file . "<br>";
            // Запись результатов в файл log.txt
            file_put_contents("log/log.txt", "Не удалось удалить файл: " . $file . PHP_EOL, FILE_APPEND);
        }
    }
}

// Закрытие подключения к базе данных
$conn->close();

Comments

[maluky]

Спасибо попробую

[Ипатьев]

Это приводит к удалению всех файлов, так как они поочередно не находятся в базе данных на каждой итерации.

Не приводит. Код автора делает то же самое, что что и ваш, причем так же неоптимально.
И надо не еще один вариант говнокода писать, а учить автора отлаживать свой код

[DooX]

Ипатьев, можете написать более верный код.

и указать на ошибки, а то со стороны говорить что что-то не так умеет каждый.

[DooX]

maluky, так чтоб дядя Куратор PHP не ругался. ты скажи а в БД у тебя с расширением имя файла хранится?

Так же проверил все работает.

[maluky]

DooX, да, есть хранение с расширением .webp

[maluky]

DooX, испробовал. Он все-равно чистит под 0 папку

[DooX]

я проверил у себя

знаешь что бы сделал?

$files = array_map('basename', glob($imagesFolder . "*.webp"));

сразу же очистил бы массив только имена файлов с расширением.

дальше перед удалением просто вывел бы
$filesInDatabase и $files

print_r или var_dump

и проверил есть ли вообще схождения

глазами так сказать глянул.

или можешь использовать array_diff

Может быть реально в БД имена не соответствуют файлам

напиши результат вывода. что пишет.

[maluky]

DooX, Удален файл: media/images/63f11fb84b660.webp

[maluky]

DooX, а он прикреплен в бд к статье

[DooX]

выводить var_dump($filesInDatabase,$files );

и там уже сравнивать есть ли похожие элементы

[tukreb]

А разве не логично удалять файл сразу, как было удалено из БД? Не, конечно иногда нужна синхронизация БД и ФС, но если сразу сделать по человечески, например реализовать через очереди, чтобы не было расинхрона, если что-то упадёт в момент какого-то действия.

Или у нас автор не хранит в БД путь к файлам? Ну такое не возможно :)

[Ипатьев]

DooX, если вы вообще не знаете, что такое отладка, и воображаете, будто единственное, что программист может сделать - это написать "более верный код", то вам тем более не стоит отвечать.

[Ипатьев]

$files = array_map('basename', glob($imagesFolder . "*.webp"));

Это глупость.
нет смысла городить лишний цикл на пустом месте.

дальше перед удалением просто вывел бы
$filesInDatabase и $files
print_r или var_dump
и проверил есть ли вообще схождения
глазами так сказать глянул.

А вот это уже правильно.
Собственно, это единственная умная мысль из всего, что вы тут написали.
Это и называется отладкой.

[Ипатьев]

Удален файл: media/images/63f11fb84b660.webp

если к $file применялось basename(), то должно быть не media/images/63f11fb84b660.webp а 63f11fb84b660.webp

то есть вы показываете нам не тот код, который в реальности выполняете.
Это довольно дурацкая идея