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

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

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

1. PHP (5712)
2. C++ (2746)
3. Куча (2407)
4. JavaScript (2034)
5. C# (1930)
6. Java (1503)
7. Си (1121)
8. Pascal (648)
9. Python (592)
10. 1C (541)
11. SQL (433)
12. ActionScript (292)
13. Objective C (202)
14. bash (201)
15. Perl (194)
16. Assembler (148)
17. Ruby (145)
18. VisualBasic (138)
19. Lua (49)
20. Haskell (46)
21. Go (31)
22. Swift (27)
23. Kotlin (14)

1. Лучший говнокод

   В номинации:

   • Лучший рейтинг
   • Самый комментируемый

   За время:

   • за сегодня
   • за неделю
   • за месяц
   • за всё время

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

   +160

   1. 1
   2. 2
   3. 3

   if (RENAME_FILE) {
         $dest_filename = md5(uniqid(rand(), true)) . '.' . $file_ext;
       }

   Рандом, рандом, еще рандом!

   7ion, 12 Февраля 2011

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

3. Perl / Говнокод #5644

   −123

   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

   my %tbl;
   $tbl{'koi'}="БВЧЗДЕЦЪЙКЛМНОПРТУФХЖИГЮЫЭШЩЯЬАСбвчздецъйклмнопртуфхжигюыэшщяьас";
   $tbl{'win'}="абвгдежзийклмнопрстуфхцчшщьыъэюяАБВГДЕЖЗИЙКЛМНОПРСТУФХЦЧШЩЬЫЪЭЮЯ";
   ...
       if ($filename=~/^http:\/\//) {
           my ($header, $page)=getpage($filename);
           if ($page!~/\S/) {next;}
           if ($header=~/Charset.*koi/i) {
               eval("\$page=~tr/$tbl{'koi'}/$tbl{'win'}/");            
           }
           open(OUT, ">$tmpfile");
           print OUT $page;
           close OUT;
           open(NEWS, "$tmpfile");
       } else {
           open(NEWS, "$config::rubricsroot/$filename") || do {warn "Не могу открыть файл новостей: $!"; next;};
       }

   #5624 спровоцировал. Хотел запостить только перекодировку, но не смог удержаться, тут перлы в комплексе.
   1) Свой велосипед для получения странички по http через IO::Socket (это отдельная большая песня, может спою потом).
   2) Перекодировка из koi8r в cp1251, если нужно. Обратите внимание на элегантный eval.
   3) Подстановка результата для дальнейшей построчной обработки через временный файл.
   
   И не говорите мне о том, какой плохой язык PHP.

   gegMOPO4, 11 Февраля 2011

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

