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

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

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

1. PHP (5708)
2. C++ (2740)
3. Куча (2383)
4. JavaScript (2033)
5. C# (1929)
6. Java (1502)
7. Си (1117)
8. Pascal (648)
9. Python (591)
10. 1C (539)
11. SQL (432)
12. ActionScript (292)
13. Objective C (202)
14. bash (200)
15. Perl (194)
16. Assembler (148)
17. Ruby (145)
18. VisualBasic (138)
19. Lua (49)
20. Haskell (34)
21. Go (31)
22. Swift (27)
23. Kotlin (14)

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

   +68

   1. 1
   2. 2
   3. 3

   //на всяк пожарный
   die();
   return false;

   в работающей системе (:

   Zdomb, 03 Мая 2012

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

2. Java / Говнокод #10175

   +66

   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
   68. 68
   69. 69
   70. 70
   71. 71
   72. 72
   73. 73
   74. 74
   75. 75
   76. 76
   77. 77
   78. 78
   79. 79
   80. 80
   81. 81
   82. 82
   83. 83
   84. 84
   85. 85
   86. 86
   87. 87
   88. 88
   89. 89
   90. 90
   91. 91
   92. 92
   93. 93
   94. 94
   95. 95
   96. 96
   97. 97

   package util;
   
   import play.db.jpa.JPA;
   import play.db.jpa.Model;
   import play.mvc.Http;
   import play.mvc.Router;
   import play.mvc.Scope;
   
   import javax.persistence.Query;
   import javax.persistence.EntityManager;
   import java.lang.reflect.Field;
   import java.util.Arrays;
   import java.util.HashMap;
   import java.util.List;
   import java.util.Map;
   
   /**
    * Page of results for model classes (because paginate module looks too buggy).
    * It is not generic solution but it fits the needs.
    * @author <a href="mailto:[email protected]">Roman Kashitsyn</a>
    */
   public class Page<M extends Model> implements PageBuilder<M> {
       
       public static final int MAX_PAGE_SIZE = 20;
       public static final int DEFAULT_PAGE_SIZE = 10;
       public static final int DEFAULT_PAGE_NUMBER = 1;
   
       public static final String PAGE_NUMBER_PARAM = "page";
       public static final String PAGE_SIZE_PARAM = "psize";
       public static final String ORDER_PARAM = "order";
       public static final String ORDER_BY_PARAM = "orderBy";
       
       private static final String ASC = "asc";
       private static final String DESC = "desc";
       private static final List<String> ALLOWED_ORDERS = Arrays.asList(ASC, DESC);
   
       private int pageSize = DEFAULT_PAGE_SIZE;
       private int pageNumber = DEFAULT_PAGE_NUMBER;
       private int total;
       private String orderBy;
       private String order;
       private final Http.Request request;
       private final Map<String, Object> params;
       private List<M> results;
       private final Class<M> clazz;
   
       public class SizeSwitcher {
           private final Map<String, Object> params;
   
           private SizeSwitcher() {
               // making defensive copy
               params = new HashMap<String, Object>(Page.this.params);
               // list of different size should always begin with 1 page
               params.put(PAGE_NUMBER_PARAM, 1);
           }
   
           public String urlToSwitchSize(int newSize) {
               params.put(PAGE_SIZE_PARAM, newSize);
               return Router.reverse(Page.this.request.action, params).url;
           }
   
           public int currentSize() {
               return Page.this.pageSize;
           }
       }
   
       private Page(Class<M> clazz) {
           this.clazz = clazz;
           request = Http.Request.current();
           params = new HashMap<String, Object>(Scope.Params.current().allSimple());
       }
       
       public static <M extends Model> PageBuilder<M> of(Class<M> modelClass) {
           return new Page<M>(modelClass);
       }
       
       public PageBuilder<M> withParams(Map<?, ?> params) {
           pageNumber = limit(params.get(PAGE_NUMBER_PARAM), pageNumber, Integer.MAX_VALUE);
           pageSize = limit(params.get(PAGE_SIZE_PARAM), pageSize, MAX_PAGE_SIZE);
           Object orderByParam = params.get(ORDER_BY_PARAM);
           if (orderByParam != null) {
               orderBy(getSingleValue(orderByParam).toString());
           }
           Object orderParam = params.get(ORDER_PARAM);
           if (orderParam != null) {
               String proposedValue = getSingleValue(orderParam).toString();
               if (ALLOWED_ORDERS.contains(proposedValue)) {
                   order = getSingleValue(orderParam).toString();
               }
           }
           return this;
       }
   
       public PageBuilder<M> withNumber(int num) {
           this.pageNumber = num;
           return this;
       }

   Govnisti_Diavol, 03 Мая 2012

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

3. JavaScript / Говнокод #10174

   +161

   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

   $('documetnt').ready (function () {
   	 $('#navcat ul li ul').addClass('over');
    		if ('/faq'.indexOf('/faq') == 0) {
   			$('#navcat #child40 > ul').addClass('active');
   		}
   		if ('massbuying' == 'aion') {
   			$('#navcat ul li.scand ul').addClass('active');
   		}
   		if ('massbuying' == '...') {
   			$('#navcat ul li.scand ul').addClass('active');
   		}
   		if ('massbuying' == '...') {
   			$('#navcat ul li.kaz ul').addClass('active');
   		}
   		if ('massbuying' == 'cao') {
   			$('#navcat ul li.kaz > ul').addClass('active');
   		}
   		if ('/massbuying/faq'.indexOf('...') == 0) {
   			$('#navcat #child7 > ul').addClass('active');
   		}
   		if ('massbuying' == 'site') {
   			$('#navcat ul li.mini ul').addClass('active');
   		}
   		if ('massbuying' == 'portal') {
   			$('#navcat ul li.portals ul').addClass('active');
   		}
   		if ('massbuying' == '...') {
   			$('#navcat ul li.portals ul').addClass('active');
   		}
   		if ('massbuying' == 'shop') {
   			$('#navcat ul li.shop ul').addClass('active');
   		}
   		if ('massbuying' == 'design') {
   			$('#navcat ul li.design ul').addClass('active');
   		}
   		if ('massbuying' == 'services') {
   			$('#navcat ul li.services ul').addClass('active');
   		}
   		if ('massbuying' == 'rekl') {
   			$('#navcat ul li.contextAdv ul').addClass('active');
   		}
   			
   		...
   		if ('/massbuying/faq' == '/faq') {
   			$('#child40 > a').addClass('active2');
   		}
   ...

   код для выпадающего меню))))) И такого говна там 400 строк ))) (оторвать прогеру руки и засунуть туда от куда они у него растут)

   corsair_gt, 03 Мая 2012

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

