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

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

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

1. PHP (5706)
2. C++ (2739)
3. Куча (2363)
4. JavaScript (2030)
5. C# (1928)
6. Java (1502)
7. Си (1116)
8. Pascal (648)
9. Python (586)
10. 1C (538)
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. Swift (27)
22. Haskell (26)
23. Kotlin (14)

1. Java / Говнокод #17813

   +74

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

   public void setSkyboxName(String skyboxName) {
       if (skyboxName != null && skyboxName.length() > 2) {
         this.setSkybox(new SkyBox(skyboxName));
       } else {
         this.setSkybox(null);
       }
     }

   если имя неба длинна > 2 значит создать новое небо,
   а иначе неба не будет вообще

   dmli, 18 Марта 2015

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

2. C# / Говнокод #17812

   +2

   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

   public static DateTime DateStringConvert(string dateString)
           {
               var dd = dateString.Substring(0, dateString.IndexOf('-'));
               var mm = dateString.Substring(dateString.IndexOf('-') + 1).Substring(0, dateString.IndexOf('-'));
               var yyyy = dateString.Substring(dateString.IndexOf('-') + 1).Substring(dateString.IndexOf('-') + 1)
                   .Substring(0, dateString.Substring(dateString.IndexOf('-') + 1)
                   .Substring(dateString.IndexOf('-') + 1).IndexOf(' '));
               var time = dateString.Substring(dateString.IndexOf('-') + 1).Substring(dateString.IndexOf('-') + 1)
                   .Substring(dateString.Substring(dateString.IndexOf('-') + 1)
                   .Substring(dateString.IndexOf('-') + 1).IndexOf(' ') + 1);
   
               var hh = time.Substring(0, time.IndexOf(':'));
               var MM = time.Substring(time.IndexOf(':') + 1);
               return new DateTime(
                   Convert.ToInt16(yyyy),
                   Convert.ToInt16(mm),
                   Convert.ToInt16(dd),
                   Convert.ToInt16(hh),
                   Convert.ToInt16(MM),
                   0);
           }

   Вот что происходит, когда не знаешь, как парсить string в DateTime. Говнокод получен от украинских фрилансеров.

   RaTT, 18 Марта 2015

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

3. C++ / Говнокод #17811

   +54

   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

   std::list<CCPoint> GameMap::stringToCCPArray(std::string points_string, float _x, float _y)
   {
   	points_string.push_back(' ');
   	std::list<CCPoint> CCPList = list<CCPoint>();
   	float x = 0.0f;
   	float y = 0.0f;
   	unsigned int len = 0;
   	bool isCurrentX = true;
   	for(std::string::iterator it = points_string.begin(); it != points_string.end(); ++it){
   		int q = 0;
   		len > 0 ? 
   			--len: (isdigit(*it) ? 
   				(q = atoi(&(*it)), len = countNumbersInInt(q), 
   				isCurrentX ?
   					x = atoi(&(*it)): y = atoi(&(*it))) : 
   				*it == ',' ? 
   					(isCurrentX = false): (CCPList.push_back(ccp(x,(-1 * y))), x = 0.0f, y = 0.0f, isCurrentX = true)) ;
   	}
   
   	return CCPList;
   }

   t0rmund, 18 Марта 2015

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

4. C++ / Говнокод #17810

   +56

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

   int count = 0;
   while (condition) {
       if (count == 5) { // Kind of like sleeping for 0.2 milliseconds
           Sleep(1);
           count = 0;
       }
       // Do whatever.
       count++;
   }

   Последний ответ, показывает способ усыпить комп на 0.2 ms. И это после ответов со всякими сокетами, boost-таймерами и прочей мутью. Ответ такой типа, эй, чуваки вы ничо не понимаете, вот как надо, удобно же че бы нет))
   http://stackoverflow.com/questions/85122/how-to-make-thread-sleep-less-than-a-millisecond-on-windows/11456112#11456112

   vitaly, 17 Марта 2015

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

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

   +167

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

   $a = true;
   $b = $a xor true;
   
   var_dump($b);

   ПЫХОПРОБЛЕМЫ
   
   http://ideone.com/cRMQor
   
   bool(true)

   someone, 17 Марта 2015

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

6. Си / Говнокод #17808

   +143

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

   /*
    * BIG UGLY WARNING! This is so damn ugly I wanna puke.  Unfortunately, some
    * macros that use ASN1_const_CTX still insist on writing in the input
    * stream.  ARGH! ARGH! ARGH! Let's get rid of this macro package. Please? --
    * Richard Levitte      
    */

   OpenSSL...

   bormand, 17 Марта 2015

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

