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

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

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

1. PHP (5706)
2. C++ (2739)
3. Куча (2371)
4. JavaScript (2030)
5. C# (1928)
6. Java (1502)
7. Си (1117)
8. Pascal (648)
9. Python (586)
10. 1C (539)
11. SQL (432)
12. ActionScript (292)
13. Objective C (202)
14. bash (198)
15. Perl (194)
16. Assembler (146)
17. Ruby (145)
18. VisualBasic (138)
19. Lua (49)
20. Go (31)
21. Haskell (29)
22. Swift (27)
23. Kotlin (14)

1. Pascal / Говнокод #17219

   +80

   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
   37. 37
   38. 38
   39. 39
   40. 40
   41. 41
   42. 42
   43. 43
   44. 44
   45. 45
   46. 46
   47. 47
   48. 48
   49. 49
   50. 50
   51. 51
   52. 52
   53. 53
   54. 54
   55. 55
   56. 56
   57. 57
   58. 58
   59. 59
   60. 60
   61. 61
   62. 62
   63. 63
   64. 64
   65. 65
   66. 66
   67. 67

   procedure TInventory.removeItem(userid: integer; invid: integer;amount: integer = 1);
   var
     MyRes: TInvItemArray;
     CurrItem: TInvItem;
     I,len: integer;
   begin
     if (amount <= 0) then
       exit;
     if (amount = 1) then
       begin
        db.removeItemFromUserInventory(GetItems(GetIndexByInvId(invid)),userid);
        FInventoryList.Remove(GetIndexByInvId(invid));
       end else
       begin
         if (amount >= finventorylist.Size) then
           exit;
         MyRes:=GetResources(invid);
         len:=length(myres);
         if (len = 0) then
           exit;
         if (amount > len) then
           exit;
         for i:=0 to len - 1 do
          begin
            curritem:=MyRes[i];
            db.removeItemFromUserInventory(GetItems(GetIndexByInvId(curritem.InvId)),userid);
            FInventoryList.Remove(GetIndexByInvId(curritem.invid));
          end;
        Setlength(myres,0);
       end;
   
   end;
   
   function TInventory.GetResources(invid: integer): TInvItemArray;
   function ToResult(var Return: TInvItemArray; const Item: TInvItem): Integer;
   begin
     Result := Length(Return);
     SetLength(Return, Result + 1);
     Return[Result] := Item;
   end;
   var
     item,sameitem: TInvItem;
     Resource: TItem;
     i: integer;
   begin
     SetLength(result,0);
     item:=GetItem(invid);
     if (item = nil) then
       exit;
     if (item.item <> nil) then
       begin
        // if (item.itemю = 30) then
        //   begin
             for i:=0 to FInventoryList.Size - 1 do
              begin
                sameitem:=TInvItem(FInventoryList.Items[i]);
                 if (sameitem = nil) then
                   continue;
                resource:=sameitem.item;
                 if (resource = nil) then
                   continue;
                 if (item.item.itemid = resource.itemid) then
                   ToResult(result,sameitem);
              end;
          // end;
       end;
   end;

   По сути - удалить объект из списка. OMG!

   Cynicrus, 29 Ноября 2014

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

2. JavaScript / Говнокод #17218

   +154

   1. 1

   var coutryOption = commonSettings.parent().parent().parent().find("tr:contains('Country')");

   очень поиск...

   fess, 29 Ноября 2014

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

3. C# / Говнокод #17217

   +134

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

   private bool CompareLvlCats(string[] cat,List<string[]> cats, int lvl)
   {
       if (lvl == 1) return cats.Find(x => x[0] == cat[0] && (x[1] != cat[1] ||x[1]!="")) != null;
       if (lvl == 2) return cats.Find(x => x[0] == cat[0] && x[1] == cat[1] && (x[2] != cat[2] || x[2] != "")) != null;
       if (lvl == 3) return cats.Find(x => x[0] == cat[0] && x[1] == cat[1] && x[2] == cat[2] && (x[3] != cat[3] || x[3] != "")) != null;
       if (lvl == 4) return cats.Find(x => x[0] == cat[0] && x[1] == cat[1] && x[2] == cat[2] && x[3] == cat[3] && (x[4] != cat[4] || x[4] != "")) != null;
       if (lvl == 5) return cats.Find(x => x[0] == cat[0] && x[1] == cat[1] && x[2] == cat[2] && x[3] == cat[3] && x[4] == cat[4] && (x[5] != cat[5] || x[5] != "")) != null;
       if (lvl == 6) return false;
       return false;
   }

   Здесь мы идём снова.

   pushistayapodmyshka, 29 Ноября 2014

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

