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

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

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

1. PHP (5706)
2. C++ (2739)
3. Куча (2373)
4. JavaScript (2031)
5. C# (1929)
6. Java (1502)
7. Си (1117)
8. Pascal (648)
9. Python (588)
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 (30)
22. Swift (27)
23. Kotlin (14)

1. PHP / Говнокод #2246

   +160.1

   1. 1
   2. 2
   3. 3

   public function getLastThreeIdeas($limit=3) {
       return $this->DB->select('SELECT id, title, user_name FROM ideas ORDER BY date DESC LIMIT ?d', $limit);
   }

   Отличный пример несоответствия названия метода и того, что он делает. По названию метода можно предположить, что он возвращает последние 3 записи. На деле можно получить больше записей блягодаря параметру $limit.

   vavaka, 08 Декабря 2009

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

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

   +126.1

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

   switch (dType) { 
            case "2":
                       return Title + "(" + dTitle + ")";
                       break;
            case "3":
                       return Title + " за " + dDate.ToString("d");
                       break;
            default: return Title;
                       break;
   }

   Чтобы уж наверняка :)

   pe4enov, 08 Декабря 2009

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

3. PHP / Говнокод #2244

   +169.8

   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

   <?php
   $message=str_replace("q","<img src=\"bukv_".$cvet."/q.gif\" alt=\"\">",$message); 
   $message=str_replace("w","<img src=\"bukv_".$cvet."/w.gif\" alt=\"\">",$message); 
   $message=str_replace("e","<img src=\"bukv_".$cvet."/e.gif\" alt=\"\">",$message); 
   $message=str_replace("r","<img src=\"bukv_".$cvet."/r.gif\" alt=\"\">",$message); 
   $message=str_replace("t","<img src=\"bukv_".$cvet."/t.gif\" alt=\"\">",$message); 
   $message=str_replace("y","<img src=\"bukv_".$cvet."/y.gif\" alt=\"\">",$message); 
   $message=str_replace("u","<img src=\"bukv_".$cvet."/u.gif\" alt=\"\">",$message); 
   $message=str_replace("i","<img src=\"bukv_".$cvet."/i.gif\" alt=\"\">",$message); 
   $message=str_replace("o","<img src=\"bukv_".$cvet."/o.gif\" alt=\"\">",$message); 
   $message=str_replace("p","<img src=\"bukv_".$cvet."/p.gif\" alt=\"\">",$message);
   
   //вывожу так : 
   
   $namessss=$message; 
   echo "<div class=\"header\"><center><b> $namessss </b></center></div>";
   
   //и получается так
   /*
   g srg src="g src="bukv_greeg src="bukv_green/n.gif" alt="">/b.gif" alt="">ukg src="g src="bukv_greeg src="bukv_green/n.gif" alt="">/b.gif" alt="">ukv_greeg src="bukv_green/n.gif" alt="">/v.gif" alt="">_greeg src="bukv_green/n.gif" alt="">/c.gif" alt="">="g src="bukv_greeg src="bukv_green/n.gif" alt="">
   */
   ?>

   на одном из форумов мое внимание привлекла загадочная тема с названием "что не правильно в стр_реплаке?"

   [email protected], 08 Декабря 2009

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

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

   +54.7

   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

   int NOD(int a,int b)
   		{
   			if(a==0)
   			{
   				return b;
   			}
   			if(b==0)
   			{
   				return a;
   			}
   			if(a==b)
   			{
   				return a;
   			}
   			if((a%2==0)&&(b%2==0))
   			{
   				return 2*NOD(a/2,b/2);
   			}
   			else if((a%2==0)&&(b%2!=0))
   			{
   				return NOD(a/2,b);
   			}
   			else if((a%2!=0)&&(b%2==0))
   			{
   				return NOD(a,b/2);
   			}
   			else if((a%2!=0)&&(b%2!=0))
   			{
   				return NOD(b,abs(a-b));
   			}
   			else return 1;
   
   			/*
   			   1. НОД(0, n) = n; НОД(m, 0) = m; НОД(m, m) = m;
   			   2. НОД(1, n) = 1; НОД(m, 1) = 1;
   			   3. Если m, n чётные, то НОД(m, n) = 2*НОД(m/2, n/2);
   			   4. Если m чётное, n нечётное, то НОД(m, n) = НОД(m/2, n);
   			   5. Если n чётное, m нечётное, то НОД(m, n) = НОД(m, n/2);
   			   6. Если m, n нечётные, то НОД(m, n) = НОД(n, |m - n|).
   
   			*/
   		}

   Алгоритм Евклида - прошлый век!
   Нарыл в Wiki некий алгоритм)

   zelenov.pavel, 07 Декабря 2009

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

5. Python / Говнокод #2242

   −99.7

   1. 1

   length = int((re.findall(r"top:[1-9]\d*",temp))[-1][4:])*2.54/72-0.6

   wackum, 07 Декабря 2009

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

