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

    +168

    1. 01
    2. 02
    3. 03
    4. 04
    5. 05
    6. 06
    7. 07
    8. 08
    9. 09
    10. 10
    11. 11
    12. 12
    13. 13
    14. 14
    function UploadAvatar($FILE_tmp, $FILE_name){
    	/*куча тупого кода*/
    	if($width >= $height)
    	    $kf = (float) $width/$height;
    	if($width < $height)		    	  
    	   $kf = (float) $height/$width;
    		   
    	//Проверка на квадратность! ппц идиотизм, идея не моя.
    	if($kf>1.1){
    		$error = "Аватар неквадратен. Пожалуйста выберите квадратный или обрежьте текущий в графическом редакторе.";
    		return 0;
    	}
    	/*еще куча тупого кода*/
    }

    Вырезал кусок из собственного старого сайта. Комменты оставил уникальными.
    Когда дизайнер потребовал чтоб аватары были только квадратны, я наваял такое.
    Допуск в 1,1 - это поблажка юзерам)))

    Запостил: Skull, 25 Марта 2011

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

    • Я тут говна не вижу. Программист правильно написал. Если загружен аватар размером 80 на 81 пиксель, то разве это не квадрат? 1 пиксель зажмется при помощи HTML <img width="80" height="80" /> 1.1 допускает 87*80.
      Ответить
    • А к чему коэффициенты какие-то? Нельзя было просто сравнить width и height с нужными допусками?
      Ответить
    • а почему не круговой интеграл? оптимизация что-ли?
      Ответить
    • А почему самому не ресайзить авки ?? И пользователю показать в превьюве не нравится пусть сам делает.
      Ответить
    • > "Аватар неквадратен. Пожалуйста выберите квадратный или обрежьте текущий в графическом редакторе."

      ага, "Для разлагинивания почистите куки браузером"
      Ответить
      • настоящий псих, удумал ставить себе аватарку в которой соотношение сторон превышает 1 к 1.1
        Ответить
        • А как же авки с Вконтакта?)
          Ответить
          • Какие "вконтакты"!!1.. Давно пора переходить на СК42 или Pulkovo-1942, на основе эллипсоида Красовского.
            Ответить
    • if($width==$height)
      echo 'Так не прокатит разве?)))';
      Ответить
      • Нет. В php так не принято.
        Ответить
      • только если так if($width===$height)
        Ответить
      • век живи, век учись, что сказать
        Ответить
      • прокатит.
        Ответить
        • да, но лучше конечно ===
          Ответить
          • применяя дедуктивные московычисления, находим, что:

            ещё лучше, конечно, ====
            Ответить
          • не собственно === не лучше єто лишняя проверка на соответствие типу которая пашет только в php 4. rtfm товарищи
            Ответить
    • Мне сначала показалось "Автор не квадратен"...
      Ответить
      • мне показалось "автор неадекватен"
        Ответить
        • Да, тоже так показалось,
          вообще формулировка весьма доставляет "Аватар неквадратен" - "Стол некругл"
          Ответить
          • а какого дьявола, спрашивается? Если с помощью того же жкуери можно обрезать аватар как нравится - как это сделано на многих популярных сайтах
            Ответить
            • Видимо, налицо недопонимание принципов автоматизации. Вместо того, чтобы сделать робота, который будет что-то делать за человека, делается робот, который будет заниматься надзором. Само собой, это проще.
              Ответить
            • Тогда мое умение работы с jquery заключалось в .slideToogle();
              Ответить
              • я мельком видел плагин, простота его подключения не сложнее slideToggle()
                Ответить
                • Ну так его ж надо увидеть))))) Я прочел все статьи с Хабра по jquery недавно - такого плагина не увидел.
                  Ответить
                  • плохо ищете - 3 минуты лазания по гуглу дали например такой вариант: http://odyniec.net/projects/imgareaselect/
                    по-моему, красиво
                    Ответить
                    • Он же "прочёл все статьи с Хабра", какой на фиг гугл... ("Гугл? Не, не слышал...")
                      Ответить
                      • ну у вас же в россии яндекс, да....
                        Ответить
                        • да уж... и вконтактах... и рутубы...
                          собственно кругом сплошные "вкусозаменители"...
                          Ответить
        • аналогично
          Ответить
      • +1 тоже самое прочитал
        Ответить

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