0

У меня идут такие проверки:

ПРОВЕРКА ISSET($_FILES['IMG'])

ПРОВЕРКА НА КОЛИЧЕСТВО ЗАГРУЖАЕМЫХ ФАЙЛОВ

ПРОВЕРКА ПО МАССИВУ ОШИБОК

ПРОВЕРКА ВЕСА (КБ, МБ)

ПРОВЕРКА ФОРМАТОВ

ПРОВЕРКА РАЗМЕРОВ КАРТИНКИ (РАЗМЕР СТОРОН КАРТИНКИ)

После этого я хочу сделать ресайз картигки, и снова проверить ФОРМАТ файла. Нужна ли эта проверка?

Имена загружаемых не проверяю - не нужны. Типы проверять ($_FILES['type']) - нет никакого смысла.

Зачем ЕЩЕ раз, после ресайза, проверять формат картинки? А вдруг кто-то внедрит туда злой код?

ВСЯ графика (кроме того что используется для дизайна) на сайте сохраняется БЕЗ расширений, выше корня сайта. Повторюсь: Нужно ли еще раз проверять формат картинки, после ресайза?

links:

upload проверка загружаемых файлов

upload странное поведение браузеров

Загружаемый файл находится во временной папке после окончания работы скрипта?

Куда сервер деалет upload? (нет файла во временной папке). upload_tmp_dir

минимальный размер файла для корректной работы exif_imagetype

Не отображается рисунок в браузере

5
  • Одного ресайза вообще достаточно. Если он прошел, то это картинка и все ок, если нет, то загрузили что-то другое.
    – Visman
    15 дек 2016 в 3:12
  • не вполне согласен. Если файл не картинка, то сразу же будет выход и сообщение об ошибке. а если делать только ресайз(без указанных проверок), то если файл не картинка, то, НАВЕРНОЕ, будет затрачено чуть больше ресурсов на ресайз не картинки (ИЛИ перед попыткой ресайза функция imagecopyresampled проверяет файл на соответствие граф формату, если ошибка, то не расайзит)? Количество файлов, вес и резмер картинки тоже нужно проверять - так что ТОЛЛЬКО РЕСАЙЗ в данном случае не уместен.
    – Aliskin
    15 дек 2016 в 10:07
  • 1
    В ресайзе первая функция, которая вызывается это getimagesize(), которая даст первичный ответ картинка это или нет, формат картинки, размер. Или вы ресайз одной функцией imagecopyresampled() делаете? :)
    – Visman
    15 дек 2016 в 10:36
  • та не одной. Мне также нужно проверять минимальные размеры картинки. Ну вообзем у меня, кажись, остался один вопрос относительно аплоада. И после его решения, наверное, всё.(?) Сейчас нужно проверить кое-что. Потом придется разместить крослинки на мои вопросы.
    – Aliskin
    15 дек 2016 в 13:27
  • Вопросов уже нет!
    – Aliskin
    15 дек 2016 в 14:24

1 ответ 1

1

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

1
  • У меня функц ресайза отдельная и функц созранение тож отдельная. А указанные выше проверки все в одной функции. Я нахожусь за полшага к пониманию ООП, но я специаольно не хочу вникать. Потому что если начну детально вникать в ООП, то никогда не смогу ДОписать сайт. Вопрос, кажись, решен.
    – Aliskin
    15 дек 2016 в 10:14

Ваш ответ

Нажимая на кнопку «Отправить ответ», вы соглашаетесь с нашими пользовательским соглашением, политикой конфиденциальности и политикой о куки

Всё ещё ищете ответ? Посмотрите другие вопросы с метками или задайте свой вопрос.