4. PHP / Говнокод #10172

   +91

   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

   // Организовываем вывод сообщений
   do 
   {
       printf ("<div class='border'> \n
       <p class='pa'>    
       Имя отправителя: %s; &nbsp;&nbsp;&nbsp; Дата отправления: %s; &nbsp;&nbsp;&nbsp; <a href='admin.php?exe=answer&id=%s'>Ответ администрации</a> &nbsp;&nbsp;&nbsp; <a href='admin.php?exe=edite&id=%s'>Редактировать</a> &nbsp;&nbsp;&nbsp;  <a href='admin.php?exe=delete&id=%s'>Удалить</a></p>\n<p align='justify'>%s</p>\n<span class='spaa'><i>%s</i></span>\n</div>",  
       $myrow_gb1['name'], $myrow_gb1['dmy'], $myrow_gb1['id'], $myrow_gb1['id'], $myrow_gb1['id'], $myrow_gb1['mes'], $myrow_gb1['kom']);
   }
   while ($myrow_gb1 = mysql_fetch_array($result_gb1)); 
   
   // Проверяем нужны ли стрелки назад
   if ($page != 1) $pervpage = '<a href=admin.php?exe=main&page=1>Первая</a> | <a href=admin.php?exe=main&page='. ($page - 1) .'>Предыдущая</a> | ';
   // Проверяем нужны ли стрелки вперед
   if ($page != $total) $nextpage = ' | <a href=admin.php?exe=main&page='. ($page + 1) .'>Следующая</a> | <a href=admin.php?exe=main&page=' .$total. '>Последняя</a>';
   
   // Находим две ближайшие станицы с обоих краев, если они есть
   if($page - 5 > 0) $page5left = ' <a href=admin.php?exe=main&page='. ($page - 5) .'>'. ($page - 5) .'</a> | ';
   if($page - 4 > 0) $page4left = ' <a href=admin.php?exe=main&page='. ($page - 4) .'>'. ($page - 4) .'</a> | ';
   if($page - 3 > 0) $page3left = ' <a href=admin.php?exe=main&page='. ($page - 3) .'>'. ($page - 3) .'</a> | ';
   if($page - 2 > 0) $page2left = ' <a href=admin.php?exe=main&page='. ($page - 2) .'>'. ($page - 2) .'</a> | ';
   if($page - 1 > 0) $page1left = ' <a href=admin.php?exe=main&page='. ($page - 1) .'>'. ($page - 1) .'</a> | ';
   
   if($page + 5 <= $total) $page5right = ' | <a href=admin.php?exe=main&page='. ($page + 5) .'>'. ($page + 5) .'</a>';
   if($page + 4 <= $total) $page4right = ' | <a href=admin.php?exe=main&page='. ($page + 4) .'>'. ($page + 4) .'</a>';
   if($page + 3 <= $total) $page3right = ' | <a href=admin.php?exe=main&page='. ($page + 3) .'>'. ($page + 3) .'</a>';
   if($page + 2 <= $total) $page2right = ' | <a href=admin.php?exe=main&page='. ($page + 2) .'>'. ($page + 2) .'</a>';
   if($page + 1 <= $total) $page1right = ' | <a href=admin.php?exe=main&page='. ($page + 1) .'>'. ($page + 1) .'</a>';
   
   // Вывод меню если страниц больше одной
   
   if ($total > 1)
   {
   Error_Reporting(E_ALL & ~E_NOTICE);
   echo "<center>";
   echo $pervpage.$page5left.$page4left.$page3left.$page2left.$page1left.'<b>'.$page.'</b>'.$page1right.$page2right.$page3right.$page4right.$page5right.$nextpage;
   echo "</center>";
   }

   Продолжение этого: http://govnokod.ru/10169

   Мартин, 03 Мая 2012

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

