Автоматическое удаление фото из папки

@cooliyev · Регистрация: 13.02.2015

Студворк — интернет-сервис помощи студентам

Здрасти. Я написал функцию которая автоматически должна удалять фото а потом и запись в бд по истечению 3 месяцев но она почему то не работает. Подскажите где ошибка?

PHP
1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
17
18
19
20
21
22
23
24
25
26
27
28
function imgname(){
    $query = "SELECT img, img_slide FROM `goods` AS `g`
            RIGHT JOIN `brands` AS `b` 
            ON `b`.`brand_id`=`g`.`goods_brandid`
 
            WHERE  `b`.`parent_id` !='94' AND`g`.`date` < NOW() - INTERVAL 3 MONTH";
    $res = mysql_query($query);
    $goods = array();
    while($row = mysql_fetch_assoc($res)){
        $goods[] = $row;
    }
    return $goods;
        
        $img = "images/".$goods['img'];
    if($goods['img_slide']){
        $goods['img_slide'] = explode("|", $goods['img_slide']);
    return $goods['img_slide'];
    }
    $img2 = "images/productimg/".$goods['img_slide'];
    
    if(unlink($img, $img2)){
        mysql_query("DELETE `g` FROM `goods` AS `g`
            LEFT JOIN `brands` AS `b` 
            ON `b`.`brand_id`=`g`.`goods_brandid`
 
            WHERE  `b`.`parent_id` !='94' AND`g`.`date` < NOW() - INTERVAL 3 MONTH");
        }
    }

@nrobert · 14.02.2015, 17:27

Где вы учились программированию, cooliyev?
1. Функция может возвращать (return) значение только один раз. Весь код после return уже не работает после ее (return) срабатывания.
2. unlink() каждый раз удаляет только 1 файл.

@cooliyev · 15.02.2015, 08:36 **[ТС]**

А как вы думаете раз пропускаю такие нелепые ошибки? Нигде! Я самоучка. Не пинайте сильно

приняв во внимание то, что вы написали я изменил функцию. Так корректно?

PHP
1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
function imgname(){
    $query = "SELECT img, img_slide FROM `goods` AS `g`
            RIGHT JOIN `brands` AS `b` 
            ON `b`.`brand_id`=`g`.`goods_brandid`
 
            WHERE  `b`.`parent_id` !='94' AND`g`.`date` < NOW() - INTERVAL 3 MONTH";
    $res = mysql_query($query);
    $goods = array();
    while($row = mysql_fetch_assoc($res)){
        $goods[] = $row;
        if($goods['img_slide']){
        $goods['img_slide'] = explode("|", $goods['img_slide']);
    }
    return $goods;
    }

PHP
1
2
3
4
5
6
7
8
9
10
function del_img ($goods) {
        $img = "/img/".$goods['img'];
        $img2 = "/img/productimg/".$goods['img_slide'];
    
    if(unlink($img) && unlink($img2)){
        mysql_query("DELETE `g` FROM `goods` AS `g`
            LEFT JOIN `brands` AS `b` 
            ON `b`.`brand_id`=`g`.`goods_brandid`
 
            WHERE  `b`.`parent_id` !='94' AND`g`.`date` < NOW() - INTERVAL 3 MONTH")

@nrobert · 15.02.2015, 15:44

Без обид, ОК? Я тоже самоучка и не самого хорошего мнения о нынешних студентах.
Лучше так

PHP
1
2
3
4
5
6
7
8
9
10
$goods=array();
while($row=mysql_fetch_assoc($res)){
    $goods[]='/img/'.$row['img'];
    if($row['img_slide']){
        $row=explode("|",$row['img_slide']);
        foreach($row as $val){
            $goods[]='/img/productimg/'.$val;
        }
    }
}

Потом

PHP
1
2
3
4
5
6
foreach($goods as $img){
    if(unlink($img)){
        blahblahblah;
    }
    blahblahblah;
}

и не нужно разбивать на 2 функции.

@cooliyev · 16.02.2015, 12:53 **[ТС]**

Так или я что то не так понял?

PHP
1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
17
18
19
20
21
22
23
24
25
26
27
28
29
30
31
32
33
function imgname(){
    $query = "SELECT img, img_slide FROM `goods` AS `g`
            RIGHT JOIN `brands` AS `b` 
            ON `b`.`brand_id`=`g`.`goods_brandid`
 
            WHERE  `b`.`parent_id` !='94' AND`g`.`date` < NOW() - INTERVAL 3 MONTH";
    $res = mysql_query($query);
    
    $goods=array();
    while($row=mysql_fetch_assoc($res)){
        $goods[]='/img/'.$row['img'];
            if($row['img_slide']){
                $row=explode("|",$row['img_slide']);
                foreach($row as $val){
                $goods[]='/img/productimg/'.$val;
                }
    }
}
 
    foreach($goods as $img){
        if(unlink($img)){
            mysql_query("DELETE `g` FROM `goods` AS `g`
            LEFT JOIN `brands` AS `b` 
            ON `b`.`brand_id`=`g`.`goods_brandid`
 
            WHERE  `b`.`parent_id` !='94' AND`g`.`date` < NOW() - INTERVAL 3 MONTH");
    }else{
        echo "Something wrong".mysql_error();
    }
    
    }
 
}

@nrobert · 16.02.2015, 13:23

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

@cooliyev · 16.02.2015, 13:24 **[ТС]**

Он не работает поэтому и спросил

@nrobert · 16.02.2015, 14:36

Ставьте проверку, чтобы проверять, где косяк. Например, перед foreach() ставьте

PHP
1
print_r($goods);

@cooliyev · 16.02.2015, 14:42 **[ТС]**

Спасибо большое заработало! А если поместить его в cron в таком виде он будет работать?

@nrobert · 16.02.2015, 15:04

Да, cron всего лишь запускает скрипт в определенное время.

Новые блоги и статьи Все статьи Все блоги /
Access VikBal 11.12.2025 Помогите пожалуйста !! Как объединить 2 одинаковые БД Access с разными данными.	Новый ноутбук volvo 07.12.2025 Всем привет. По скидке в "черную пятницу" взял себе новый ноутбук Lenovo ThinkBook 16 G7 на Амазоне: Ryzen 5 7533HS 64 Gb DDR5 1Tb NVMe 16" Full HD Display Win11 Pro	Музыка, написанная Искусственным Интеллектом volvo 04.12.2025 Всем привет. Некоторое время назад меня заинтересовало, что уже умеет ИИ в плане написания музыки для песен, и, собственно, исполнения этих самых песен. Стихов у нас много, уже вышли 4 книги, еще 3. . .	От async/await к виртуальным потокам в Python IndentationError 23.11.2025 Армин Ронахер поставил под сомнение async/ await. Создатель Flask заявляет: цветные функции - провал, виртуальные потоки - решение. Не threading-динозавры, а новое поколение лёгких потоков. Откат?. . .	Поиск "дружественных имён" СОМ портов Argus19 22.11.2025 Поиск "дружественных имён" СОМ портов На странице: https:/ / norseev. ru/ 2018/ 01/ 04/ comportlist_windows/ нашёл схожую тему. Там приведён код на С++, который показывает только имена СОМ портов, типа,. . .
Сколько Государство потратило денег на меня, обеспечивая инсулином. Programma_Boinc 20.11.2025 Сколько Государство потратило денег на меня, обеспечивая инсулином. Вот решила сделать интересный приблизительный подсчет, сколько государство потратило на меня денег на покупку инсулинов. . . .	Ломающие изменения в C#.NStar Alpha Etyuhibosecyu 20.11.2025 Уже можно не только тестировать, но и пользоваться C#. NStar - писать оконные приложения, содержащие надписи, кнопки, текстовые поля и даже изображения, например, моя игра "Три в ряд" написана на этом. . .	Мысли в слух kumehtar 18.11.2025 Кстати, совсем недавно имел разговор на тему медитаций с людьми. И обнаружил, что они вообще не понимают что такое медитация и зачем она нужна. Самые базовые вещи. Для них это - когда просто люди. . .	Создание Single Page Application на фреймах krapotkin 16.11.2025 Статья исключительно для начинающих. Подходы оригинальностью не блещут. В век Веб все очень привыкли к дизайну Single-Page-Application . Быстренько разберем подход "на фреймах". Мы делаем одну. . .	Фото: Daniel Greenwood kumehtar 13.11.2025

@nrobert 171 / 153 / 67 Регистрация: 11.02.2015 Сообщений: 450
	14.02.2015, 17:27
	Где вы учились программированию, cooliyev? 1. Функция может возвращать (return) значение только один раз. Весь код после return уже не работает после ее (return) срабатывания. 2. unlink() каждый раз удаляет только 1 файл. 0

@nrobert 171 / 153 / 67 Регистрация: 11.02.2015 Сообщений: 450
	16.02.2015, 13:23
	В принципе, да. Самый ценный способ - способ проб и ошибок. Всегда проверяйте работоспособность кода прежде, чем задавать вопросы. 0

@cooliyev 0 / 0 / 1 Регистрация: 13.02.2015 Сообщений: 42
	16.02.2015, 13:24 [ТС]
	Он не работает поэтому и спросил 0

@cooliyev 0 / 0 / 1 Регистрация: 13.02.2015 Сообщений: 42
	16.02.2015, 14:42 [ТС]
	Спасибо большое заработало! А если поместить его в cron в таком виде он будет работать? 0

@nrobert 171 / 153 / 67 Регистрация: 11.02.2015 Сообщений: 450
	16.02.2015, 15:04
	Да, cron всего лишь запускает скрипт в определенное время. 0

Автоматическое удаление фото из папки

Решение