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

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

+160

1

$VAR['server'] = trim(str_replace("www.","",strtolower($_SERVER['SERVER_NAME'])));

Угадайте что этот код должен делать, и что он делает на самом деле. Да, это ужас. Это один из многих ужасов, доставшихся мне от предыдущего программиста. До этого самым большим был explode имени файла через точку и подставление второго элемента как расширение.

BiSe_Trojanov, 02 Марта 2011

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

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

+146

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

// vba-c.h
#include <stdio.h>
#include <conio.h>

#ifndef VBAC
#   define VBAC
#   define Sub void
#   define EndSub }
#   define main main() {
#   define Dim __int32
#   define As
#   define Integer ;
#   define Print( a ) printf( "%i\n", a );
#   define Wait getch();
#endif

//main.c
#include "vba-c.h"

Sub main
    Dim a As Integer
    Dim b As Integer
    Dim c As Integer
    Dim d As Integer
    Dim mb, mc As Integer

    a = 10;
    b = 25;
    c = 34;
    d = 34;
    mb = 0x00000002;
    mc = 0x00000030;

    b = b - 1;
    c = c - 1;
    a = a + b + c;
    a = a / 3;
    b = b / 4;
    a = a - 1;
    b = b + 1;
    c = c - a + b;
    d = ( b & mb ) & ( c & mc );

    Print( a )
    Print( b )
    Print( c )
    Print( d )
    Wait
EndSub

После полугода изучения VBA, привыкание к хорошим, годным языкам, как Си очень проблематично. Что и порождает такие вот костыли...

DarkWolf, 02 Марта 2011

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

SQL / Говнокод #5849

−863

1
2
3
4
5

...........
UNION ALL
(SELECT CONCAT("Помещене: ", rm_pomeshenie.nazvanie, ". Этаж: ", rm_etaj.nomer, ". Комната: ", rm_komnata.nazvanie, ". Раб. место: ", rm_rab_mesto.nazvanie, ". Сист. блок: ", sb_tipi_sist_blokov.nazvanie) AS raspolojenie, u_tip_oborudovaniya.id AS tip_id, u_tip_oborudovaniya.nazvanie AS tip_nazvanie, u_modeli_oborudovaniya.id AS model_id, CONCAT(u_modeli_oborudovaniya.nazvanie, " ", u_modeli_oborudovaniya.parametri) AS model_nazvanie, u_ustroystva.id AS ustr_id, u_ustroystva.seriyniy_nomer AS ustr_seriya, u_ustroystva.dop_svedeniya AS ustr_dop_svedeniya FROM rm_pomeshenie, rm_etaj, rm_komnata, rm_rab_mesto, sb_tipi_sist_blokov, sb_sistemnie_bloki, u_tip_oborudovaniya, u_modeli_oborudovaniya, u_ustroystva WHERE (rm_pomeshenie.id = rm_etaj.id_pomesheniya) AND (rm_etaj.id = rm_komnata.id_etaja) AND (rm_komnata.id = rm_rab_mesto.id_komnati) AND (sb_sistemnie_bloki.id_v_tablice_naznacheniya = rm_rab_mesto.id) AND (sb_sistemnie_bloki.id_tipa = sb_tipi_sist_blokov.id) AND (sb_sistemnie_bloki.imya_tablici_naznacheniya="rm_rab_mesto") AND (u_ustroystva.id_v_tablice_naznacheniya = sb_sistemnie_bloki.id) AND (u_tip_oborudovaniya.id = u_modeli_oborudovaniya.id_tipa_oborudovaniya) AND (u_modeli_oborudovaniya.id = u_ustroystva.id_modeli) AND (u_ustroystva.imya_tablici_naznacheniya = "sb_sistemnie_bloki") ORDER BY tip_nazvanie, model_nazvanie)
UNION ALL
...........

Выборка устройств. Таких блоков шесть штук и все "склеены" через UNION ALL.
Наговнокодил от души ))

nikitasnv, 02 Марта 2011

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

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

+124

1

while (!Pr.HasExited) System.Threading.Thread.Sleep(250);

Ожидание завершения запущенного процесса

Мартин, 02 Марта 2011

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

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

+137

1

