- 01
- 02
- 03
- 04
- 05
- 06
- 07
- 08
- 09
- 10
try
{
if (empty($this->kinds) && $this->kinds !== false)
throw new Exception('', 1);
}
catch (Exception $e)
{
if (empty($this->kinds))
$this->kinds = false;
}
Нашли или выдавили из себя код, который нельзя назвать нормальным, на который без улыбки не взглянешь? Не торопитесь его удалять или рефакторить, — запостите его на говнокод.ру, посмеёмся вместе!
+147
try
{
if (empty($this->kinds) && $this->kinds !== false)
throw new Exception('', 1);
}
catch (Exception $e)
{
if (empty($this->kinds))
$this->kinds = false;
}
Забавное использование try catch, почти goto
+147
private function getExperts($where = '') {
if ($where) {
$where = "WHERE $where";
}
$query = "SELECT * FROM $this->expertsTable $where";
$experts = self::$DB->select($query);
// workarround кривой структуры бд
// выбрасываем повторяющихся экспертов
$experts_unique = array();
foreach($experts as $expert)
{
$key = md5($expert['expertName'].$expert['photo'].$expert['text']);
if(!isset($experts_unique[$key]))
$experts_unique[$key] = $expert;
}
return array_values($experts_unique);
}
+147
if(isset($_POST['logout']))
{
$_POST['name']=trim($_POST['name']);
$_POST['pass']=trim($_POST['pass']);
if($_POST['name']!="" && $_POST['pass']!="")
{
$cons = "bcdfghjklmnpqrstvwxyz";
$bons = "BCDFGHJKLMNPGRSTVWXYZ";
$vocs = "aeiou";
$noms = "123456789";
for($x=0; $x < 33; $x++){
mt_srand ((double) microtime() * 1000000);
$con[$x] = substr($cons, mt_rand(0, strlen($cons)-1), 1);
$bon[$x] = substr($bons, mt_rand(0, strlen($bons)-1), 1);
$voc[$x] = substr($vocs, mt_rand(0, strlen($vocs)-1), 1);
$nom[$x] = substr($noms, mt_rand(0, strlen($noms)-1), 1);
}
$makerand= $con[0] . $voc[3] . $con[1] . $nom[2] . $nom[5] .$bon[2] . $con[4];
@chmod("../admin/conf/config.inc", 0777);
$cont=file("../admin/conf/config.inc");
$f=fopen("../admin/conf/config.inc","w");
if (!$f) {
echo "Невозможно открыть файл кофигурации";
exit;
}
@$writefile = fwrite($f,$cont[0].$cont[1].$makerand);
if (!$writefile) {
echo "Невозможно записать файл кофигурации";
exit;
}
@chmod("../admin/conf/config.inc", 0604);
fclose($f);
session_unset ();
session_destroy ();
$info="Вы вышли!";
}
}
Столкнулся я с правками на неизвесный мне до этого Kandidat CMS (http://www.kan-studio.ru/).
Кусок выбран на угад... там все феерически шедеврально.
Качайте, читайте... помойгайте разработчикам (кошельки на оф. сайте), походу им очень надо на образование подкинуть.
В остальном коде собачки стоят через строчку.
PS: я сюда зарегался ради ентого поста.
PPS: не так важно, но табуляции тоже сохранены.
+147
if (file_exists("C:\autoexec.bat"))
mysql_query('CREATE TABLE eto_windows');
...гдето в другом файле
if(mysql_query('DROP TABLE eto_windows')) {
echo "это виндоус ;(";
mysql_query('CREATE TABLE eto_windows');
}
...
// не забываем чистить шлак! а то будут утечки!
mysql_query('DROP TABLE eto_windows')
вот такие вот дела...
+147
// Базовые проверки на тип переменных
if ((isset($this->params['month']) && (string)$this->params['month']!==(string)(int)$this->params['month'])
|| (isset($this->params['season']) && (string)$this->params['season']!==(string)(int)$this->params['season'])
|| (isset($this->params['year']) && (string)$this->params['year'] !== (string)(int)$this->params['year']) // проверки на вхождение в диапзон дат
|| (isset($this->params['year']) && !in_array($this->params['year'], $this->data['years']))
|| (isset($this->params['season']) && !isset($this->data['seasons'][(string)(int)$this->params['season']]))
|| (isset($this->params['month']) && !isset($this->data['monthes'][(string)(int)$this->params['month']])) || false
|| (isset($this->params['month']) && isset($this->params['season']) && !in_array((string)(int)$this->params['month'], $this->data['seasons_monthes'][(string)(int)$this->params['season']])))
+147
While ($componentPage != "ticket_edit")
$componentPage = "ticket_edit";
if ($componentPage == "ticket_edit")
$componentPage = "ticket_edit";
нет, я всё понимаю, но зачем такое насилие и контрольный в голову?
+147
if (!$case['item_type'])
unset($case['item_type']);
else
unset($case['content_id']);
+146
// поменяем местами get и post
$_REQUEST=$_GET;
$_POST=$_GET;
$_GET=$_REQUEST;
хорошее начало. Уже после комментария я стал бояться.
Нет, ну блять,
1. нахуя пытаться менять местами _GET и _POST
2. через _REQUEST (что, переменных что ли мало?)
3. Попутно перетирая нахуй его и чем-то провинившийся _POST со всеми остановками по дороге.
PS каким-то образом кары избежал _GET. Порадуемся хоть за его.
+165
if ($flag == false) {
# на всякий случай
if (false == true) exit;
include «execute.php»;
}
На просторах инета... Какой ещё ВСЯКИЙ СЛУЧАЙ?
+154
$andZip = mysql_real_escape_string(nl2br($filterObj->process(trim(intval($_GET['zip'])))));