4. Pascal / Говнокод #5642

   +110

   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

   procedure TMainForm.ApplicationEventsShortCut(var Msg: TWMKey;
     var Handled: Boolean);
   begin
     if msg.CharCode <> vk_F11 then
       exit;
     raise Exception.Create('Вы хотите выйти из программы?');
   end;
   
   procedure TMainForm.ApplicationEventsException(Sender: TObject;
     E: Exception);
   begin
     if Application.MessageBox(pchar('Произошла ошибка:'#13#10 + e.Message
       +
       #13#10'Нажмите Retry продолжить работу c программой.'#13#10'Нажмите Сancel чтобы завершить работу программы.'),
       'Ошибка', MB_RETRYCANCEL or MB_ICONINFORMATION or MB_SYSTEMMODAL) =
       ID_CANCEL then try
       MainData.MainDataBaseBeforeDisconnect(nil);
       Close;
     finally
       Application.Terminate;
     end;
     if not CaptureError(E) then
       Close;
   end;

   предыдущий автор одного проекта таким вот нетривиальным образом сделал подтверждение выхода из программы. так понравилось, что не стал убивать, просто закомментарил. сейчас весть этот код заменен одной строчкой в OnCloseQuery.
   
   поясню, на всякий случай. по нажатию F11 поднимается эспепшен "Вы хотите выйти из программы?", этот экспепшен перехватывается на уровне TApplication, выводится сообщение "Произошла ошибка: "Вы хотите выйти из программы?"" с педалями "Retry" и "Cancel", и при нажатии на отмену прога тупо рубиться по Application.Terminate.
   
   непростое детство было у песателя, по ходу.

   x77, 11 Февраля 2011

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

5. JavaScript / Говнокод #5635

   +154

   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

   // Parse strings looking for color tuples [255,255,255]
   function getRGB(color) {
     var result;
     if (color && isArray(color) && color.length == 3)
       return color;
     if (result = /rgb\(\s*([0-9]{1,3})\s*,\s*([0-9]{1,3})\s*,\s*([0-9]{1,3})\s*\)/.exec(color))
       return [parseInt(result[1]), parseInt(result[2]), parseInt(result[3])];
     if (result = /rgb\(\s*([0-9]+(?:\.[0-9]+)?)\%\s*,\s*([0-9]+(?:\.[0-9]+)?)\%\s*,\s*([0-9]+(?:\.[0-9]+)?)\%\s*\)/.exec(color))
       return [parseFloat(result[1])*2.55, parseFloat(result[2])*2.55, parseFloat(result[3])*2.55];
     if (result = /#([a-fA-F0-9]{2})([a-fA-F0-9]{2})([a-fA-F0-9]{2})/.exec(color))
       return [parseInt(result[1],16), parseInt(result[2],16), parseInt(result[3],16)];
     if (result = /#([a-fA-F0-9])([a-fA-F0-9])([a-fA-F0-9])/.exec(color))
       return [parseInt(result[1]+result[1],16), parseInt(result[2]+result[2],16), parseInt(result[3]+result[3],16)];
   }

   InstantI, 11 Февраля 2011

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

6. Куча / Говнокод #5617

   +137

   1. 1

   <div style="height: 1px; line-height: 0.1; overflow: hidden; font-size: 12px; color: #fff;">999 999 999 999 999 999 999 999 999 999 999 999 999 999 999 999 999 999 999 999 999 999 999 999 999 999 999 999 999 999 </div><!--Распорка для float:left-->

   yvu, 10 Февраля 2011

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

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

   +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

   // из объявления класса
   ...
       float   _x_3DMin;         /**< Min X value */
       float   _x_3DMax;         /**< Max X value */
       float   _z_3DMin;         /**< Min Z depth value */
       float   _z_3DMax;         /**< Max Z depth value */
   
       /* 2D seismic variables */
       float   _x_2DMin;         /**< Min X value */
       float   _x_2DMax;         /**< Max X value */
       float   _z_2DMin;         /**< Min Z depth value */
       float   _z_2DMax;         /**< Max Z depth value */
   
       /* No seismic selection variables */
       float   _xMin;         /**< Min X value */
       float   _xMax;         /**< Max X value */
       float   _zMin;         /**< Min Z depth value */
       float   _zMax;         /**< Max Z depth value */
   
   ...
   
   void SeismicSelectionDG::initMembers()
   {
     // initialise values
     _x_3DMin = 0.0F;
     _x_3DMax = 0.0F;
     _z_3DMin = 0.0F;
     _z_3DMax = 0.0F;
   
     _x_2DMin = 0.0F;
     _x_2DMax = 0.0F;
     _z_2DMin = 0.0F;
     _z_2DMax = 0.0F;
   
     _xMin = 0.0F;
     _xMax = 0.0F;
     _zMin = 0.0F;
     _zMax = 0.0F;
   
     ...
   }

   Ну и весь остальной код состоит из этих говно-паттернов. Об абстракциях товарищ понятия не имеет.

   kitaec, 09 Февраля 2011

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

8. C++ / Говнокод #5603

   +162

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

   SeismicDriver* SeismicSelectionDG::seismicDriver()
   {
     Exf2dMW* mw = Exf2dMW::getMW(this);
     _seismicDriver = mw->seismicDriver();
     if ( !mw || !_seismicDriver)
       return 0;
   
     return _seismicDriver;
   }

   Все тот же коллега китаец. Exf2dMW - класс главного окна приложения. Во-первых бесмыссленно проверять его на 0, потому что без него вообще бы ничего не работало, но и как видно проверяет его он все равно не там. Ну и проверка на 0 _seismicDrvier тоже лишняя. Итог никакие проверки не нужны да и вообще первую строку в методе можно перенести в конструктор а возвращать член класса _seismicDriver.

   kitaec, 09 Февраля 2011

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

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

   +163

   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

   <?
   class Thread {
   <...>
   	function Thread($proc_id) {
   		$this->db=new ezSQL_mssql(s_login, s_password, s_db_name_threads, s_host);
   		$this->proc_id=$proc_id;
   		$this->timeout=500;
   		$this->last_busy=0;
   		$this->notactive_num=0;
   		$query="INSERT INTO threads(proc_id, last_beat) VALUES('".$this->proc_id."','".(time()+60)."');";
   		$this->db->query($query);
   	}
   	static function Create($url,$proc_id) {
   		$t = new Thread($proc_id);
   		
   		//### execute thread
   		//NB!!!
   		//BE CAREFUL WITH LOG PATHS, IF YOU MISS OR MISSPEL THE PATH, IT IS HARDLY POSSIBLE TO DEBUG
   		//IF YOU MISSPELL THE PATH YOU CAN FACE THE PROBLEM OF THREADS SIMPLY DO NOT START OR DO NOT LOG WITHOUT ANY NOTIFICATION
   		//USE YOUR OWN PATHS FOR PHP, LOGS AND COMMAND LINE COMMANDS AD PARAMETERS FOR YOUR SPECIFIC OS, WINDOWS EXAMPLE IS BELOW
   		//start /B will execute background process in windows, > symbol will store the output of current process into log file
   		//you can call threads from another server via http request etc.
   		pclose(popen("start /B \"$proc_id\" C:\php\php.exe D:\wwwroot\\newimport\elko\import_ignitor_thread.php > D:\globalimport\logs\\".$proc_id.".txt $proc_id","r"));		
   		
   		//give some time to start the thread
   		Sleeper(1000);
   		return $t;
   	}
   	
   	//check is Thread active or not
   	//check active, busy, last beat etc.
   	//you can put here your own business logic how thread should be checked for statused etc.
   	function isActive () {
       if($this->state==3){
   			return false;
   		}elseif ($this->last_busy==1){
   			return true;
   		}
   		$cur_time=time();
   		if($cur_time>$this->last_beat){
   			$result=$this->db->get_var("SELECT last_beat FROM threads WHERE proc_id=".$this->proc_id);
   			$this->state=$this->db->get_var("SELECT state FROM threads WHERE proc_id=".$this->proc_id);
   			if($cur_time<$result){
   				return true;
   			}
   		}else{
   			return true;
   		}
   		return true;
   	}
   	
   	//check is Thread is busy or not, in order to give a new task/job
   	//it is similat to the previous procedure
   	function isBusy() {
   		//$this->tell("ping"); - this could be implemented in the future
   		$cur_time=time();
   		if($cur_time>$this->last_beat or $this->last_busy==0){
   			$result=$this->db->get_var("SELECT busy FROM threads WHERE proc_id=".$this->proc_id);
   			$this->last_busy=$result;
   			if($result==1){
   				return true;
   			}else{
   				return false;
   			}
   		}else{
   			return false;
   		}
   	}
   	
   	//tells a command to the thread
   	function tell($thought, $params = NULL) {
   		$param=base64_encode(serialize($params));
   		$query="INSERT INTO cmd(proc_id, cmd, param) VALUES('".$this->proc_id."','".$thought."','".$param."');";
   		$this->db->query($query);
   	}
   }

   'многопоточность'

   xXx_totalwar, 07 Февраля 2011

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

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

    +157

    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

    <?php
    function Sleeper($mSec)
    {
    	//    For dummies like me who spent 5 minutes
    	//    wondering why socket_create wasn't defined
    	if(!function_exists('socket_create')){
    		die("Please enable extension php_sockets.dll");
    	}
    	//    So the socket is only created once
    	static $socket=false;
    	if($socket===false){
    		$socket=array(socket_create(AF_INET,SOCK_RAW,0));
    	}
    	$pSock=$socket;
    	//    Calc time
    	$uSex = $mSec * 1000;
    	//    Do the waiting
    	socket_select($read=NULL,$write=NULL,$pSock,0,$uSex);
    	//    OCD
    	return true;
    }

    что бы гк делал без этого чудесного языка..

    xXx_totalwar, 07 Февраля 2011

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

11. ActionScript / Говнокод #5534

    −116

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

    var strLabel:String = (_numCurrentUpload + 1) + "/" + _filesToUpload.length;
    strLabel = (_numCurrentUpload + 1 <= _filesToUpload.length && numPerc > 0 && numPerc < 100) ? numPerc + "% - " + strLabel : strLabel;
    strLabel = (_numCurrentUpload + 1 == _filesToUpload.length && numPerc == 100) ? "Upload Complete - " + strLabel : strLabel;
    strLabel = (_filesToUpload.length == 0) ? "" : strLabel;

    Вот так просто и незатейливо обновляется статус загруженного файла :)

    dimas_art, 04 Февраля 2011

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

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

• 1978
• …
• 471
• 470
• 469
• 468
• 467
• 466
• 465
• 464
• 463
• …
• 1