5. 1C / Говнокод #10171

   −179

   1. 1
   2. 2
   3. 3

   Если Не КВыгрузке Тогда
       КВыгрузке = НЕ КВыгрузке;
   КонецЕсли;

   inspired, 03 Мая 2012

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

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

   −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

   sscanf_s(buf,
   		"%f%f%f%f%f%d%d%d%d%d%d"
   		"%f%f%f%f%f%d%d%d%d%d%d"
   		"%f%f%f%f%f%d%d%d%d%d%d"
   		"%f%f%f%f%f%d%d%d%d%d%d",
   		&_race_state._car[0]._x, &_race_state._car[0]._y, &_race_state._car[0]._vx,
   		&_race_state._car[0]._vy, &_race_state._car[0]._angle, &_race_state._car[0]._nitro_time,
   		&_race_state._car[0]._oiled_bus, &_race_state._car[0]._nirtos, &_race_state._car[0]._bonus,
   		&_race_state._car[0]._lap, &_race_state._car[0]._next_seg,
   		&_race_state._car[1]._x, &_race_state._car[1]._y, &_race_state._car[1]._vx,
   		&_race_state._car[1]._vy, &_race_state._car[1]._angle, &_race_state._car[1]._nitro_time,
   		&_race_state._car[1]._oiled_bus, &_race_state._car[1]._nirtos, &_race_state._car[1]._bonus,
   		&_race_state._car[1]._lap, &_race_state._car[1]._next_seg,
   		&_race_state._car[2]._x, &_race_state._car[2]._y, &_race_state._car[2]._vx,
   		&_race_state._car[2]._vy, &_race_state._car[2]._angle, &_race_state._car[2]._nitro_time,
   		&_race_state._car[2]._oiled_bus, &_race_state._car[2]._nirtos, &_race_state._car[2]._bonus,
   		&_race_state._car[2]._lap, &_race_state._car[2]._next_seg,
   		&_race_state._car[3]._x, &_race_state._car[3]._y, &_race_state._car[3]._vx,
   		&_race_state._car[3]._vy, &_race_state._car[3]._angle, &_race_state._car[3]._nitro_time,
   		&_race_state._car[3]._oiled_bus, &_race_state._car[3]._nirtos, &_race_state._car[3]._bonus,
   		&_race_state._car[3]._lap, &_race_state._car[3]._next_seg );

   парсим строчку...

   rip, 03 Мая 2012

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

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

   +91

   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

   //Постраничная навигация
   $result77 = mysql_query("SELECT str FROM ..."); 
   $myrow77 = mysql_fetch_array($result77);
   $num = $myrow77["str"];
   
   // Извлекаем из URL текущую страницу
   @$page = $_GET['page'];
   
   // Определяем общее число сообщений в базе данных
   $result00 = mysql_query("SELECT COUNT(*) FROM ...");
   $temp = mysql_fetch_array($result00);
   $posts = $temp[0];
   
   // Находим общее число страниц
   $total = (($posts - 1) / $num) + 1;
   $total =  intval($total);
   
   // Определяем начало сообщений для текущей страницы
   $page = intval($page);
   // Если значение $page меньше единицы или отрицательно
   // переходим на первую страницу
   // А если слишком большое, то переходим на последнюю
   if(empty($page) or $page < 0) $page = 1;
   if($page > $total) $page = $total;
   // Вычисляем начиная с какого номера
   // следует выводить сообщения
   $start = $page * $num - $num;
   // Выбираем $num сообщений начиная с номера $start                                     
   $result_gb1 = mysql_query("SELECT * FROM . ORDER BY id DESC LIMIT $start, $num");
   $myrow_gb1 = mysql_fetch_array($result_gb1);
   $list = 1;
   
   switch($page)
   {
       case "2":
       $list = $num + 1;
       break;
       case "3":
       $list = $num * 2 + 1;
       break;
       case "4":
       $list = $num * 3 + 1;
       break;
       case "5":
       $list = $num * 4 + 1;
       break;
       case "6":
       $list = $num * 5 + 1;
       break;
       case "7":
       $list = $num * 6 + 1;
       break;
       case "8":
       $list = $num * 7 + 1;
       break;
   }

   Навигация по страницам, написанная человеком с гуманитарным складом ума...

   Мартин, 02 Мая 2012

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

