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

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

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

1. PHP (5703)
2. C++ (2740)
3. Куча (2337)
4. JavaScript (2030)
5. C# (1929)
6. Java (1503)
7. Си (1103)
8. Pascal (649)
9. Python (585)
10. 1C (536)
11. SQL (434)
12. ActionScript (294)
13. Objective C (204)
14. bash (200)
15. Perl (196)
16. Assembler (148)
17. Ruby (144)
18. VisualBasic (140)
19. Lua (51)
20. Go (32)
21. Swift (29)
22. Haskell (28)
23. Kotlin (15)

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

   +3

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

   public function favoriteCount($userId) {
   		$query = $this->db->query("SELECT * FROM ".DB_PREFIX."favorite WHERE customer_id =".$this->db->escape($userId));
   		$rows = $query->rows;
   		return count($rows);
   }

   Opencart.
   Потому что агрегатные функции для слабаков!

   Panda, 13 Июля 2016

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

2. PHP / Говнокод #20366

   +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
   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

   <table>       
           <?php 
           $showShowroomsIds =  array();
           foreach($available_sizes as $sizeName):
           $available  =  false;
           foreach($warehouses as $warehouse) :
               $uuid = $warehouse['uuid'];
               $name = $warehouse['name'];
               $quantity = $size_quantity[$uuid][trim($sizeName)];
               if($quantity >0) {
                   $available  =  true;
               }
           endforeach;
           if(!$available) continue;
           ?>
           <?php
           foreach($warehouses as $warehouse) :
           $uuid = $warehouse['uuid'];
           //echo $uuid."<br>";
           
           if(isset($showrooms[$uuid])) {
               $showShowroomsIds[$uuid] = $uuid;
           }
           
           $name = $warehouse['name'];
           $quantity = $size_quantity[$uuid][trim($sizeName)];
           if($quantity) {
               $eshopAvailable = '';
               if($uuid == $eshopUuid && $quantity > 0) $eshopAvailable = 'eshop_available=\'y\'';
               $quantity  = ModelCommonWarehouse::getFormattedQuantity($quantity,$uuid);
               $qtA  =  "";
           } else {
               continue;
               $qtA  =  "not-available";
               $quantity  =  "нет в наличии";
           }
           ?>
           <tr <?php echo $eshopAvailable?> available='y'>
               <td class="td">
                   <?php if(isset($showrooms[$uuid])):?><a href="#" class="showroom-info"  uuid="<?php echo $uuid;?>" icId="uuid" rel="nofollow"><?php endif;?>
                   <?php echo $name?>
                   <!--<?php echo $uuid;?>-->
                   <?php if(isset($showrooms[$uuid])):?></a><?php endif;?>
               </td>
               <td class="qt <?php echo $qtA;?>"   ><?php echo $quantity;?></td>
           </tr>
           <?php endforeach;?>
           <tr>
               <td style="height: 10px;"></td>
           </tr>
           <?php endforeach;?>
       </table>

   Opencart. Прямо во вьюхе ровным слоем размазаны бизнес-логика и js.
   Что самое интересное, оборот магазина составляет 7 млн. руб (о__О)

   Panda, 13 Июля 2016

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

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

   +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
   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

   // p2.cpp : Defines the entry point for the console application.
   //   Язык Visual C++ 7.0
   //   Консольное приложение
   //   13.07.2016
   
   #include "stdafx.h"
   #include <conio.h>
   
   int aa (int, int, int);
   void ab (int);
   
   
   int _tmain(int argc, _TCHAR* argv[])
   {
   	int a, b, c, d, e;
   	int f;
   
   	for (a = 0; a < 4; a++)
   		for (b = 0; b < 4; b++)
   			for (c = 0; c < 4; c++)
   				for (d = 0; d < 4; d++)
   					for (e = 0; e < 4; e++)
   					{
   						f = aa (1, a, 2);
   						f = aa (f, b, 3);
   						f = aa (f, c, 4);
   						f = aa (f, d, 5);
   						f = aa (f, e, 6);
   
   						if (f == 35)
   						{
   							printf ("((((1 "); ab (a);
   							printf ("2) "); ab (b);
   							printf ("3) "); ab (c);
   							printf ("4) "); ab (d);
   							printf ("5) "); ab (e);
   							printf ("6 = 35.\n");
   						}
   					}
   
   	getch ();
   	return 0;
   }
   
   
   int aa (int a, int b, int c)
   {
   	switch (b)
   	{
   		case 0: return a + c;
   		case 1: return a - c;
   		case 2: return a * c;
   		case 3: return a / c;
   	}
   
   	return 0;
   }
   
   void ab (int a)
   {
   	switch (a)
   	{
   		case 0: printf ("+ "); break;
   		case 1: printf ("- "); break;
   		case 2: printf ("* "); break;
   		case 3: printf ("/ "); break;
   	}
   }

   Задача: В написанном выражении ((((1 ? 2) ? 3) ? 4) ? 5) ? 6 вместо каждого знака ? вставить знак одного из четырёх арифметических действий: +, -, *, / так, чтобы результат вычислений равнялся 35.

   FrontlineReporter, 13 Июля 2016

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

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

   +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

   $t0=mktime(
               preg_replace("/^....\-..\-..\ (..):..:..$/","$1",$task['end_date']),
               preg_replace("/^....\-..\-..\ ..:(..):..$/","$1",$task['end_date']),
               preg_replace("/^....\-..\-..\ ..:..:(..)$/","$1",$task['end_date']),
               preg_replace("/^....\-(..)\-..\ ..:..:..$/","$1",$task['end_date']),
               preg_replace("/^....\-..\-(..)\ ..:..:..$/","$1",$task['end_date']),
               preg_replace("/^(....)\-..\-..\ ..:..:..$/","$1",$task['end_date'])
           );
   
           $t1=mktime(
               preg_replace("/^....\-..\-..\ (..):..:..$/","$1",$task['real_end_date']),
               preg_replace("/^....\-..\-..\ ..:(..):..$/","$1",$task['real_end_date']),
               preg_replace("/^....\-..\-..\ ..:..:(..)$/","$1",$task['real_end_date']),
               preg_replace("/^....\-(..)\-..\ ..:..:..$/","$1",$task['real_end_date']),
               preg_replace("/^....\-..\-(..)\ ..:..:..$/","$1",$task['real_end_date']),
               preg_replace("/^(....)\-..\-..\ ..:..:..$/","$1",$task['real_end_date'])
           );
   
           for($i=$t0;$i<=$t1;$i=$i+(60*60))
           {

   Вот такой разбор дат по крупицам обнаружился...

   Lblss, 12 Июля 2016

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

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

   +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
   22. 22
   23. 23
   24. 24

   function implode() {
       $str = '';
       if(func_num_args()==1) {
           $array  = (is_array(func_get_arg(0))) ? func_get_arg(0) : false;
           $symbol = '';
       } elseif(func_num_args()==2) {
           $array = (is_array(func_get_arg(0))) ? func_get_arg(0): func_get_arg(1);
           $symbol = (!is_array(func_get_arg(0))) ? func_get_arg(0) : func_get_arg(1);
       } else {
           return false;
       }
       if($array===false) {
           return false;
       } else {
           foreach($array as $k=>$v) {
               if(is_array($v)) {
                   return false;
               } else {
                   $str = $str.$symbol.$v;
               }
           }
       return $str;
       }
   }

   Свой аналог функции implode. И такое бывает

   McLotos, 11 Июля 2016

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

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

   +4

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

   /**
     * @param int $day
     * @return bool
     */
   protected function isWorkingDay($day)
   {
       return $day % 8 <= 5;
   }

   Cпособ определить, является ли указанный день (передается номер дня в неделе от 1 до 7) рабочим. Чем автора не устроил return $day <= 5 - непонятно..

   feversocial, 11 Июля 2016

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

7. C++ / Говнокод #20359

   +4

   1. 1
   2. 2

   auto highPriority = static_cast<bool>(features(w)[5]);
   // Тип features(w) - std::vector<bool>

   Скотт Майерс. Эффективный и современный С++.

   Antervis, 10 Июля 2016

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

8. Куча / Говнокод #20358

   −4

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

   Челлендж
   
   Написать на любом языке программирования код, который будет генерировать и выводить ASCII-гоатсе
   Использование константного гоатсе запрещено

   Тот, у кого получится самый короткий хуй код, получает очко 3_14darа в качестве приза

   cykablyad, 10 Июля 2016

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

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

   +5

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

   [color=blue][u]https://otvet.imgsmail.ru/download/1bceee23eb24bf87101d513013c175aa_i-215.jpg[/u][/color]
   
   [color=blue][u]https://hsto.org/files/fb4/363/3e4/fb43633e4de04988b9f64eeb0c521f68.png[/u][/color]
   
   [color=blue][u]https://cdn.scratch.mit.edu/scratchr2/static/images/hp-video-screenshot.png[/u][/color]

   Можете бить ногами за передачу кода по ссылке, но на ГК картинки по значению не проходят.

   inkanus-gray, 09 Июля 2016

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

10. C++ / Говнокод #20355

    +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
    13. 13
    14. 14
    15. 15
    16. 16
    17. 17
    18. 18

    enum SomeEnum
    {
        // ...
        SomeShit = 0xD6,
        // ...
    };
    
    // ....
    
        Byte opcode = ReadSomeShit<Byte>();  // функция читающая raw memory в нужном представлении
        // из raw memory считано значение эквивалентное 0xD6
        // ...
        if (opcode == SomeShit) // условие не выполнилось
        {
            // do stuff
        }
    
    // ...

    почему? а потому что кто додумался до
    typedef char Byte;
    который (хоть и не обязан быть, но) знаковый
    
    и даже сраного ворнинга не выдало
    причина правда обнаружилась достаточно быстро, ибо в дебагере в opcode красовалось -42 а в SomeShit 214
    
    https://ideone.com/02TpT7 на первый взгляд вызывает когнитивный диссонанс
    обожаю кресты

    meinf, 08 Июля 2016

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

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

• 2103
• …
• 1686
• 1685
• 1684
• 1683
• 1682
• 1681
• 1680
• 1679
• 1678
• …
• 1