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

• Все
• Лучший
• Сток
• Глупый поиск
• Наговнокодить!
• 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. C# / Говнокод #11671

   +137

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

   string[] CfgStrings = File.ReadAllLines(path);
   title = CfgStrings[0].Replace("Тема: ", "");
   message = CfgStrings[1].Replace("Сообщение: ", "");
   ThreadsCount = Convert.ToInt32(CfgStrings[2].Replace("Потоков: ", ""));
   passwd = CfgStrings[3].Replace("Пароль удаления: ", "");
   board = CfgStrings[4].Replace("Доска: ", "");
   thread = CfgStrings[5].Replace("Тред(если вайпаете доску, поставьте 0): ", "");
   repeat = Convert.ToBoolean(CfgStrings[6].Replace("Зациклить вайп: ", ""));
   proxies = LoadProxies(CfgStrings[7].Replace("Файл с проксями: ", ""));

   хуита, 29 Августа 2012

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

2. Си / Говнокод #11670

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

   int closeestsockptr( SOCKET* pSocket )
   {
   	char chBuf[ 100 ];
   	fd_set rdevents, exevents;
   	struct timeval tv;
   	int iRetVal = 1;
   	int optlen;
   	SOCKET s = INVALID_SOCKET;
   	BOOL bLinger;
   	LINGER lng;
   	if( NULL == pSocket || INVALID_SOCKET == *pSocket )
   		return SOCKET_ERROR;
   	s = *pSocket;
   	optlen = sizeof( bLinger );
   	iRetVal = getsockopt( s, SOL_SOCKET, SO_DONTLINGER, ( char* )&bLinger, &optlen );
   	if( 0 == iRetVal && TRUE == bLinger ) //linger is off
   	{
   		lng.l_onoff = 1; //set linger on
   		lng.l_linger = 1; //set linger timeout to 1 second
   		iRetVal = setsockopt( s, SOL_SOCKET, SO_LINGER, ( char* )&lng, sizeof( lng ) );
   		if( 0 == iRetVal )
   		{
   			if( 0 == shutdown( s, SD_SEND ) )
   			{
   				tv.tv_sec = 3; //seconds
   				tv.tv_usec = 0; //microseconds
   				while( 1 )
   				{
   					FD_ZERO( &rdevents );
   					FD_ZERO( &exevents );
   					addsock2fd( &rdevents, NULL, &exevents, s ); //FD_SET( s, &rdevents ), FD_SET( s, &exevents );
   					iRetVal = select( 1, &rdevents, NULL, &exevents, &tv );
   					if( SOCKET_ERROR != iRetVal && 0 != iRetVal && FD_ISSET ( s, &rdevents ) && !FD_ISSET ( s, &exevents ) )
   					{
   						iRetVal = recv( s, chBuf, sizeof( chBuf ) / sizeof( *chBuf ), 0 );
   						if( iRetVal > 0 ) //Some data received
   							continue;
   						if( 0 == iRetVal ) //Receive FD_CLOSE
   							break;
   						else //SOCKET_ERROR returned
   							break;
   					}
   					else if( 0 == iRetVal ) //exceeded the timeout
   					{
   						WSASetLastError( WSAETIMEDOUT );
   						break;
   					}
   					else //SOCKET_ERROR returned
   						break;
   				}
   			}
   		}
   	}
   
   	if( NULL == pSocket || INVALID_SOCKET == *pSocket )
   		return SOCKET_ERROR;
   	iRetVal = closesocket( *pSocket );
   	*pSocket = INVALID_SOCKET;
   	return iRetVal;
   }

   Пуресишник наваял

   sokol, 29 Августа 2012

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

3. PHP / Говнокод #11669

   +53

   1. 1
   2. 2
   3. 3

   $post_slug = explode("/", trim($_SERVER["REQUEST_URI"], " /"));
   $the_query = new WP_Query(array("name" => $post_slug[1],
                                   "post_type" => "conference"));

   И снова WordPress!'
   
   URL вида http://somesite.ru/conference/pervaya-konferentsiya-anesteziologov/program/

   varg242, 29 Августа 2012

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

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

   +51

   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

   <? 
   return true;
   				@ $db=mysql_connect('akapyan.localhost','akapyan','akapyan123');
   				if ( !$db)
   				 {
   echo "<h2 align=center>Приносим извенения !<br>Произошел сбой соединения,попробуйте зайти на сайт позднее...</h2>
   ";
   exit;
   				}
   				
   	$quer1="select * from obekt  ORDER BY `mm` DESC LIMIT 20  ";   
   												
   												mysql_select_db("db_akapyan");

   Весь код смысла нет приводить, там вообще ад не для слабонервных.
   Хочу уточнить файл инклудится не в функцию.
   Кроме того автор создает константы для того чтобы проверить загружался файл ранее или нет. Он просто не знал, что есть include_once.

   oooZinka, 29 Августа 2012

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

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

   +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

   while (!cp.isEnd()) {
              if (!pause(200)) {
                  break;
              }
          }
   
   private boolean pause(int millisecond) {
          try {
              Thread.sleep(200);
          } catch (InterruptedException ex) {
              Logger.getLogger(Visualizer.class.getName()).log(Level.ERROR, null, ex);
              return false;
          }
          return true;
      }

   Dougrinch, 29 Августа 2012

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

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

   +129

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

   <li title="document.location='http://handleopenurl.com/scheme/buffer' null" href="http://handleopenurl.com/scheme/buffer" class="scheme" onclick="document.location='http://handleopenurl.com/scheme/buffer'">
   <h1>Buffer</h1>
   <h2 class="code">bufferapp</h2>
   </li>

   И это всё вместо <a>.

   Vindicar, 29 Августа 2012

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

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

   +45

   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($data['status'] == 1 ) {
            $data['status_a'] = 'selected="selected"';
            $data['status_d'] = false;
       } else {
             $data['status_a'] = false;
             $data['status_d'] = 'selected="selected"';
        }
    
   echo '<select id="status">
          <option value="1" '.$data['status_a'].'>Активный</option>
           <option value="0" '.$data['status_d'].'>Не активный</option>
   </select>';

   Мой говнокод, виноват! Но все же интересно было бы, как с таким справляются ?

   nepster, 29 Августа 2012

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

