Логика if - else PHP нужна помощь - Веб-программирование

killhunter · 16.10.2009, 00:53

День добрый.
Помогите, пожалуйста, разобраться с логикой многоуровнего if - else.
Есть код загрузки картинок на сервер + пути в базу.
Многочисленные проверки работают и всю грузиться куда надо, если проходит проверки. Но вот текст ошибки по проверке размера фаила в Кб не выдаётся, а выдается ошибка по пикселям. Хотя фаил большого чем надо размера не пропускает. В общем хочу понять почему не выдаёт ошибку по размеру.

Заранее спасибо.

PHP код:


			
//коннектимся к базе

$max_image_width = 150;

$max_image_height = 301;

$max_image_width2 = 801;

$max_image_height2 = 1001;

$max_image_size = 20 * 1024;

$max_image_size2 = 200 * 1024;

$valid_types = array("gif","jpg", "png", "jpeg", "JPG", "JPEG", "GIF", "PNG");



if($post=="Y")

  {

   



   // Каталог, в который мы будем принимать файл:

   $uploaddir1 = 'files/';

   $filename = $_FILES['image_path1']['name'];

   $filename_1 = $_FILES['image_path2']['name'];

     if (eregi("[а-яА-Я]", $filename) || eregi("[а-яА-Я]", $filename_1))

       {

         $error_1="<b>Ошибка:</b> <font color=#FF0000>В имени фаила не должно быть русских букв.</font>";

       }

     else

       {

         //генерируем новые имена

         $add=rand(1,10).rand(5,20).rand(3,40).rand(1,100);

         $name=substr($filename, 0, strrpos($filename, "."));

         $name_hex=bin2hex($name);

         $add2=rand(1,10).rand(5,20).rand(3,40).rand(1,100);

         $name2=substr($filename_1, 0, strrpos($filename_1, "."));

         $name_hex2=bin2hex($name2);

         $ext = substr($filename, 1 + strrpos($filename, "."));

         $ext2 = substr($filename_1, 1 + strrpos($filename_1, "."));

         $filename2 = $_FILES['image_path1']['tmp_name'];

         $filename2_2 = $_FILES['image_path2']['tmp_name'];

         if (strrpos($filename, ".") > 8 && strrpos($filename_1, ".") > 8)

           {

            $error_2="<b>Ошибка:</b> <font color=#FF0000>Слишком длинное имя фаила. Допустимо 8 символов до точки.</font>";

           } 

         else

           {

         $image_path1 = $uploaddir1.basename($add.'_'.$name_hex.'.'.$ext);

         $image_path2 = $uploaddir1.basename($add2.'_'.$name_hex2.'.'.$ext2);

          //вот что-то здесь не так

           [B]if (filesize($filename2) > $max_image_size || filesize($filename2_2) > $max_image_size2) 

             {

               $error_3="<b>Ошибка:</b> <font color=#FF0000Размер фаила слишком большой. Допустимо не более 20Kб для фото-миниатюры и 200Кб для полной фотографии.</font>";[/B]             } 

           elseif (!in_array($ext, $valid_types) || !in_array($ext2, $valid_types)) 

             {

               $error_4="<b>Ошибка:</b> <font color=#FF0000>Неправильный тип фаила. Допустимо (jpg, jpeg) или фаил не указан.</font>";

             } 

           else 

             {

               $size = GetImageSize($filename2);

               $size2 = GetImageSize($filename2_2);

               if (($size) && ($size[0] == $max_image_width) && ($size[1] < $max_image_height) && ($size2) && ($size2[0] < $max_image_width2) && ($size2[1] < $max_image_height2)) 

                   {

                      // Копируем файл из каталога для временного хранения файлов:

                         if (copy($filename2, $image_path1) && copy($filename2_2, $image_path2))

                           {

                             //грузим в базу данные и путь

                                        

                            }

                         else $error_5="<b>Ошибка:</b> <font color=#FF0000>Не удалось загрузить файл на сервер!</font>"; 

                   } 

                 else $error_6="<b>Ошибка:</b> <font color=#FF0000>Размеры картинки слишком большие. Для фото-миниатюры ширина = 150px, высота максимально - 300px, для полной фотографии ширина максимально 800px, высота максимально - 1000px.</font>";

             }

          }

       }   

  }

echo $error_1;

echo $error_2;

echo $error_3;

echo $error_4;

echo $error_5;

echo $error_6;

?>

<form action="<? $PHP_SELF; ?>" method="post" enctype=multipart/form-data>

<input type="hidden" name="post" value="Y">

