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

• Все
• Лучший
• Сток
• Глупый поиск
• Наговнокодить!
• Oтзывы

Нашли или выдавили из себя код, который нельзя назвать нормальным, на который без улыбки не взглянешь? Не торопитесь его удалять или рефакторить, — запостите его на говнокод.ру, посмеёмся вместе!

1. PHP (5703)
2. C++ (2740)
3. Куча (2337)
4. JavaScript (2030)
5. C# (1929)
6. Java (1503)
7. Си (1103)
8. Pascal (649)
9. Python (585)
10. 1C (536)
11. SQL (434)
12. ActionScript (294)
13. Objective C (204)
14. bash (200)
15. Perl (196)
16. Assembler (148)
17. Ruby (144)
18. VisualBasic (140)
19. Lua (51)
20. Go (32)
21. Swift (29)
22. Haskell (28)
23. Kotlin (15)

1. SQL / Говнокод #17623

   −165

   1. 1
   2. 2
   3. 3

   SELECT `score` FROM `score`
   WHERE `score` = (SELECT DISTINCT `score` FROM `score`
     ORDER BY `score` DESC LIMIT 1,1)

   Про именование таблиц и полей...

   smart, 11 Февраля 2015

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

2. SQL / Говнокод #17499

   −170

   1. 1
   2. 2
   3. 3
   4. 4
   5. 5

   CREATE TRIGGER TR_Table1 ON Table1
   INSTEAD OF INSERT
   AS
   INSERT INTO Table1
   SELECT * FROM INSERTED

   Диалект MS SQL
   INSTEAD OF INSERT - триггер, отменяющий вставку и передающий список значений, указанных в запросе в псевдотаблице INSERTED.
   
   т.е. автор вместо того чтобы позволить северу вставлять строк решил каждый раз вставлять их лично.

   German_1984, 23 Января 2015

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

