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

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

+164

1
2
3
4
5
6
7

$str="T:/home//localhost///MyScripts/////";
$pos=strpos($str,"//");
while($pos!=false)
{
    $str=str_replace("//","/",$str);
    $pos=strpos($str,"//");
}

функция удаляет повторяющиеся слэши в строке

Sulik78, 17 Февраля 2011

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

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

+173

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

$pattern = '/[^A-Za-zА-Яа-я0-9]/u';
$phrase = $this->data['ProjectsPhrase']['phrase'];

$phrase = preg_replace($pattern, ' ', $phrase);

$phrase = str_replace('  ', ' ', $phrase);
$phrase = str_replace('  ', ' ', $phrase);
$phrase = str_replace('  ', ' ', $phrase);
$phrase = str_replace('  ', ' ', $phrase);
$phrase = str_replace('  ', ' ', $phrase);
$phrase = str_replace('  ', ' ', $phrase);
$phrase = str_replace('  ', ' ', $phrase);
$phrase = trim($phrase);

Вот такое говнокодище встретил в проекте. Паранойя такая паранойя…

govnouzver, 17 Февраля 2011

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

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

+144

1

Вопрос...

Ребят, у меня такой вопрос, тут или посмейтесь, или скажите как правельней, сайт не для вопросов согласен, но помогите не говнакодить)))
Ядро системы у меня, сканирует директории модулей и плагинов таким образом что у имени деректории есть префикс mod and plg где если это модуль то вот имя директории mod[news], нет оно всё не в куче разумеется, порядок соблюдаю разумеется, но и исправить метод можно не проблема, но я пишу это к чему, недавно готовый сайт выложил на хостинг на котором ранее небыл, и всё конечно идеально, за исключением одного нюанса, FTP менеджеры не могут сканировать директории в имене которых находятся знаки [ ] и только единственный менеджер может сканировать отлично это FAR... И у меня просьба, посмеятся какой я дурак если это реально проблема, или это плюсом, что важные директории проблемно прочитать? Работа системы проходит на ура хоть где не в этом вопрос, а именно имена директорий с знакоми [ ], и проблема их сканировать из "ВНЕ"... Стоит ли их переименовывать?

nethak, 16 Февраля 2011

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

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

+163

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

<?php

if($_SERVER['HTTP_X_REQUESTED_WITH'] == 'XMLHttpRequest') {

sleep(3);

if($_POST[parent_id]) $parent_id = preg_replace('/\D+/i','', $_POST[parent_id]);
else $parent_id = 0;

$author = trim($_POST[author]);
$comment = trim($_POST[comment]);



if(!$author) $error[author] = 'Введите имя!';
if(!$comment) $error[comment] = 'Напишите комментарий!';

if($error)
exit(json_encode($error));


require_once 'blocks/bd.php';


$sql = "INSERT INTO comments (parent_id, name, comment, date_add) VALUES ($parent_id, '$author', '$comment', NOW())";
$result = mysql_query($sql);
if(!$result)
{
$error[] = 'Произошла ошибка, комментарий не сохранен';
exit(json_encode($error));
}
exit();
}

?>

qbasic, 16 Февраля 2011

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

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

+158

1
2
3
4
5
6
7
8
9

public function search() {
    $ret = array();
    foreach($this->dir() as $file)
        if($file->folder())
            $ret = array_merge($ret,$file->search()->asArray());
        else
            $ret[] = $file;
    return new mod_file_list($ret);
}

Красота, в результате работы функции мы получаем объект, состоящий из массива объектов класса file (этот метод в этом классе)... "чудо-поиска" (хотя это вовсе и не поиск, а банальный список файлов всех дочерних папок), особенно будет забавно, если в списке будет несколько тысяч файлов...
В этом классе в принципе много мест, которые наровят насоздавать кучу экземпляров этого же класса, вместо замены одной приватной переменной...

GhOsTMZ, 15 Февраля 2011

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

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

+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
29
30
31
32
33
34
35
36
37
38
39
40
41
42
43

public static function perform($table, $data, $action = 'insert', $parameters = '') {
		reset($data);
		if ($action == 'insert') {
			$query = 'INSERT INTO ' . $table . ' (';
			while (list($columns, ) = each($data)) {
				$query .= $columns . ', ';
			}
			$query = substr($query, 0, -2) . ') values (';
			reset($data);
			while (list(, $value) = each($data)) {
				switch ((string)$value) {
					case 'now()':
						$query .= 'NOW(), ';
						break;
					case 'null':
						$query .= 'NULL, ';
						break;
					default:
						$query .= '\'' . mysql_escape_string($value) . '\', ';
						break;
				}
			}
			$query = substr($query, 0, -2) . ')';
		} elseif ($action == 'update') {
			$query = 'UPDATE ' . $table . ' SET ';
			while (list($columns, $value) = each($data)) {
				switch ((string)$value) {
					case 'now()':
						$query .= $columns . ' = NOW(), ';
						break;
					case 'null':
						$query .= $columns .= ' = NULL, ';
						break;
					default:
						$query .= $columns . ' = \'' . mysql_escape_string($value) . '\', ';
						break;
				}
			}
			$query = substr($query, 0, -2) . ' WHERE ' . $parameters;
		}

		self::request($query);
	}

Вот так у нас вставляют в базу :)

wmmorgun, 15 Февраля 2011

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

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

+159

1
2
3
4
5
6
7
8
9

$this->setLayout('empty');
        $this->setTemplate('step3');

        if ($this->getBrokerId() == 19 || $this->getBrokerId() == 29)
        {
            $this->setLayout('empty');
            $this->setTemplate('step3');
            $this->brokerId = $this->getBrokerId();
        }

Нашел в чужом проекте. При этом $this->getBrokerId() не просто возвращает переменную, а производит сравнение строк (то есть вызывать этот метод 3 раза - явный перебор).

blaster999, 15 Февраля 2011

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

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

+159

1
2
3

$ar=$arFields['GROUP_ID'];
$arFields['GROUP_ID']=array();
foreach($ar as $a) $arFields['GROUP_ID'][]=$a;

Смысл данной конструкции не ясен, если так нужно обнулить ключи для массива то array_values() в помощь.

tkf, 15 Февраля 2011

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

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