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

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

В номинации:

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

За время:

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

0

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

WriteBuffer = enc.GetBytes("$ FETCH " + number + " BODY[HEADER]\r\n");
netStream.Write(WriteBuffer, 0, WriteBuffer.Length);
ListMessage = strReader.ReadLine();
//...
while ((ListMessage.IndexOf("$ OK ") == -1) && (ListMessage.IndexOf("$ BAD ") == -1))
{
        //Типа там читаем всякие заголовки и прочие бла-бла
        if (ListMessage.IndexOf("filename=") > -1)
        {
                temp = ListMessage.Replace("filename=", "");
                temp = temp.Trim();
                file_name = temp;
                is_file = 1; temp = ""; is_att = 1;
        }
        //...
        ListMessage = strReader.ReadLine();
        continue;
}
//...
WriteBuffer = enc.GetBytes("$ FETCH " + number + " BODY[TEXT]\r\n");
netStream.Write(WriteBuffer, 0, WriteBuffer.Length);
ListMessage = strReader.ReadLine();

temp = "";
end_file = 0;
body = "";
if (ListMessage.IndexOf("$ BAD ") == -1) ListMessage = strReader.ReadLine();
if (ListMessage == "")
        is_body = 0;
else
        is_body = 1;
while ((ListMessage.IndexOf("$ OK ") == -1) && (ListMessage.IndexOf("$ BAD ") == -1))
{
        if (is_body == 1)
        {
                if (ListMessage == "")
                        is_body = 0;
                else
                        body = body + ListMessage;
        }
        if ((is_att == 1) && (end_file == 0))
        {
                if (is_file == 1)
                {
                        if (ListMessage.IndexOf("--") > -1)
                                end_file = 1;
                        else
                                temp = temp + ListMessage;
                }
                if (ListMessage == "")
                {
                        is_file = 1;
                }
        }
        if (ListMessage.IndexOf("Content-Disposition: attachment;") > -1)
        {
                is_att = 1;
        }
        //...
        ListMessage = strReader.ReadLine();
        continue;
}
if (is_att==0)
{
        //А тут ничего интересного, просто декодирование из Base64 и сохранение в файл
}

Следите за булями (точнее, их суррогатами).

ЗЫ: да, я уже выкладывал кусок из того же модуля, и там уже были некоторые куски, которые попали и сюда тоже.

tucvbif, 02 Апреля 2016

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

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

+1

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

document.getElementById('order').addEventListener('submit', function(e){
  document.getElementById('order').addEventListener('keyup', function(e){
      var appBtns = document.getElementsByClassName('btn'), i;
      for (var i = 0; i < appBtns.length; i ++)
        appBtns[i].style.disabled = false;
      form.submit.disabled = false;
  });
  var appBtns = document.getElementsByClassName('btn'), i;
  for (var i = 0; i < appBtns.length; i ++)
    appBtns[i].style.disabled = true;
  form.submit.disabled = true;
});

без комментариев

vestnik, 24 Марта 2016

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

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

−4

1
2
3

<Style>
    html { overflow: visible !important; }
</Style>

CSS конечно, но...

young, 09 Марта 2016

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

1C / Говнокод #19554

−97

01
02
03
04
05
06
07
08
09
10

tmpDt = ДобавитьМесяц(СсылкаНаОбъект.Дата, -3);
Если Месяц(СсылкаНаОбъект.Дата)<4 Тогда
	txt = "Д.в. "+Месяц(tmpDt)+"/"+Год(tmpDt)+"-"+ "0"+Месяц(СсылкаНаОбъект.Дата)+"/"+Формат(Год(СсылкаНаОбъект.Дата),"ЧГ=0");
Иначе
	Если Месяц(СсылкаНаОбъект.Дата)<10 Тогда
		txt="Д.в. "+"0"+Месяц(tmpDt)+"-"+"0"+Месяц(СсылкаНаОбъект.Дата)+"/"+Формат(Год(СсылкаНаОбъект.Дата),"ЧГ=0");
	Иначе    
		txt="Д.в. "+"0"+Месяц(tmpDt)+"-"+Месяц(СсылкаНаОбъект.Дата)+"/"+Формат(Год(СсылкаНаОбъект.Дата),"ЧГ=0");
	КонецЕсли;
КонецЕсли;

Неумение пользоваться форматной строкой приводит к перлам наподобие "если месяц меньше 10, то добавить к месяцу нолик слева".
Это не считая разборки в числа и сборки назад в строку.

WildSery, 01 Марта 2016

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

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

0

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

