Нашли или выдавили из себя код, который нельзя назвать нормальным, на который без улыбки не взглянешь? Не торопитесь его удалять или рефакторить, — запостите его на говнокод.ру, посмеёмся вместе!
Всего: 13
+78
PreparedStatement statement = createStatement();
for (Value value: values) {
if (statement == null) {
statement = createStatement();
}
try {
if (statement.isClosed()) {
statement = createStatement();
}
} catch (Exception e) {
logger.error("...", e);
}
// ...
}"бог любит троицу"
user654321,
23 Сентября 2011
+166
for(var i=obj.parentNode.parentNode.childNodes.length;i>=0;i--)
{
if(obj.parentNode.parentNode.childNodes[i]!=undefined)
{
if(obj.parentNode.parentNode.childNodes[i].tagName=="TR")
{
if (obj.parentNode.parentNode.childNodes[i].childNodes[0]!=undefined){
for (var j=obj.parentNode.parentNode.childNodes[i].childNodes.length;j>=0;j--)
{
if(obj.parentNode.parentNode.childNodes[i].childNodes[j]!=undefined){
if(obj.parentNode.parentNode.childNodes[i].childNodes[j].childNodes[0].tagName=="TABLE"){
obj.parentNode.parentNode.childNodes[i].childNodes[j].childNodes[0].childNodes[0].childNodes[0].childNodes[0].attributes.getNamedItem("background").value = "images/category_n.png";
}
}
}
}
}
}
}Случайно заметил, что на samsung.ru в файрбаг валятся ошибки. Сходил посмотреть и нашел это.
user654321,
13 Апреля 2011
+169
<?php
if($key === false){
return false;
}else{
return $key;
}
user654321,
03 Ноября 2010
+162
<?php
class Model_Crypt_FactoryCipher {
const BlOWFISH = 'Model_Crypt_BlowFishCipher';
/**
* создает экземпляр cipher для указанного алгоритма
*
* @return Model_Crypt_BlowFishCipher
*/
public static function createCipher($crypt) {
$cipher = new $crypt;
if ($cipher instanceof Model_Crypt_BlowFishCipher) {
return new Model_Crypt_BlowFishCipher ();
} else {
throw new Exception ( "This algoritm <$crypt> not implement yet" );
}
}
}Гениальная фабрика. Используется один раз путем передачи константы "BlOWFISH" в метод createCipher()
user654321,
29 Октября 2010
+156
<?php
// ...
$buffer='$buffer';
$text = '$text';
// ...
$file = <<<CODE
<?php
// ...
function prepareText($text)
{
$text = str_replace("\\n\\r", " ", $text);
$text = str_replace("\\r\\n", " ", $text);
$text = str_replace("\\n", " ", $text);
return $text;
}
// ...
CODE;
Суть такова: нужно было генерировать пхп код, который впоследствии должен был выполняться на клиенте.
Угадайте, зачем нужны переменные $buffer, $text и еще много таких, в которых лежит имя самой переменной в строке?
user654321,
26 Октября 2010
+163
<?php
// ...
////////////////
$_engines = unserialize(SENGINES);
if (0 == count($loading)) return -3;
$select=-1;
$delta=0;
$me=count($_engines);
$ms=count($args);
if (count($loading))
foreach ($loading as $k=>$v)
{
if (!isset($v['all'])) continue;
if (0==$v['all']) continue;
$tmp=0;
switch(1)
{
case $ms > 0:
foreach ($args as $engine)
{
if (isset($v[$engine]))
$tmp += (float) $v[$engine]/(float) $v['all'];
}
$tmp=$tmp/$ms;
break;
case $me > 0:
foreach ($_engines as $engine)
{
if (isset($v[$engine]))
$tmp += (float) $v[$engine]/(float) $v['all'];
}
$tmp=$tmp/$me;
break;
}
if ($tmp > $delta) {$delta=$tmp;$select=$k;}
}
if (-1 == $select) return false;
////////////////
// ...
Автор этого убеждает, что тут где-то производится сортировка.
Кстати, это, #4071 и #4215 кодил один и тот же автор. Поэтому тут можно наблюдать десериализацию массива из константы, возвращение непонятных отрицательных статусов, очень информативное именование переменных, копипаст и самое классное — "if для лохов, я использую switch(1)"
user654321,
20 Октября 2010
+161
<!--<?php if( $updatesData[ $updateCol->getId() ]['remainingToIndex'] < 10 ): ?>
<?php $indexedToday = $updatesData[ $updateCol ]['remainingToIndex']; ?>
<?php else: ?>
<?php $indexedToday = rand( 5, $updatesData[ $updateCol ]['remainingToIndex'] ); ?>
<?php endif; ?>-->Закомментировал.
user654321,
11 Октября 2010
+160
<?php
define ('SENGINES',serialize(
array(
'yandex',
'google',
// ... тут их еще несколько
)));
class <засекречено> {
/**
* engines
*
* @var array
*/
private $_engines = unserialize(SENGINES);
// ...
}
user654321,
10 Сентября 2010
+160
<?php
if (preg_match("/\.[gjpi][ipnc][fgo]/i", $_SERVER['REQUEST_URI'])) exit;
user654321,
08 Сентября 2010
+157
//
// <?php if (!Yii::app()->user->checkAccess('partner')) { ?>
a+='<td'+cl+'>'+msg['keywords'][i][mord] + '</td>';
// <?php } ?>
//'</strong></td><td'+cl+'>'+currentMordCost+
a+='<td'+cl+'>'+(msg['keywords'][i]['costM'])
+'</td>'+
'<td'+cl+'>'+$.getTiming(msg['domainTrust'], msg['keywords'][i][mord]) + '</td>' +
'<td'+cl+'><img class="info" src=\"/images/information.png\" alt=\"'+i+'\" id=\"'+i+'\" > <B>' +msg['keywords'][i]['notice']+'</B></td></tr>';
msg['keywords'][i]['domainTrust'] = msg['domainTrust'];
hints[i]=jQuery.getHint(msg['keywords'][i]);
if (msg['keywords'][i]['bad']==true)
badCount++;
//mordNestedCost = mord/2 / msg['keywords'][i]['nested'].length;
//costMNested=(msg['keywords'][i]['costM']) / 2 / msg['keywords'][i]['nested'].length;
//if (halfMord == "-")
// mordNestedCost = costMNested = "-";
mod='';modA='';
for (k=0;k<msg['keywords'][i]['nested'].length;k++)
{
if (k==parseInt(msg['keywords'][i]['up']))
{
mod = '<span style="color:#626262;font-weight:bold;">';
modA = '<\/span>';
}
if (msg['keywords'][i]['nested'][k]['freq'] <= 50 && msg['keywords'][i]['nested'][k]['mordCost'] >= 1)
{
msg['keywords'][i]['nested'][k]['notice'] += " <BR><span style='color:red'>Возможно посчитано неверно!</span>";
hasCriticalErrors = true;
}
a+= '<tr><td'+cl+'>' + mod + ' '+(msg['keywords'][i]['nested'][k]['title'])+ modA +
'</td>';
// // <?php if (!Yii::app()->user->checkAccess('partner')) { ?>
a+='<td'+cl+'>'+(msg['keywords'][i]['nested'][k][mord])+'</td>';
// <?php } ?>
a+='<td'+cl+'>'+
//'</td><td'+cl+'>'+currentNestedMordCost+'</td><td'+cl+'>'+
(msg['keywords'][i]['nested'][k]['costM'])+'</td>'+
'<td'+cl+'>'+$.getTiming(msg['domainTrust'], msg['keywords'][i]['nested'][k][mord]) + '</td>' +
'<td'+cl+'>'+'<img class="info" src=\"/images/information.png\" alt=\"info_'+i+'_'+k+'\" id=\"'+i+'_'+k+'\" >'+' <B>'+msg['keywords'][i]['nested'][k]['notice']+'</B></td></tr>';
msg['keywords'][i]['nested'][k]['domainTrust'] = msg['domainTrust'];
hints[i+'_'+k]=jQuery.getHint(msg['keywords'][i]['nested'][k]);
if (msg['keywords'][i]['nested']['bad'] == true)
badCount++;
}Как вам такая каша из яваскрипта, пхп и хтмл?
user654321,
31 Августа 2010
Follow us!