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

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

+157

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

foreach ( $category->getChildrensCategory ( $_GET['id']+0 ) as $value ) {

	$count = count ( $db->fetchAll ( $db->select ()->from ( array ('I' => 'ItemShop' ) )
		    		->join ( array ('O' => 'ItemOptions' ), 'I.1cId=O.1cId' )
				->where ( 'I.Category IN (?)', array_merge ( array ($value ['Id'] ), $category->getAllChildrensCategory ( $value ['Id'] ) ) )
				->where ( 'O.New = 1' )
				->orWhere ( 'I.Category IN (?)', array_merge ( array ($value ['Id'] ), $category->getAllChildrensCategory ( $value ['Id'] ) ) )
				->where ( 'O.Insert > ?', date ( 'Y-m-d', mktime ( 0, 0, 0, date ( 'm' ), date ( 'd' ) - $date, date ( 'Y' ) ) ) )
 				->group ( 'I.1cId' ) ) );


	// ...

}

ZEND. Oh, wait... ZOMG!

Мало того, что тащим в память результат запроса, только чтобы его пересчитать, так в процессе формирования запроса два раза (!) дёргаем тяжелую рекурсивную функцию, которая собирает всех детей ноды (да, да, структура в БД имеет только Parent Node, никаких Nested Nodes и пр.)

sheershoff, 29 Сентября 2011

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

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

+161

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

