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

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

+154

1
2
3
4
5
6
7
8

public static function checkPath($path)
{
	if (substr($path, -1) == '/')
		$path = $path;
	else
		$path = $path.'/';
	return $path;
}

facepalm.jpg

DrFreez, 24 Марта 2013

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

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

+158

01
02
03
04
05
06
07
08
09
10

public static function checkWriteToTorrentPath($path)
{
	if (file_put_contents($path.'file.txt', ' '))
	{
		unlink($path.'file.txt');
		return TRUE;
	}
	else
		return FALSE;
}

is_writable ? Не, не слышал!
https://github.com/ElizarovEugene/TorrentMonitor/blob/master/class/System.class.php#L48 Эпик!

DrFreez, 24 Марта 2013

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

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

+151

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
30
31
32
33
34
35
36
37
38
39
40
41
42
43
44
45
46
47
48
49
50
51
52
53
54
55
56
57
58
59
60
61
62
63
64
65
66
67
68
69
70
71
72
73
74
75
76
77
78
79
80
81
82
83
84
85
86

if (anidub::$exucution)
		{
			//получаем страницу для парсинга
			$page = anidub::getContent($torrent_id, anidub::$sess_cookie);

			if ( ! empty($page))
			{
				//ищем на странице дату регистрации торрента
				if (preg_match("/<td width=\"\" class=\"heading\" valign=\"top\" align=\"right\">Добавлен<\/td><td valign=\"top\" align=\"left\">(.*)<\/td>/", $page, $array))
				{
					//проверяем удалось ли получить дату со страницы
					if (isset($array[1]))
					{
						//если дата не равна ничему
						if ( ! empty($array[1]))
						{
							//сбрасываем варнинг
							Database::clearWarnings($tracker);
							//приводим дату к общему виду
							$date = $array[1];
							$date_str = anidub::dateNumToString($array[1]);
							//если даты не совпадают, перекачиваем торрент
							if ($date != $timestamp)
							{
                                preg_match('/<a href=\"download\.php\?id=(\d{2,6})&name=(.*)\">/U', $page, $array);
                                $torrent_id = $array[1];
                                $torrent_id_name = $array[2];
								//сохраняем торрент в файл
								$torrent = anidub::getTorrent($torrent_id, $torrent_id_name, anidub::$sess_cookie);
								$client = ClientAdapterFactory::getStorage('file');
								$client->store($torrent, $id, $tracker, $name, $torrent_id, $timestamp);
								//обновляем время регистрации торрента в базе
								Database::setNewDate($id, $date);
								//отправляем уведомлении о новом торренте
								$message = $name.' обновлён.';
								Notification::sendNotification('notification', $date_str, $tracker, $message);
							}
						}
						else
						{
							//устанавливаем варнинг
							if (anidub::$warning == NULL)
							{
								anidub::$warning = TRUE;
								Errors::setWarnings($tracker, 'not_available');
							}
							//останавливаем процесс выполнения, т.к. не может работать без кук
							anidub::$exucution = FALSE;
						}
					}
					else
					{
						//устанавливаем варнинг
						if (anidub::$warning == NULL)
						{
							anidub::$warning = TRUE;
							Errors::setWarnings($tracker, 'not_available');
						}
						//останавливаем процесс выполнения, т.к. не может работать без кук
						anidub::$exucution = FALSE;
					}
				}
				else
				{
					//устанавливаем варнинг
					if (anidub::$warning == NULL)
					{
						anidub::$warning = TRUE;
						Errors::setWarnings($tracker, 'not_available');
					}
					//останавливаем процесс выполнения, т.к. не может работать без кук
					anidub::$exucution = FALSE;
				}
			}			
			else
			{
				//устанавливаем варнинг
				if (anidub::$warning == NULL)
				{
					anidub::$warning = TRUE;
					Errors::setWarnings($tracker, 'not_available');
				}
				//останавливаем процесс выполнения, т.к. не может работать без кук
				anidub::$exucution = FALSE;
			}
		}

Классическая лесенка пыхомакаки.

Stallman, 23 Марта 2013

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

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

+149

1

preg_match_all('/<td class=\"f\">\n\t\t\t\t\n\t\t\t\t\t(.*)\n\t\t\t\t<\/td>/', $page, $section)

