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

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

−122

1
2

ALTER TABLE [dbo].[IssueGroup]  WITH CHECK ADD  CONSTRAINT [FK_IssueGroup_Group] FOREIGN KEY([ProjectId], [GroupId])
REFERENCES [dbo].[Group] ([GroupId],[ProjectId])

Часа 2 искал проблему в чужом коде - оказалось, что просто поля ProjectId и GroupId были записаны в разных порядках сначала FOREIGN KEY([ProjectId], [GroupId]), а потом REFERENCES [dbo].[Group] ([GroupId],[ProjectId]).

vadimliman, 16 Ноября 2010

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

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

+100

1
2
3
4
5

{Pascal}
function f( a: boolean ) : boolean;
begin
if a then return := f(a) else return := false;
end;

При а - истина программа вылетит с переполненным стеком.

Govnocoder#0xFF, 16 Ноября 2010

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

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

+173

1
2
3

$S=preg_replace('/(\')|(\")|(\,)|(\()|(\))|(\.)|(\‹)|(\:)|(\;)|(\$)|(\#)|(\/)|(\{)|(\})|(\*)|(\ )|(\|)|(\>)|(\<)|(\=)|(\-)|(\[)|(\])|(\!)|(\+)|(\☺)|(\☻)'
                             .'|(\♥)|(\♦)...................../', '', $S);
$S=trim($S);

Регулярные выражения! Супер!

Yakud, 16 Ноября 2010

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

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

+143

1
2
3

if (url == null) {
    throw new RuntimeException("Error reading resource " + url.getFile());
}

abatishchev, 16 Ноября 2010

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

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

−117

1

SELECT * FROM CompanyData d WHERE d.Name = @companyName AND d.EntityID NOT IN (SELECT COUNT(*) FROM...)

Оригинал потерялся, но особый способ отсевания компаний запомнился. Хорошо, что COUNT(*) возвращал всегда ноль :)

AlexKhomich, 16 Ноября 2010

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

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

+177

1
2

if(strlen($text)) return $text;
return "";

ferry-very-good, 16 Ноября 2010

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

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

+126

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

private void SetBorderColor()
{
            if (AuthInfo.BorderColor)
            {
                LinearGradientBrush lcBrush = new LinearGradientBrush();
                lcBrush.StartPoint = new Point(0.5, 0);
                lcBrush.EndPoint = new Point(0.5, 1);
                lcBrush.GradientStops.Add(new GradientStop() { Color = Color.FromArgb(100, 249, 249, 249) });
                lcBrush.GradientStops.Add(new GradientStop() {Color = Color.FromArgb(100,75,139,180), Offset = 0.331});
                lcBrush.GradientStops.Add(new GradientStop() { Color = Color.FromArgb(100, 75, 139, 180), Offset = 0.853 });
                lcBrush.GradientStops.Add(new GradientStop() { Color = Color.FromArgb(100, 249, 249, 249), Offset = 1 });
                this.rectBackGround.Fill = lcBrush;
            }
            else
            {
                LinearGradientBrush lcBrush = new LinearGradientBrush();
                lcBrush.StartPoint = new Point(0.5, 0);
                lcBrush.EndPoint = new Point(0.5, 1);
                lcBrush.GradientStops.Add(new GradientStop() { Color = Color.FromArgb(100, 249, 249, 249) });
                lcBrush.GradientStops.Add(new GradientStop() { Color = Color.FromArgb(100, 206, 109, 109), Offset = 0.331 });
                lcBrush.GradientStops.Add(new GradientStop() { Color = Color.FromArgb(100, 206, 109, 109), Offset = 0.853 });
                lcBrush.GradientStops.Add(new GradientStop() { Color = Color.FromArgb(100, 249, 249, 249), Offset = 1 });
                this.rectBackGround.Fill = lcBrush;
            }
}

ОПТИМИЗИРОВАНО!!!
я плакаль)

HellMaster_HaiL, 16 Ноября 2010

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

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

+106

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

procedure FirstStep (var Text: string);
// Удаление комментариев <!--...-->, переводов строк, раскрытие тегов <br>, удаление парных пробелов:
var
  P:     Integer;
  State: Integer;
  Idx:   Integer;
  Cnt:   Integer;
  NL:    Boolean;
