+24.9
- 01
- 02
- 03
- 04
- 05
- 06
- 07
- 08
- 09
- 10
if(empty($real_name)) $real_name = $file_name;
if(empty($real_name)) $real_name="fileid_".$id;
....
if($mode=="send") {
$head[] = array("link" => "customer.php?area=$area&mode=send", "title" => func_get_langvar_by_name("lbl_send_to_mail"), active=>"Y");
} else {
$head[] = array("link" => "customer.php?area=$area&mode=send", "title" => func_get_langvar_by_name("lbl_send_to_mail"), active=>"n");
}
Разбирая один из скриптов на досуге, обнаружил, что некоторые программисты уж совсем с головой не дружат.
1. Думаю, все тут ясно.
2. Чувак явно не знает что такое логика... Мало того, что в одну строку ложится массив, так еще и ВЕЗДЕ строковые 'Y'/'N' использует вместо булевых труфолсов. И такого говна - тыщи 2 строк. Ы =)
Не позавидую тем, кто будет дописывать / переделывать.
ЗЫ: Скрипт - рип корпоративной системы.
Запостил: guest,
16 Декабря 2008
guest 17.12.2008 04:09 # 0
битрикс-стайл
[url]http://govnokod.ru/8[/url] — вот кто его научил :)
guest 18.12.2008 17:12 # 0
а в остальном сойдет.
Y или N вместо true и false иногда применимы, например, если есть 3-ий какой-нибудь вариант
guest 19.12.2008 09:45 # 0
> ЗЫ: Скрипт - рип корпоративной системы.
до боли знакомый код. в каком городе находится компания?
guest 19.12.2008 19:05 # 0
kay, не могу распространять информацию.
Говногость, да ну? я же говорю про булевы переменные. Их у нас три, да?
>> 'Y'/'N' использует вместо булевых труфолсов
guest 23.12.2008 20:09 # 0
1. а где active дальше используется? точно не в выводе?
2. проверка на пустоту в первой строчке точно не нужна?
guest 24.12.2008 12:18 # 0
anmiles, внимательнее читай комменты. Тут обсуждение идет введение тупых Y/N вместо нативных true/false.
Y/N в строковом представлении дальше нигде не видел :)
guest 28.12.2008 19:18 # 0
>>1. а где active дальше используется? точно не в выводе?
а если бы и в выводе, уж наверно лучше там было бы if поставить чем тут такой хуетищщей заниматься. потом есличе хоть в одном месте только править пришлось бы
guest 22.04.2009 11:26 # 0
Это говнокод изза повторения куска кода, а y/n присваиваются просто полю и вообще тут не причём.