8. PHP / Говнокод #11664

   +64

   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

   case 5: {
   			$im = imagecreatetruecolor(1024, 1024);
   			imagefill($im, 0, 0, imagecolorallocate($im, 255, 255, 255));
   			imagecopymerge($im, imagecreatefromjpeg("core/temp/0.jpg"), 0, 0, 0, 0, 255, 255, 100);
   			imagecopymerge($im, imagecreatefromjpeg("core/temp/1.jpg"), 0, 255, 0, 0, 255, 255, 100);
   			imagecopymerge($im, imagecreatefromjpeg("core/temp/2.jpg"), 0, 510, 0, 0, 255, 255, 100);
   			imagecopymerge($im, imagecreatefromjpeg("core/temp/3.jpg"), 0, 765, 0, 0, 255, 255, 100);
   			imagecopymerge($im, imagecreatefromjpeg("core/temp/4.jpg"), 255, 0, 0, 0, 255, 255, 100);
   			imagecopymerge($im, imagecreatefromjpeg("core/temp/5.jpg"), 255, 255, 0, 0, 255, 255, 100);
   			imagecopymerge($im, imagecreatefromjpeg("core/temp/6.jpg"), 255, 510, 0, 0, 255, 255, 100);
   			imagecopymerge($im, imagecreatefromjpeg("core/temp/7.jpg"), 255, 765, 0, 0, 255, 255, 100);
   			imagecopymerge($im, imagecreatefromjpeg("core/temp/8.jpg"), 510, 0, 0, 0, 255, 255, 100);
   			imagecopymerge($im, imagecreatefromjpeg("core/temp/9.jpg"), 510, 255, 0, 0, 255, 255, 100);
   			imagecopymerge($im, imagecreatefromjpeg("core/temp/10.jpg"), 510, 510, 0, 0, 255, 255, 100);
   			imagecopymerge($im, imagecreatefromjpeg("core/temp/11.jpg"), 510, 765, 0, 0, 255, 255, 100);
   			imagecopymerge($im, imagecreatefromjpeg("core/temp/12.jpg"), 735, 0, 0, 0, 255, 255, 100);
   			imagecopymerge($im, imagecreatefromjpeg("core/temp/13.jpg"), 735, 255, 0, 0, 255, 255, 100);
   			imagecopymerge($im, imagecreatefromjpeg("core/temp/14.jpg"), 765, 510, 0, 0, 255, 255, 100);
   			imagecopymerge($im, imagecreatefromjpeg("core/temp/15.jpg"), 765, 765, 0, 0, 255, 255, 100);
   			imagepng($im, "core/".$name_tovar."/1.png");
   			$im2 = imagecreatetruecolor(1024, 1024);
   			imagefill($im2, 0, 0, imagecolorallocate($im2, 255, 255, 255));
   			imagecopymerge($im2, imagecreatefromjpeg("core/temp/16.jpg"), 0, 0, 0, 0, 255, 255, 100);
   			imagecopymerge($im2, imagecreatefromjpeg("core/temp/17.jpg"), 0, 255, 0, 0, 255, 255, 100);
   			imagecopymerge($im2, imagecreatefromjpeg("core/temp/18.jpg"), 0, 510, 0, 0, 255, 255, 100);
   			imagecopymerge($im2, imagecreatefromjpeg("core/temp/19.jpg"), 0, 765, 0, 0, 255, 255, 100);
   			imagecopymerge($im2, imagecreatefromjpeg("core/temp/20.jpg"), 255, 0, 0, 0, 255, 255, 100);
   			imagecopymerge($im2, imagecreatefromjpeg("core/temp/21.jpg"), 255, 255, 0, 0, 255, 255, 100);
   			imagecopymerge($im2, imagecreatefromjpeg("core/temp/22.jpg"), 255, 510, 0, 0, 255, 255, 100);
   			imagecopymerge($im2, imagecreatefromjpeg("core/temp/23.jpg"), 255, 765, 0, 0, 255, 255, 100);
   			imagecopymerge($im2, imagecreatefromjpeg("core/temp/24.jpg"), 510, 0, 0, 0, 255, 255, 100);
   			imagecopymerge($im2, imagecreatefromjpeg("core/temp/25.jpg"), 510, 255, 0, 0, 255, 255, 100);
   			imagecopymerge($im2, imagecreatefromjpeg("core/temp/26.jpg"), 510, 510, 0, 0, 255, 255, 100);
   			imagecopymerge($im2, imagecreatefromjpeg("core/temp/27.jpg"), 510, 765, 0, 0, 255, 255, 100);
   			imagecopymerge($im2, imagecreatefromjpeg("core/temp/28.jpg"), 735, 0, 0, 0, 255, 255, 100);
   			imagecopymerge($im2, imagecreatefromjpeg("core/temp/29.jpg"), 735, 255, 0, 0, 255, 255, 100);
   			imagecopymerge($im2, imagecreatefromjpeg("core/temp/30.jpg"), 765, 510, 0, 0, 255, 255, 100);
   			imagecopymerge($im2, imagecreatefromjpeg("core/temp/31.jpg"), 765, 765, 0, 0, 255, 255, 100);
   			imagepng($im2, "core/".$name_tovar."/2.png");
   			// еще строк 60
   			imagepng($im5, "core/".$name_tovar."/5.png");
   		}

   ..и так далее.
   Прислал знакомый с комментарием:
   
   [18:18:48] брал 16 картинок
   [18:18:52] нужно было склеить в одну
   [18:19:04] ну и эти 16 картинок спарсить и т.д. и т.п.
   
   Это, как бы, только один кейс. Там таких 6, в 6 вообще чертовщина ужасная строк на 200.
   Всё это чудо занимает 858 строк несвежего кала весом в ~80кб.

   AlexT, 28 Августа 2012

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

