+162
- 01
- 02
- 03
- 04
- 05
- 06
- 07
- 08
- 09
- 10
- 11
if(!function_exists('getImg')){
function getImg($isVip)
{
$type = false;
if (isset($_GET['f']) && $_GET['f']) $type = 1;
if (isset($_GET['bl']) && $_GET['bl']) $type = 16;
if (isset($_GET['lw']) && $_GET['lw']) $type = 'author';
if (isset($_GET['sym']) && $_GET['sym'] && $isVip) $type = 2;
return $type;
}
}
Функция getImg: хм... название как бэ намекает нам, что должно на выходе получиться что-нибудь типа картинки или её адреса, НО НЕТ! нам возвращается какой-то тип...
Что это за тип (ну да, да, php бестиповой язык, но нужно же как-то хотя бы намёки на тип делать) - одному Богу известно, НО! эта переменная изначально логическая, потом ВНЕЗАПНО превпращается в число, ну и под конец может быть и строкой! (ну а почему бы и нет)
Причём конечно же всем очевидна последовательность: 1, 16, (опять-таки ВНЕЗАПНО) 'author', ну и по уже "понятной" логике в конце 2
Да, ну и разумеется вокруг оооочень много комментариев поясняющие этот кусок кода...
А вы на индусов ругаетесь... Так-то!
Запостил: Zemtall,
06 Сентября 2010
Анонимус 07.09.2010 01:48 # +1
7ion 07.09.2010 01:49 # 0
Изящная логика, создание функции, которой передается параметр, используемый лишь в одном условии - отличная реализация функционала PHP.
А Img скорее всего означает что-то другое.
Этот код прекрасен для понимания и в комментариях не нуждается.
[sarcasm_mode=off]
Zemtall 07.09.2010 10:44 # −1
для [sarcasm_mode=off] - [sarcasm_mode=on]???
Lure Of Chaos 07.09.2010 19:05 # 0
зато боимся, что функцию мы уже где-то написали (см 1 строку)
и кодоиндусы - понятие интернациональное
istem 07.09.2010 23:02 # −1
if(!function_exists('getImg')) {
function getImg($isVip) {
$ar1=array(false, 1, 16, 'author', 2 );
$ar2=array(false, @$_GET['f'], @$_GET['bl'], @$_GET['lw'], @$_GET['sym']&&$isVip);
return $ar1[ array_search(true, array_reverse( $ar2, true ) ) ];
}
}
// [ ??? ]