3. SQL / Говнокод #17477

   −167

   1. 1
   2. 2
   3. 3

   CREATE TABLE users (
     --...
     active VARCHAR(1) NOT NULL DEFAULT 'N'

   Lowezar, 19 Января 2015

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

4. SQL / Говнокод #17451

   −164

   1. 01
   2. 02
   3. 03
   4. 04
   5. 05
   6. 06
   7. 07
   8. 08
   9. 09
   10. 10
   11. 11
   12. 12
   13. 13
   14. 14
   15. 15
   16. 16
   17. 17
   18. 18
   19. 19

   CREATE TABLE NEWS (
    ID INT NOT NULL AUTO_INCREMENT
    , NEWS_NAME VARCHAR(300) NOT NULL
    , NEWS_SHORT_NAME VARCHAR(80) NOT NULL
    , NEWS_TEXT VARCHAR(40000) NOT NULL
    , NEWS_SHORT_TEXT VARCHAR(400) NOT NULL
    , NEWS_IMG_1 VARCHAR(60)
    , NEWS_IMG_2 VARCHAR(60)
    , NEWS_IMG_3 VARCHAR(60)
    , NEWS_IMG_4 VARCHAR(60)
    , NEWS_IMG_5 VARCHAR(60)
    , NEWS_IMG_6 VARCHAR(60)
    , NEWS_IMG_7 VARCHAR(60)
    , NEWS_IMG_8 VARCHAR(60)
    , NEWS_IMG_9 VARCHAR(60)
    , NEWS_IMG_10 VARCHAR(60)
    , NEWS_DATE DATE
    , PRIMARY KEY (ID)
   );

   Таблица новостей из одной очень крутой и никому неизвесной CMS на java.
   Сейчас автор думает над тем что делать если пользователяю надо будет загрузить 11 изображений, варианты: увеличить количество полей до 15, ограничить количество изображений на новость.

   lisiy50, 15 Января 2015

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

5. SQL / Говнокод #17318

   −165

   1. 1
   2. 2
   3. 3
   4. 4
   5. 5
   6. 6
   7. 7

   update #tmpTours1 set pricelink = ('<a target="_blank" title="Прайс-лист на ближайшие заезды" href="../pricelist/Complex.aspx?country=' + ltrim(str(cnkey)) + '&tour=' + ltrim(str(tourkey)) + '&dateFrom=' + dbo.mwFirstTourDate(tlkey) + '">' + ltrim(str(tourprice)) + ' ' + tourrate + '</a>')
     
    set @sql = N'select * from #tmpTours1 where tourprice is not null and quota <> case ' + ltrim(str(@quotaYes)) + ' when -1 then -10 else 0 end '
    if len(@sort) > 0
     set @sql = @sql + ' order by ' + @sort
   	print @sql
    exec sp_executesql @sql

   Хранимочки по 1000 строк подобного кода.

   tablecell, 15 Декабря 2014

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

6. SQL / Говнокод #17237

   −169

   1. 1

   LEFT JOIN ($regions_table LEFT JOIN ($states_table LEFT JOIN

   мы встроили вам лефт джоин в лефт джоин чтобы лефтжоинить лефт джоин...

   xAlz, 02 Декабря 2014

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

7. SQL / Говнокод #17168

   −864

   1. 01
   2. 02
   3. 03
   4. 04
   5. 05
   6. 06
   7. 07
   8. 08
   9. 09
   10. 10
   11. 11
   12. 12
   13. 13
   14. 14
   15. 15
   16. 16
   17. 17
   18. 18
   19. 19
   20. 20
   21. 21
   22. 22
   23. 23
   24. 24
   25. 25
   26. 26

   sResult:= '';
      select  code||' '||h.longname into sResult
           from tbl1 mo, tbl2 ob, g_cli c, tbl3 h
         where mo.m_dep_id = ob.obj_dep_id and mo.m_id = ob.obj_id
             and ob.dep_id = nDepId and ob.id = nId
             and  mo.cli_dep_id = c.dep_id and mo.cli_id = c.id
             and c.id = h.id and c.dep_id = h.dep_id and p_operday between h.FROMDATE and h.TODATE;
            return sResult; 
     exception
       when NO_DATA_FOUND then return null;
       when TOO_MANY_ROWS then          
             for rec in(   select  distinct code as cli_code, h.longname as cli_name 
           from tbl1 mo, tbl2 ob, g_cli c, tbl3 h
         where mo.m_dep_id = ob.obj_dep_id and mo.m_id = ob.obj_id
             and ob.dep_id = nDepId and ob.id = nId
             and  mo.cli_dep_id = c.dep_id and mo.cli_id = c.id
             and c.id = h.id and c.dep_id = h.dep_id and dT between h.FROMDATE and h.TODATE)
               loop
                 if nvl(length(sResult),0) = 0 then
                   sResult:= rec.cli_code||' '||rec.cli_name;
                 else
                   sResult:= sResult||', '||rec.cli_code||' '||rec.cli_name;
                 end if;
               end loop;
               return sResult;   
        when others then return null;

   ColvirBydlokod, 25 Ноября 2014

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

8. SQL / Говнокод #17060

   −863

   1. 1
   2. 2
   3. 3
   4. 4
   5. 5

   SELECT *
   	FROM session s
   	WHERE s.platform IS NOT NULL
   	GROUP BY s.user_id
   	HAVING MAX(s.open_utc)

   Но хоть можно понять что сказать пытались, и на том спасибо.

   Lowezar, 03 Ноября 2014

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

9. SQL / Говнокод #16905

   −863

   1. 01
   2. 02
   3. 03
   4. 04
   5. 05
   6. 06
   7. 07
   8. 08
   9. 09
   10. 10
   11. 11

   select top 1 v.Id
      from dbo.Verification v
      where v.ApplicationId = a.Id
       and v.ResultId = 'D69E0B3A-C5CA-42D0-A8CA-FA7FF63BC414'
       and not exists (
        select 1 from Verification vo
        where vo.ApplicationId = v.ApplicationId
         and vo.ResultId = v.ResultId
         and vo.Id != v.Id
         and vo.Position > v.Position
       )

   MS SQL. Видимо, автору не рассказали об order by

   alex123098, 21 Октября 2014

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

10. SQL / Говнокод #16870

    −860

    1. 01
    2. 02
    3. 03
    4. 04
    5. 05
    6. 06
    7. 07
    8. 08
    9. 09
    10. 10
    11. 11
    12. 12
    13. 13
    14. 14
    15. 15
    16. 16
    17. 17
    18. 18
    19. 19
    20. 20
    21. 21
    22. 22
    23. 23
    24. 24
    25. 25
    26. 26
    27. 27
    28. 28
    29. 29
    30. 30
    31. 31
    32. 32
    33. 33
    34. 34
    35. 35
    36. 36

    SELECT 
     iif(isnull((SELECT H_ALL FROM S_KODGR WHERE GR.D1 = S_KODGR.K_KOD)),0,(SELECT H_ALL FROM S_KODGR WHERE GR.D1 = S_KODGR.K_KOD)) +
     iif(isnull((SELECT H_ALL FROM S_KODGR WHERE GR.D2 = S_KODGR.K_KOD)),0,(SELECT H_ALL FROM S_KODGR WHERE GR.D2 = S_KODGR.K_KOD)) +
     iif(isnull((SELECT H_ALL FROM S_KODGR WHERE GR.D3 = S_KODGR.K_KOD)),0,(SELECT H_ALL FROM S_KODGR WHERE GR.D3 = S_KODGR.K_KOD)) +
     iif(isnull((SELECT H_ALL FROM S_KODGR WHERE GR.D4 = S_KODGR.K_KOD)),0,(SELECT H_ALL FROM S_KODGR WHERE GR.D4 = S_KODGR.K_KOD)) +
     iif(isnull((SELECT H_ALL FROM S_KODGR WHERE GR.D5 = S_KODGR.K_KOD)),0,(SELECT H_ALL FROM S_KODGR WHERE GR.D5 = S_KODGR.K_KOD)) +
     iif(isnull((SELECT H_ALL FROM S_KODGR WHERE GR.D6 = S_KODGR.K_KOD)),0,(SELECT H_ALL FROM S_KODGR WHERE GR.D6 = S_KODGR.K_KOD)) +
     iif(isnull((SELECT H_ALL FROM S_KODGR WHERE GR.D7 = S_KODGR.K_KOD)),0,(SELECT H_ALL FROM S_KODGR WHERE GR.D7 = S_KODGR.K_KOD)) +
     iif(isnull((SELECT H_ALL FROM S_KODGR WHERE GR.D8 = S_KODGR.K_KOD)),0,(SELECT H_ALL FROM S_KODGR WHERE GR.D8 = S_KODGR.K_KOD)) +
     iif(isnull((SELECT H_ALL FROM S_KODGR WHERE GR.D9 = S_KODGR.K_KOD)),0,(SELECT H_ALL FROM S_KODGR WHERE GR.D9 = S_KODGR.K_KOD)) +
     iif(isnull((SELECT H_ALL FROM S_KODGR WHERE GR.D10 = S_KODGR.K_KOD)),0,(SELECT H_ALL FROM S_KODGR WHERE GR.D10 = S_KODGR.K_KOD)) +
     iif(isnull((SELECT H_ALL FROM S_KODGR WHERE GR.D11 = S_KODGR.K_KOD)),0,(SELECT H_ALL FROM S_KODGR WHERE GR.D11 = S_KODGR.K_KOD)) +
     iif(isnull((SELECT H_ALL FROM S_KODGR WHERE GR.D12 = S_KODGR.K_KOD)),0,(SELECT H_ALL FROM S_KODGR WHERE GR.D12 = S_KODGR.K_KOD)) +
     iif(isnull((SELECT H_ALL FROM S_KODGR WHERE GR.D13 = S_KODGR.K_KOD)),0,(SELECT H_ALL FROM S_KODGR WHERE GR.D13 = S_KODGR.K_KOD)) +
     iif(isnull((SELECT H_ALL FROM S_KODGR WHERE GR.D14 = S_KODGR.K_KOD)),0,(SELECT H_ALL FROM S_KODGR WHERE GR.D14 = S_KODGR.K_KOD)) +
     iif(isnull((SELECT H_ALL FROM S_KODGR WHERE GR.D15 = S_KODGR.K_KOD)),0,(SELECT H_ALL FROM S_KODGR WHERE GR.D15 = S_KODGR.K_KOD)) +
     iif(isnull((SELECT H_ALL FROM S_KODGR WHERE GR.D16 = S_KODGR.K_KOD)),0,(SELECT H_ALL FROM S_KODGR WHERE GR.D16 = S_KODGR.K_KOD)) +
     iif(isnull((SELECT H_ALL FROM S_KODGR WHERE GR.D17 = S_KODGR.K_KOD)),0,(SELECT H_ALL FROM S_KODGR WHERE GR.D17 = S_KODGR.K_KOD)) +
     iif(isnull((SELECT H_ALL FROM S_KODGR WHERE GR.D18 = S_KODGR.K_KOD)),0,(SELECT H_ALL FROM S_KODGR WHERE GR.D18 = S_KODGR.K_KOD)) +
     iif(isnull((SELECT H_ALL FROM S_KODGR WHERE GR.D19 = S_KODGR.K_KOD)),0,(SELECT H_ALL FROM S_KODGR WHERE GR.D19 = S_KODGR.K_KOD)) +
     iif(isnull((SELECT H_ALL FROM S_KODGR WHERE GR.D20 = S_KODGR.K_KOD)),0,(SELECT H_ALL FROM S_KODGR WHERE GR.D20 = S_KODGR.K_KOD)) +
     iif(isnull((SELECT H_ALL FROM S_KODGR WHERE GR.D21 = S_KODGR.K_KOD)),0,(SELECT H_ALL FROM S_KODGR WHERE GR.D21 = S_KODGR.K_KOD)) +
     iif(isnull((SELECT H_ALL FROM S_KODGR WHERE GR.D22 = S_KODGR.K_KOD)),0,(SELECT H_ALL FROM S_KODGR WHERE GR.D22 = S_KODGR.K_KOD)) +
     iif(isnull((SELECT H_ALL FROM S_KODGR WHERE GR.D23 = S_KODGR.K_KOD)),0,(SELECT H_ALL FROM S_KODGR WHERE GR.D23 = S_KODGR.K_KOD)) +
     iif(isnull((SELECT H_ALL FROM S_KODGR WHERE GR.D24 = S_KODGR.K_KOD)),0,(SELECT H_ALL FROM S_KODGR WHERE GR.D24 = S_KODGR.K_KOD)) +
     iif(isnull((SELECT H_ALL FROM S_KODGR WHERE GR.D25 = S_KODGR.K_KOD)),0,(SELECT H_ALL FROM S_KODGR WHERE GR.D25 = S_KODGR.K_KOD)) +
     iif(isnull((SELECT H_ALL FROM S_KODGR WHERE GR.D26 = S_KODGR.K_KOD)),0,(SELECT H_ALL FROM S_KODGR WHERE GR.D26 = S_KODGR.K_KOD)) +
     iif(isnull((SELECT H_ALL FROM S_KODGR WHERE GR.D27 = S_KODGR.K_KOD)),0,(SELECT H_ALL FROM S_KODGR WHERE GR.D27 = S_KODGR.K_KOD)) +
     iif(isnull((SELECT H_ALL FROM S_KODGR WHERE GR.D28 = S_KODGR.K_KOD)),0,(SELECT H_ALL FROM S_KODGR WHERE GR.D28 = S_KODGR.K_KOD)) +
     iif(isnull((SELECT H_ALL FROM S_KODGR WHERE GR.D29 = S_KODGR.K_KOD)),0,(SELECT H_ALL FROM S_KODGR WHERE GR.D29 = S_KODGR.K_KOD)) +
     iif(isnull((SELECT H_ALL FROM S_KODGR WHERE GR.D30 = S_KODGR.K_KOD)),0,(SELECT H_ALL FROM S_KODGR WHERE GR.D30 = S_KODGR.K_KOD)) +
     iif(isnull((SELECT H_ALL FROM S_KODGR WHERE GR.D31 = S_KODGR.K_KOD)),0,(SELECT H_ALL FROM S_KODGR WHERE GR.D31 = S_KODGR.K_KOD)) AS H_ALL,
    GR.ID_GRAF
    FROM GRAF_RAB AS GR
    WHERE GR.GOD = 2014
    AND GR.K_T_GR = 1

    1) Говеная архитектура БД: В базе месяц лежит в одной таблице с полями D1, D2, D3 и т.д. Потому нельзя просто взять и одним запросом связать две таблицы, а приходится делать 31 подзапрос
    2) Говеня БД: используется access, который не может и шагу ступить без vba. В итоге функция nz, написанная на vba просто не видна OleDbDataAdapter-у и приходится использовать iif. И 31 подзапрос превращается в 62
    3) Надо получить значения из двух полей. 62 подзапроса превращаются в 124(Здесь только одно поле. В проекте код в 2 раза длиннее)

    yrHeTaTeJlb, 17 Октября 2014

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

• ← влево
• вправо →

• 44
• …
• 41
• 40
• 39
• 38
• 37
• 36
• 35
• 34
• 33
• …
• 1