<?php
	preprint($_POST);
	if (
		(
			isset($_POST['id']) &&
			isset($_POST['kart']) &&	
			isset($_POST['kart']) &&
			isset($_POST['print'])&&
			isset($_POST['num'])&&
			isset($_POST['resourse'])&&
			isset($_POST['pr1'])&&
			isset($_POST['pr2'])&&
			isset($_POST['pr3'])&&
			isset($_POST['pr4'])&&
            isset($_POST['pr5'])
		) 
	&& 
		(
			(count($_POST['id']) == count($_POST['kart'])) &&
			(count($_POST['id']) == count($_POST['print']))  &&
			(count($_POST['id']) == count($_POST['num']))  &&
			(count($_POST['id']) == count($_POST['resourse']))  &&
			(count($_POST['id']) == count($_POST['pr1']))  &&
			(count($_POST['id']) == count($_POST['pr2']))  &&
			(count($_POST['id']) == count($_POST['pr3']))  &&
			(count($_POST['id']) == count($_POST['pr4']))  &&
            (count($_POST['id']) == count($_POST['pr5'])) 
		)
	)
	{
		$momass = array();
			if ($this->url_array[5] == 'ink')
			{
				$type = "1";
				
			}
			elseif ($this->url_array[5] == 'laser')
			{
				$type = "2";
				
			}
		foreach($_POST['id'] as $key => $val)
		{
			$momass[$key] = new cPrinterElements($val);
			$momass[$key]->setData('type',$type);
			$pr_id = trim(strip_tags($this->url_array[4]));
			$momass[$key]->setData('printer_id',$pr_id);
			
			
		}
		foreach($_POST['kart'] as $key => $val)
		{
			$momass[$key]->setData('kart',$val);
		}
		foreach($_POST['print'] as $key => $val)
		{
			$momass[$key]->setData('print',$val);
		}
		foreach($_POST['num'] as $key => $val)
		{
			$momass[$key]->setData('num',$val);
		}
		foreach($_POST['resourse'] as $key => $val)
		{
			$momass[$key]->setData('resourse',$val);
		}
		foreach($_POST['pr1'] as $key => $val)
		{
			$momass[$key]->setData('pr1',$val);
		}
		foreach($_POST['pr2'] as $key => $val)
		{
			$momass[$key]->setData('pr2',$val);
		}
		foreach($_POST['pr3'] as $key => $val)
		{
			$momass[$key]->setData('pr3',$val);
		}
		foreach($_POST['pr4'] as $key => $val)
		{
			$momass[$key]->setData('pr4',$val);
		}
		foreach($_POST['pr5'] as $key => $val)
		{
			$momass[$key]->setData('pr5',$val);
		}
		foreach($_POST['komment'] as $key => $val)
		{
			$momass[$key]->setData('komment',$val);
		}
	
	
        foreach($_POST['link'] as $key => $val)
        {
            $momass[$key]->setData('link',$val);
        }
		foreach($momass as $uno)
		{
			if ($uno->getID())
			{

Ранеее я постил всьюху.. это ее контроллер. пошел пить водку.

amazondev, 29 Сентября 2011

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

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

+160

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

<div class="pmi_cont pr" style="width:60px;float: left; min-height: 20px;z-index: 5;"><?php $k = 0;?>
            <?php if ($uno_el['pr1']): ?>
                <div class="price_1"
                     style="height:25px;"><?php echo $uno_el['pr1']; $k = 1; echo is_numeric($uno_el['pr1']) ? ' руб.'
                        : ''; ?></div><?php endif;?>
            <?php if ($uno_el['pr2']): ?>
                <div class="price_2"
                     style="height:25px;"><?php echo $uno_el['pr2']; $k = 1; echo is_numeric($uno_el['pr2']) ? ' руб.'
                        : ''; ?></div><?php endif;?>
            <?php if ($uno_el['pr3']): ?>
                <div class="price_3"
                     style="height:25px;"><?php echo $uno_el['pr3']; $k = 1; echo is_numeric($uno_el['pr3']) ? ' руб.'
                        : ''; ?></div><?php endif;?>
            <?php if ($uno_el['pr4']): ?>
                <div class="price_4"
                     style="height:25px;"><?php echo $uno_el['pr4']; $k = 1; echo is_numeric($uno_el['pr4']) ? ' руб.'
                        : ''; ?></div><?php endif;?>
            <?php if ($uno_el['pr5']): ?>
                <div class="price_5"
                     style="height:25px;"><?php echo $uno_el['pr5']; $k = 1; echo is_numeric($uno_el['pr5']) ? ' руб.'
                        : ''; ?></div><?php endif;?>
            <?php if ($k): ?>
                <?php if ($uno_el['link']): ?>
                    <div style="margin-left: -60px; width: 200px;"><?php echo $uno_el['link'];?></div><?php endif; ?>
                <?php else: ?>
                <?php if ($uno_el['link']): ?>
                    <div style="position:relative;left: -80px; width: 140px;"><?php echo $uno_el['link'];?></div><?php endif; ?>
                <?php endif;?>
        </div>

Эта хуита выводит цену. И так все шаблоны. Разработчик - мудак.

amazondev, 29 Сентября 2011

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

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

+146

1
2
3
4
5
6

foreach( $vote->get() as $i => $variant )
    {
        . . . 
        $vote = +@$variant[ 'value' ];
        . . .
    }

Чтобы наверняка!

litechat, 28 Сентября 2011

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

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

+150

1
2
3
4
5
6

foreach( $vote->get() as $i => $variant )
    {
        . . . 
        $vote = +@$variant[ 'value' ];
        . . .
    }

Чтобы наверняка!

litechat, 28 Сентября 2011

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

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

+169

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

class Banners_object extends DomainObject
{
    function setDefaultProperties()
    {
        $this->properties = array(
          'oid' => null,
          `priority` => null,
          `title` => null,
          `link` => null,
        );
    }
}

При попытке создать новый объект в STDOUT вылезает что-то типа этого:

sh: priority: command not found
sh: title: command not found
link: missing operand
Try `link --help' for more information.

Очень долго ржали

travka, 28 Сентября 2011

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

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

+159

1

$sql = 'START TRANSACTION;'.rtrim($sql, "; \t\r\n\0").'; COMMIT;';

Jommla 1.6, вот так вот у нас обращаются с транзакцией при управление запросами в СУБД, ещё любопытно то, что не все таблицы в «субд» у нас типа InnoDB, более того доминирующему количеству таблиц присвоен тип MyISAM.

nethak, 28 Сентября 2011

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

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

+169

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

//$idtov получается из БД
if ($idtov==101008116)
        $book["name"]="Костюм мушкетера";
elseif ($idtov==101009116)
        $book["name"]="Костюм ниндзя";
elseif ($idtov==101010116)
        $book["name"]="Костюм пирата";
elseif ($idtov==2200003)
        $book["name"]="Amazonas Слинг-шарф для новорожденных";
elseif ($idtov==100011672)
        $book["name"]="Санки Метелица универсальные с п/м сиденьем и ручкой";
elseif ($idtov==90600)
        $book["name"]="Woody Кукольный домик Ева - маленький";

С сайта tetyamotya.ru

Skiv, 27 Сентября 2011

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

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

+166

1
2
3

function GetUsersStatus($n){
return "Авторизованный пользователь";
}

просто и без лишних слов

andruxin, 27 Сентября 2011

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

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