// Private method of server, which dispatches active incoming connection.
// Function receives address string and uses it as key to retrieve cached connection.
// Fetched connection is getting read by bufio.Reader, parsed to header and data string if it's size was pointed in header.
// Next, the parsed data handles by protocol and writes a response message.
// The process turns in loop until whether input stream will get an EOF or an error will be occurred.
// In the last case it will be return some error message to a client.
// Anyway, at the end connection will be broken up.
func (server *Server) dispatch(address string) {
	defer server.free_chan()
	if server.Stat.Connections[address] != nil {
		server.Stat.Connections[address].State = "conn_new_cmd"
	}
	connection := server.connections[address]
	connectionReader := bufio.NewReader(connection)
	// let's loop the process for open connection, until it will get closed.
	for {
		// let's read a header first
		if server.Stat.Connections[address] != nil {
			server.Stat.Connections[address].State = "conn_read"
		}
		received_message, n, err := readRequest(connectionReader, -1)
		if err != nil {
			if server.Stat.Connections[address] != nil {
				server.Stat.Connections[address].State = "conn_swallow"
			}
			if err == io.EOF {
				server.Logger.Info("Input stream has got EOF, and now is being closed.")
				server.breakConnection(connection)
				break
			}
			server.Logger.Warning("Dispatching error: ", err, " Message: ", received_message)
			if !server.makeResponse(connection, []byte("ERROR\r\n"), 5){
				break
			}
		} else {
			if server.Stat.Connections[address] != nil {
				server.Stat.Connections[address].Cmd_hit_ts = time.Now().Unix()
			}
			// Here the message should be handled
			server.Stat.Read_bytes += uint64(n)
			parsed_request := protocol.ParseProtocolHeader(string(received_message[ : n - 2]))
			server.Logger.Info("Header: ", *parsed_request)

			if (parsed_request.Command() == "cas" || parsed_request.Command() == "gets") && server.cas_disabled ||
			   parsed_request.Command() == "flush_all" && server.flush_disabled{
				err_msg := parsed_request.Command() + " command is forbidden."
				server.Logger.Warning(err_msg)
				if server.Stat.Connections[address] != nil {
					server.Stat.Connections[address].State = "conn_write"
				}
				err_msg = strings.Replace(protocol.CLIENT_ERROR_TEMP, "%s", err_msg, 1)
				server.makeResponse(connection, []byte(err_msg), len(err_msg))
				continue
			}

			if parsed_request.DataLen() > 0 {
				if server.Stat.Connections[address] != nil {
					server.Stat.Connections[address].State = "conn_nread"
				}
				received_message, _, err := readRequest(connectionReader, parsed_request.DataLen())
				if err != nil {
					server.Logger.Error("Error occurred while reading data:", err)
					server.breakConnection(connection)
					break
				}
				parsed_request.SetData(received_message[0 : ])
			}
			server.Logger.Info("Start handling request:", *parsed_request)
			response_message, err := parsed_request.HandleRequest(server.storage, server.Stat)
			server.Logger.Info("Server is sending response:\n", string(response_message[0 : len(response_message)]))
			// if there is no flag "noreply" in the header:
			if parsed_request.Reply() {
				if server.Stat.Connections[address] != nil {
					server.Stat.Connections[address].State = "conn_write"
				}
				server.makeResponse(connection, response_message, len(response_message))
			}
			if err != nil {
				server.Logger.Error("Impossible to send response:", err)
				server.breakConnection(connection)
				break
			}
		}
		if server.Stat.Connections[address] != nil {
			server.Stat.Connections[address].State = "conn_waiting"
		}
	}
}

memcache на go

alek0585, 27 Февраля 2016

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

1C / Говнокод #19493

−98

1
2
3

СтрокаТовары.Качество					       = СтрокиОст.Качество;
СтрокаТовары.ЕдиницаИзмерения			= СтрокиОст.ЕдИзм;
СтрокаТовары.ЕдиницаИзмеренияМест		= СтрокиОст.ЕдИзмМест;

Поражает сам подход. А почему бы не выдумать для (поля/объекта/коллекции) ещё одно уникальное имя?! Ведь столько ещё свободных и красивых уникальных имён в мире есть! К тому же "ЕдИзм" - это ведь короче чем "ЕдиницаИзмерения", не правда ли? Дурак перепутает только, правда? Писать ведь будет удобней, правда?
Ладно... про ЗаполнитьЗначенияСвойств писать уже баян. Но ведь СтрокиОст - это строка какой-то таблицы? А в исходной таблице это называлось "ЕдиницаИзмерения", наверное, да? Так зачем лишний раз придумывать псевдоним?!!! Зачем? Почему просто не соединить таблицы друг с другом в запросе? Почему?! Почему нужно постоянно дрочить циклами?!
Циклы надо применять тогда, когда нельзя по-другому, а это узкий круг задач, например те, где нужен CAST которого в 1С нет, или распределение одной таблицы на другую по какому какой нибудь "хитровыдуманной и уникальной" бизнес логике. Даже списание по ФИФО уже придумали как сделать запросом... Не хочу! Хочу циклы...

levxxx, 19 Февраля 2016

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

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