PHP — Говнокод.ру

PHP / Говнокод #3426

+149

1
2
3
4

<?php
      @require_once('something-that-doesnot-exists.php');
      echo 'My etogo uzhe ne prochitaem((';
?>

Баг ПХП. 4 часа искал методом научного echo. Ошибки не выдает, завершает выполнение как будто вызвали exit() or die(). Растолкуйте в чем суть?
Но всё-таки виновати разработчики CodeIgniter. Они нарушили принцип KISS (keep it simple). Я например, никогда не использую @ (даже незнаю к чему этот отросток).

increazon, 08 Июня 2010

Комментарии (56)

PHP / Говнокод #3425

+164

001
002
003
004
005
006
007
008
009
010
011
012
013
014
015
016
017
018
019
020
021
022
023
024
025
026
027
028
029
030
031
032
033
034
035
036
037
038
039
040
041
042
043
044
045
046
047
048
049
050
051
052
053
054
055
056
057
058
059
060
061
062
063
064
065
066
067
068
069
070
071
072
073
074
075
076
077
078
079
080
081
082
083
084
085
086
087
088
089
090
091
092
093
094
095
096
097
098
099
100

foreach($listuserson as $uinfo => $pinfo) {
	$class_check = TRUE;
	list($oid, $oname) = explode(".", $uinfo, 2);
	$sql2 = new db;
	$sql2->db_Select("user", "user_id", "user_admin > 0 ");
	$row_2 = $sql2->db_Fetch();
	$online_location = $pinfo;
	$online_location_page = substr(strrchr($online_location, "/"), 1);
	if (!strstr($online_location, "forum_") || !strstr($online_location, "content.php") || !strstr($online_location, "comment.php")) {
		$online_location_page = str_replace(".php", "", substr(strrchr($online_location, "/"), 1));
	}
	if ($online_location_page == "log" || $online_location_page == "error") {
		$online_location = "news.php";
		$online_location_page = NEWS;
	}
	if ($online_location_page == "request") {
		$online_location = "download.php";
	}
	if ($online_location_page == "article") {
		$online_location_page = ARTICLEPAGE;
	}
	if ($online_location_page == "chat") {
		$online_location_page = CHAT;
	}
	//if($online_location_page == "comment"){$online_location_page = COMMENT;}
	if ($online_location_page == "content") {
		$online_location_page = CONTENT;
	}
	if ($online_location_page == "download") {
		$online_location_page = DOWNLOAD;
	}
	if ($online_location_page == "email") {
		$online_location_page = EMAIL;
	}
	if ($online_location_page == "forum") {
		$online_location_page = FORUM;
	}
	if ($online_location_page == "links") {
		$online_location_page = LINKS;
	}
	if ($online_location_page == "news") {
		$online_location_page = NEWS;
	}
	if ($online_location_page == "oldpolls") {
		$online_location_page = OLDPOLLS;
	}
	if ($online_location_page == "print") {
		$online_location_page = PRINTPAGE;
	}
	if ($online_location_page == "login") {
		$online_location_page = LOGIN;
	}
	if ($online_location_page == "search") {
		$online_location_page = SEARCH;
	}
	if ($online_location_page == "stats") {
		$online_location_page = STATS;
	}
	if ($online_location_page == "submitnews") {
		$online_location_page = SUBMITNEWS;
	}
	if ($online_location_page == "upload") {
		$online_location_page = UPLOAD;
	}
	if ($online_location_page == "user") {
		$online_location_page = USERPAGE;
	}
	if ($online_location_page == "usersettings") {
		$online_location_page = USERSETTINGS;
	}
	if ($online_location_page == "new") {
		$online_location_page = LISTNEW;
	}
	if ($online_location_page == "online") {
		$online_location_page = ONLINE;
	}
	if ($online_location_page == "userposts") {
		$online_location_page = USERPOSTS;
	}
	if ($online_location_page == "subcontent") {
		$online_location_page = SUBCONTENT;
	}
	if ($online_location_page == "top") {
		$online_location_page = TOP;
	}
	//commonly used plugin pages
	if ($online_location_page == "bugtracker") {
		$online_location_page = BUGTRACKER;
	}
	if ($online_location_page == "event") {
		$online_location_page = EVENT;
	}
	if ($online_location_page == "calendar") {
		$online_location_page = CALENDAR;
	}
	if ($online_location_page == "faq") {
		$online_location_page = FAQ;
	}
	if ($online_location_page == "pm") {
		$online_location_page = PM;

Шарился по винту, наткнулся на архив с CMS e107, машинально зашёл в него и открыл первый попавшийся файл...
ROBUST
@
SCALABLE

telnet, 08 Июня 2010

Комментарии (7)

PHP / Говнокод #3410

+164

01
02
03
04
05
06
07
08
09
10

mysql_query("INSERT INTO `classes` (`name`) VALUES('".$_POST["class_name"]."')");
		$Mresult=mysql_query("SELECT MAX(`id`) FROM `classes`");
		$row=mysql_fetch_row($Mresult);
		$class_num=$row[0];
		$date1=$_POST["class_start"];
		$date2=$_POST["class_end"];
		mysql_query("CREATE TABLE `class_".$class_num."_lessons` (`id` INT, `data` TEXT)");
		mysql_query("CREATE TABLE `class_".$class_num."_video` (`id` INT, `data` TEXT)");
		mysql_query("CREATE TABLE `class_".$class_num."_ege` (`id` INT, `data` TEXT)");
		mysql_query("CREATE TABLE `class_".$class_num."_rasp` (`id` INT NOT NULL AUTO_INCREMENT PRIMARY KEY, `data` TEXT)");

DSL88, 07 Июня 2010

Комментарии (165)

PHP / Говнокод #3409

+162

01
02
03
04
05
06
07
08
09
10
11
12
13
14
15
16
17
18
19
20
21
22
23
24
25
26
27
28

function plusDay($data){
	$dat=explode(".",$data);
	if ($dat[0]<28){
		$dat++;
		} else {
			if (($dat[1]!=2)&&($dat[0]<31)){
				$dat[0]++;
				}
			if (($dat[1]==2)&&($dat[0]==28)){
				$dat[1]++;
				$dat[0]=1;
				}
			if (($dat[1]==1)&&($dat[1]==3)&&($dat[1]==5)&&($dat[1]==7)&&($dat[1]==8)&&($dat[1]==10)&&($dat[1]==12)&&($dat[0]==31)){
				$dat[0]=1;
				if ($dat[1]==12){
					$dat[2]++;
					$dat[1]=1;
					}else{
						$dat[1]++;
						}
				}
			if (($dat[1]==4)&&($dat[1]==6)&&($dat[1]==9)&&($dat[1]==11)&&($dat[0]==30)){
				$dat[0]=1;
				$dat[1]++;
				}
			}
	return implode(".",$dat);
}

DSL88, 07 Июня 2010

Комментарии (21)

PHP / Говнокод #3399

+170

1
2

require_once('configure.php'); 
require('configure.php');

надо быть _увереным_ в своем коде

sogekoder, 05 Июня 2010

Комментарии (36)

PHP / Говнокод #3398

+162

1
2
3
4

$res = $db->loadResult();
if($res); else return false;
if($res!=$user->id) return false;
return true;

vsu, 04 Июня 2010

Комментарии (43)

PHP / Говнокод #3380

+166

1
2
3
4
5
6
7

<?php
session_start();
// супер-мега защита от ддоса
$_SESSION['antiddos']==time() ? exit() : $_SESSION['antiddos']=time();
//конец супер мега защиты
....
?>

Найдено в старом проекте, код не мой.
Но вы знаете, от толпы школьников с кнопками F5 это реально помогает!

mr.The, 02 Июня 2010

Комментарии (21)

PHP / Говнокод #3379

+158

01
02
03
04
05
06
07
08
09
10
11
12
13
14
15
16
17
18
19
20
21
22
23

/**
 * Метод проверяет права доступа на использование этого модуля, возвращает результат проверки.
 * Кроме этого, происходит запись в лог
 *
 * @return boolean
 */
private function check_access() {
	Logger::log_exception($_SERVER['REQUEST_URI'].'?'.$_SERVER['QUERY_STRING'], $this->log_file_name);
	return true;
}

// [...]

/**
 * Попытка переключить рабочую базу данных
 */
public function actionChange_db() {
	if (!$this->check_access()) {
		echo strval($this->err_failure_access);
		return;
	}

// [...]

Вот такое иногда встречается в коде коллеги, про название метода "log_exception" я промолчу, т.к. это тоже его рук дело, а общий для всех метод Logger::log() ему почему-то не понравился? Я уже просто устал ругаться с ним...

cr0t, 02 Июня 2010

Комментарии (19)

PHP / Говнокод #3370

+159

1
2
3
4
5
6
7
8

public static function getSizeformat($iNumber){
    $value = "-";
    if($iNumber){
      $iNumber = $iNumber/100000;
      $value = number_format($iNumber, 2, '.', '');
    }
    return $value;
  }

Кусочек который пришел по наследству, судя по подписям в таблице на фронтэнде в итоге мы должны получить размер файлов в мегабайтах, но как то не получается... :)

psych, 01 Июня 2010

Комментарии (23)

PHP / Говнокод #3365

+175

01
02
03
04
05
06
07
08
09
10
11
12
13
14
15
16
17
18
19
20
21
22
23
24
25
26
27
28
29

if($hour == 1){
$hour = "01";
}
if($hour == 2){
$hour = "02";
}
if($hour == 3){
$hour = "03";
}
if($hour == 4){
$hour = "04";
}
if($hour == 5){
$hour = "05";
}
if($hour == 6){
$hour = "06";
}
if($hour == 7){
$hour = "07";
}
if($hour == 8){
$hour = "08";
}
if($hour == 9){
$hour = "09";
}if($hour == 0){
$hour = "00";
}

Marker689, 01 Июня 2010

Комментарии (13)

Говнокод: по колено в коде.