По колено в коде — Говнокод.ру

JavaScript / Говнокод #8032

+172

1

var id = document.getElementsById('color').id;

омг

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

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

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)

C# / Говнокод #8028

+122

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

private void webBrowser1_DocumentCompleted(object sender, WebBrowserDocumentCompletedEventArgs e)
        {
            if (this.cbDocumentComleted.Checked) WriteText("DocumentCompleted " + e.Url.ToString()); ;

            if (e.Url.ToString() == "about:blank") return;
            
            this.myCountCompleted++;
            
            if ((this.myCurrentBootState == BootState.FirstBoot) && (this.myCountCompleted == 2))
            {

                this.myCountCompleted = 0;
		FirstBoot();

            }

            if((this.myCurrentBootState == BootState.BootAfterChangePageSize) && (this.myCountCompleted == 2))
            {

                this.myCountCompleted = 0;
                this.myFirstAppStart = false;
                this.timer3.Start();

            }

            if ((this.myCurrentBootState == BootState.BootAfterCapcha) && (this.myCountCompleted == 2))
            {

                this.myCountCompleted = 0;
                this.timer1.Start();

            }

            if ((this.myCurrentBootState == BootState.BootAfterNavigation) && (this.myCountCompleted == 2))
            {

                this.myCountCompleted = 0;
                this.timer2.Start();

            }

        }

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

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

Куча / Говнокод #8027

+144

1
2
3
4
5
6
7

/**
 * 30 сентября в России отмечают день Интернета и по этому поводу
 * разнообразные IT-компании выдвигают разнообразные инициативы.
 * Я хочу предложить провести перепись активных на данный момент 
 * пользователей ГК.
 **/
start();

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

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

Ruby / Говнокод #8026

−93

1
2
3
4

def properties
  Hash.send :[], *(self.class.column_names & self.class::PROPERTIES).
    inject([]) { |a, p| a << p.to_sym << send(p) }
end

это же очевидно

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

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

C++ / Говнокод #8025

+150

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

Люди, помогите решить такой вопрос. Приведу пример: 
   Есть программист Петя. Он прочитал очень много книг по программированию,
и вообще умный парень :). И, так как, Петя прочитал много умных книг (Александреску к примеру),
он соответственно пишет умный код, используя различные фишки с++, например,
очень любит шаблоны (такие вещи типа статического полиморфизма и еще много
всего связанного с шаблонами и обобщенным программированием) или использует
сложные тернарные операторы… ну и т.д. … Соответственно код работает и все хорошо. 
Как-то в разговоре с Петей ему сказали, что если так прогать, то может получиться
непрозрачный код. 
  На что он ответил: «Если алгоритм написан верно, и Каму-то непонятен код, то
проблема в том что человек просто плохо образован в плане программирования.
Ибо кто-то, например, не знает что такое классы и ему будет не прозрачен код,
в котором используют классы или обычное наследование. И соответственно не
писать же все процедурно, из-за того, что кто-то не знает что такое классы.
А само понятие прозрачности кода - ерунда ».
   От сюда вопрос, Прав ли Петя и что такое вообще «Прозрачность кода»?
Или действительно код не может быть  написан «Заумно», а тот, кто так
считает, просто плохо знает стандарт языка.

Вопрос совершенно серьёзный. //Не холивар.

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

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

C++ / Говнокод #8024

+167

1
2

for(byte i = 3;i<129;++i)
  (state.programImpl[i])(state);

Мне достался на поддержку этот проект. Это основной цикл программы, хотя их несколько. Есть некоторый массив, заполненный алгоритмом программыфункциями через

PI programImpl= {

  beginPrg,

  //...

};

В зависимости от выполнения различных условий state.programImpl самомодифицируется во время выполнения этого цикла. Мне хочется сказать: "Возьми Lisp и никогда более не берись за С++!". А вообще, в этом проекте эта самомодификация просто не нужна. А теперь вопрос: "Что у человека с головой?"

Говногость, 29 Сентября 2011

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

C++ / Говнокод #8023

+178

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

static long get_ver(unsigned short *src, unsigned char *dst)
{
	u8 tmp_buf[16];
	s32 i = 0;

	while(1)
	{
		if(dst[i] < '0' ||  dst[i] > '9')
			break;

		tmp_buf[i] = dst[i] % 0x30;
		i++;
	}

	if(i == 1)
		*src = tmp_buf[0];
	else if(i == 2)
		*src = (tmp_buf[0] * 10) + tmp_buf[1];
	else if(i == 3)
		*src = (tmp_buf[0] * 100) + (tmp_buf[1] * 10) + tmp_buf[2];
	else if(i == 4)
		*src = (tmp_buf[0] * 1000) + (tmp_buf[1] * 100) + (tmp_buf[2] * 10) + tmp_buf[3];

	return i;
}

Функция читает из dst и пишет в src O_O

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

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

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