- 1
<div class="futer">
Нашли или выдавили из себя код, который нельзя назвать нормальным, на который без улыбки не взглянешь? Не торопитесь его удалять или рефакторить, — запостите его на говнокод.ру, посмеёмся вместе!
+154
<div class="futer">
vxod, vixod и прочие шедевры. У меня глаза изливаются кровью
+150
if(!$this->user->isVip())
{
$balls_us=$this->user->balls_video;
$balls_comm=$this->user->balls_comm;
$balls_summ=$balls_comm+$balls_us;
if ($balls_summ>=1000)
{
$this->user->add('roles',ORM::factory('role',2));
}
else if($this->user->isVip())
{
if($balls_summ<1000)
{
$this->user->remove('roles',ORM::factory('role',2));
}
else
{}
}
}
как работает этот else if($this->user->isVip()), если вначале условие что если не вип..
ппц..
+157
/**
* Появилась из-за опечатки. См. getCurrentEnterpriseIds()
*/
public function getCurrnetEnterpriseIds($user_id) {
return $this->getCurrentEnterpriseIds($user_id);
}
рефакторинг? не, не слышал
+149
$data = "";
if (isset($_POST["json"]))
$data = $_POST["json"];
$send_str = "";
$jsonObj=json_decode($data);
if(json_last_error() == JSON_ERROR_NONE)
{
if ($jsonObj)
{
if (strlen($jsonObj->pass) != 32 || strlen($jsonObj->login)==0 || strlen($jsonObj->userName)==0)
{
$send_str = '{"RegistrationActionError":{"type":"2"}}';
}
else
{
// проверка на совпадение логина
if ($db->userExists($jsonObj->login))
$send_str = '{"RegistrationActionError":{"type":"1"}}';
else
{
$user = $db->createUser($jsonObj->login, $jsonObj->pass, $jsonObj->userName, $jsonObj->mail);
$send_str = '{"RegistrationActionComplete"}';
}
}
}
else
$send_str = '{"RegistrationActionError":{"type":"2"}}';
}
else
$send_str = '{"RegistrationActionError":{"type":"2"}}';
echo $send_str;
Регистрация пользователя написанная человеком который себя позиционирует как пхпе миддл )
+150
if (!isset($pmas_table[0][3])) $pmas_table[0][3]='';
if (!isset($pmas_table[0][5])) $pmas_table[0][5]='';
if (!isset($pmas_table[0][6])) $pmas_table[0][6]='';
if (!isset($pmas_table[0][7])) $pmas_table[0][7]='';
UserSide v2.48 again. Так как там каждая строчка имеет право быть на этом сайте, выкладываю The best of the best =)
+161
while (!$db->query("INSERT INTO lots (lot_num) VALUES (".($unicnum=rand(1,99999)).")"));
Проверка и создание уникального рандомного идентификатора
+150
public function run() {
$stime = microtime(TRUE);
$this->header('Content-Type: text/html');
$this->setcookie('testcookie', '1');
$this->registerShutdownFunction(function() {
?></html><?php
});
?><!DOCTYPE html PUBLIC "-//W3C//DTD XHTML 1.0 Transitional//EN" "http://www.w3.org/TR/xhtml1/DTD/xhtml1-transitional.dtd">
<html xmlns="http://www.w3.org/1999/xhtml">
<head>
<meta http-equiv="Content-Type" content="text/html; charset=utf-8" />
<title>It works!</title>
</head>
<body>
<h1>It works! Be happy! ;-)</h1>
............
............
............
phpDaemon Example.php line 63
+173
/*
if($_REQUEST["act_id"]==8){
а нет. маньяки так не пишут...
*/
define("Э", $_REQUEST["act_id"]);
if( 8===Э ){
...
}
оператор члена :)
+161
<?php
//...
while (($data = fgetcsv($handle, 1000, $this->CSV_SEPARATOR, $this->CSV_ENCLOSURE)) !== FALSE) {
$num = count($data);
$item = array();
for ($c=0; $c < count($data); $c++) {
$item[] = $data[$c];
}
//некие действия с $item. $data больше нигде используется
?>
Итого, $num не используется в цикле, да и сам по себе цикл - бесполезен, можно было сделать просто $item = $data; Не говоря уже о том, что можно просто использовать $data.
Код из одного модуля для opencart.
+149
<?php
function check_brackets($s)
{
$brackets = array('(' => ')' , '[' => ']', '{' => '}');
$stack = array();
for($i = 0; $i < strlen($s); $i++)
{
if(in_array($s[$i], array_keys($brackets)))
{
array_push($stack, $brackets[$s[$i]]);
}
elseif(in_array($s[$i], array_values($brackets)))
{
if(empty($stack) || array_pop($stack) != $s[$i])
{
return false;
}
}
}
if(empty($stack)) return true;
}
if($_POST["bracket_string"] && $_POST["bracket_string"] <= 30)
{
if(check_brackets($_POST["bracket_string"]))
{
$message = "Check passed";
}
else $message = "Check failed";
}
?>
Финальная версия.