"PHP" / Говнокод #6094 Ссылка на оригинал

0

  1. 1
  2. 2
  3. 3
  4. 4
  5. 5
  6. 6
  7. 7
  8. 8
  9. 9
  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 Skull, (Updated )

Комментарии (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 'Так не прокатит разве?)))';
    Ответить
  • Мне сначала показалось "Автор не квадратен"...
    Ответить
      • Да, тоже так показалось,
        вообще формулировка весьма доставляет "Аватар неквадратен" - "Стол некругл"
        Ответить
        • а какого дьявола, спрашивается? Если с помощью того же жкуери можно обрезать аватар как нравится - как это сделано на многих популярных сайтах
          Ответить
          • Видимо, налицо недопонимание принципов автоматизации. Вместо того, чтобы сделать робота, который будет что-то делать за человека, делается робот, который будет заниматься надзором. Само собой, это проще.
            Ответить
          • Тогда мое умение работы с jquery заключалось в .slideToogle();
            Ответить
            • я мельком видел плагин, простота его подключения не сложнее slideToggle()
              Ответить
              • Ну так его ж надо увидеть))))) Я прочел все статьи с Хабра по jquery недавно - такого плагина не увидел.
                Ответить

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

Семь раз отмерь — один отрежь, guest!

    А не использовать ли нам bbcode?


    8