begin
  State := 0;
  P     := 1;
  NL    := True;
  while P <= Length (Text) do
  begin
    if Text[P] in [#9,#10,#13] then Text[P] := ' ';
    case State of
      0: case Text[P] of
           '<': State := 1;
           ' ': begin
                  Idx := P;
                  State := 9;
                end; { }
           else NL := False;
         end; {case}
      1: case Text[P] of
           '!': State := 2;
           'b': State := 7;
           else
           begin
             Dec (P);
             State := 10;
           end; {else}
         end; {case}
      2: if Text[P] <> '-' then
         begin
           Dec (P);
           State := 10;
         end else State := 3;
      3: if Text[P] = '-' then
         begin
           Idx   := P - 3;
           State := 4;
         end else
         begin
           Dec (P);
           State := 10;
         end; {if}
      4: if Text[P] = '-' then State := 5;
      5: if Text[P] = '-' then State := 6 else State := 4;
      6: if Text[P] = '>' then
         begin
           Delete (Text, Idx, P - Idx + 1);
           P := Idx - 1;
           while (P >= 1) and (Text[P] = ' ') do Dec (P);
           State := 0;
         end else State := 4;
      7: if Text[P] = 'r' then State := 8 else State := 0;
      8: begin
           if Text[P] = '>' then
           begin
             Text[P-3] := #13;
             Text[P-2] := #10;
             Delete (Text, P - 1, 2);
             Dec (P, 2);
             NL := True;
           end; {if}
           State := 0;
         end; {8}
      9: if Text[P] <> ' ' then
         begin
           Cnt := P - Idx;
           if NL then
           begin
             Delete (Text, Idx, Cnt);
             Dec (Idx);
           end else if Cnt > 1 then Delete (Text, Idx, Cnt - 1);
           P := Idx;
           State := 0;
         end; {if}
      10: if Text[P] = '>' then State := 0;
    end; {case}
    Inc (P);
  end; {while}
end; {proc FirstStep}

Кусок парсера HTML.

Arigato, 16 Ноября 2010

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

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

+135

1
2
3
4
5
6
7

<p id="entrance">
            Нашли или ...
</p>

<p id="entrance" style="background: #fff; padding: 8px; -webkit-border-radius: 8px; -moz-border-radius: 8px; border-radius: 8px; -webkit-box-shadow: 0 0 5px #aaa">
	Дорогие пользователи
</p>

Решил я написать скрипт для GreaseMonkey, который убирает сообщение "Дорогие пользователи, "
И вот что обнаружил: в коде есть два одинаковых айдишника id="entrance".

Используется стиль p#entrance.

Если надо использовать стили в нескольких элементах, используй классы.

А разные айдишники оставь для джаваскрипта,
а то не выбрать определенный элемент с помощью document.getElementById.

В крайнем случае используй атрибут name. Для него есть метод document.getElementsByName

Oleg_quadro, 16 Ноября 2010

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

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

+112

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

sSQLTitle := 'SELECT  CASE WHEN (VTSS.WRKORDNO IS NOT NULL)'
    +' THEN VTSS.WRKORDNO ELSE VTW.WRKORDNO END AS '''+RSNumDok+''','
    +' CASE WHEN (VTSS.GRECNO IS NOT NULL) THEN VTSS.GRECNO ELSE VTW.GRECNO END AS '''+RSSchet+''','
    +' CASE WHEN (VTSS.STYPE IS NOT NULL) THEN VTSS.STYPE ELSE VTW.STYPE END AS '''+RSDocType+''','
    +' CASE WHEN (VTSS.ServD IS NOT NULL) THEN VTSS.ServD ELSE VTW.ServD END AS '''+RSOpen+''','
    +' CASE WHEN (VTSS.BILLD IS NOT NULL) THEN VTSS.BILLD ELSE VTW.BILLD  END AS '''+RSClose+''','
    +' CASE WHEN (VTSS.FULLNAME IS NOT NULL) THEN VTSS.FULLNAME ELSE VTW.FULLNAME  END AS '''+RSClient+''','
    +' CASE WHEN (VTSS.PAYNAME IS NOT NULL) THEN VTSS.PAYNAME ELSE VTW.PAYNAME  END AS '''+RSPlatelwik+''','
    +' CASE WHEN (VTSS.VIN IS NOT NULL) THEN VTSS.VIN ELSE VTW.VIN  END AS '''+RSVIN+''','
    +' CASE WHEN (VTSS.GNUMER IS NOT NULL) THEN VTSS.GNUMER ELSE VTW.GNUMER  END AS '''+RSGosNomer+''','
    +' CASE WHEN (VTSS.HSMANID IS NOT NULL) THEN VTSS.HSMANID ELSE VTW.HSMANID  END AS '''+RSOtvetstv+''','
    +' CASE WHEN (VTSS.LBACK IS NOT NULL) THEN VTSS.LBACK ELSE VTW.LBACK  END AS '''+RSReturn+''','
    +' VTSS.FULLSUMMA AS '''+RSSumItems+''','
    +' VTW.SUMMA AS '''+RSSumWork+'''';
with DataMdl.adsADODataSet do
  begin
    Active := false;
    if rbAllDate.Checked then
    {$REGION 'Поиск по всем датам'}
      case rgZakazType.ItemIndex of
        0: //поиск по открытым заказ-заявкам
        CommandText:=sSQLTitle
          +' FROM'
          +' (SELECT  VTS.WRKORDNO,'
       (...)
          +' SUM(SUMMA) AS FULLSUMMA'
          +' FROM'
          +' (SELECT GS.WRKORDNO,'
          +' GB.GRECNO,'
          + sSQLCase
          +' CONVERT(varchar(10), GS.ServD, 104) AS ServD,'
          +' CONVERT(varchar(10), GB.BILLD,104) AS BILLD,'
          +' CASE'
          +'	WHEN C.FNAME IS NULL THEN C.LNAME'
          +'	WHEN C.LNAME IS NULL THEN C.FNAME'
          +' ELSE C.FNAME+'' ''+C.LNAME'
          +'	END AS FULLNAME,'
//          +' C.LNAME + '' '' + C.FNAME AS FULLNAME,'
          +' CS.PNAME AS PAYNAME,'
          +' V.SERIALNO AS VIN,'
          +' V.LicNo AS GNUMER,'
          +' GB.HSMANID,'
          +' CASE WHEN (GS.CREDIT IS NULL)OR (GS.CREDIT <> 1) THEN NULL ELSE '''+RSReturn+''' END AS LBACK,'
          +' GW.NAME,'
          +' SUM(GW.ORDNUM) AS '''+RSQuantity+''','
          +' SUM(GW.ORDNUM)*GW.UNITPR AS SUMMA'
          +' FROM GSAL'+sPostfix+' GS LEFT OUTER JOIN'
          +' GROW'+sPostfix+' GW ON (GS.GSALID = GW.GSALID)  LEFT OUTER JOIN'
          +' GBIL'+sPostfix+' GB ON (GS.GSALID = GB.GSALID) LEFT OUTER JOIN'
          +' VEHI V ON (GS.VEHIID = V.VEHIID) LEFT OUTER JOIN'
          +' CUST C ON (GS.CUSTNO = C.CUSTNO) RIGHT OUTER JOIN'
          +' (SELECT     CustNo, Lname + '' '' + FNAME AS PNAME'
          +' FROM CUST WITH (NOLOCK))'
          +' CS ON (GB.CUSTNO = CS.CUSTNO)'
          +' WHERE (GS.STATUS = ''A'') AND (GW.RTYPE IN (1,2))'  +sSQLWhere
          +' GROUP BY  GW.ITEM, GW.NAME, GW.UNITPR,'
          +' GS.WRKORDNO, GB.Grecno, GS.ServD, GB.BILLD, C.LNAME, C.FNAME,'
          +' CS.PNAME, V.SERIALNO, V.LicNo, GB.HSMANID, GS.CREDIT, GW.RECMTIME, GS.STYPE)'
          +' VTS'
          +' CASE WHEN (GS.CREDIT IS NULL)OR (GS.CREDIT <> 1) THEN NULL ELSE '''+RSReturn+''' END AS LBACK,'
          +' GW.NAME,'
          +' SUM(GW.ORDNUM) AS '''+RSQuantity+''','
          +' SUM(GW.ORDNUM)*GW.UNITPR AS SUMMA'
          +' FROM GSAL'+sPostfix+' GS LEFT OUTER JOIN'
          +' GROW'+sPostfix+' GW ON (GS.GSALID = GW.GSALID)  LEFT OUTER JOIN'
          +' GBIL'+sPostfix+' GB ON (GS.GSALID = GB.GSALID) LEFT OUTER JOIN'
          +' VEHI V ON (GS.VEHIID = V.VEHIID) LEFT OUTER JOIN'
          +' CUST C ON (GS.CUSTNO = C.CUSTNO) RIGHT OUTER JOIN'
          +' (SELECT     CustNo, Lname + '' '' + FNAME AS PNAME'
       
     ...

... и так далее. НАПИСАНО ВРУЧНУЮ! Всего 420 строк, 4 подзапроса, связанных через full outer join. 16 тысяч записей 4-x ядерный mssql server обрабатывал таким образом минуты две. Я уж промолчу, как он по строкам в Delphi разбит...
Два дня врубался, как оно работает, переписал за два часа. 58 строк, включая каждое поле из select с новой строки :)

artemlight, 16 Ноября 2010

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

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

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

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

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

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

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

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

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

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

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

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