7. PHP / Говнокод #17807

   +163

   1. 1
   2. 2
   3. 3

   <?php
   namespace application;
   header('Content-Type: text/html; charset=windows-1251');

   Кто-то любит BDSM, а кто-то windows-1251

   Onotole, 17 Марта 2015

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

8. JavaScript / Говнокод #17806

   +160

   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

   function displayFilterElements(typeList){
   	_filter.form.elements["name1"].closest("tr").hidden = (typeList === "Type1") ? false : true;
   	_filter.form.elements["name2"].closest("tr").hidden = (typeList === "Type1") ? false : true;
   	_filter.form.elements["nam3"].closest("tr").hidden = (typeList === "Type1") ? false : true;
   	_filter.form.elements["name4"].closest("tr").hidden = (typeList === "Type1") ? false : true;
   	_filter.form.elements["name5"].closest("tr").hidden = (typeList === "Type1") ? false : true;
   	_filter.form.elements["name6"].closest("tr").hidden = (typeList === "Type1") ? false : true;
   	_filter.form.elements["name7"].closest("tr").hidden = (typeList === "Type1") ? false : true;
   	_filter.form.elements["name8"].closest("tr").hidden = (typeList === "Type2") ? false : true;
   	_filter.form.elements["name9"].closest("tr").hidden = (typeList === "Type2") ? false : true;
   	_filter.form.elements["name10"].closest("tr").hidden = (typeList === "Type2") ? false : true;
   	_filter.form.elements["name11"].closest("tr").hidden = (typeList === "Type2") ? false : true;
   	_filter.form.elements["name12"].closest("tr").hidden = (typeList === "Type2") ? false : true;
   	_filter.form.elements["name13"].closest("tr").hidden = (typeList === "Type2") ? false : true;
   	_filter.form.elements["name14"].closest("tr").hidden = (typeList === "Type2") ? false : true;
   	_filter.form.elements["name15"].closest("tr").hidden = (typeList === "Type2") ? false : true;
   	_filter.form.elements["name16"].closest("tr").hidden = (typeList === "Type2") ? false : true;
   	_filter.form.elements["name17"].closest("tr").hidden = (typeList === "Type2") ? false : true;
   	_filter.form.elements["name18"].closest("tr").hidden = (typeList === "Type2") ? false : true;
   	_filter.form.elements["name19"].closest("tr").hidden = (typeList === "Type2") ? false : true;
   	_filter.form.elements["name20"].closest("tr").hidden = (typeList === "Type2") ? false : true;
   	_filter.form.elements["name21"].closest("tr").hidden = (typeList === "Type2") ? false : true;
   	_filter.form.elements["name22"].closest("tr").hidden = (typeList === "Type2") ? false : true;
   	_filter.form.elements["name23"].closest("tr").hidden = (typeList === "Type2") ? false : true;
   	_filter.form.elements["name24"].closest("tr").hidden = (typeList === "Type2") ? false : true;
   	_filter.form.elements["name25"].closest("tr").hidden = (typeList === "Type2") ? false : true;
   	_filter.form.elements["name26"].closest("tr").hidden = (typeList === "Type2") ? false : true;
   	_filter.form.elements["name27"].closest("tr").hidden = (typeList === "Type2") ? false : true;
   	_filter.form.elements["name28"].closest("tr").hidden = (typeList === "Type3") ? true : false;
   	_filter.form.elements["name29"].closest("tr").hidden = (typeList === "Type3") ? false : true;
   };
   	
   function stateFilterElements(){
   	let select = false;
   	["name1", "name2", "name3", "name4", "name5", "name6", "name7", "name8", "name9", "name10", "name11", "name12", "name13"].forEach(function(fieldName){
   		if ($.query.hashJSON.has(fieldName))
   			select = true;
   	});
   	return select;
   }

   Получил в доработку модуль сотрудника. Идентификаторы изменены для публикации, не в них суть.

   torbasow, 17 Марта 2015

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

