1. PHP / Говнокод #5715

    +163

    1. 01
    2. 02
    3. 03
    4. 04
    5. 05
    6. 06
    7. 07
    8. 08
    9. 09
    10. 10
    } elseif ($go == 4) {
	include("function/no-cache.php");
	include("config/config_uploads.php");
	switch($go) {
		default:
		$con = explode("|", $confup[$mod]);
		upload(2, "uploads/".$mod."", $con[0], $con[2], $mod, $con[3], $con[4]);
		break;
	}
}

    http://www.slaed.net/files-view-1103.html

    111111, 18 Февраля 2011

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

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

    +164

    1. 1
    2. 2
    3. 3
    4. 4
    5. 5
    6. 6
    7. 7
    $str="T:/home//localhost///MyScripts/////";
$pos=strpos($str,"//");
while($pos!=false)
{
    $str=str_replace("//","/",$str);
    $pos=strpos($str,"//");
}

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

    Sulik78, 17 Февраля 2011

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

  3. PHP / Говнокод #5705

    +161

    1. 01
    2. 02
    3. 03
    4. 04
    5. 05
    6. 06
    7. 07
    8. 08
    9. 09
    10. 10
    11. 11
    12. 12
    13. 13
    14. 14
    15. 15
    16. 16
    17. 17
    18. 18
    19. 19
    20. 20
    21. 21
    22. 22
    23. 23
    24. 24
    25. 25
    26. 26
    27. 27
    28. 28
    29. 29
    30. 30
    31. 31
    32. 32
    33. 33
    34. 34
    35. 35
    36. 36
    37. 37
    38. 38
    39. 39
    40. 40
    41. 41
    42. 42
    43. 43
    <?php
        define('PATH_LEN', 32);
        define('ALPH_SIZE', 66);
        
        class Security
        {
                public static $rus_alphabet = array('А', 'а', 'Б', 'б', 'В', 'в', 'Г', 'г', 'Д', 'д', 'Е', 'е', 'Ё', 'ё',   'Ж', 'ж',   'З', 'з', 'И', 'и', 'Й', 'й',   'К', 'к', 'Л', 'л', 'М', 'м', 'Н', 'н', 'О', 'о', 'П', 'п', 'Р', 'р', 'С', 'с', 'Т', 'т', 'У', 'у', 'Ф', 'ф', 'Х', 'х', 'Ц', 'ц', 'Ч', 'ч',   'Ш', 'ш',   'Щ', 'щ',   'Ъ', 'ъ', 'Ы', 'ы', 'Ь', 'ь',     'Э', 'э', 'Ю', 'ю',     'Я', 'я');
                public static $eng_alphabet = array('A', 'a', 'B', 'b', 'V', 'v', 'G', 'g', 'D', 'd', 'E', 'e', 'Yo', 'yo', 'Zh', 'zh', 'Z', 'z', 'I', 'i', 'Ji', 'ji', 'K', 'k', 'L', 'l', 'M', 'm', 'N', 'n', 'O', 'o', 'P', 'p', 'R', 'r', 'S', 's', 'T', 't', 'Y', 'y', 'F', 'f', 'H', 'h', 'C', 'c', 'Ch', 'ch', 'Sh', 'sh', 'Ch', 'ch', '_', '_', '_', '_', '_', '_', '_', '_', 'U', 'u', 'Ya', 'ya');
                
                static public function Transform($string)
                {
                        $string = htmlspecialchars($string);
                        $string = strip_tags($string);
                        
                        return $string;
                }
                
                static public function TransformName($string)
                {
                        $temp = '';
                        
                        for($i = 0; $i < strlen($string); $i++)
                        {
                                if($string[$i] === ' ')
                                        {
                                                $temp += '_';
                                                continue;
                                        }
                                
                                for($j = 0; $j < ALPH_SIZE; $j++)
                                        if(Security::$rus_alphabet[$j] === Security::$string[$i])
                                                {
                                                        $temp += Security::$eng_alphabet[$j];
                                                        continue 2;                                                     
                                                }
                                $temp += $string[$i];                   
                        }
                        
                        return $temp;
                }
                
        }
?>

    qbasic, 17 Февраля 2011

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

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

    +173

    1. 01
    2. 02
    3. 03
    4. 04
    5. 05
    6. 06
    7. 07
    8. 08
    9. 09
    10. 10
    11. 11
    12. 12
    13. 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)

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

    +144

    1. 1
    Вопрос...

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

    nethak, 16 Февраля 2011

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

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

    +163

    1. 01
    2. 02
    3. 03
    4. 04
    5. 05
    6. 06
    7. 07
    8. 08
    9. 09
    10. 10
    11. 11
    12. 12
    13. 13
    14. 14
    15. 15
    16. 16
    17. 17
    18. 18
    19. 19
    20. 20
    21. 21
    22. 22
    23. 23
    24. 24
    25. 25
    26. 26
    27. 27
    28. 28
    29. 29
    30. 30
    31. 31
    32. 32
    33. 33
    34. 34
    35. 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)

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

    +158

    1. 1
    2. 2
    3. 3
    4. 4
    5. 5
    6. 6
    7. 7
    8. 8
    9. 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)

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

    +162

    1. 01
    2. 02
    3. 03
    4. 04
    5. 05
    6. 06
    7. 07
    8. 08
    9. 09
    10. 10
    11. 11
    12. 12
    13. 13
    14. 14
    15. 15
    16. 16
    17. 17
    18. 18
    19. 19
    20. 20
    21. 21
    22. 22
    23. 23
    24. 24
    25. 25
    26. 26
    27. 27
    28. 28
    29. 29
    30. 30
    31. 31
    32. 32
    33. 33
    34. 34
    35. 35
    36. 36
    37. 37
    38. 38
    39. 39
    40. 40
    41. 41
    42. 42
    43. 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)

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

    +159

    1. 1
    2. 2
    3. 3
    4. 4
    5. 5
    6. 6
    7. 7
    8. 8
    9. 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)

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

    +159

    1. 1
    2. 2
    3. 3
    $ar=$arFields['GROUP_ID'];
$arFields['GROUP_ID']=array();
foreach($ar as $a) $arFields['GROUP_ID'][]=$a;

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

    tkf, 15 Февраля 2011

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