printf("%u\t%u\t%u\t%u\t%u\t%u\t%u\t%u\t%u\t%u\t%f\t%u\t%u\t%u\t%u\t%u\t%u\t%u\t%u\t%u\t%u\t%u\t%u\t%u\t%u\t", ...

vayerx, 02 Марта 2011

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

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

+98

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

Procedure TreeNodeBuilding(Tree: TTreeView; Query: TZQuery; DestinationNode: TTreeNode;
                          IdColumn, NameColumn, FilterColumn, TableType: String; Filtering:Boolean; ImageIndex:Integer);
  //-------------------------------------------------//
  Function AddOrNot:Boolean; //Фильтрация по ИД
  var
    Add: Boolean;
  begin
  Add:=False;
  If Filtering=False then Add:=True else
    begin
    if DestinationNode=nil then Add:=True else
      begin
      MyNodeParamPtr:=DestinationNode.Data;
      if MyNodeParamPtr^.id=Query.FieldValues[FilterColumn] then Add:=True;
      end;
    end;
  Result:=Add;
  end;
  //-------------------------------------------------//
  Procedure AddTreeNodeToViewer; //Добавление ветки
  var
    MyNode: TTreeNode;
  begin
  //Добавляем новую запись в массив и заполняем её
//  Inc(ArrMyNodeParamLength);
//  SetLength(ArrMyNodeParam, ArrMyNodeParamLength);
  New(MyNodeParamPtr);
  MyNodeParamPtr^.id:=Query.FieldValues[IdColumn];
  MyNodeParamPtr^.table:=TableType;
  //Добавляем новый узел дерева и указатель на соотв запись
  if DestinationNode=nil then
  MyNode:=Tree.Items.AddObject(DestinationNode, VarToStr(Query.FieldValues[NameColumn]), MyNodeParamPtr)
    else MyNode:=Tree.Items.AddChildObject(DestinationNode, VarToStr(Query.FieldValues[NameColumn]), MyNodeParamPtr);
  MyNode.ImageIndex:=ImageIndex;
  MyNode.SelectedIndex:=ImageIndex;
  end;
  //-------------------------------------------------//
begin
//
Query.First;
if AddOrNot=True then AddTreeNodeToViewer;
Query.Next;
while not Query.Eof do
  begin
  if AddOrNot=True then AddTreeNodeToViewer;
  Query.Next;
  end;
end;

Добавление ветки в TreeView. Я тут явно перемудрил

nikitasnv, 02 Марта 2011

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

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

+163

1
2
3
4
5
6
7

if ($type=='input') $params['class'] = 'styleInput';
if ($type=='password') $params['class'] = 'stylePassword';
if ($type=='checkbox') $params['class'] = 'styleCheckBox';
if ($type=='select') $params['class'] = 'styleSelect';
if ($type=='multipleSelect') $params['class'] = 'styleMultipleSelect';
if ($type=='textarea') $params['class'] = 'styleTextarea';
if ($type=='groupSelect') $params['class'] = 'styleGroupSelect';

Конкатенация - враг! Учитесь трудолюбию!

ifeya, 02 Марта 2011

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

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

+104

1
2
3
4
5
6
7

if(res < dateFrom)
    {
        res = dateFrom;
    }
    else
        return dateFrom;
    return res;

Выгребаю из проекта...

ccoder, 02 Марта 2011

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

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

+169

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

<?php
	exit
	(
		empty($_POST['key']) ||
		empty($_POST['time']) ||
		empty($_POST['email']) ||
		(int)$_POST['time'] != $_POST['time'] ||
		strlen((int)$_POST['time']) != strlen($_POST['time']) ||
		!preg_match('/^[^@]+@[a-zA-Z0-9._-]+\.[a-zA-Z]+$/', $_POST['email']) ||
		($_SERVER['REQUEST_TIME'] - $_POST['time']) > 60 * 60 ||
		md5(str_rot13(md5($_POST['time']))) != $_POST['key']
		?'-':'+'
	);
?>

Вот Ajax ответ

wyand, 02 Марта 2011

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

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

+115

1
2
3
4
5
6
7

RepaymentEvent e = DatabaseHelper.GetString("event_type", pReader) == "RBLE"
	                ? new BadLoanRepaymentEvent {Id = DatabaseHelper.GetInt32("rpe_id", pReader)}
	                : (DatabaseHelper.GetString("event_type", pReader) == "RRLE"
	                       ? new RescheduledLoanRepaymentEvent {Id = DatabaseHelper.GetInt32("rpe_id", pReader)}
	                       :(DatabaseHelper.GetString("event_type", pReader).StartsWith("P") 
                                ? new PendingRepaymentEvent (DatabaseHelper.GetString("event_type", pReader)) {Id = DatabaseHelper.GetInt32("rpe_id", pReader)}
                                : new RepaymentEvent {Id = DatabaseHelper.GetInt32("rpe_id", pReader)}));

Вот такую "элегантную" строчку нашел сегодня коллега в коде нашего проекта :)
Создаем событие пойди разбери какое :)

_Ru55_, 02 Марта 2011

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

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