- 1
- 2
- 3
- 4
- 5
function get_comments($ident,$id,$type,$feedback=false){
...
$tree_id = $id*10+$type;
...
$TL = TreeLoad(0,$tree_id,-1,$ident,$feedback);
Нашли или выдавили из себя код, который нельзя назвать нормальным, на который без улыбки не взглянешь? Не торопитесь его удалять или рефакторить, — запостите его на говнокод.ру, посмеёмся вместе!
+145
function get_comments($ident,$id,$type,$feedback=false){
...
$tree_id = $id*10+$type;
...
$TL = TreeLoad(0,$tree_id,-1,$ident,$feedback);
Стыдно
+133.4
// parse from the uninstall, the actuall installation path
try {
int jarLoc = uninstallDir.indexOf("-jar");
String s = uninstallDir.substring(jarLoc + 5);
// turn it into a normalized file
int uniLoc = s.indexOf("uninstaller");
s = s.substring(1,uniLoc);
s = s.replace("\\", "/");
if(s.endsWith("/"))
s = s.substring(0,s.length()-1);
File f = new File(s);
if(!f.exists()) {
throw new Exception("installation directory does not exist: " + s);
}
return f;
}
catch(Exception e) {
throw new Exception("installation directory could not be parsed from: " + uninstallDir);
}
Typical code of one highly paid "tech expert" from USA
Exception handling makes me cry, this guy doesn't even now about any other exception classes
+89.9
static char DigitToChar(
int _Digit
)
{
switch (_Digit)
{
case 0: return '0';
case 1: return '1';
case 2: return '2';
case 3: return '3';
case 4: return '4';
case 5: return '5';
case 6: return '6';
case 7: return '7';
case 8: return '8';
case 9: return '9';
default: return ':';
}
}
в результате долгих размышлений всетаки заменили на return '0' + _Digit;
+35
<tr>
<td width="21"><a href="javascript:couleur(0);" onMouseOver="window.status='<?php echo __("noir")?>';return true;"><img name="cou0" src="<?php echo HTTP_CHAT?>images/noir-select.gif" width=21 height=21 border=0 alt=""></a></td>
<td width="21"><a href="javascript:couleur(1);" onMouseOver="window.status='<?php echo __("violet")?>';return true;"><img name="cou1" src="<?php echo HTTP_CHAT?>images/violet.gif" width=21 height=21 border=0 alt=""></a></td>
<td width="21"><a href="javascript:couleur(2);" onMouseOver="window.status='<?php echo __("bleu")?>';return true;"><img name="cou2" src="<?php echo HTTP_CHAT?>images/bleu.gif" width=21 height=21 border=0 alt=""></a></td>
<td width="21"><a href="javascript:couleur(3);" onMouseOver="window.status='<?php echo __("vert")?>';return true;"><img name="cou3" src="<?php echo HTTP_CHAT?>images/vert.gif" width=21 height=21 border=0 alt=""></a></td>
<td width="21"><a href="javascript:couleur(4);" onMouseOver="window.status='<?php echo __("rouge")?>';return true;"><img name="cou4" src="<?php echo HTTP_CHAT?>images/rouge.gif" width=21 height=21 border=0 alt=""></a></td>
<td width="21"><a href="javascript:couleur(5);" onMouseOver="window.status='<?php echo __("orange")?>';return true;"><img name="cou5" src="<?php echo HTTP_CHAT?>images/orange.gif" width=21 height=21 border=0 alt=""></a></td>
</tr>
маленький кусочек говна из большого куска французского движка
+24
<html>
<body>
<button onclick="document.getElementById('f').size=1" >222</button>
<select id="f" size="3">
<option>1</option>
<option>3</option>
<option>4</option>
<option>5</option>
<select>
</body>
</html>
−14
uint i;
…
if (i.ToString().Length == 1)
{
...
}
Не сразу можно понять, что в этом коде просто-напросто выполняется проверка i < 10. Алгоритм достаточно прост: выполняется преобразование i в строку, после чего вычисляется ее длина. Если число больше 9, то его десятичная запись содержит больше одного символа. Отрицательные числа переменная типа uint содержать не может. Проверку проходят лишь числа от 0 до 9.
Алгоритм ресурсоемок, неочевиден и не поддается сопровождению даже теоретически.
взято с lurkmore.ru, там вообще много про "индусский код"
+27.8
function createErrorObj($strField, $bDisplay = 'false', $format = false, $strTitle = null) {
global $Error, $__error;
if (!$Error) {
ErrorsHandling::_create();
}
if ($bDisplay !== 'true') {
$bDisplay = 'false';
}
....
страшный для кого-то был $bDisplay
+72.8
mov ebx, eax
mov eax, ebx
Такое борландовский кодогенератор иногда выдаёт
+34.7
} else {
$json_array = array('error' => $this->log_error('busy', true));
}
} else {
$json_array = array('error' => $this->log_error('busy', true));
}
} else {
$json_array = array('error' => $this->log_error('busy', true));
}
} else {
$json_array = array('error' => $this->log_error('busy', true));
}
} else {
$json_array = array('error' => $this->log_error('bad_symbols', false));
}
} else {
$json_array = array('error' => $this->log_error('check_error', false));
// 'errorStr' => $this->validation->error_string
}
Кусок метода в контроллере регистрации пользователя на одном из наших проектов. Вверху хренова гора ифов а внизу во такое вот счастье. "Авторская" табуляция сохранена
+32
/**
* Сабж: Проверка формы
* Для каждого поля в форме было понаколбашено по отдельной(!) функции для проверки
* Но и это цветочки...
*/
// Поехали!(С)
// запуск проверки всех полей формы <- это древний комент
function checkAll()
{
var elements = new Array('avto', 'duration', 'start_adress', 'end_adress', 'orderer_name', 'orderer_phone', 'orderer_mail');
for (i = 0; i < 7; i++)
{
var curEl = document.getElementById(elements[i]);
curEl.focus(); // тут, suka, особенно жестоко!
curEl.blur(); //
}
var StatSum = "";
for (j = 0; j < 7; j++)
{
// в форме сделаны были ккрытые инпупуты, куда
// проверяющая функция писала 0 или 1 в зависимости от результата проверки
var curStat = document.getElementById(elements[j]+"Status");
if (curStat.value == "1")
{StatSum = StatSum + "1";}
if (curStat.value == "0" || curStat.value == "")
{StatSum = StatSum + "";}
}
// ну тут ваще пиздец)))
// семь нулей - это семь удачно проверенных полей формы
if (StatSum == "" || StatSum == "0000000")
{
// щёлкаем, suka, по самбмиту..
var btn = document.getElementById("submit");
btn.click();
}
return (false);
}
// занавес...
/************************************************************/
Блин, попёрло сейчас покурить свои извращения на JS двухлетней давности...
У меня истерика = )