- 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-запрос