1. PHP / Говнокод #10225

    +76

    1. 1
    2. 2
    3. 3
    4. 4
    5. 5
    6. 6
    7. 7
    8. 8
    9. 9
    $sizes = array(50,60,70,80,90,100,110,120,130,140,150,160,170,180,190,200);
foreach ($sizes as $v2)
{
    $file3 = $file1."-".$v2."x".$v2.".".$ext;
    if(file_exists($file3)) unlink($file3);
    $c = "convert $file2 -resize ".$v2."x".$v2." -quality 80 -strip $file3";
    $last_line = system($c, $retval);
    chmod($file3, 0777);
}

    Мне одному кажется, что порождать пару десятков процессов из php скрипта при загрузке небольшой картинки немного... толсто?

    Запостил: roman-kashitsyn, 09 Мая 2012

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

    • ava zloirock 09.05.2012 12:13 # +1

      мне вот более интересно - все эти размеры реально необходимы?
      Ответить

      • ava roman-kashitsyn 09.05.2012 13:01 # 0

        неа, автор использует всегда 200x200, а размер подгоняет в width и height атрибутах img.
        Ответить

    • ava wvxvw 09.05.2012 12:34 # +1

      convert это ж imagemagic если я правильно понимаю. Для него ж вроде и обертка была для ПХП? Или я чего-то не знаю?
      Ответить

      • ava roman-kashitsyn 09.05.2012 13:00 # +1

        Есть, конечно. NIH синдром в действии.
        Ответить

      • ava eth0 09.05.2012 14:35 # +2

        Не буду врать, но, возможно, PHP собран без поддержки, вот такой грязный хак получается.
        Ответить

      • ava ZX_Spectrum 09.05.2012 14:56 # +2

        ну да, есть IMagick для PHP, только вот в этой ситуации вряд ли будет проще кодить, лучше склеить нужную команду для convert
        Ответить

    • ava rat4 09.05.2012 13:17 # −2

      >chmod 0777

      Придумали тут, понимаешь ли, какие-то права...
      Ответить

    • ava Anon4e4Y7YbA 10.05.2012 22:55 # −3

      Не толсто, мне с imagick-ом тоже удобнее через консольную программу. Так что (-1)
      Ответить

      • ava inkanus-gray 10.05.2012 23:47 # +1

        А генерировать 16 превьюшек, из которых будет использована только одна, ничуть не толсто?
        Ответить

    Добавить комментарий