- 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;
}