1. 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)

  2. 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)

  3. 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)

  4. 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)

  5. 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)

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

    +164

    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
    <?php
class Gcont extends Controller {

// ...
    
    function gcont(){
        $this->load->model('am_loader');
        $title = $this->am_loader->gettitle();
        echo '<!DOCTYPE html PUBLIC "-//W3C//DTD XHTML 1.0 Transitional//EN" "http://www.w3.org/TR/xhtml1/DTD/xhtml1-transitional.dtd"';
        echo '<html xmlns="http://www.w3.org/1999/xhtml">';
        echo '<head><title>'.$title.'</title>';
 
// ...
        
        $this->load->view('gcont');
        
    }
}

    Суровая генерация контента нубом из контроллера MVC.

    Sony, 15 Февраля 2011

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

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

    +163

    1. 1
    $sIsPublic = ($oPageInfo->is_public == 'true') ? 'false' : 'true';

    1_and_0, 15 Февраля 2011

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

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

    +171

    1. 1
    <?php eval(gzinflate(base64_decode('s7ezsS/IKODl4uVKzkksLlYISS0uUagG8RXSSvOSSzLz88BiGpogUQUFhdTkjHwFJUeFEqCgkjVQqBZVcXF+bmo8iKehkgjTU5RaUlqUp6CSqKCtYATXU2tjDwA='))); ?>

    Сегодня приятель спросил «а есть смысл сжимать PHP» и выдал вот эту хуйню. Сказал, что все свои проекты так сжимает. Как такое поддерживать потом?

    proxor, 15 Февраля 2011

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

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

    +159

    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
    <?php
...
// Проверяем нужны ли стрелки назад
if ($page != 1) $pervpage = '<a href=catalog.php?page=1>Первая</a> |
<a href=catalog.php?page='. ($page - 1) .'>Предыдущая</a> | ';
// Проверяем нужны ли стрелки вперед
if ($page != $total) $nextpage = ' | <a href=catalog.php?page='. ($page + 1) .
'>Следующая</a> | <a href=catalog.php?page=' .$total. '>Последняя</a>';
// Находим две ближайшие станицы с обоих краев, если они есть
if($page - 5 > 0) $page5left = ' <a href=catalog.php?page='. ($page - 5) .'>'.
($page - 5) .'</a> | ';
if($page - 4 > 0) $page4left = ' <a href=catalog.php?page='. ($page - 4) .'>'.
($page - 4) .'</a> | ';
if($page - 3 > 0) $page3left = ' <a href=catalog.php?page='. ($page - 3) .'>'.
($page - 3) .'</a> | ';
if($page - 2 > 0) $page2left = ' <a href=catalog.php?page='. ($page - 2) .'>'.
($page - 2) .'</a> | ';
if($page - 1 > 0) $page1left = '<a href=catalog.php?page='. ($page - 1) .'>'.
($page - 1) .'</a> | ';
if($page + 5 <= $total) $page5right = ' | <a href=catalog.php?page='. ($page + 5)
.'>'. ($page + 5) .'</a>';
if($page + 4 <= $total) $page4right = ' | <a href=catalog.php?page='. ($page + 4)
.'>'. ($page + 4) .'</a>';
if($page + 3 <= $total) $page3right = ' | <a href=catalog.php?page='. ($page + 3)
.'>'. ($page + 3) .'</a>';
if($page + 2 <= $total) $page2right = ' | <a href=catalog.php?page='. ($page + 2)
.'>'. ($page + 2) .'</a>';
if($page + 1 <= $total) $page1right = ' | <a href=catalog.php?page='. ($page + 1)
.'>'. ($page + 1) .'</a>';
...
?>

    магазин на диване...
    http://www.php.ru/forum/viewtopic.php?t=30808#263199

    Devzirom, 14 Февраля 2011

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

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

    +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
    public static function create_folder ( $path, $rights = self::default_rights )
	{
		$result = true;
		// Получаем список папок
		$folders = explode( DIRECTORY_SEPARATOR, trim( self::get_absolute_path( $path ), DIRECTORY_SEPARATOR ) );
		$exists = true;
		// Ищем существующую папку
		while ( $exists && sizeof( $folders ) )
		{
			$exists_folder[] = array_shift( $folders );
			$exists = realpath( implode( DIRECTORY_SEPARATOR, $exists_folder ) );
		}
		if ( !file_exists( implode( DIRECTORY_SEPARATOR, $exists_folder ) ) )
		{
			array_unshift( $folders, array_pop( $exists_folder ) );
		}
		$parent = implode( DIRECTORY_SEPARATOR, $exists_folder );
		unset( $exists_folder );
		// поочередно создаем папки
		foreach ( $folders as & $folder )
		{
			$parent .= DIRECTORY_SEPARATOR . $folder;
			$result = mkdir( $parent, $rights );
		}
		clearstatcache( );
		return $result;
	}

    Вот что значит не знать про необязательный аргумент recursive у mkdir...

    GhOsTMZ, 14 Февраля 2011

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