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

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

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

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

   −116

   1. 1
   2. 2

   DECLARE @SysDatoFull datetime
   SET @SysDatoFull = CAST(YEAR(getdate()) as varchar) + RIGHT('00'+CAST(MONTH(getdate()) as varchar), 2) + RIGHT('00'+CAST(DAY(getdate()) as varchar), 2)

   SQL, даты... nuff said
   Вспомнился http://govnokod.ru/9211. Может есть где-то специальное учебное заведение, где обучают таким техникам?

   DBdev, 12 Июля 2012

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

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

   +70

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

   if (s.startsWith("job."))
   {
       s = s.trim().replaceFirst("job\\.", "");
       String name = s.split("\\.")[0];
       String paramName = s.split("\\.")[1];
       // Ещё немного говнокода, не сильно интересного
   }

   konsoletyper, 12 Июля 2012

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

3. Java / Говнокод #11403

   +68

   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

   public abstract class AbstractJob implements Job
   {
       private static ConcurrentHashMap<String, Boolean> mutexes = new ConcurrentHashMap<String, Boolean>();
   
       public abstract void job(JobExecutionContext jec, Logger logger) throws JobExecutionException;
   
       public void execute(JobExecutionContext jec) throws JobExecutionException
       {
           Logger logger = LoggerLocator.getLogger(this.getClass().getSimpleName());
           if (mutexes.containsKey(this.getClass().getSimpleName()))
           {
               logger.info("Job "+jec.getJobDetail().getName()+" already running");            
           }
           else
           {
               try
               {
                   mutexes.put(this.getClass().getSimpleName(), true);
                   job(jec, logger);
               }
               finally
               {
                   mutexes.remove(this.getClass().getSimpleName());
               }
           }
       }
   }

   Race condition и передача логгера в параметрах очень порадовали. А ведь вместо написания кривого велосипеда можно было просто @DisallowConcurrentExecution

   konsoletyper, 12 Июля 2012

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