8. Java / Говнокод #10168

   +71

   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

   public class ... {
   
   public Double toDoubleValue(String s){
   	Double value;
   	if (s!=""){
   		value = Double.valueOf(s);
   		return value;
   	}
   	else {
   		value = 0.00;
   		return value;
   	}
           ....
           toDoubleValue(row.getColumnValue(col.getId()).toString());
   }

   1. На null не проверяется
   2. Можно было тоже самое написать короче, не объявляя дополнительный метод, с помощью ?:

   -EZ-, 02 Мая 2012

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

9. PHP / Говнокод #10167

   +90

   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

   if(!empty($_POST))
   {
   	include("dbconnect.php");
   
   	$shown_name=$_POST['shown_name'];
   	$username=$_POST['username'];
   	$header=$_POST['header'];
   	$text=$_POST['text'];
   
   	$date_array=getdate(time());
   
   	$date=$date_array['hours'].":".$date_array['minutes'].":".$date_array['seconds']." ".$date_array['weekday']." ".$date_array['mday']." ".$date_array['month']." ".$date_array['year'];
   
   
   	$filename=$date_array['hours']."-".$date_array['minutes']."-".$date_array['seconds']."-".$date_array['mday'].$date_array['month']."-".$date_array['year']."-".$username.".php";
   
   	$path="posts/".$filename;
   
   	$addpost="INSERT INTO posts(author, header, text, date, path) VALUES ('$shown_name', '$header', '$text', '$date', '$path')";
   	$add=$db->prepare($addpost);
   	$add->execute();
   
   
   
   
   	$file=fopen($path, 'w');
   	$content="<?php \$shown_name=\"".$shown_name."\"; \$header=\"".$header."\"; \$text=\"".$text."\";  include(\"template.html\"); ?>";
   
   	$write=fwrite($file, $content);
   
   	fclose($file);
   
   
   	header('Location: index.php');
   	$db=null;
   }

   Добавление поста в блог. Радует каждая строчка.

   nucleartux, 02 Мая 2012

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

10. PHP / Говнокод #10166

    +88

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

    if ( ($cache = isset($this->cache[0]) ? $this->cache[0] : null) != null ) {
            $isNeedCache = $cache->isNeedCache($plug);
        }
        
        if ( !$isNeedCache || !$cache->getPlugCache($plug, $plugInfo) ) {
          
          /** processing: running plug */
          $this->dispatcher->processPlug($request, $response);
          
          if ($isNeedCache)
            $cache->setPlugCache($plug, $plugInfo);
        }

    hwtech, 02 Мая 2012

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

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

• 2110
• …
• 972
• 971
• 970
• 969
• 968
• 967
• 966
• 965
• 964
• …
• 1