<input type="hidden" name="MAX_FILE_SIZE" value="200000">

....

<input type="file" name="image_path1" value="<? echo $image_path1; ?>" size=26>

<input type="file" name="image_path2" value="<? echo $image_path2; ?>" size=26

...

<input type="submit" value="Добавить">

</form>

//закрываем коннект к базе

*Plague* · 16.10.2009, 01:38

хммм

Цитата:

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

<font color=#FF0000Размер фаила слишком большой. Допустимо не более 20Kб....

тег-то допиши уж, зачем экономить...

<font color=#FF0000>Размер...

killhunter · 16.10.2009, 03:21

Спасибо.

Однако вопрос до конца не решён.
Проверил еще раз.
Дано два тестовых фаила:
первый - маленький 150*112 (по геометр. размерам подходит). Сохранён специально без сжатия - 44 Кб
(по Кб не подходит) - загружаем только его - PHP честно пишет - не подходит - ура!

второй - большой 600*450 (по геометр. размерам подходит). Сохранён специально без сжатия - 323 Кб (по Кб не подходит) - загружаем только его - PHP пишет - "Размеры картинки слишком большие. Для фото-миниатюры ширина = 150px, высота максимально - 300px, для полной фотографии ширина максимально 800px, высота максимально - 1000px." - не то - косяк!

Грузим оба - PHP честно пишет - не подходит - ура!
(ссылаясь скорее всего на первый фаил)

Жмем фаилы по размеру - ошибок нет - грузим на сервер и в базу.