4. JavaScript / Говнокод #17216

   +158

   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
   37. 37
   38. 38

   var firstpostime = 1417254005;var numpos = 1; //номер посещения
   var sredvrem  = 0; //среднее время на каждой странице
   var lasdo15sek = 0;
   
   function debug444(ff) {
   	console.log(ff);
   	//alert(ff);
   }
   var newTxt="!!";
   var oldTxt=document.title;
   function migalka(){
       if(document.title==oldTxt){
           document.title=newTxt;
       }else{
           document.title=oldTxt;
       }
   } 
   function neuspeli() {
   	var element55577 = document.createElement('div');
   	element55577.innerHTML = '<img style="width:1px;height:1px;" src="http://rbcall.com/neuspeli.php?&tel='+document.getElementById('wi_tele233').value+'&site=47&user=159">', 
   	element55577.id = 'wi_r223';
   	document.body.appendChild(element55577);
   	document.getElementById("superjalobal").style.display='block';
   } 
   
   function jduzvonka() {
   	if (zvonim == 1) { return true; }
   	if(document.getElementById("wi_tele233").value.length>=8 || document.getElementById("wi_tele233").value == 400){
   		wiyacall();
   		
   		insertscript(document.getElementById("wi_tele233").value);
   		
   	} else { 
   		document.getElementById("wi_tele233").style.border="1px solid red"; 
   	}
   }
   
   ...... и весь скрипт такой

   http://rbcall.com/ ROBOCALL - технологии будущего (они же: технологии ИИ)

   xjester0, 29 Ноября 2014

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

5. PHP / Говнокод #17215

   +162

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

   (strrev "hello")                #=> "olleh"
   (array_sum [array 1 2 3])       #=> 6
   (array-product [array 4 5 6])   #=> 120
   (substring "world" 2)           #=> "rld"

   Теперь ваш Lisp и в PHP: https://github.com/lisphp/lisphp

   volter9, 29 Ноября 2014

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

6. C++ / Говнокод #17214

   +51

   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

   PVOID ZDbgDoc::getVaByName(PCSTR name)
   {
   	PLIST_ENTRY head = &_dllListHead, entry = head;
   
   	while ((entry = entry->Flink) != head)
   	{
   		ZDll* pDll = static_cast<ZDll*>(entry);
   
   		if (PWSTR ImageName = pDll->_ImageName)
   		{
   			PCSTR sz = name;
   __loop:
   			int c = tolower(*sz++);
   
   			if (!c || c != towlower(*ImageName++))
   			{
   				continue;
   			}
   
   			if (c != '.')
   			{
   				goto __loop;
   			}
   
   			pDll->Parse(this);
   
   			return pDll->getVaByName(sz, this);
   		}
   	}
   
   	return 0;
   }

   zhukas, 29 Ноября 2014

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

7. JavaScript / Говнокод #17213

   +164

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

   if (jq('#' + id).attr('checked')) {
       jq('#' + id).attr('checked', true);
   } else {
       jq('#' + id).attr('checked', false);
   }

   Чужой код, года 11, переписываю, натыкаюсь на такое вот повсеместно

   tagrim, 28 Ноября 2014

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

