Говнокод

list($width, $height, $type, $attr) = getimagesize($file1);
     
if($width == $height)
{    
    set_time_limit(270);
    if(filesize($file1) > 102400)  return array(false, "Размер больше 100Кб!");
    if($type == 1)      {$file2 = $file1.'.gif'; $out = $tizer_id.'.gif'; $type = 'gif'; $ext = 'gif';}
    elseif($type == 2)  {$file2 = $file1.'.jpg'; $out = $tizer_id.'.jpg'; $type = 'jpg'; $ext = 'jpg';}
    elseif($type == 3)  {$file2 = $file1.'.png'; $out = $tizer_id.'.png'; $type = 'png'; $ext = 'png';}
    else return array(false, "Это не картинка!");
    /* ... */
}
// no square
else
{
    return array(array('img',false, "Изображение не квадратное!"));
}

Зачем жить, если no square

Запостил:

roman-kashitsyn, 19.08.2012 (Updated 25.03.2018)

Комментарии (8) RSS

scriptin 14 лет ago # 0
>Зачем жить, если no square
Тогда должно быть:
```
die("Изображение не квадратное!");
```
Ответить
- 3.14159265 14 лет ago # 0 ⇈
  
  >если no square
  Насколько я понял это тонкий намёк сюда:
  http://www.youtube.com/watch?v=eKxEvN8LETA
  Ответить
eth0 14 лет ago # 0

Возвращает массив массивов - однозначно сильно.
Ответить
- bormand 14 лет ago # 0 ⇈
  
  Туплов то нет, вот и приходится...
  
  P.S. Гораздо хуже, что походу $file1 и $file2 глобальные, как и $out, $type и $ext.
  Ответить
  - Edd 14 лет ago # 0 ⇈
    
    function aaa($file1, &$file2, &$out, &$type, &$ext){
    .....
    }
    Ответить
    - bormand 14 лет ago # 0 ⇈
      
      Ну да... а зачем тогда этот странный массив возвращать? В конце концов автор мог бы добавить еще один выходной параметр под текст ошибки...
      Ответить
  - roman-kashitsyn 14 лет ago # 0 ⇈
    
    > $file1 и $file2 глобальные
    Нет, в 11 строке намёк, что дальше что-то происходит (там есть глобальные переменные, но они ссылаются на текущее соединение к БД).
    Ответить
anonimb84a2f6fd141 14 лет ago # 0

Ответить