Не могу понять почему вот так не работает запрос

@Bertram · Регистрация: 15.10.2013

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

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

HTML5
1
<input multiple="multiple" name="images[]" type="file" accept="image/*">

обработчик

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
if(filter_input_array(INPUT_POST)){
        $uploaddir = '../'.USERIMAGES;
        $query='';
        //если хоть одно изображение загружено
        if ($_FILES['images']['size'][0]!=0){
            //для каждого изображения
            foreach ($_FILES['images']['name'] as $key => $value) {
                $extension=end(explode('.', $value));//получаем расширение
                $uploadfile = $uploaddir . basename('project_'.$id.'_'.$key.'.'.$extension);//директория и имя нового файла
                //если файл загрузился
                if(is_uploaded_file($_FILES['images']['tmp_name'][$key])){
                    //перемещаем файл в новую директорию под новым именем, если перемещение удачно
                    if (move_uploaded_file($_FILES['images']['tmp_name'][$key], $uploadfile)) {
                        $query.='|project_'.$id.'_'.$key.'.'.$extension.'';//добавляем к запросу бд имя файла
                    } else {
                        $_SESSION['warning']='<div class=warning>Изображение не было загружено. Попробуйте ещё раз</div>';
                    }
                }
            }
        $query=  substr($query, 1);//обрезаем лишний | у запроса    
        }  else {
            $query='noimage.png';//если пользователь не загружал изображений, ставим изображение по умолчанию
        }
        mysql_query('UPDATE projects SET images="'.$query.'" WHERE id='.$id);//обновляем БД
}

Теперь. Есть вид "изменение проекта". Там тоже можно загружать картинки и код обработчика надо изменить с учетом того, что в базе и на сервере уже есть какие-то картинки проекта. То есть нужно подгрузить новые картинки к старым. Вот что я пишу

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
if(filter_input_array(INPUT_POST)){
if($projectforedit['images']=='noimage.png'){
        $query='';
        $count=0;
    }  else {
        $query=$projectforedit['images'];
        $count=  count(explode('|', $projectforedit['images']));
    }
    //если хоть одно изображение загружено
    if ($_FILES['images']['size'][0]!=0){
        //для каждого изображения
        foreach ($_FILES['images']['name'] as $key => $value) {
            $extension=end(explode('.', $value));//получаем расширение
            $uploadfile = $uploaddir . basename('project_'.$id.'_'.$count.'.'.$extension);//директория и имя нового файла
            //если файл загрузился
            if(is_uploaded_file($_FILES['images']['tmp_name'][$key])){
                //перемещаем файл в новую директорию под новым именем, если перемещение удачно
                if (move_uploaded_file($_FILES['images']['tmp_name'][$key], $uploadfile)) {
                    $query.='|project_'.$id.'_'.$count.'.'.$extension.'';//добавляем к запросу бд имя файла
                } else {
                    $_SESSION['warning']='<div class=warning>Изображение не было загружено. Попробуйте ещё раз</div>';
                }
            }
            $count++;
        }
    $query=  substr($query, 1);//обрезаем лишний | у запроса    
    }  else {
        $query='noimage.png';//если пользователь не загружал изображений, ставим изображение по умолчанию
    }
    mysql_query('UPDATE projects SET images="'.$query.'" WHERE id='.$id) or die(mysql_error());//обновляем БД
}

В самом начале условие. Если в проекте не было картинок (тогда автоматом ставится картинка по-умолчанию), $query='';, то есть картинку по-умолчанию мы не сохраняем, она нам не нужна. Иначе мы берем строку из базы и к ней уже будем добавлять.

Переменная $count тут показывает какое количество картинок есть в базе для этого проекта. Дальше в цикле foreach в $uploadfile и в $query мы используем переменную $count (в виде добавление проекта там была переменная $key) и в конце делаем $count++;

Теперь сама суть. Первый обработчик работает полностью отлично. А вот второй сохраняет файлы на сервер под нужным названием, но НЕ изменяет запись в БД вообще. Почему так происходит? Переменная $count же int, я вроде правильно вставляю её в запрос. mysql_error никаких ошибок не выдаёт.

Надеюсь все понятно расписал

@nrobert · 17.03.2015, 20:16

Проверьте $id, скорее всего, не выполняется проверка на нее.

@Bertram · 18.03.2015, 11:09 **[ТС]**

Была такая мысль, делал echo $id;, он правильный, с БД совпадает

Именно после того, как в запросе заменил $key на $count он перестал выполняться. Причина кроется где-то в $count, только непонятно где

Добавлено через 14 часов 49 минут
Неужели ни у кого больше нет идей? Я уже намучался с этим, а сделать надо

@lenk0belk0 · 18.03.2015, 17:03

Пробовали проверять значения переменных var_dump ? Попробуйте перед отправкой запроса в бд вывести его и вручную исполнить его (например через phpmyadmin)