9. Куча / Говнокод #17805

   +131

   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

   Я даже не могу сказать, что не так с "PHP", потому что… Окей. Представьте себе, эмм, коробку с инструментами.
   Набор инструментов. Выглядит нормально, инструменты как инструменты.
   
   Вы берёте отвёртку и видите крестовину с тремя лепестками.
   Окей, не очень полезно для вас, но может когда-нибудь понадобиться.
   
   Берёте молоток и ужасаетесь тому, что он раздвоен с двух сторон.
   Он, конечно, всё ещё пригоден, я имею ввиду, что вы можете забивать гвозди серединой бойка, держась за ручку наоборот.
   
   Берёте плоскогубцы, у которых нет зазубрин; они плоские и гладкие.
   Не так полезно, как могло бы быть, но ими всё ещё можно выкручивать болты.
   
   И так далее. Все инструменты чем-то странные и вывернутые, но не настолько, чтобы
   быть совсем бесполезными. И во всём наборе нет конкретной проблемы; в нём есть все инструменты.
   
   Теперь представьте себе миллионы плотников, использующих такой вот набор
   инструментов и говорящих вам: «А что не так с этими инструментами? Я никогда не использовал ничего другого и они отлично работают!».
   И плотники показывают вам, построенные ими дома с пятиугольными комнатами и крышей кверху ногами.
   Вы стучитесь в дверь, она просто падает внутрь и они орут на вас за то, что вы сломали их дверь.
   
   Вот что не так с "PHP".

   Время баянчиков.

   gost, 17 Марта 2015

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

10. Си / Говнокод #17804

    +138

    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

    #define poff    acurp + curp * (nSQs+2)        /*probs: offset*/
    long psum() {return  c(poff+ 1);}    long pmax()    {return c(poff+2);}
    long pn     (long n)                {return          c(poff+2+n);   }
    void setp   (long n, long x){set    (poff+2+n, x);} /*prob changes:*/ 
    void setmax         (long x){set    (poff+2,   x);} /*metasearching*/
    void setsum         (long x){set    (poff+1,   x);}
    long get2ndmax()    {long n= poff+2, x=pmax(), m=n,  s=0,end=n+nSQs;   
         do     {n++;}   while  (c(n) != x);  /*find 2nd-highest <=pmax*/     
         do     {m++;    if (s < c(m) && m !=n) s= c(m);}/*jue tncheck?*/
                 while      (s < x    && m < end);       return  s;}
    void addtoSQ(long i, long val) {long x;   /*increase prob of ith SQ*/
         if  (val < 1 || i >nSQs || i<1) return;        /*not possible!*/
         x =  psum()+val;if (x > maxint) return;   /*has max normalizer*/
         setsum (x);    /*normalizer +*/ x =  pn(i)+val; setp(i, x); 
         if (x > pmax()) setmax (x);}   /*maximal SQprob has increased */    
    void subofSQ(long i, long val) {long x;   /*decrease prob of ith SQ*/
         if  (val < 1 || i >nSQs || i<1) return;        /*not possible!*/
         x =  pn(i)-val; if (x < 0)      return;/*no neg probabilities!*/
         setsum (psum()- val);           setp(i,x);/*don't check if =0!*/   
         if (x + val  == pmax()) setmax (get2ndmax());} /*change of max*/
    void incSQ()        {long i= top();  addtoSQ(i,1);} /*top, not pop!*/
    void decSQ()        {long i= top(),  x,y,z; /*decrem prob of the SQ*/
         if  (i> nSQs   || i< 1) return;/*no such search Q number known*/
         x =  pn(i);     if     (x==0)   return;        /*SQ: already 0*/
         y  = psum();    z =     pmax();              
         if  (x == 1 &&  y    <= z+1)    stop    /*leave at least 2 SQs*/
         if  (x == z)    setmax         (get2ndmax());  /*change of max*/
         setp(i, x-1);   setsum (y-1);}                 /*normalizer -1*/
    long upSQ;          /*SQ probability:enumerator  +=  upSQ: increase*/
    void oldSQ()   {long a=pop()+ndecl,  n,i; if(a<0||a> oldp)stop/*bad*/
         n=old[a].size;  a=old[a].start;/*all SQs of old nondecl: +upSQ*/
         tncheck  n+=a;  for(i=a;i<n;i++)addtoSQ(SQ[q[i].Q],     upSQ);}
    
    void setpat()       {long i= pop(); /*instantiate my search pattern*/
         if  (i<0 || i > patp)     stop /*no such search pattern exists*/
         set (acurp,i);}/*next SQ-search defined via new probabilities!*/
    void pupat()        {long    i  =    apatp;   /*push search pattern*/ 
         if  (i>maxpat)  stop    i++;    set(apatp,i);  /*not too many?*/
         cpabn  (poff+1, acurp + 1 + i *(2+ nSQs),       2 + nSQs,  
                                         acurp + 1,      aendpats);     }  
    void popat()        {long i= apatp;  if(i==0)   stop
         set(apatp,i-1); push(i); }                /*pop search pattern*/

    Полный исходник http://www.idsia.ch/~juergen/oopscode.c

    j123123, 17 Марта 2015

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

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

• 2105
• …
• 1512
• 1511
• 1510
• 1509
• 1508
• 1507
• 1506
• 1505
• 1504
• …
• 1