Лучший говнокод за всё время

Лучший говнокод

В номинации:

Лучший рейтинг
Самый комментируемый

За время:

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

+154

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

$this->load->model('localisation/country');
$country_info = $this->model_localisation_country->getCountry($this->request->post['shipping_country_id']);

if ($country_info && $country_info['postcode_required'] && (utf8_strlen($this->request->post['shipping_postcode']) < 2) || (utf8_strlen($this->request->post['shipping_postcode']) > 10)) {
     $json['error']['shipping']['postcode'] = $this->language->get('error_postcode');
}
     
if ($this->request->post['shipping_country_id'] == '') {
     $json['error']['shipping']['country'] = $this->language->get('error_country');
}
            
if ($this->request->post['shipping_zone_id'] == '') {
      $json['error']['shipping']['zone'] = $this->language->get('error_zone');
}
                         
$this->load->model('localisation/country');
$country_info = $this->model_localisation_country->getCountry($this->request->post['shipping_country_id']);
             
if ($country_info && $country_info['postcode_required'] && (utf8_strlen($this->request->post['shipping_postcode']) < 2) || (utf8_strlen($this->request->post['shipping_postcode']) > 10)) {
     $json['error']['shipping']['postcode'] = $this->language->get('error_postcode');
}

строки 279 - 301
файл catalog/controller/checkout/manual.php
opencart 1.5.5.1.1

Зачем два идентичных куска кода (1 - 6 и 16 - 21), в одном файле идущих друг за другом, я так и не понял.

gnom_virtuoz, 17 Января 2014

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

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

+160

1
2
3

if ( String(typeof Ext).toLowerCase() === "object") {		
    Ext.BLANK_IMAGE_URL = '/style/ext/resources/images/default/s.gif';		
}

Ох. Штабильненько.

Elvenfighter, 28 Декабря 2013

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

Си / Говнокод #14263

+135

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
44
45
46
47
48
49
50
51
52
53
54
55
56
57
58
59
60
61
62
63
64
65
66
67
68
69
70
71
72
73
74
75
76
77
78
79
80
81
82
83
84
85
86
87
88

Структура для хранения данных:
typedef struct rs_script_t {
    int status; // on/off
    int next_stage;
    int next_time;
} rs_script_t;

Макросы:

