- 1
 
http://habrahabr.ru/post/172015/
                                    Нашли или выдавили из себя код, который нельзя назвать нормальным, на который без улыбки не взглянешь? Не торопитесь его удалять или рефакторить, — запостите его на говнокод.ру, посмеёмся вместе!
+167
http://habrahabr.ru/post/172015/
                                    
            Наш клиент. 
Не смотря на заголовок статьи, он упёрся в ограничение на количество параметров в функции.
        
+167
$this->loger->info($this->loger);
                                    
+167
var pic_jpg = /(\.jpg)$/ig;
var pic_jpeg = /(\.jpeg)$/ig;
var pic_png = /(\.png)$/ig;
var pic_bmp = /(\.bmp)$/ig;
 var a = $("a");
$.each(a, function (i, val) {
		var src = $(val).attr("href");
		if (src !== undefined) {
				if (src.match(pic_jpg)) {
						AppendImage(i, src, 'a');
				}
				if (src.match(pic_jpeg)) {
						AppendImage(i, src, 'a');
				}
				if (src.match(pic_png)) {
						AppendImage(i, src, 'a');
				}
				if (src.match(pic_bmp)) {
						AppendImage(i, src, 'a');
				}
		}
});
                                    
+167
unlink(__FILE__);
                                    
+167
function quests_pickup_found_item($questParam,$userId,$itemId)
{
  $DBConnection=connect();
  //
  $sqlQuery="UPDATE quests_states SET ".
		" quest_found_items=CONCAT_WS(',',quest_found_items,'".mysql_real_escape_string($itemId)."'), ".
		" quest_last_found_time=".time()." ".
		" WHERE quest_id=".mysql_real_escape_string($questParam['quest_id'])." AND ".
		" user_id=".mysql_real_escape_string($userId)." ".
		" LIMIT 1";
  $resultId=@mysql_query($sqlQuery, $DBConnection);
  $result=@mysql_affected_rows($resultId);
  return true;
}
// Функция проверяет,может ли юзер в данный момент вообще брать квесты (может он уже взял квест).
// Внимание,функция НИ ХУЯ НЕ ПРОВЕРЯЕТ текущую локацию юзера,так что это на вашей совести (я понимаю,что ее нет,но все же).
function quests_check_user_any_quests_awailable($userInfo)
{
    // И ЧТО ТЫ ТУТ ХОТЕЛ УВИДЕТЬ?
}
function quests_set_dead_npc($userId,$deadNpc)
{
    $DBConnection=connect();
    $sqlQuery="INSERT INTO quests_dead_npc SET user_id='".mysql_real_escape_string($userId)."', npc_id='".mysql_real_escape_string($deadNpc)."' ";
    $resultId=@mysql_query($sqlQuery, $DBConnection);
    if(!$resultId)
    {
	return false;
                                    
            А вам слабо в каждой функции устанавливать соединение с базой?
P.S. Ну и коменты к функции quests_check_user_any_quests_awailable() тоже хороши. Сразу видна забота о других программистах,котрым это все поддерживать.
        
+167
<p class="last_pay">
                        <?if($arResult["history"]["PROPERTIES"]["payment10_date"]["VALUE"]):?>
                             <?=$arResult["history"]["PROPERTIES"]["payment10_date"]["VALUE"]?> Произведен платеж на ... бла бла
                        <?else:?>
                            <?if($arResult["history"]["PROPERTIES"]["payment9_date"]["VALUE"]):?>
                                 <?=$arResult["history"]["PROPERTIES"]["payment9_date"]["VALUE"]?> Произведен платеж на сумму ... 
                            <?else:?>
                                <?if($arResult["history"]["PROPERTIES"]["payment8_date"]["VALUE"]):?>
                                     <?=$arResult["history"]["PROPERTIES"]["payment8_date"]["VALUE"]?> Произведен платеж на сумму ... 
                                <?else:?>
                                    <?if($arResult["history"]["PROPERTIES"]["payment7_date"]["VALUE"]):?>
                                         <?=$arResult["history"]["PROPERTIES"]["payment7_date"]["VALUE"]?> Произведен платеж на сумму  ... 
                                    <?else:?>
                                        <?if($arResult["history"]["PROPERTIES"]["payment6_date"]["VALUE"]):?>
                                             <?=$arResult["history"]["PROPERTIES"]["payment6_date"]["VALUE"]?> Произведен платеж на сумму  ...
                                        <?else:?>
                                            <?if($arResult["history"]["PROPERTIES"]["payment5_date"]["VALUE"]):?>
                                                 <?=$arResult["history"]["PROPERTIES"]["payment5_date"]["VALUE"]?> Произведен платеж на сумму  ...
                                            <?else:?>
                                                <?if($arResult["history"]["PROPERTIES"]["payment4_date"]["VALUE"]):?>
                                                     <?=$arResult["history"]["PROPERTIES"]["payment4_date"]["VALUE"]?> Произведен платеж на сумму  ... 
                                                <?else:?>
                                                    <?if($arResult["history"]["PROPERTIES"]["payment3_date"]["VALUE"]):?>
                                                         <?=$arResult["history"]["PROPERTIES"]["payment3_date"]["VALUE"]?> Произведен платеж на сумму  ... 
                                                    <?else:?>
                                                        <?if($arResult["history"]["PROPERTIES"]["payment2_date"]["VALUE"]):?>
                                                             <?=$arResult["history"]["PROPERTIES"]["payment2_date"]["VALUE"]?> Произведен платеж на сумму  ... 
                                                        <?else:?>
                                                            <?if($arResult["history"]["PROPERTIES"]["payment1_date"]["VALUE"]):?>
                                                                 <?=$arResult["history"]["PROPERTIES"]["payment1_date"]["VALUE"]?> Произведен платеж на сумму  ... 
                                                            <?else:?>
                                                                Вы пока не производили платежи
                                                            <?endif;?>
                                                        <?endif;?>
                                                    <?endif;?>
                                                <?endif;?>
                                            <?endif;?>
                                        <?endif;?>
                                    <?endif;?>
                                <?endif;?>
                            <?endif;?>
                        <?endif;?>
                    <?endif;?>
 
                    </p>
                                    красиво
+167
$sredn['Итог']['Е']=$sredn['Итог']['Е']+$sredn['Цех монтажа, ремонта вооружения и ремонта электрооборудования N 2']['Е'];$sredn['Итог']['Я']=$sredn['Итог']['Я']+$sredn['Цех монтажа, ремонта вооружения и ремонта электрооборудования N 2']['Я']; $sredn['Итог']['Яч']=$sredn['Итог']['Яч']+$sredn['Цех монтажа, ремонта вооружения и ремонта электрооборудования N 2']['Яч']; $sredn['Итог']['К']=$sredn['Итог']['К']+$sredn['Цех монтажа, ремонта вооружения и ремонта электрооборудования N 2']['К']; $sredn['Итог']['О']=$sredn['Итог']['О']+$sredn['Цех монтажа, ремонта вооружения и ремонта электрооборудования N 2']['О']; $sredn['Итог']['Ж']=$sredn['Итог']['Ж']+$sredn['Цех монтажа, ремонта вооружения и ремонта электрооборудования N 2']['Ж']; $sredn['Итог']['Б']=$sredn['Итог']['Б']+$sredn['Цех монтажа, ремонта вооружения и ремонта электрооборудования N 2']['Б']; $sredn['Итог']['Р']=$sredn['Итог']['Р']+$sredn['Цех монтажа, ремонта вооружения и ремонта электрооборудования N 2']['Р']; $sredn['Итог']['ОЖ']=$sredn['Итог']['ОЖ']+$sredn['Цех монтажа, ремонта вооружения и ремонта электрооборудования N 2']['ОЖ']; $sredn['Итог']['У']=$sredn['Итог']['У']+$sredn['Цех монтажа, ремонта вооружения и ремонта электрооборудования N 2']['У']; $sredn['Итог']['А']=$sredn['Итог']['А']+$sredn['Цех монтажа, ремонта вооружения и ремонта электрооборудования N 2']['А']; $sredn['Итог']['Г']=$sredn['Итог']['Г']+$sredn['Цех монтажа, ремонта вооружения и ремонта электрооборудования N 2']['Г']; $sredn['Итог']['ОАД']=$sredn['Итог']['ОАД']+$sredn['Цех монтажа, ремонта вооружения и ремонта электрооборудования N 2']['ОАД']; $sredn['Итог']['ПР']=$sredn['Итог']['ПР']+$sredn['Цех монтажа, ремонта вооружения и ремонта электрооборудования N 2']['ПР']; $sredn['Итог']['В']=$sredn['Итог']['В']+$sredn['Цех монтажа, ремонта вооружения и ремонта электрооборудования N 2']['В']; $sredn['Итог']['дни']=$sredn['Итог']['дни']+$sredn['Цех монтажа, ремонта вооружения и ремонта электрооборудования N 2']['дни']; $sredn['Итог']['днич']=$sredn['Итог']['днич']+$sredn['Цех монтажа, ремонта вооружения и ремонта электрооборудования N 2']['днич']; $sredn['Итог']['srednespisRaschet']=$sredn['Итог']['srednespisRaschet']+round($sredn['Цех монтажа, ремонта вооружения и ремонта электрооборудования N 2']['srednespisRaschet']/$datn,1); $sredn['Итог']['neiavki']=$sredn['Итог']['neiavki']+$sredn['Цех монтажа, ремонта вооружения и ремонта электрооборудования N 2']['neiavki'];
                                    
            это только маленькая частичка того поистине огромного наследства, которое досталось мне и моему коллеге.
человек писавший это отличался полным нехотение использовать хотя бы оператор += 
и циклы конечно же.
p.s. да-да это одна строка
p.p.s. и, как вы понимаете, она там не одна
        
+167
$zfunctv=0;
if ($zfunctv==1 || $zfunctv==2 || $zfunctv==3 || $zfunctv==4) require_once('function1.php');
if ($zfunctv==2 || $zfunctv==3) require_once('function2.php');
if ($zfunctv==3 || $zfunctv==4) require_once('function3.php');
if ($zfunctv==''){
	require_once('function1.php');//Для всех окон
	require_once('function2.php');//Для полноразмерных окон
	require_once('function3.php');//Для оборудования
	require_once('admfunct.php');
}
                                    Какого хрена? Как это работает?! Код коммерческой системы UserSide v2.48
+167
<?php /*
host=localhost
db_name=db
user=db_user
password=db_pass
charset=utf8
*/ ?>
                                    
            Совершенно гениальный и единственно верный способ хранить конфиги в PHP.
Имя файла тоже, должно быть именно таким -  db.ini.php
        
+167
if (!$xhr)
{
	//...
} 
else
{
	echo $goodUpload ? 'ok' : 'ne ok';							
}
                                    Ответ формы на AJAX-запрос