https://github.com/ElizarovEugene/TorrentMonitor/blob/master/trackers/tfile.me.search.php#L34
ну и много всякой другой вкуснятины

DrFreez, 22 Марта 2013

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

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

+156

1
2

if( SITE == 'http://dev.example.com' ) die( file_get_contents( 'http://www.example.com/error.php?error=xml' ) );
die( file_get_contents( SITE.'/error.php?error=xml' ) );

некоторые 404-ую выводят вот так

dead_star, 22 Марта 2013

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

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

+146

01
02
03
04
05
06
07
08
09
10
11
12
13
14
15
16

getInsideText($part1[0],'<string>','</string>',1,true);

function getInsideText($str,$fstr,$lstr,$limit=0,$trim=true){
    $temp_arr=array();
    $lcnt=0;
    while(strpos($str,$fstr)!==false && ($limit ? $lcnt<$limit : true)){
        $fpos=($fstr ? strpos($str,$fstr)+strlen($fstr) : 0);
        $str=substr($str,$fpos);
        $lpos=strpos($str,$lstr);
        $val=($lpos!==false ? substr($str,0,$lpos) : $str);
        $temp_arr[]=($trim ? trim($val) : $val);
        $str=substr($str,$lpos+strlen($lstr));
        $lcnt++;
    }
    return ($limit==1 ? (isset($temp_arr[0]) ? $temp_arr[0] : '') : $temp_arr);
}

Конечный автомат своими силами.

alexx, 21 Марта 2013

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

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

+149

1
2

$ev = '$atHtml[] = template::translateTemplate('.$atArr[0].'::getHTML($atArr[1]));';
eval($ev);

Самый простой способ заставить работать статический метод класса из переменной в PHP ниже 5.3.

Lander, 20 Марта 2013

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

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

+157

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

function _subqery_helper ($uri){
static $i;
		$i++;

		if ((is_array($uri)) && (!empty($uri))) {
			$keyword = array_pop($uri); //извлекаем последний элемент
			
			if ($i==1) 
			{
				$subqery = 'AND item.keyword= '.$this->db->escape($keyword).' '.(count($uri)? 'AND parent IN 
				('.$this->_subqery_helper($uri).')' : '');
			}
			
			elseif ($i > 1) 
			{
				$subqery = 'SELECT id FROM '.$this->db->dbprefix($this->table).' WHERE keyword = '.$this->db->escape($keyword).' AND parent '.(count($uri)? 'IN 
				('.$this->_subqery_helper($uri).')' : ' = 0');
			}
		}
		return $subqery;
}

массив $url очень простой:
1=>"str1",2=>"str2",3=>"str3"

это непонятная итерационная функция, которая вообще непонятно зачем такая сложная....

xoptov, 15 Марта 2013

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

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

+159

01
02
03
04
05
06
07
08
09
10
11
12
13

$threadusers = $db->query_read("
  SELECT user.username, user.usergroupid, user.membergroupids,
   session.userid, session.inthread, session.lastactivity, session.badlocation,
   IF(user.displaygroupid = 0, user.usergroupid, user.displaygroupid) AS displaygroupid, infractiongroupid,
   IF(user.options & " . $vbulletin->bf_misc_useroptions['invisible'] . ", 1, 0) AS invisible
  FROM " . TABLE_PREFIX . "session AS session
  LEFT JOIN " . TABLE_PREFIX . "user AS user ON(user.userid = session.userid)
  WHERE  session.lastactivity > $datecut
 ");

...

while ($loggedin = $db->fetch_array($threadusers))

Я уже несколько раз писал о гениальности ребят которые пишут форум vBulletin. Продолжу эту традицию.
Выше приведен огрызок кода, которым ребятки собирают список активных пользователей в текущей теме. Думаю тут все понятно, но все же объясню на примере того форума, где довелось увидеть это чудо. В таблице session около 7к записей, активных записей в среднем 4к. Т.е. на выходе мы имеем запрос, который кладет сервак при 64 гигах оперативы + последовательный перебор 4к записей для того что бы выбрать парочку юзеров, которые активны в этой теме.
Ну в общем ребятки поставили себе прижизненный памятник, который я спешу предоставить на лицезрение сообщества.

vagrand, 13 Марта 2013

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

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