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

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

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

1. PHP (5706)
2. C++ (2739)
3. Куча (2371)
4. JavaScript (2031)
5. C# (1928)
6. Java (1502)
7. Си (1117)
8. Pascal (648)
9. Python (587)
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. PHP / Говнокод #17716

   +159

   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

   if (!isset($_GET['outbox']) and !isset($_GET['deleted']) and !isset($_GET['verification'])){
   				$res2 = mysql_query("SELECT l.* FROM letters l LEFT JOIN `customers` ct ON ct.id=l.from_user WHERE l.deleted<>1 AND ct.`Sex` = 'male' {$addon_sql} ORDER BY l.time_insert DESC");
   				for($arr=array(); $t = mysql_fetch_assoc($res2); $arr[]=$t);
   
   				smarty_put_var('zagolovok', 'Входящие письма');
   		}
   
   		if (isset($_GET['verification'])) {
   		$res2 = mysql_query("SELECT * FROM `letters` where `deleted` != 1 AND `checked` = 'N' {$addon_sql} order by `time_insert` DESC");
   		for($arr = array(); $t = mysql_fetch_assoc($res2); $arr[] = $t);
   
   		smarty_put_var('zagolovok', 'Ожидающие проверку');
   		}
   
   		if (isset($_GET['outbox'])) {
       	$res2 = mysql_query("SELECT l.* FROM `letters` l
   		    	LEFT JOIN `customers` ct ON ct.`id`=l.from_user
   		    	WHERE l.deleted != 1 AND ct.`Sex` = 'female' {$addon_sql}
   		    	ORDER BY l.time_insert DESC LIMIT 200");
   		    	for ($arr = array(); $t = mysql_fetch_assoc($res2); $arr[]=$t);
   
   		    	smarty_put_var('zagolovok', 'Исходящие письма');
   		}
   
   		    	if (isset($_GET['deleted'])){
       	$res2 = mysql_query("SELECT * FROM `letters` WHERE `deleted` = 1
   		    	{$addon_sql} ORDER BY `time_insert` DESC");
   		    	for($arr=array(); $t=mysql_fetch_assoc($res2); $arr[]=$t);
   
   		    	smarty_put_var('deleted', true);
   		    	smarty_put_var('zagolovok', 'Удаленные письма');
   		    	}
   
   		    	$letters = array_slice($arr, $first, $limit);
   
   		foreach ($letters as &$letter) {
   			$letter['from_user_details'] = Model_Table_Customers::model()
   				->fetchByPk($letter['from_user']);
   			$letter['to_user_details'] = Model_Table_Customers::model()
   				->fetchByPk($letter['to_user']);
   		}
   
   if (isset($_GET['outbox'])){smarty_put_var('pages', pages_admin("?module=admin_letters&outbox&page=", $arr, $page, $limit)); smarty_put_var('type', 'outbox');}
   elseif (isset($_GET['verification'])){smarty_put_var('pages', pages_admin("?module=admin_letters&verification&page=", $arr, $page, $limit)); smarty_put_var('type', 'verification');}
   elseif (isset($_GET['deleted'])){smarty_put_var('pages', pages_admin("?module=admin_letters&deleted&page=", $arr, $page, $limit)); smarty_put_var('type', 'deleted');}
   else {smarty_put_var('pages', pages_admin("?module=admin_letters&page=", $arr, $page, $limit)); smarty_put_var('type', '');}
   
   ...
   
   function pages_admin($link, $arr, $page, $vis)
   {
       $radius=4;
       $pages=floor(count($arr)/$vis);
       $return="";
       $str='<table cellpadding="0" cellspacing=3 border="0">
       <tr><td height=7></td></tr>
       <tr>';
   
       if ($pages > 1) {
           $first = 'First';
           $prev = 'Prev';
           $next = 'Next';
           $last = 'Last';
   
           if($page<$radius*2){
              .....
         трэшак в том же духе

   Очередной шедевр с сохранением авторского стиля и табуляции. Это подготовка для smarty представления таблицы с постраничной разбивкой. Возможны 4 типа писем - 4 вкладки. pages_admin() - возвращает пагинатор в виде готового html кода.
   
   Клиент жаловался, что страница сильно тормозит. Там более 150 000 писем. Я догадывался, что код можно чутка оптимизирвоать, но не думал, что существенно. А когда посмотрел вижу - п-ц ))) Интересно насколько получить ускорить?

   stechkh, 01 Марта 2015

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

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

   +76

   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

   private void spreadBullets() {
           bulletItemsWip.clear();
           for (AbstractInventoryItem item : inventoryItemsSortedByValue) {
               if (item instanceof RangeBulletItem) {
                   bulletItemsWip.add((RangeBulletItem) item);
               }
           }
   
           rangeUnitsQueueWip.clear();
           for (Unit unit : areaObject.getUnits()) {
               if (unit.getSpecialization().equals(Unit.UnitSpecialization.RANGE)) {
                   rangeUnitsQueueWip.add(unit);
               }
           }
   
           while (rangeUnitsQueueWip.size > 0 && bulletItemsWip.size > 0) {
               Iterator<Unit> unitsIterator = rangeUnitsQueueWip.iterator();
               while (unitsIterator.hasNext()) {
                   // find suitable bullets or remove unit from queue
                   Unit rangeUnit = unitsIterator.next();
   
                   RangeBulletItem foundBulletItemInInventoryForUnit = searchBulletFor(rangeUnit);
                   if (foundBulletItemInInventoryForUnit == null) {
                       // remove unit from bullets queue
                       unitsIterator.remove();
                   } else {
                       //  bullet was found
                       RangeBulletItem unitBulletsItem = rangeUnit.getEquipment().getRangeBulletsItem();
                       if (unitBulletsItem == null) {
                           unitBulletsItem = foundBulletItemInInventoryForUnit.split(1);
                           rangeUnit.getEquipment().setRangeBulletsItem(unitBulletsItem);
                       } else {
                           if (unitBulletsItem.getClass().equals(foundBulletItemInInventoryForUnit.getClass())){
                               if (!unitBulletsItem.isFull()) {
                                   foundBulletItemInInventoryForUnit.moveQuantity(1, unitBulletsItem);
                               } 
                           } else {
                               rangeUnit.getEquipment().dropRangeBullets();
                               unitBulletsItem = foundBulletItemInInventoryForUnit.split(1);
                               rangeUnit.getEquipment().setRangeBulletsItem(unitBulletsItem);
                           }
                       }
   
                       if (foundBulletItemInInventoryForUnit.isEmpty()) {
                           // remove item from inventory
                           bulletItemsWip.removeValue(foundBulletItemInInventoryForUnit, true);
                           remove(foundBulletItemInInventoryForUnit);
                       }
   
                       if (unitBulletsItem.isFull()) {
                           // remove unit from bullets queue
                           unitsIterator.remove();
                       }
   
                       if (bulletItemsWip.size == 0) {
                           // stop spreading
                           break;
                       }
                   }
               }
           }
   
           // sync quantities to inventory
           for (AbstractInventoryItem item : bulletItemsWip) {
               getItem(item.getClass()).setQuantity(item.getQuantity());
           }
       }

   Раздать пули отряду...

   dmli, 01 Марта 2015

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

3. Куча / Говнокод #17714

   +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

   elseif selectedObj::mouthOnMenuRect then
     if dir == "left" then
       inv = true
       changeMouth()
     elseif dir == "right" then
       inv = false
       changeMouth()
     end
   elseif selectedObj::eyesOnMenuRect then
     if dir == "left" then
       inv = true
       changeEyes()
     elseif dir == "right" then
       inv = false
       changeEyes()
     end
   elseif

   Вступительная заставка к фильму "Добро пожаловать в рай", промелькнуло с десяток таких вот блоков "elseif selectedObj::чтоТоТамOnMenuRect".
   В начале фильма под сеансом видеосвязи с аналоговыми помехами закольцевали фрагмент телефонного справочника на си с полотнами printf.
   Где-то в середине в консольном окошке с заголовком "DATABASE_QUERY" продемонстрировали код функции groups_alloc из ядра linux, а чуть позже - groups_from_user и groups_sort из того же файла.
   Между делом, хакер отличается от других тем, что у него не терминал, а SECURE_TERMINAL, сам он носит ублюдошные очки в толстой оправе, а код в терминале тот же сишный, но красного цвета вместо зеленого.
   Под конец на "главный сервер" заливали виндовый батничек, над которым явно работали больше, чем над сценарием фильма. Нагуглился исходник батничка: http://www.wikihow.com/Make-a-Prank-Virus-(Windows)

   Xom94ok, 01 Марта 2015

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

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

   +159

   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 function get_from_text($text){ 
         foreach($text as $x){ 
           $a = array('A', '?', 'A', '?', '?', '?', '?', 'C', 'E', 'E', 'E', 'E', '?', '?', 'I', 'I', '?', '?', '?', '?', 'O', '?', '?', '?', 'U', '?', 'U', 'U', '?', '?', 'à', '?', 'â', '?', '?', '?', '?', 'ç', 'è', 'é', 'ê', 'ë', '?', '?', 'î', 'ï', '?', '?', '?', 'ô', '?', '?', '?', 'ù', '?', 'û', 'ü', '?', '?', '?', '?', '?', '?', '?', '?', '?', '?', '?', '?', '?', '?', '?', '?', '?', '?', '?', '?', '?', '?', '?', '?', '?', '?', '?', '?', '?', '?', '?', '?', '?', '?', '?', '?', '?', '?', '?', '?', '?', '?', '?', '?', '?', '?', '?', '?', '?', '?', '?', '?', '?', '?', '?', '?', '?', '?', '?', '?', '?', '?', '?', '?', '?', '?', '?', '?', '?', '?', '?', '?', '?', '?', '?', '?', '?', '?', '?', '?', '?', 'Œ', 'œ', '?', '?', '?', '?', '?', '?', '?', '?', '?', '?', '?', '?', '?', '?', '?', '?', '?', '?', '?', '?', '?', '?', '?', '?', '?', '?', '?', '?', '?', '?', '?', '?', '?', '?', '?', '?', '?', '?', '?', '?', '?', '?', '?', '?', 'ƒ', '?', '?', '?', '?', '?', '?', '?', '?', '?', '?', '?', '?', '?', '?', '?', '?', '?', '?', '?', '?', '?', '?', '?', '?', '?', '?'); 
           $b = array('A', 'A', 'A', 'A', 'A', 'A', 'AE', 'C', 'E', 'E', 'E', 'E', 'I', 'I', 'I', 'I', 'D', 'N', 'O', 'O', 'O', 'O', 'O', 'O', 'U', 'U', 'U', 'U', 'Y', 's', 'a', 'a', 'a', 'a', 'a', 'a', 'ae', 'c', 'e', 'e', 'e', 'e', 'i', 'i', 'i', 'i', 'n', 'o', 'o', 'o', 'o', 'o', 'o', 'u', 'u', 'u', 'u', 'y', 'y', 'A', 'a', 'A', 'a', 'A', 'a', 'C', 'c', 'C', 'c', 'C', 'c', 'C', 'c', 'D', 'd', 'D', 'd', 'E', 'e', 'E', 'e', 'E', 'e', 'E', 'e', 'E', 'e', 'G', 'g', 'G', 'g', 'G', 'g', 'G', 'g', 'H', 'h', 'H', 'h', 'I', 'i', 'I', 'i', 'I', 'i', 'I', 'i', 'I', 'i', 'IJ', 'ij', 'J', 'j', 'K', 'k', 'L', 'l', 'L', 'l', 'L', 'l', 'L', 'l', 'l', 'l', 'N', 'n', 'N', 'n', 'N', 'n', 'n', 'O', 'o', 'O', 'o', 'O', 'o', 'OE', 'oe', 'R', 'r', 'R', 'r', 'R', 'r', 'S', 's', 'S', 's', 'S', 's', 'S', 's', 'T', 't', 'T', 't', 'T', 't', 'U', 'u', 'U', 'u', 'U', 'u', 'U', 'u', 'U', 'u', 'U', 'u', 'W', 'w', 'Y', 'y', 'Y', 'Z', 'z', 'Z', 'z', 'Z', 'z', 's', 'f', 'O', 'o', 'U', 'u', 'A', 'a', 'I', 'i', 'O', 'o', 'U', 'u', 'U', 'u', 'U', 'u', 'U', 'u', 'U', 'u', 'A', 'a', 'AE', 'ae', 'O', 'o'); 
           $x= str_replace($a, $b, $x); 
           $x= str_replace("!", "", $x);$x= str_replace("@", "", $x);$x= str_replace("#", "", $x); 
           $x= str_replace("$", "", $x);$x= str_replace("%", "", $x);$x= str_replace("^", "", $x); 
           $x= str_replace("&", "", $x);$x= str_replace("*", "", $x);$x= str_replace("(", "", $x); 
           $x= str_replace(")", "", $x);$x= str_replace("-", "", $x);$x= str_replace("_", "", $x); 
           $x= str_replace("=", "", $x);$x= str_replace("+", "", $x);$x= str_replace("+", "", $x); 
           $x= str_replace("nbsp;", "", $x);$x= str_replace(";", "", $x);$x= str_replace(".", "", $x); 
           $x= str_replace("{", "", $x);$x= str_replace("[", "", $x);$x= str_replace("]", "", $x); 
           $x= str_replace("}", "", $x);$x= str_replace(":", "", $x);$x= str_replace("'", "", $x); 
           $x= str_replace("\"", "", $x);$x= str_replace(",", "", $x);$x= str_replace("<", "", $x); 
           $x= str_replace("\\", "", $x);$x= str_replace("?", "", $x);$x= str_replace(">", "", $x); 
           $x= str_replace("/0-9/", "", $x);$x= str_replace("?", "", $x);$x= str_replace(">", "", $x); 
           $x=trim(str_replace(range(0,9),'',$x));$x=explode(" ",$x); 
                   $vok[]=$x; 
         } 
         return $vok; 
   }

   http://www.phpclasses.org/package/8932-PHP-Filter-text-to-remove-non-SEO-friendly-characters.html#view_files/files/57793

   shilgen, 01 Марта 2015

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

5. Си / Говнокод #17712

   +137

   1. 1

   (a != b) ? a ? b = true : b = !b : b = (a == b) && a;

   refactor, 01 Марта 2015

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

6. Java / Говнокод #17711

   +81

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

   class GetDataFromMySQL extends AsyncTask<String, String, String> {
   
    protected String doInBackground(String[] params) {
   
        runOnUiThread(new Runnable() {
           public void run() {
           ...
           ... 
         //абсолютно вся логика здесь.
           }

   Мастер многопоточного ПОГРОМмирования под Ондроид...

   argamidon, 01 Марта 2015

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

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

   +158

   1. 1
   2. 2

   $arr = mysql_fetch_assoc(mysql_query("select * from customers where Email1='".$_POST['email']."'"));
   if ($arr and count($arr)!=0) {

   Проверка, что массив не пустой

   stechkh, 28 Февраля 2015

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

8. C# / Говнокод #17709

   +95

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

   [Flags]
       public enum UserTypes : uint
       {
           None = 0,
           xxx = uint.MaxValue / 2 + 1,
           yyy= uint.MaxValue / 4 + 1,
           All = xxx | yyy
       }

   Кчему мелочиться

   ichi404, 27 Февраля 2015

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

9. JavaScript / Говнокод #17708

   +159

   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

   <p>
                   <script>// <![CDATA[
                       if (navigator.appVersion.indexOf("Win") != -1) {
                           document.getElementById("windows").style.display = "inline";
                       } else if (navigator.appVersion.indexOf("Mac") != -1) {
                           document.getElementById("osx").style.display = "inline";
                       } else if (navigator.appVersion.indexOf("Linux") != -1) {
                           document.getElementById("linux").style.display = "inline";
                       } else {
                           document.getElementById("windows").style.display = "inline";
                           document.getElementById("osx").style.display = "inline";
                           document.getElementById("linux").style.display = "inline";
                       }
                       // ]]&gt;</script>
               </p>

   Недавно один Java-pазработчик не смог разобраться, как пофиксеть баг в WordPress и решил переписать наш корпоративный сайт на Java мотивируя это тем, что будет намного легче поддерживать и развивать новую ситсему. Что из этого получилось видно на наглядном примере js-кода в верстке.

   etual, 27 Февраля 2015

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

10. JavaScript / Говнокод #17707

    +158

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

    <p><!-- Need to do some UI cleanup --><br>
                    <script type="text/javascript">// <![CDATA[
                        document.getElementById("badge-images").style.visibility = "hidden";
                        document.getElementById("about-contact").style.visibility = "hidden";
                        document.getElementById("header-menu").style.visibility = "hidden";
                        var hideContactDetails = true;
                        // ]]&gt;</script>
                    <br>
                </p>

    Недавно один Java-pазработчик не смог разобраться, как пофиксеть баг в WordPress и решил переписать наш корпоративный сайт на Java мотивируя это тем, что будет намного легче поддерживать и развивать новую ситсему. Что из этого получилось видно на наглядном примере наложение стилей на элементы страницы(верстка).

    etual, 27 Февраля 2015

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

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

• 2107
• …
• 1503
• 1502
• 1501
• 1500
• 1499
• 1498
• 1497
• 1496
• 1495
• …
• 1