#define SEQ(scr1,t)    \
    const int scr_zero = __COUNTER__;   \
    rs_script_t *scr = scr1; \
    if ( (scr->status) && (t > scr->next_time) ) {  \
        switch(scr->next_stage) {    \
    
#define END_SEQ } };             


#define SEQ_FIRST \
        case 0: \
                scr->next_stage = scr_zero;
                

#define SEQ_DT(dt) \
                scr->next_stage = __COUNTER__ + 1;   \
                scr->next_time += dt;  \
                break;  \
            case __COUNTER__:   \
            
#define SEQ_DT_TAGGED(dt,tag) \
                scr->next_stage = __COUNTER__ + 1; \
                scr->next_time += dt;    \
                break; \
            case __COUNTER__:    \
            case (100000+tag):   \

#define SEQ_LAST_DEFAULT    \
                scr->status = 0; \
                break;  \
                \
            default:    \
                DEBUG10f(":: ERROR, unknown stage position: %d \n", scr->next_stage);    \
                break;

#define SEQ_GOTO(dt,tag) \
                scr->next_stage = 100000+tag; \
                scr->next_time += dt; \
                break;
                
#define SEQ_GOTO_INSTANT(tag) \
                scr->next_stage = 100000+tag; \
                break;

Пользоваться примерно так:
void process() { // вызывать каждый кадр

    SEQ(&scrtest, rs_app.app_time)
            
        SEQ_FIRST
            printf("Ноль\n");
            
        SEQ_DT(1000)
            printf("Один\n");
            
        SEQ_DT(1000)
            printf("Два\n");
            
        SEQ_DT_TAGGED(1000, 555)
            printf("Три (метка №555) \n");
            
        SEQ_DT(1000)
            printf("Четыре\n");
            
        SEQ_DT(1000)
            printf("Пять\n");
            
        SEQ_DT(1000)
            printf("Шесть, повторяем с метки №555, если нажат Шифт\n");
            if (rs_keyboard_is_key_down(RS_KEY_SHIFT_L)) {
                SEQ_GOTO(1000, 555);
            };
            printf("Завершаем. \n");
            
        SEQ_LAST_DEFAULT     
        
    END_SEQ;

};

Страдания по yield

TarasB, 23 Декабря 2013

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

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

+152

1

public function getNearestRoadsOrderedByDistanceFromOverpassAPI ($point, $radius = 1000, $highway = FALSE, $raw = FALSE, $json = FALSE)

за то со смыслом!

nonamez, 04 Декабря 2013

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

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

+129

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

[Serializable]
 private class NameValue<N, V>
 {
      public N Name { get; set; }
      public V Value { get; set; }
      public NameValue() { }
      public NameValue(N name, V value)
      {
          Name = name;
          Value = value;
      }
 }

 private System.Collections.Generic.List<NameValue<string, string>> productList =
            new System.Collections.Generic.List<NameValue<string, string>>();

не шутка

taburetka, 02 Декабря 2013

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

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

+166

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

function CompareDates($date1, $date2)
{
	$s_date1 = $this->CharToDateFunction($date1);
	$s_date2 = $this->CharToDateFunction($date2);
	$strSql = "
		SELECT
			if($s_date1 > $s_date2, 1,
				if ($s_date1 < $s_date2, -1,
					if ($s_date1 = $s_date2, 0, 'x')
			)) as RES
		";
	$z = $this->Query($strSql, false, "FILE: ".__FILE__."<br> LINE: ".__LINE__);
	$zr = $z->Fetch();
	return $zr["RES"];
}

Bitrix. Функция прекрасно живет как минимум с 10 версии по сей день.

Homess, 26 Ноября 2013

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

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

+12

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

#include <iostream>

class base
{
	virtual void on_create() { }
public:
	int value;
	float another_value;

	void create()
	{
		static base test;
		if(*reinterpret_cast<unsigned int *>(this) != *reinterpret_cast<unsigned int *>(&test))
		{
			std::cout << "please, do not override on_create()\n";
		}
		on_create();
	}
};

class derived : public base
{
	virtual void on_create(){}
};

int main(int argc, char * argv[])
{
	std::cout << "base\n";
	base b;
	b.create();

	std::cout << "derived\n";
	derived d;
	d.create();
}

Родилось в попытке ограничить переопределение виртуального метода.
Работает, цуко.
http://ideone.com/gUN9OA

Xom94ok, 20 Ноября 2013

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

Pascal / Говнокод #14109

+138

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

// прога у нас многопоточная, данный код выполняется в потоке. Создание и настройка Idhttp прописана в конструкторе потока.
while not terminated do
    begin
       synchronize(ReadProxy);
        if ThreadExit then break;
        try
           memorystream.Position:=0;
          try
            idHTTP.Get(linknew,memorystream);
            idhttp.disconnect;
          except
            idhttp.Disconnect;
          end;
          finally
            temp:=memorystreamtostring(memorystream); // если этого не сделать, память будет утекать.
            memorystream.clear;
          end;

Я нашел способ предотвратить утечки памяти в Indy, при Get-запросе -
многие грешат на Indy, мол, нестабильная она в потоках, провоцирует утечки.
Оказалось, что дело-то в tmemorystream, я его маму бля ебал!!!

Stertor, 19 Ноября 2013

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

Objective C / Говнокод #13902

−105

1
2
3
4

- (void)reloadTableViewOnSecondaryThread
{
	[self performSelectorOnMainThread:@selector(reloadTableViewOnMainThread) withObject:nil waitUntilDone:NO];
}

Разглядывая код от заказчика нашёл вот такой перл!

ProFFeSSoR, 06 Октября 2013

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

Java / Говнокод #13884

+80

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

/**
   * Форматирует double, c arround кол-вом знаков после запятой
   * @param d double
   * @param arround int
   * @return String
   */
  public static String formatDouble(double d, int arround) {
    if (arround > 6) {
      throwIllegalFormat();
    }
    StringBuffer sb = new StringBuffer(10);
    boolean minus = d < 0;
    if (minus) {
      d = -d;
    }
    for (int i = 0; i < arround; i++) {
      d *= 10;
    }
    long l = (long) (d + 0.000000001);
    do {
      char digit = (char) ('0' + (l % 10));
      l = l / 10;
      sb.append(digit);
      if (--arround == 0) {
        sb.append('.');
      }
    } while (l != 0 || arround >= 0);
    if (minus) {
      sb.append('-');
    }
    return sb.reverse().toString();
  }

glprizes, 02 Октября 2013

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

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