9. C# / Говнокод #11663

   +141

   1. 1

   if (this.textControl1 != null && !"".Equals(text))

   String.Empty? Нет, не слышал.

   minuzZ, 28 Августа 2012

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

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

    +57

    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

    <?php
    //...
    foreach ($month as $key=>$value)
    	{
    		if($value!=0)
    		{
    			$leased[$year[$key]] = $pay_leased->get_leased_pay($year[$key],$value);
    			$dialup[$year[$key]] = $pay_dialup->get_billing_pay($year[$key],$value);
    			$domain[$year[$key]] = $pay_domain->get_domain_pay($year[$key],$value);
    			$web[$year[$key]] = $pay_web->get_hosting_pay($year[$key],$value);
    
    			for ($d=1;$d<=31;$d++) // во всех месяцах по 31 дню, да
    			{
    				$leased[$year[$key]][$value][$d] = isset($leased[$year[$key]][$value][$d]) ? $leased[$year[$key]][$value][$d] : 0;
    				$dialup[$year[$key]][$value][$d] = isset($dialup[$year[$key]][$value][$d]) ? $dialup[$year[$key]][$value][$d] : 0;
    				$domain[$year[$key]][$value][$d] = isset($domain[$year[$key]][$value][$d]) ? $domain[$year[$key]][$value][$d] : 0;
    				$web[$year[$key]][$value][$d] = isset($web[$year[$key]][$value][$d]) ? $web[$year[$key]][$value][$d] : 0;
    			}
    			ksort($leased[$year[$key]][$value]);
    			ksort($dialup[$year[$key]][$value]);
    			ksort($domain[$year[$key]][$value]);
    			ksort($web[$year[$key]][$value]);
    
    			for ($c=1;$c<=31;$c++) // и еще
    			{
    				$leased[$year[$key]][$value][$c] = $leased[$year[$key]][$value][$c-1]+$leased[$year[$key]][$value][$c];
    				$dialup[$year[$key]][$value][$c] = $dialup[$year[$key]][$value][$c-1]+$dialup[$year[$key]][$value][$c];
    				$domain[$year[$key]][$value][$c] = $domain[$year[$key]][$value][$c-1]+$domain[$year[$key]][$value][$c];
    				$web[$year[$key]][$value][$c] = $web[$year[$key]][$value][$c-1]+$web[$year[$key]][$value][$c];
    				$sum[$year[$key]][$value][$c] = $leased[$year[$key]][$value][$c]+$dialup[$year[$key]][$value][$c]+$domain[$year[$key]][$value][$c]+$web[$year[$key]][$value][$c]; // вау, какая большая строка
    			}
    		}
    	}
    //...

    Досталось в подарок от предыдущих разработчиков. Наверное, я хреновый программист, если ничего не могу здесь понять...
    Комментарии мои, исходник вообще лишен комментариев.

    rsvasilyev, 28 Августа 2012

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

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

• 2107
• …
• 1052
• 1051
• 1050
• 1049
• 1048
• 1047
• 1046
• 1045
• 1044
• …
• 1