6. PHP / Говнокод #2241

   +153

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

   $res1 = mysql_query("INSERT INTO tbl_orders
   (date_order,name,id_city,city_text,kod_tel_1,tel_1,kod_tel_2,tel_2,description,subject_text,id_teacher,id_category,category_text,check_student,place_student,check_teacher,place_teacher,price,email,icq,order_close,id_status,to_send,id_whence_came,text_whence_came,one_teacher)
   VALUES ('$ts','$name','$city','$other_city','$kod_tel_1','$tel_1','$kod_tel_2','$tel_2','$prim','$other','$id_rep','$category','$other_cat','$check_uch','$district_uch','$check_prep','$district_prep','$stoim','$email','$icq','0','1','1','$whence_came','$text_whence_came','$one_teacher')")
   or die ("Query failed".mysql_error());

   Услада ума, любящего искать связи. Автор чутка лажанул — некоторые переменные всё же совпадают с полями таблицы.

   defann, 07 Декабря 2009

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

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

   +158.6

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

   if(isset($_POST['submit']))
     {
        send($_POST['id_rep'],$_POST['name'],$_POST['city'],$_POST['other_city'],$_POST['kod_tel_1'],$_POST['tel_1'],$_POST['kod_tel_2'],$_POST['tel_2'],$_POST['prim'],$_POST['res'],$_POST['category'],$_POST['other_cat'],$_POST['check_uch'],$_POST['check_prep'],$_POST['district_uch'],$_POST['district_prep'],$_POST['stoim'],$_POST['email'],$_POST['icq'],$_POST['whence_came'],$_POST['text_whence_came'],$_POST['one_teacher']);
     }
     else
     {
        form($id_teacher,$_POST['name'],$_POST['city'],$_POST['other_city'],$_POST['kod_tel_1'],$_POST['tel_1'],$_POST['kod_tel_2'],$_POST['tel_2'],$_POST['prim'],$_POST['res'],$_POST['category'],$_POST['other_cat'],$_POST['check_uch'],$_POST['check_prep'],$_POST['district_uch'],$_POST['district_prep'],$_POST['stoim'],$_POST['email'],$_POST['icq'],$_POST['whence_came'],$_POST['text_whence_came'],$_POST['one_teacher']);
     }

   Как на самом деле правильно обрабатывать формы.

   defann, 07 Декабря 2009

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

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

   +165.4

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

   <script language="JavaScript">
   		function change_city() 
   		{
   			document.getElementById("header_18").style.display='none';
   			document.getElementById("header_18_add").style.display='block';
   			document.getElementById("header_18_add").id='header_18'; //т.к. стили прописаны для id
   		}
   	</script>
   </head>
   <body>
   
   <div id="mega_scroller">

   <div id="mega_scroller">
   document.getElementById("header_18_add") .id='header_18'; //т.к. стили прописаны для id
   http://www.cinemapark.ru/ - гении делали

   pompomru, 07 Декабря 2009

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

9. C# / Говнокод #2238

   +96.3

   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

   public class zTimer
   {
       object SomeTag;
       int StartTime;
       int TimeOut;
   
       public int Start(int Value, object tag)
       {
           SomeTag = tag;
           TimeOut = Value;
           StartTime = MakeLngTime(DateTime.Now);
           return StartTime;
       }
   
       public int MakeLngTime(DateTime Tm)
       {
          return ((Tm.Day * 24 + Tm.Hour) * 60 + Tm.Minute) * 60 + Tm.Second;
       }
   
       public bool IsTimeOut(object tag)
       {
           if (tag != null && SomeTag != null)
           {
               if (tag == SomeTag)
               {
                   if (MakeLngTime(DateTime.Now) > StartTime + TimeOut)
                   {
                       return true;
                   }
                   else
                   {
                       return false;
                   }
               }
               else
               {
                   StartTime = MakeLngTime(DateTime.Now);
                   SomeTag = tag;
                   return false;
               }
           }
           else
           {
               if (MakeLngTime(DateTime.Now) > StartTime + TimeOut)
               {
                   return true;
               }
               else
               {
                   return false;
               }
           }
       }
   }

   Могучий таймер. Особое внимание стоит уделить методу MakeLngTime.

   crelian, 07 Декабря 2009

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

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

    +130.8

    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

    private static bool EvaluteExpression(string expression, out object result)
            {
                string source =
                    string.Format("public static class Evalutor{{public static object GetValue(){{return {0};}}}}",
                        expression);
    
                using (Microsoft.CSharp.CSharpCodeProvider provider = new Microsoft.CSharp.CSharpCodeProvider())
                {
                    System.CodeDom.Compiler.CompilerParameters compilerParams =
                        new System.CodeDom.Compiler.CompilerParameters
                    {
                        GenerateInMemory = true,
                        GenerateExecutable = false
                    };
    
                    System.CodeDom.Compiler.CompilerResults compileResults = provider.CompileAssemblyFromSource(compilerParams, source);
    
                    if (compileResults.NativeCompilerReturnValue == 0)
                    {
                        Type type = compileResults.CompiledAssembly.GetType("Evalutor");
                        MethodInfo method = type.GetMethod("GetValue");
                        result = method.Invoke(null, null);
                        return true;
                    }
    
                }
    
                result = null;
                return false;
            }

    некогда было писать эвалютор))
    работает и так

    exe-dealer, 07 Декабря 2009

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

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

• 2107
• …
• 219
• 218
• 217
• 216
• 215
• 214
• 213
• 212
• 211
• …
• 1