@Bertram · 18.03.2015, 17:21 **[ТС]**

Значения переменных, в том числе $count правильные. Запрос вручную в phpmyadmin выполняется без проблем

@root · 18.03.2015, 18:04

Сообщение от Bertram

mysql_query

говорят от анахронизма волосы на ладонях растут. это раз. где вы устанавливаете значение переменной $id прежде чем пихать её в запросы? это два. остальное неважно. данный код нерабочий в принципе..

@Bertram · 18.03.2015, 18:12 **[ТС]**

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

PHP
1
$id=filter_input(INPUT_GET, 'id');

Это же значение переменной $id отлично себя чувствует в имени файла, который успешно загружается на сервер.

@root · 18.03.2015, 18:44

вот еще крутяк:

Сообщение от Bertram

$query.='|project_'.$id.'_'.$count.'.'.$ extension.'';//добавляем к запросу бд имя файла

Добавлено через 1 минуту
старайтесь писать код читабельным, а не помойку в которой чёрт ногу сломит

Новые блоги и статьи Все статьи Все блоги /
SDL3 для Web (WebAssembly): Обработчик клика мыши в браузере ПК и касания экрана в браузере на мобильном устройстве 8Observer8 02.02.2026 Содержание блога Для начала пошагово создадим рабочий пример для подготовки к экспериментам в браузере ПК и в браузере мобильного устройства. Потом напишем обработчик клика мыши и обработчик. . .	Философия технологии iceja 01.02.2026 На мой взгляд у человека в технических проектах остается роль генерального директора. Все остальное нейронки делают уже лучше человека. Они не могут нести предпринимательские риски, не могут. . .	SDL3 для Web (WebAssembly): Вывод текста со шрифтом TTF с помощью SDL3_ttf 8Observer8 01.02.2026 Содержание блога В этой пошаговой инструкции создадим с нуля веб-приложение, которое выводит текст в окне браузера. Запустим на Android на локальном сервере. Загрузим Release на бесплатный. . .	SDL3 для Web (WebAssembly): Сборка C/C++ проекта из консоли 8Observer8 30.01.2026 Содержание блога Если вы откроете примеры для начинающих на официальном репозитории SDL3 в папке: examples, то вы увидите, что все примеры используют следующие четыре обязательные функции, а. . .
SDL3 для Web (WebAssembly): Установка Emscripten SDK (emsdk) и CMake для сборки C и C++ приложений в Wasm 8Observer8 30.01.2026 Содержание блога Для того чтобы скачать Emscripten SDK (emsdk) необходимо сначало скачать и уставить Git: Install for Windows. Следуйте стандартной процедуре установки Git через установщик. . . .	SDL3 для Android: Подключение Box2D v3, физика и отрисовка коллайдеров 8Observer8 29.01.2026 Содержание блога Box2D - это библиотека для 2D физики для анимаций и игр. С её помощью можно определять были ли коллизии между конкретными объектами. Версия v3 была полностью переписана на Си, в. . .	Инструменты COM: Сохранение данный из VARIANT в файл и загрузка из файла в VARIANT bedvit 28.01.2026 Сохранение базовых типов COM и массивов (одномерных или двухмерных) любой вложенности (деревья) в файл, с возможностью выбора алгоритмов сжатия и шифрования. Часть библиотеки BedvitCOM Использованы. . .	SDL3 для Android: Загрузка PNG с альфа-каналом с помощью SDL_LoadPNG (без SDL3_image) 8Observer8 28.01.2026 Содержание блога SDL3 имеет собственные средства для загрузки и отображения PNG-файлов с альфа-каналом и базовой работы с ними. В этой инструкции используется функция SDL_LoadPNG(), которая. . .

@nrobert 171 / 153 / 67 Регистрация: 11.02.2015 Сообщений: 450
	17.03.2015, 20:16
	Проверьте $id, скорее всего, не выполняется проверка на нее. 0

@Bertram 0 / 0 / 0 Регистрация: 15.10.2013 Сообщений: 33
	18.03.2015, 11:09 [ТС]
	Была такая мысль, делал echo $id;, он правильный, с БД совпадает Именно после того, как в запросе заменил $key на $count он перестал выполняться. Причина кроется где-то в $count, только непонятно где Добавлено через 14 часов 49 минут Неужели ни у кого больше нет идей? Я уже намучался с этим, а сделать надо 0

@lenk0belk0 7 / 7 / 5 Регистрация: 24.12.2014 Сообщений: 33
	18.03.2015, 17:03
	Пробовали проверять значения переменных var_dump ? Попробуйте перед отправкой запроса в бд вывести его и вручную исполнить его (например через phpmyadmin) 0

@Bertram 0 / 0 / 0 Регистрация: 15.10.2013 Сообщений: 33
	18.03.2015, 17:21 [ТС]
	Значения переменных, в том числе $count правильные. Запрос вручную в phpmyadmin выполняется без проблем 0