- 1
$baner->disabled = !empty($input['disabled'])?!$input['disabled']:TRUE;
Нашли или выдавили из себя код, который нельзя назвать нормальным, на который без улыбки не взглянешь? Не торопитесь его удалять или рефакторить, — запостите его на говнокод.ру, посмеёмся вместе!
+144
$baner->disabled = !empty($input['disabled'])?!$input['disabled']:TRUE;
огонь)
+144
if(count($proxy) === 0 ){
echo 'SMTH went wrong or all proxy pososali vse' . PHP_EOL;
exit;
}
Останавливаемся если закончились прокси
+144
sleh = UpCase(92); // просто слеш
Просто слеш.
+144
private String compId(final String name) {
final Pattern compiled = Pattern.compile("^on([_A-Za-z0-9]+)From([_A-Za-z0-9]+)$");
final Matcher matcher = compiled.matcher(name);
return matcher.matches() ? matcher.group(2) : null;
}
private String compEvent(final String name) {
final Pattern compiled = Pattern.compile("^on([_A-Za-z0-9]+)(From([_A-Za-z0-9]+))?$");
final Matcher matcher = compiled.matcher(name);
return matcher.matches() ? matcher.group(1) : null;
}
+144
...
GameItem* new_game_item = GetGameItem(id);
bool flag = !new_game_item;
if (flag)
{
new_game_item = new GameItem();
}
...
Меня больше всего поразило название локальной переменной new_game_item, которая чаще даже совсем не new. Но кого это волнует...
+144
<!--Женская одежда Блузки-->
<?if($arResult["IBLOCK_SECTION_ID"] == 487 ):?>
<table class="tproperty">
<tr><td class="tdproperty"><?=$arResult["PROPERTIES"]["SOSTAV"]["NAME"];?></td><td><?=$arResult["PROPERTIES"]["SOSTAV"]["VALUE"];?></td></tr>
<tr><td class="tdproperty" ><?=$arResult["PROPERTIES"]["TSVET"]["NAME"];?></td><td><?=$arResult["PROPERTIES"]["TSVET"]["VALUE"];?></td></tr>
<tr><td class="tdproperty" ><?=$arResult['PROPERTIES']['RAZMER']['NAME'];?></td><td><?=$arResult["PROPERTIES"]["RAZMER"]["VALUE"];?></td></tr>
<tr><td class="tdproperty" ><?=$arResult['PROPERTIES']['UPAKOVKA']['NAME'];?></td><td><?=$arResult["PROPERTIES"]["UPAKOVKA"]["VALUE"];?></td></tr>
<tr><td class="tdproperty"><?=$arResult['PROPERTIES']['POKROY']['NAME'];?></td><td><?=$arResult["PROPERTIES"]["POKROY"]["VALUE"];?></td></tr>
<tr><td class="tdproperty"><?=$arResult['PROPERTIES']['DLINNA_RUKAVA']['NAME'];?></td><td><?=$arResult["PROPERTIES"]["DLINNA_RUKAVA"]["VALUE"];?></td></tr>
</table>
<!--Женская одежда Брючные костюмы-->
<?elseif($arResult["IBLOCK_SECTION_ID"] == 489 ):?>
<table class="tproperty">
<tr><td class="tdproperty"><?=$arResult["PROPERTIES"]["SOSTAV"]["NAME"];?></td><td><?=$arResult["PROPERTIES"]["SOSTAV"]["VALUE"];?></td></tr>
<tr><td class="tdproperty"><?=$arResult["PROPERTIES"]["TSVET"]["NAME"];?></td><td><?=$arResult["PROPERTIES"]["TSVET"]["VALUE"];?></td></tr>
<tr><td class="tdproperty"><?=$arResult['PROPERTIES']['RAZMER']['NAME'];?></td><td><?=$arResult["PROPERTIES"]["RAZMER"]["VALUE"];?></td></tr>
<tr><td class="tdproperty"><?=$arResult['PROPERTIES']['UPAKOVKA']['NAME'];?></td><td><?=$arResult["PROPERTIES"]["UPAKOVKA"]["VALUE"];?></td></tr>
</table>
<!--Женская одежда Жакеты-->
<?elseif($arResult["IBLOCK_SECTION_ID"] == 491 ):?>
<table class="tproperty" >
<tr><td class="tdproperty"><?=$arResult["PROPERTIES"]["SOSTAV"]["NAME"];?></td><td><?=$arResult["PROPERTIES"]["SOSTAV"]["VALUE"];?></td></tr>
<tr><td class="tdproperty"><?=$arResult["PROPERTIES"]["TSVET"]["NAME"];?></td><td><?=$arResult["PROPERTIES"]["TSVET"]["VALUE"];?></td></tr>
<tr><td class="tdproperty"><?=$arResult['PROPERTIES']['RAZMER']['NAME'];?></td><td><?=$arResult["PROPERTIES"]["RAZMER"]["VALUE"];?></td></tr>
<tr><td class="tdproperty"><?=$arResult['PROPERTIES']['UPAKOVKA']['NAME'];?></td><td><?=$arResult["PROPERTIES"]["UPAKOVKA"]["VALUE"];?></td></tr>
<tr><td class="tdproperty"><?=$arResult['PROPERTIES']['POKROY']['NAME'];?></td><td><?=$arResult["PROPERTIES"]["POKROY"]["VALUE"];?></td></tr>
<tr><td class="tdproperty"><?=$arResult['PROPERTIES']['DLINNA_RUKAVA']['NAME'];?></td><td><?=$arResult["PROPERTIES"]["DLINNA_RUKAVA"]["VALUE"];?></td></tr>
</table>
........................................
И так еще 900 строчек.... Как такие люди рождаются вообще? :)
+144
string q = DateTime.Now.ToString().Substring(3, 3);
string w = DateTime.Now.ToString().Substring(0, 2);
string e = DateTime.Now.ToString().Substring(5, DateTime.Now.ToString().Length - 5);
string dsasd = q + w + e;
CrmDateTimeProperty _new_date_fitst_update_rstatus = new CrmDateTimeProperty();
_new_date_fitst_update_rstatus.Name = "new_date_first_update_rstatus";
_new_date_fitst_update_rstatus.Value = new CrmDateTime();
_new_date_fitst_update_rstatus.Value.Value = dsasd;
dynamicEntity.Properties.Add(_new_date_fitst_update_rstatus);
Попался проект на фрилансе. Попросили исправить ошибки. Начал исправлять и вот такое.
+144
/* Convert IP address to unsigned long int. */
function ip2ulong($ip) {
return sprintf("%u", ip2long32($ip));
}
https://github.com/pfsense/pfsense/blob/master/etc/inc/util.inc#L414-L417
Продолжаем осторожно идти дальше. Это нормально вообще? Да, я видел всякие смехуёчки с магическим приведением типов в PHP и JS, так что наверняка оно где-то правильно конвертится и все работает несмотря ни на что и вопреки. Может, это даже нормальная практика в мире PHP, но мне страшно.
+144
if ($ipaddr == $ip_reverse) {
return true;
} else {
return false;
}
https://github.com/pfsense/pfsense/blob/master/etc/inc/util.inc#L625-L629
Давно хотел спросить: это как-нибудь здравым смыслом оправдывается? Часто встречаю похожее, когда чужой код смотрю. Может, это профи делают для… читаемости какой-то, я не знаю. Или я ищу хорошее там, где его нет?
+144
int spisok::kolvo()
{
return spisok::N;
}