Всё бы ничего, если бы делал интерфейс для себя. А так пользователь будет биться головой о стенку (хоть весь лист испиши help'ом) почему размеры большой картинки по пикселям верны, а картинка не грузится и выдаёт что неправильные размеры по пикселям.

P.S. Сделал то же самое для одного фаила
При выполнение условий всё грузиться.
Если взять картинку меньшую по пикселям, но большую по Кб, пишет большая по Кб.
Если взять нужную по пикселям, но большую по Кб, пишет неправильные размеры по пикселям.

PHP код:


			
<?

$max_image_width = 150;

$max_image_height = 301;

$max_image_size = 20 * 1024;

$valid_types = array("jpg", "jpeg", "JPG", "JPEG");



 

if($post=="Y")

  {

               

    // Каталог, в который мы будем принимать файл:

    $uploaddir = 'upload/';

    $filename1_1 = $_FILES['image_path']['name'];



         if (eregi("[а-яА-Я]", $filename1_1))

      {

        $error_1="<b>Ошибка:</b> <font color=#FF0000>В имени фаила не должно быть русских букв.</font>";

      }

     else

      {

       $add=rand(1,10).rand(5,20).rand(3,40).rand(1,100);

       $name=substr($filename1_1, 0, strrpos($filename1_1, "."));

       $name_hex=bin2hex($name);

       $filename2_1 = $_FILES['image_path']['tmp_name'];

         if (strrpos($filename1_1, ".") > 9)

           {

            $error_2="<b>Ошибка:</b> <font color=#FF0000>Слишком длинное имя фаила. Допустимо 8 символов до точки.</font>";

           } 

         else

           {

         $ext = substr($filename1_1, 1 + strrpos($filename1_1, "."));

         $image_path = $uploaddir.basename($add.'_'.$name_hex.'.'.$ext);

 

        if (filesize($filename2_1) > $max_image_size) 

           {

            $error_3="<b>Ошибка:</b> <font color=#FF0000>Размер фаила слишком большой. Допустимо не более 20Кб.</font>";

           } 

         elseif (!in_array($ext, $valid_types)) 

           {

            $error_4="<b>Ошибка:</b> <font color=#FF0000>Неправильный тип фаила. Допустимо (jpg, jpeg) или фаил не указан.</font>";

           } 

         else 

           {

            $size = getimagesize($filename2_1);

             if (($size) && ($size[0] == $max_image_width) && ($size[1] < $max_image_height)) 

              {



                // Копируем файл из каталога для временного хранения файлов:

                  if (copy($filename2_1, $image_path))

                    {

                       //грузим в базу и на сервер                                  

                     } else   $error_5="<b>Ошибка:</b> <font color=#FF0000>Не удалось загрузить файл на сервер!</font";  

              } else   $error_6="<b>Ошибка:</b> <font color=#FF0000>Неправильные размеры изображения. Измените, пожалуйста, размеры изображения до: ширина = 150px, высота максимально - 300px.</font>";

           }

         }

       }   

    }

echo $error_1;

echo $error_2;

echo $error_3;

echo $error_4; 

echo $error_5; 

echo $error_6;

?>

<form action="<? $PHP_SELF; ?>" method="post" enctype=multipart/form-data>

<input type="hidden" name="post" value="Y">

<input type="hidden" name="MAX_FILE_SIZE" value="20000">

....

<input type="file" name="image_path" value="<? echo $image_path; ?>" size=25>

.....

<input type="submit" value="Сохранить" >

</form>

*Plague* · 16.10.2009, 03:36

бррр. многа букф. особенно для третьего часа ночи. я честно пытался трижды прочесть и собрать в кучу. ниасилил.

файл a - проходит по dimension и не проходит по size
файл b - проходит по dimension и не проходит по size

дальше что?

killhunter · 16.10.2009, 03:45

Сделал дополнение на один фаил.

По вопросам, да именно так.

Оба фаила правильно проверяются на условия, но первый выдаёт правильные else (dimension, size). Второй выдает только dimension, на проверку как dimension, так и size.

EvroStandart · 16.10.2009, 12:27

Делай рядом с каждой проверкой выписку в хтмл. Чтобы видеть что там скрипт реально насчитал.

killhunter · 16.10.2009, 13:10

Сделал.
Проверял скрипт одного фаила.
Взял фаил 150*105, т.е. размеры подходят, но 30 Кб (не подходит).
Дальше началось странное:
три загрузки этого фаила - выводит неправильные размеры по ширине и высоте и не выводит echo размера по Кб и по ширине.
Четвертая и далее загрузки пишет большой размер по Кб и выводит echo filesize($filename2_1); 30150 как и должен.

Т.е. получается первые три загрузки getimagesize вообще не сработал.
Я правильно понимаю?
Если это так, то как можно с этим бороться - это же код, а не гадание на кофейной гуще

boor · 16.10.2009, 13:45

Версия php какая?
формат картинок разный или один?

killhunter · 16.10.2009, 13:53

Проверяю на локальном компе
Apache/2.0.59 PHP/4.3.4
Картинка одна (если рассматривать скрипт загрузки одного фаила), формат .jpg (обработка в Photoshop)

Hubbitus · 19.10.2009, 01:45

Хм, скажу честно, код жуткий (начиная с прямых проверок, странных получений имени файла и расширения и copy вместо move_uploaded_file ну и других конечно), но суть Вашей проблемы полагаю что Вы просто не проверяете ошибку аплоада ($_FILES['image_path']['error']) в Вашем случае получается, когда размер больше указанного в MAX_FILE_SIZE, php выставляет ошибку 2 ( UPLOAD_ERR_FORM_SIZE ), и соответственно $filename2_1 = $_FILES['image_path']['tmp_name']; пустые.

Правда getimagesize потом честно говорит об этом:

Код:

Warning: getimagesize() [function.getimagesize]: Filename cannot be empty in /home/hubbitus/_SITES/ru.bir.ru/_temp/imho.ws/Upload/1.php on line 35

странно как Вы этого не заметили. Полагаю забыли включить уровень вывода всех ошибок.

killhunter · 19.10.2009, 14:43

Т.е. надо ввести проверку на размер временного фаила, чтобы не ноль.
Понял, спасибо.

И ещё вопрос в контексте

Все мы в общем-то учимся постоянно. Вот сейчас я изучаю php, мне интересно почему код жуткий

Дело в том, что пытаюсь таким образом максимально обезопасить загрузку картинок пользователями (там есть еще ограничения на вывод информации из БД).

Однако, буду признателен любой ссылки на мануал или пример максимально безопасной загрузки картинок на сервер

Hubbitus · 19.10.2009, 15:04

Цитата:

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

Т.е. надо ввести проверку на размер временного фаила, чтобы не ноль.

То есть надо проверить корректность загрузки, я же написал! $_FILES['image_path']['error'], is_uploaded_file...

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

Код:

$name=substr($filename1_1, 0, strrpos($filename1_1, "."));

Если в $filename1_1 будет имя файла "a.b.c.d.jpg"? Почему не воспользоваться стандартными http://php.net/pathinfo, http://ru2.php.net/basename? Там же и расширение файла. Про небезопасное использование copy я говорил. Ну и также далее, про длину файла в 8 символов (вообще не понятно откуда, ну да Бог с ней, может Вы в DOSе работаете, или того требует бизнес-логика) проверка по первой точке также не корректна...

killhunter · 21.10.2009, 10:08

Вот вариант с учётом, анализа ошибок загрузки и проверки временного фаила на ноль. Всё работает.
Кстати фаил a.b.c.d.jpg тоже правильно грузится

Поясню про 8 символов - сделано по двум причинам:
1. ограничить имя конечного фаила после bin2hex()
2. можно было и 5,7; 8 символов просто небольшая хитрость и пыль в глаза

PHP код:


			
$max_image_width = 150;

$max_image_height = 301;

$valid_types = array("jpg", "jpeg", "JPG", "JPEG");



 

if($post=="Y")

  {

        

    // Каталог, в который мы будем принимать файл:

    $uploaddir = 'upload/';

    $filename1_1 = $_FILES['image_path']['name'];



         if (eregi("[а-яА-Я]", $filename1_1))

      {

        $error_1="<b>Ошибка:</b> <font color=#FF0000>В имени фаила не должно быть русских букв.</font>";

      }

     else

      {

       $add=rand(1,10).rand(5,20).rand(3,40).rand(1,100);

       $name=substr($filename1_1, 0, strrpos($filename1_1, "."));

       $name_hex=bin2hex($name);

       $filename2_1 = $_FILES['image_path']['tmp_name'];

         if (strrpos($filename1_1, ".") > 9)

           {

            $error_2="<b>Ошибка:</b> <font color=#FF0000>Слишком длинное имя фаила. Допустимо 8 символов до точки.</font>";

           } 

         else

           {

         $ext = substr($filename1_1, 1 + strrpos($filename1_1, "."));

         $image_path = $uploaddir.basename($add.'_'.$name_hex.'.'.$ext);

 

        if (!in_array($ext, $valid_types)) 

           {

            $error_4="<b>Ошибка:</b> <font color=#FF0000>Неправильный тип фаила. Допустимо (jpg, jpeg) или фаил не указан.</font>";

           } 

         else 

           {



            if($_FILES['image_path']['error'] === 0)

             {

               if(isset($_FILES['image_path']) && is_uploaded_file($_FILES['image_path']['tmp_name']) && $_FILES['image_path']['size'])

               {

                $size = getimagesize($filename2_1);

                 if (($size) && ($size[0] == $max_image_width) && ($size[1] < $max_image_height)) 

                  {

                  // Копируем файл из каталога для временного хранения файлов:

                  if (copy($filename2_1, $image_path))

                    {

                         //грузим на сервер и в базу                                

                             

                     } else  $error_5="<b>Ошибка:</b> <font color=#FF0000>Не удалось загрузить файл на сервер!</font"; 



                  } else  $error_6="<b>Ошибка:</b> <font color=#FF0000>Неправильные размеры изображения. Измените, пожалуйста, размеры изображения до: ширина = 150px, высота максимально - 300px.</font>"; 

              } else $error_3="<b>Ошибка:</b> <font color=#FF0000>Такого фаила не существует</font>";

            } 

          else 

            {

               if($_FILES['image_path']['error'] === 1)

               { $msg="<b>Ошибка:</b> <font color=#FF0000>Размер файла ".BASENAME( $_FILES['image_path']['name'])." больше максимально допустимого размера, разрешённого директивой upload_max_filesize конфигурационного файла php.ini.</font>"; }

               if($_FILES['image_path']['error'] === 2)

               { $msg="<b>Ошибка:</b> <font color=#FF0000>Размер фаила ".BASENAME( $_FILES['image_path']['name'])." слишком большой. Допустимо не более 50Kб.</font>"; }

               if($_FILES['image_path']['error'] === 3)

               { $msg="<b>Ошибка:</b> <font color=#FF0000>Файл ".BASENAME( $_FILES['image_path']['name'])." был получен только частично.</font>"; }

               if($_FILES['image_path']['error'] === 4)

               { $msg="<b>Ошибка:</b> <font color=#FF0000>Файл не был загружен. Вероятнее всего Вы не выбрали файл.</font>"; }

            }

          }

        }

      }   

    }

.....

<? echo $error_1; ?>

<? echo $error_2; ?>

<? echo $error_3; ?>

<? echo $error_4; ?>

<? echo $error_5; ?>

<? echo $error_6; ?>

<? echo $msg; ?>

.....

<form action="<? $PHP_SELF; ?>" method="post" enctype=multipart/form-data>

<input type="hidden" name="post" value="Y">

<input type="hidden" name="MAX_FILE_SIZE" value="50000">

....

<input type="file" name="image_path" value="<? echo $image_path; ?>" size=25>

....

<input type="submit" value="Добавить"> 

</form>

Всем спасибо за помощь и ссылки на мануал.
Пойду выяснять про отличие copy и move_uploaded_file в аспекте безопасности.

16.10.2009, 00:53	# 1
killhunter Junior Member Регистрация: 24.10.2002 Сообщения: 68	Логика if - else PHP нужна помощь День добрый. Помогите, пожалуйста, разобраться с логикой многоуровнего if - else. Есть код загрузки картинок на сервер + пути в базу. Многочисленные проверки работают и всю грузиться куда надо, если проходит проверки. Но вот текст ошибки по проверке размера фаила в Кб не выдаётся, а выдается ошибка по пикселям. Хотя фаил большого чем надо размера не пропускает. В общем хочу понять почему не выдаёт ошибку по размеру. Заранее спасибо. PHP код: //коннектимся к базе $max_image_width = 150; $max_image_height = 301; $max_image_width2 = 801; $max_image_height2 = 1001; $max_image_size = 20 * 1024; $max_image_size2 = 200 * 1024; $valid_types = array("gif","jpg", "png", "jpeg", "JPG", "JPEG", "GIF", "PNG"); if($post=="Y") { // Каталог, в который мы будем принимать файл: $uploaddir1 = 'files/'; $filename = $_FILES['image_path1']['name']; $filename_1 = $_FILES['image_path2']['name']; if (eregi("[а-яА-Я]", $filename) \|\| eregi("[а-яА-Я]", $filename_1)) { $error_1="<b>Ошибка:</b> <font color=#FF0000>В имени фаила не должно быть русских букв.</font>"; } else { //генерируем новые имена $add=rand(1,10).rand(5,20).rand(3,40).rand(1,100); $name=substr($filename, 0, strrpos($filename, ".")); $name_hex=bin2hex($name); $add2=rand(1,10).rand(5,20).rand(3,40).rand(1,100); $name2=substr($filename_1, 0, strrpos($filename_1, ".")); $name_hex2=bin2hex($name2); $ext = substr($filename, 1 + strrpos($filename, ".")); $ext2 = substr($filename_1, 1 + strrpos($filename_1, ".")); $filename2 = $_FILES['image_path1']['tmp_name']; $filename2_2 = $_FILES['image_path2']['tmp_name']; if (strrpos($filename, ".") > 8 && strrpos($filename_1, ".") > 8) { $error_2="<b>Ошибка:</b> <font color=#FF0000>Слишком длинное имя фаила. Допустимо 8 символов до точки.</font>"; } else { $image_path1 = $uploaddir1.basename($add.'_'.$name_hex.'.'.$ext); $image_path2 = $uploaddir1.basename($add2.'_'.$name_hex2.'.'.$ext2); //вот что-то здесь не так [B]if (filesize($filename2) > $max_image_size \|\| filesize($filename2_2) > $max_image_size2) { $error_3="<b>Ошибка:</b> <font color=#FF0000Размер фаила слишком большой. Допустимо не более 20Kб для фото-миниатюры и 200Кб для полной фотографии.</font>";[/B] } elseif (!in_array($ext, $valid_types) \|\| !in_array($ext2, $valid_types)) { $error_4="<b>Ошибка:</b> <font color=#FF0000>Неправильный тип фаила. Допустимо (jpg, jpeg) или фаил не указан.</font>"; } else { $size = GetImageSize($filename2); $size2 = GetImageSize($filename2_2); if (($size) && ($size[0] == $max_image_width) && ($size[1] < $max_image_height) && ($size2) && ($size2[0] < $max_image_width2) && ($size2[1] < $max_image_height2)) { // Копируем файл из каталога для временного хранения файлов: if (copy($filename2, $image_path1) && copy($filename2_2, $image_path2)) { //грузим в базу данные и путь } else $error_5="<b>Ошибка:</b> <font color=#FF0000>Не удалось загрузить файл на сервер!</font>"; } else $error_6="<b>Ошибка:</b> <font color=#FF0000>Размеры картинки слишком большие. Для фото-миниатюры ширина = 150px, высота максимально - 300px, для полной фотографии ширина максимально 800px, высота максимально - 1000px.</font>"; } } } } echo $error_1; echo $error_2; echo $error_3; echo $error_4; echo $error_5; echo $error_6; ?> <form action="<? $PHP_SELF; ?>" method="post" enctype=multipart/form-data> <input type="hidden" name="post" value="Y"> <input type="hidden" name="MAX_FILE_SIZE" value="200000"> .... <input type="file" name="image_path1" value="<? echo $image_path1; ?>" size=26> <input type="file" name="image_path2" value="<? echo $image_path2; ?>" size=26 ... <input type="submit" value="Добавить"> </form> //закрываем коннект к базе __________________ Через терни к звездам!

16.10.2009, 03:21	# 3
killhunter Junior Member Регистрация: 24.10.2002 Сообщения: 68	Спасибо. Однако вопрос до конца не решён. Проверил еще раз. Дано два тестовых фаила: первый - маленький 150112 (по геометр. размерам подходит). Сохранён специально без сжатия - 44 Кб (по Кб не подходит) - загружаем только его - PHP честно пишет - не подходит - ура! второй - большой 600450 (по геометр. размерам подходит). Сохранён специально без сжатия - 323 Кб (по Кб не подходит) - загружаем только его - PHP пишет - "Размеры картинки слишком большие. Для фото-миниатюры ширина = 150px, высота максимально - 300px, для полной фотографии ширина максимально 800px, высота максимально - 1000px." - не то - косяк! Грузим оба - PHP честно пишет - не подходит - ура! (ссылаясь скорее всего на первый фаил) Жмем фаилы по размеру - ошибок нет - грузим на сервер и в базу. Всё бы ничего, если бы делал интерфейс для себя. А так пользователь будет биться головой о стенку (хоть весь лист испиши help'ом) почему размеры большой картинки по пикселям верны, а картинка не грузится и выдаёт что неправильные размеры по пикселям. P.S. Сделал то же самое для одного фаила При выполнение условий всё грузиться. Если взять картинку меньшую по пикселям, но большую по Кб, пишет большая по Кб. Если взять нужную по пикселям, но большую по Кб, пишет неправильные размеры по пикселям. PHP код: <? $max_image_width = 150; $max_image_height = 301; $max_image_size = 20 * 1024; $valid_types = array("jpg", "jpeg", "JPG", "JPEG"); if($post=="Y") { // Каталог, в который мы будем принимать файл: $uploaddir = 'upload/'; $filename1_1 = $_FILES['image_path']['name']; if (eregi("[а-яА-Я]", $filename1_1)) { $error_1="<b>Ошибка:</b> <font color=#FF0000>В имени фаила не должно быть русских букв.</font>"; } else { $add=rand(1,10).rand(5,20).rand(3,40).rand(1,100); $name=substr($filename1_1, 0, strrpos($filename1_1, ".")); $name_hex=bin2hex($name); $filename2_1 = $_FILES['image_path']['tmp_name']; if (strrpos($filename1_1, ".") > 9) { $error_2="<b>Ошибка:</b> <font color=#FF0000>Слишком длинное имя фаила. Допустимо 8 символов до точки.</font>"; } else { $ext = substr($filename1_1, 1 + strrpos($filename1_1, ".")); $image_path = $uploaddir.basename($add.'_'.$name_hex.'.'.$ext); if (filesize($filename2_1) > $max_image_size) { $error_3="<b>Ошибка:</b> <font color=#FF0000>Размер фаила слишком большой. Допустимо не более 20Кб.</font>"; } elseif (!in_array($ext, $valid_types)) { $error_4="<b>Ошибка:</b> <font color=#FF0000>Неправильный тип фаила. Допустимо (jpg, jpeg) или фаил не указан.</font>"; } else { $size = getimagesize($filename2_1); if (($size) && ($size[0] == $max_image_width) && ($size[1] < $max_image_height)) { // Копируем файл из каталога для временного хранения файлов: if (copy($filename2_1, $image_path)) { //грузим в базу и на сервер } else $error_5="<b>Ошибка:</b> <font color=#FF0000>Не удалось загрузить файл на сервер!</font"; } else $error_6="<b>Ошибка:</b> <font color=#FF0000>Неправильные размеры изображения. Измените, пожалуйста, размеры изображения до: ширина = 150px, высота максимально - 300px.</font>"; } } } } echo $error_1; echo $error_2; echo $error_3; echo $error_4; echo $error_5; echo $error_6; ?> <form action="<? $PHP_SELF; ?>" method="post" enctype=multipart/form-data> <input type="hidden" name="post" value="Y"> <input type="hidden" name="MAX_FILE_SIZE" value="20000"> .... <input type="file" name="image_path" value="<? echo $image_path; ?>" size=25> ..... <input type="submit" value="Сохранить" > </form> __________________ Через терни к звездам! Последний раз редактировалось killhunter; 16.10.2009 в 03:37.

16.10.2009, 03:36	# 4
Plague Administrator Регистрация: 06.05.2003 Адрес: Московская Подводная Лодка Пол: Male Сообщения: 12 062	бррр. многа букф. особенно для третьего часа ночи. я честно пытался трижды прочесть и собрать в кучу. ниасилил. файл a - проходит по dimension и не проходит по size файл b - проходит по dimension и не проходит по size дальше что? __________________ все "спасибы" - в приват и в репутацию! не засоряйте форум!!!! ~~~~~~~~~~~~~~~~~~~~~~ The time has come it is quite clear, our antichrist is ~~almost~~ already here. M.M.

16.10.2009, 03:45	# 5
killhunter Junior Member Регистрация: 24.10.2002 Сообщения: 68	Сделал дополнение на один фаил. По вопросам, да именно так. Оба фаила правильно проверяются на условия, но первый выдаёт правильные else (dimension, size). Второй выдает только dimension, на проверку как dimension, так и size. __________________ Через терни к звездам! Последний раз редактировалось killhunter; 16.10.2009 в 03:47.

16.10.2009, 13:10	# 7
killhunter Junior Member Регистрация: 24.10.2002 Сообщения: 68	Сделал. Проверял скрипт одного фаила. Взял фаил 150105, т.е. размеры подходят, но 30 Кб (не подходит). Дальше началось странное: три загрузки этого фаила - выводит неправильные размеры по ширине и высоте и не выводит echo размера по Кб и по ширине. Четвертая и далее загрузки пишет большой размер по Кб и выводит echo filesize($filename2_1); 30150 как и должен. Т.е. получается первые три загрузки getimagesize вообще не сработал. Я правильно понимаю? Если это так, то как можно с этим бороться - это же код, а не гадание на кофейной гуще __________________ Через терни к звездам! Последний раз редактировалось killhunter; 16.10.2009 в 13:12.*

16.10.2009, 12:27	# 6
EvroStandart Full Member Регистрация: 20.01.2004 Адрес: Таллинн Пол: Male Сообщения: 623	Делай рядом с каждой проверкой выписку в хтмл. Чтобы видеть что там скрипт реально насчитал.

16.10.2009, 13:45	# 8
boor ::VIP:: Хрюндель Регистрация: 27.10.2002 Адрес: Столица блин Сообщения: 650	Версия php какая? формат картинок разный или один? __________________ Если Вас выписали из сумасшедшего дома, это не значит, что Вас вылечили. Просто Вы стали как все.

16.10.2009, 13:53	# 9
killhunter Junior Member Регистрация: 24.10.2002 Сообщения: 68	Проверяю на локальном компе Apache/2.0.59 PHP/4.3.4 Картинка одна (если рассматривать скрипт загрузки одного фаила), формат .jpg (обработка в Photoshop) __________________ Через терни к звездам!

19.10.2009, 01:45	# 10
Hubbitus мод IMHO Кодер-200(6,7,8) Регистрация: 29.03.2003 Адрес: Saint-Petersburg, Russia Пол: Male Сообщения: 2 734	Хм, скажу честно, код жуткий (начиная с прямых проверок, странных получений имени файла и расширения и copy вместо move_uploaded_file ну и других конечно), но суть Вашей проблемы полагаю что Вы просто не проверяете ошибку аплоада ($_FILES['image_path']['error']) в Вашем случае получается, когда размер больше указанного в MAX_FILE_SIZE, php выставляет ошибку 2 ( UPLOAD_ERR_FORM_SIZE ), и соответственно $filename2_1 = $_FILES['image_path']['tmp_name']; пустые. Правда getimagesize потом честно говорит об этом: Код: Warning: getimagesize() [function.getimagesize]: Filename cannot be empty in /home/hubbitus/_SITES/ru.bir.ru/_temp/imho.ws/Upload/1.php on line 35 странно как Вы этого не заметили. Полагаю забыли включить уровень вывода всех ошибок. __________________ Я делаю Линукс! Присоединяйтесь к свободным людям! Связаться со мной всегда можно по джабберу: Hubbitus@jabber.ru Pahan-Hubbitus.

19.10.2009, 14:43	# 11
killhunter Junior Member Регистрация: 24.10.2002 Сообщения: 68	Т.е. надо ввести проверку на размер временного фаила, чтобы не ноль. Понял, спасибо. И ещё вопрос в контексте Все мы в общем-то учимся постоянно. Вот сейчас я изучаю php, мне интересно почему код жуткий Дело в том, что пытаюсь таким образом максимально обезопасить загрузку картинок пользователями (там есть еще ограничения на вывод информации из БД). Однако, буду признателен любой ссылки на мануал или пример максимально безопасной загрузки картинок на сервер __________________ Через терни к звездам! Последний раз редактировалось killhunter; 19.10.2009 в 14:52.

21.10.2009, 10:08	# 13
killhunter Junior Member Регистрация: 24.10.2002 Сообщения: 68	Вот вариант с учётом, анализа ошибок загрузки и проверки временного фаила на ноль. Всё работает. Кстати фаил a.b.c.d.jpg тоже правильно грузится Поясню про 8 символов - сделано по двум причинам: 1. ограничить имя конечного фаила после bin2hex() 2. можно было и 5,7; 8 символов просто небольшая хитрость и пыль в глаза PHP код: $max_image_width = 150; $max_image_height = 301; $valid_types = array("jpg", "jpeg", "JPG", "JPEG"); if($post=="Y") { // Каталог, в который мы будем принимать файл: $uploaddir = 'upload/'; $filename1_1 = $_FILES['image_path']['name']; if (eregi("[а-яА-Я]", $filename1_1)) { $error_1="<b>Ошибка:</b> <font color=#FF0000>В имени фаила не должно быть русских букв.</font>"; } else { $add=rand(1,10).rand(5,20).rand(3,40).rand(1,100); $name=substr($filename1_1, 0, strrpos($filename1_1, ".")); $name_hex=bin2hex($name); $filename2_1 = $_FILES['image_path']['tmp_name']; if (strrpos($filename1_1, ".") > 9) { $error_2="<b>Ошибка:</b> <font color=#FF0000>Слишком длинное имя фаила. Допустимо 8 символов до точки.</font>"; } else { $ext = substr($filename1_1, 1 + strrpos($filename1_1, ".")); $image_path = $uploaddir.basename($add.'_'.$name_hex.'.'.$ext); if (!in_array($ext, $valid_types)) { $error_4="<b>Ошибка:</b> <font color=#FF0000>Неправильный тип фаила. Допустимо (jpg, jpeg) или фаил не указан.</font>"; } else { if($_FILES['image_path']['error'] === 0) { if(isset($_FILES['image_path']) && is_uploaded_file($_FILES['image_path']['tmp_name']) && $_FILES['image_path']['size']) { $size = getimagesize($filename2_1); if (($size) && ($size[0] == $max_image_width) && ($size[1] < $max_image_height)) { // Копируем файл из каталога для временного хранения файлов: if (copy($filename2_1, $image_path)) { //грузим на сервер и в базу } else $error_5="<b>Ошибка:</b> <font color=#FF0000>Не удалось загрузить файл на сервер!</font"; } else $error_6="<b>Ошибка:</b> <font color=#FF0000>Неправильные размеры изображения. Измените, пожалуйста, размеры изображения до: ширина = 150px, высота максимально - 300px.</font>"; } else $error_3="<b>Ошибка:</b> <font color=#FF0000>Такого фаила не существует</font>"; } else { if($_FILES['image_path']['error'] === 1) { $msg="<b>Ошибка:</b> <font color=#FF0000>Размер файла ".BASENAME( $_FILES['image_path']['name'])." больше максимально допустимого размера, разрешённого директивой upload_max_filesize конфигурационного файла php.ini.</font>"; } if($_FILES['image_path']['error'] === 2) { $msg="<b>Ошибка:</b> <font color=#FF0000>Размер фаила ".BASENAME( $_FILES['image_path']['name'])." слишком большой. Допустимо не более 50Kб.</font>"; } if($_FILES['image_path']['error'] === 3) { $msg="<b>Ошибка:</b> <font color=#FF0000>Файл ".BASENAME( $_FILES['image_path']['name'])." был получен только частично.</font>"; } if($_FILES['image_path']['error'] === 4) { $msg="<b>Ошибка:</b> <font color=#FF0000>Файл не был загружен. Вероятнее всего Вы не выбрали файл.</font>"; } } } } } } ..... <? echo $error_1; ?> <? echo $error_2; ?> <? echo $error_3; ?> <? echo $error_4; ?> <? echo $error_5; ?> <? echo $error_6; ?> <? echo $msg; ?> ..... <form action="<? $PHP_SELF; ?>" method="post" enctype=multipart/form-data> <input type="hidden" name="post" value="Y"> <input type="hidden" name="MAX_FILE_SIZE" value="50000"> .... <input type="file" name="image_path" value="<? echo $image_path; ?>" size=25> .... <input type="submit" value="Добавить"> </form> Всем спасибо за помощь и ссылки на мануал. Пойду выяснять про отличие copy и move_uploaded_file в аспекте безопасности. __________________ Через терни к звездам!