8. PHP / Говнокод #17212

   +157

   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
   37. 37
   38. 38
   39. 39
   40. 40
   41. 41
   42. 42
   43. 43
   44. 44
   45. 45
   46. 46
   47. 47
   48. 48
   49. 49
   50. 50
   51. 51
   52. 52
   53. 53
   54. 54
   55. 55
   56. 56
   57. 57
   58. 58
   59. 59

   Здрасьте.
   
   Есть самая верхняя строчка на странице, так сказать, самый верхный header. В ней на сайтах часто располагается контактная информация (телефоны, адреса, прочее), так вот, мне нужно сделать так, чтобы элементы <a><img></a> были видны лишь спустя сотку-две пикселей прокрутки. Я призадумалась, поискала и нашла такой код:
   
   <?php function scroll_block(){
   var stop = (document.body.scrollTop || document.documentElement.scrollTop);
   var el=document.getElementById('menublock');
   if(stop>150){
   if(el.style.display!='block'){ 
   el.style.display='block'; 
   }
   }
   else 
   el.style.none='none';
   
   return false;
   }
   
   if (window.addEventListener){
   window.addEventListener("scroll", scroll_block, false);
   window.addEventListener("load", scroll_block, false);
   }
   else if (window.attachEvent){
   window.attachEvent("onscroll", scroll_block);
   window.attachEvent("onload", scroll_block);
   }
   
   <div id='menublock' style="display:none">а тут меню</div>
   ?>
   Затем я подумала, и теперь сделала сам элемент:
   
   <div id="hideousvar"> <a href="http://dev.zoldol.ru/vremya-raboty-sklonov"> <img class="varhid" style="padding: 0px 5px 12px 5px;height: 25px;" title="Время работы склонов" alt="" src="http://dev.zoldol.ru/wp-content/themes/subway-child/images/clock.png"></a>
   
   <a href="http://dev.zoldol.ru/webcamera"> <img class="varhid" style="padding: 0px 5px 10px 5px;height: 28px;" title="WEB-камера" alt="" src="http://dev.zoldol.ru/wp-content/themes/subway-child/images/camera.png"></a>
   
   <a href="http://dev.zoldol.ru/3d-tour"> <img class="varhid" style="padding: 0px 0px 5px 0px;height: 35px;" title="3D-тур" alt="" src="http://dev.zoldol.ru/wp-content/themes/subway-child/images/tour.png"></a>
   
   <a href="http://dev.zoldol.ru/scheme"> <img class="varhid" style="padding: 0px 5px 13px 5px; height: 24px;" title="Интерактивная карта склона" alt="" src="http://dev.zoldol.ru/wp-content/themes/subway-child/images/map.png"></a> </div>
   
   Функция:
   
   <?php function scroll_block(){ var stop = (document.body.scrollTop || document.documentElement.scrollTop); var el=document.getElementById('hideousvar'); if(stop>150){ if(el.style.display!='block'){ el.style.display='block'; } } else el.style.none='none';
   
           return false;
           }
   
       if (window.addEventListener){
           window.addEventListener("scroll", scroll_block, false);
           window.addEventListener("load", scroll_block, false);
               }
       else if (window.attachEvent){
           window.attachEvent("onscroll", scroll_block);
           window.attachEvent("onload", scroll_block);
           }
   
   <div id='hideousvar' style="display:none">а тут меню</div>
   ?>
   
   Но что-то не сработало. Подскажите, пожалуйста. Буду безмерно благодарна за помощь.

   Наткнулся на хешкоде на следующее чудо.
   Хоть и топик тот, судя по всему, писала девушка, но всё ж, блин, с более-менее профессиональной моей точки зрения, это совсем ужас :D
   Всё содержание топика точно скопипащено.:D

   RSLab, 28 Ноября 2014

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

9. C++ / Говнокод #17210

   +70

   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

   QString windoliz_path(QString str)
   {
   /*
   функция "виндолизирует" QTшные пути, для передачи в неQTшные части проги
   (в QT-шные, кстати тоже можно, ибо QT вроде бы понимает и те и другие ;)
   если вызвать вне винды ничо страшного, предпроцессор сделает всё за вас
   и функция вернёт ту же строку что и получила. После вызова в виндах не
   помешает сделать .toLocal8Bit()
   //*/
   #ifdef _WIN32                      //если венда
       char s = '\\';                 //символ для замены
       char t = '/';                  //шаблон поиска
       for (int i = str.size();i;i--) //повторить столько раз сколько символов в строке
       {
           if (str[i-1] == t)           //если символ соответствует образцу поиска
           {
               str[i-1] = s;            //меняем на образец замены
           }
       }
   #endif                             //директива отметы условия "если венда"
       return str;
   }

   Этот проект не перестаёт удивлять... апи не смотри, велосипеды пиши. Коммент сохранен авторский.

   kin63camapa, 28 Ноября 2014

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

10. C# / Говнокод #17209

    +100

    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

    class Book
        {
            public void BookName(ref string file, ref int Index)
            {
                string[] ReadFile = File.ReadAllLines("NameFile.txt", Encoding.Default);
                file = ReadFile[Index];
            }
    
            public void BookAuthor(ref string file, ref int Index)
            {
                string[] ReadFile = File.ReadAllLines("AuthorFile.txt", Encoding.Default);
                file = ReadFile[Index];
            }
    
            public void BookDescription(ref string file, ref int Index)
            {
                string[] ReadFile = File.ReadAllLines("DescriptionFile.txt", Encoding.Default);
                file = ReadFile[Index];
            }
    
            public void BookTags(ref string file, ref int Index)
            {
                string[] ReadFile = File.ReadAllLines("TagFile.txt", Encoding.Default);
                file = ReadFile[Index];
            }
        }

    LightningAtom, 28 Ноября 2014

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

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

• 2107
• …
• 1455
• 1454
• 1453
• 1452
• 1451
• 1450
• 1449
• 1448
• 1447
• …
• 1