4. Java / Говнокод #11401

   +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

   public static void main(String[] args) {      
   List<String> parameters = new ArrayList<String>();
   
   if (args.length > 1) {
            int count = 0;
            for (String arg : args) {
               if (count == 0) {
                  count++;
                  continue;
               }
               parameters.add(arg);
            }
   }
   ...

   Ах Индия, Индия...
   (все имена переменных изменены, все совпадения - исключительно случайны)

   Taru4, 12 Июля 2012

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

5. Java / Говнокод #11400

   +117

   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

   public class Foo { 
    public Foo() {
     System.out.println("constructing foo: " + this);
     throw new RuntimeException();
    }
    public void doEvil() {
     System.out.println("Evil!Evil!Evil");
    }
   }
   
   public class MyFoo extends Foo {
    public static Foo x;
    @Override
    protected void finalize() throws Throwable {
     x = this;
    } 
   }
   
   try {
    Foo x = new MyFoo();
   } catch(Exception e) {
    System.out.println(""+e);
   }
   System.gc();
   System.runFinalization();
   System.out.println("MyFoo instance: " + MyFoo.x);
   MyFoo.x.doEvil();

   Вот так бесстрашные хакеры получают доступ к методу экземпляра объекта, конструктор которого выбрасывает исключение, а потом жалуются, что, мол, Java - решето.
   
   А теперь вопрос на засыпку: как сделать то же самое, не используя finalize и gc?

   someone, 12 Июля 2012

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

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

   +54

   1. 1
   2. 2

   //Вычисляем предыдущий месяц
   $bdate = q1("SELECT '$cdate' - INTERVAL 1 MONTH");

   wds, 12 Июля 2012

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

7. Си / Говнокод #11398

   +113

   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

   #include <stdio.h>
    
   int main()
   {
           void say_hello()
           {
                   puts("Hello govnokod!");
           }
    
           say_hello();
   
   	return 0;
   }

   http://ideone.com/QZipp

   rat4, 11 Июля 2012

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

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

   +156

   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

   var renderHours = function(s){
           var html = '<ul>';
           var json = Ext.decode(s);
           for(i in json)
               if(typeof json[i].from_d != 'undefined')
                   html += '<li>с <b>'
                       + hours[json[i].from_d][1]
                       + '</b> -  по <b>'
                       + hours[json[i].to_d][1]
                       + '</b> ('
                       + json[i].from_h +':'+ json[i].from_m +'-'
                       +json[i].to_h +':'+ json[i].to_m +')</li>';
               
           return html+'</ul>';
       };

   Вот такое попалось

   Sulik78, 11 Июля 2012

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

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

   +56

   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

   <html>
   <head>
   <style>
      body{
       background-color: #3366CC; /* Цвет фона веб-страницы */
      }
   </style>
   </head>
   <body>
   <?php
   	/*error_reporting(E_ALL);*/
   	class Mirror{
   		public $sizeX;
   		public $sizeY;
   		public $filenameI;
   		public $filenameO;
   		public $red;
   		public $green;
   		public $blue;
   		public $alpha;
   		function __construct($f){
   			$this->filenameI=$f.".png";
   			$this->filenameO=$f."2.png";
   			$size=getimagesize($this->filenameI);
   			$this->sizeX=$size[0];
   			$this->sizeY=$size[1];
   		}
   		function saveImg(){
   			if(!file_exists($this->filenameO)){
   				$xx=$this->sizeX;
   				$yy=($this->sizeY)*2;
   				$im = imagecreatetruecolor($xx, $yy);
   				imageAlphaBlending($im, false);
   				imageSaveAlpha($im, true);
   				for($x=0;$x<$xx;$x++){
   					for($y=0;$y<$yy;$y++){
   						$r=$this->red[$x][$y];
   						$g=$this->green[$x][$y];
   						$b=$this->blue[$x][$y];
   						$alph=$this->alpha[$x][$y];
   						$color=ImageColorAllocateAlpha($im, $r, $g, $b, $alph);
   						imagesetpixel($im, $x, $y, $color);
   					}
   				}
   				imagepng($im, $this->filenameO);
   				imagedestroy($im);
   			}
   			echo "<img src=\"".$this->filenameI."\" /><hr>\n";
   			echo "<img src=\"".$this->filenameO."\" /><br>\n";
   		}
   		function loadImg(){
   			$im = imagecreatefrompng($this->filenameI);
   			for($x=0;$x<$this->sizeX;$x++){
   				for($y=0;$y<$this->sizeY;$y++){
   					$rgb = imagecolorat($im, $x, $y);
   					$color=imagecolorsforindex($im, $rgb);
   					$this->red[$x][$y]=$color['red'];
   					$this->green[$x][$y]=$color['green'];
   					$this->blue[$x][$y]=$color['blue'];
   					$this->alpha[$x][$y]=$color['alpha'];
   				}
   			}
   			imagedestroy($im);
   		}
   		function createMirror(){
   			if(!file_exists($this->filenameO)){
   				$yy=($this->sizeY*2);
   				for($x=0;$x<$this->sizeX;$x++){
   					$alph=0;
   					for($y=$this->sizeY;$y<$yy;$y++){
   						if($alph<126){
   							$alph++;
   						}
   						$this->red[$x][$y]=$this->red[$x][$this->sizeY-($y-$this->sizeY)-1];
   						$this->green[$x][$y]=$this->green[$x][$this->sizeY-($y-$this->sizeY)-1];
   						$this->blue[$x][$y]=$this->blue[$x][$this->sizeY-($y-$this->sizeY)-1];
   						$this->alpha[$x][$y]=$alph+$this->alpha[$x][$this->sizeY-($y-$this->sizeY)-1];
   						if($this->alpha[$x][$y]>127) $this->alpha[$x][$y]=127;
   					}
   				}
   			}
   		}
   	}
   	$filename="alpha";
   	$mirror=new Mirror($filename);
   	$mirror->loadImg();
   	$mirror->createMirror();
   	$mirror->saveImg();
   ?>
   </body>
   </html>

   Делал для одного сайта зеркальное отражение картинки (photoshop использовать не умеем-с). Почему-то сервер часто падал.
   Как оно работает, можно посмотреть здесь http://xcont.com/perc/newperceptron/cell/mirror.php

   xcont, 11 Июля 2012

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

10. Pascal / Говнокод #11395

    +99

    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

    // Получение имени выполняемого метода, вызывать можно только из Published-методов класса.
    // Для обычных методов: FindClassMethodNames(ClassType()), для статических методов FindClassMethodNames(self).
    {$optimization OFF}
    function FindClassMethodNames(obj: TClass): string;
    var _AdrPtr: Pointer;
    begin
      asm
        mov eax, obj
        mov edx, dword ptr [esp + 272]
        push ebx
        push esi
        push edi
        push $7FFFFFFF
        xor edi, edi
        jmp @@haveVMT
       @@outer:
        mov eax, dword ptr [eax]
       @@haveVMT:
        mov esi, dword ptr [eax].vmtMethodTable
        test esi, esi
        je @@parent
        movzx ecx, word ptr [esi]
        add esi, 2
       @@inner:
        pop ebx
        push edx
        sub edx, dword ptr [esi + 2]
        jl @@no1
        cmp edx, ebx
        jg @@no1
        mov ebx, edx
        mov edx, dword ptr [esi + 2]
        mov edi, edx
       @@no1:
        pop edx
        push ebx
        movzx ebx, word ptr [esi]
        add esi, ebx
        dec ecx
        jnz @@inner
       @@parent:
        mov eax, dword ptr [eax].vmtParent
        test eax, eax
        jne @@outer
        mov _AdrPtr, edi
        pop edi
        pop esi
        pop ebx
      end;
      Result := obj.MethodName(_AdrPtr);
    end;
    {$optimization ON}
    
    // пример использования
    procedure TForm1.Button1Click(Sender: TObject);
    begin
      ShowMessage(FindClassMethodNames(ClassType()));
    end;

    А есть нормальный способ получить имя выполняемого метода, и строку кода заодно?

    ctm, 11 Июля 2012

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

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

• 2107
• …
• 1027
• 1026
• 1025
• 1024
• 1023
• 1022
• 1021
• 1020
• 1019
• …
• 1