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

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

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

1. PHP (5708)
2. C++ (2740)
3. Куча (2389)
4. JavaScript (2034)
5. C# (1930)
6. Java (1503)
7. Си (1117)
8. Pascal (648)
9. Python (591)
10. 1C (539)
11. SQL (432)
12. ActionScript (292)
13. Objective C (202)
14. bash (200)
15. Perl (194)
16. Assembler (148)
17. Ruby (145)
18. VisualBasic (138)
19. Lua (49)
20. Haskell (37)
21. Go (31)
22. Swift (27)
23. Kotlin (14)

1. JavaScript / Говнокод #6096

   +190

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

   // Count how many rows are in the table.
       for (var x = 0; x < tableRows.length; ++x)
       {
       	rowCount += 1;
       }

   napmor1, 26 Марта 2011

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

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

   +149

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

   // get info from db
   		$results = $this->datadb->dataForIndex();
   		$data_['mainResult'] = $results['mainResult'];
   		$data_['pics'] = $results['pics'];
   		$data_['com'] = $results['com'];
   		$data_['huuInvert'] = $results['huuInvert'];
   		$data_['dateInvert'] = $results['dateInvert'];
   		$data_['type'] = $results['type'];

   Кусок со своего CMS. Да я знаю что можно объединить массивы, но иногда не все требуется из функции.

   increazon, 26 Марта 2011

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

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

   +168

   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

   function UploadAvatar($FILE_tmp, $FILE_name){
   	/*куча тупого кода*/
   	if($width >= $height)
   	    $kf = (float) $width/$height;
   	if($width < $height)		    	  
   	   $kf = (float) $height/$width;
   		   
   	//Проверка на квадратность! ппц идиотизм, идея не моя.
   	if($kf>1.1){
   		$error = "Аватар неквадратен. Пожалуйста выберите квадратный или обрежьте текущий в графическом редакторе.";
   		return 0;
   	}
   	/*еще куча тупого кода*/
   }

   Вырезал кусок из собственного старого сайта. Комменты оставил уникальными.
   Когда дизайнер потребовал чтоб аватары были только квадратны, я наваял такое.
   Допуск в 1,1 - это поблажка юзерам)))

   Skull, 25 Марта 2011

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

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

   +151

   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

   <?php
   
   function test_menu() {
   //  $menu['test'] = array(
   //    'page callback' => 'test_page',
   //    'access callback' => TRUE,
   //  );
   
     $menu['test/%user_uid_optional'] = array(
       'page callback' => 'test_view',
       'page arguments' => array(1),
       'access callback' => 'test_access',
       'access arguments' => array(1),
     );
   
     $menu['test/%user/view'] = array(
       'title' => 'View',
       'type' => MENU_DEFAULT_LOCAL_TASK,
       'weight' => -10,
     );
   
     $menu['test/%user/edit'] = array(
       'title' => 'Edit',
       'page callback' => 'test_edit',
       'page arguments' => array(1),
       'access callback' => 'test_access',
       'access arguments' => array(1),
       'type' => MENU_LOCAL_TASK,
     );
   
     return $menu;
   }
   
   function test_page() {
     global $user;
     if ($user->uid) {
       menu_set_active_item("test/$user->uid");
       return menu_execute_active_handler();
     }
     else {
       drupal_goto('user/login');
     }
   }
   
   function test_view($account) {
     module_load_include('pages.inc', 'user');
     return user_view($account);
   }
   
   function test_edit($account) {
     module_load_include('pages.inc', 'user');
     return user_edit($account);
   }
   
   function test_access($account) {
     dpm($account);
     return TRUE;
   }

   vectoroc, 25 Марта 2011

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

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

   +150

   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

   <?php
   
   function test_menu() {
     $menu['test'] = array(
       'page callback' => 'test_page',
       'access callback' => TRUE,
     );
   
     $menu['test/%user'] = array(
       'page callback' => 'test_view',
       'page arguments' => array(1),
       'access callback' => TRUE,
     );
   
     $menu['test/%user/view'] = array(
       'title' => 'View',
       'type' => MENU_DEFAULT_LOCAL_TASK,
       'weight' => -10,
     );
   
     $menu['test/%user/edit'] = array(
       'title' => 'Edit',
       'page callback' => 'test_edit',
       'page arguments' => array(1),
       'access callback' => TRUE,
       'type' => MENU_LOCAL_TASK,
     );
   
     return $menu;
   }
   
   function test_page() {
     global $user;
     if ($user->uid) {
       menu_set_active_item("test/$user->uid");
       return menu_execute_active_handler();
     }
     else {
       drupal_goto('user/login');
     }
   }
   
   function test_view($account) {
     module_load_include('pages.inc', 'user');
     return user_view($account);
   }
   
   function test_edit($account) {
     module_load_include('pages.inc', 'user');
     return user_edit($account);
   }

   vectoroc, 25 Марта 2011

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

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

   +150

   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

   <?php
   
   function test_menu() {
     $menu['test'] = array(
       'page callback' => 'test_page',
       'access callback' => TRUE,
     );
   
     $menu['test/%user'] = array(
       'page callback' => 'test_view',
       'page arguments' => array(1),
       'access callback' => TRUE,
     );
   
     return $menu;
   }
   
   function test_page() {
     global $user;
     if ($user->uid) {
       menu_set_active_item("test/$user->uid");
       return menu_execute_active_handler();
     }
     else {
       drupal_goto('user/login');
     }
   }
   
   function test_view($account) {
     module_load_include('pages.inc', 'user');
     return user_view($account);
   }

   vectoroc, 25 Марта 2011

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

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

   +167

   1. 1
   2. 2

   <?php
   $nidPage_storage = 64578; // нид страницы "Магазины и склады" - хранится отдельно, потому что контент-менеджеры удаляют страницу "Магазинов" и нид всё время меняется

   Изначально это это был drupal.

   turdman, 25 Марта 2011

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

8. Java / Говнокод #6089

   +78

   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

   public Boolean isEqual(RepJoinCondition targetJoin )
   	{
   		if (targetJoin == null) return false;
   
   		Boolean isLeftSideEqual ;
   		Boolean isRightSideEqual ;
   					
   		isLeftSideEqual = ((this.leftJoinFiled.fieldTableName.toUpperCase() == targetJoin.leftJoinFiled.fieldTableName.toUpperCase()) &&
   							(this.leftJoinFiled.fieldName.toUpperCase() == targetJoin.leftJoinFiled.fieldName.toUpperCase()));
   							
   		isRightSideEqual = 	((this.rightJoinFiled.fieldTableName.toUpperCase() == targetJoin.rightJoinFiled.fieldTableName.toUpperCase()) &&
   							(this.rightJoinFiled.fieldName.toUpperCase() == targetJoin.rightJoinFiled.fieldName.toUpperCase()));
   							
   		if (isLeftSideEqual && isRightSideEqual) return true;
   		
   		isLeftSideEqual = ((this.leftJoinFiled.fieldTableName.toUpperCase() == targetJoin.rightJoinFiled.fieldTableName.toUpperCase()) &&
   							(this.leftJoinFiled.fieldName.toUpperCase() == targetJoin.rightJoinFiled.fieldName.toUpperCase()));
   							
   		isRightSideEqual = 	((this.rightJoinFiled.fieldTableName.toUpperCase() == targetJoin.leftJoinFiled.fieldTableName.toUpperCase()) &&
   							(this.rightJoinFiled.fieldName.toUpperCase() == targetJoin.leftJoinFiled.fieldName.toUpperCase())); 					
   		
   		if (isLeftSideEqual && isRightSideEqual) return true;					
   		
   		return false;
   	}

   Весьма сложнная проверка равенства, изпользуется некорректно ==

   vraptor, 25 Марта 2011

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

9. Java / Говнокод #6088

   +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

   package somepackage;
   
   import  java.net.URL;
   import  javax.xml.namespace.QName;
   import  javax.xml.ws.Service;
               
   public class proxyClass {
       
       URL url = new URL("http://someserver/somepath/output?wsdl");
       QName qname = new QName("http://someserver/", "someservice");
   
       Service service = Service.create(url, qname);
   
       test someVar = service.getPort(test.class);
   
   }

   И почему же Eclipse ругается?

   lyric, 25 Марта 2011

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

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

    +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

    <?php
    $z=0;
    $ti=0;
    $ti2=0;
    
    $res=array();
    $txt="";
    $res=array();
    if($argc<2)exit(0);
    $fn=$argv[1];
    $pt='/(^|\b|\s)((\w|[а-я]|\d)+(?:\.|\,)?)(\b|\.|,|-)/i';
    if(isset($fn)){
    $fc=file($fn);
    foreach($fc as $nl=>$str){
    preg_match_all($pt,$str,$res,PREG_PATTERN_ORDER);
    foreach($res[0] as $key=>$per){
     # $per=trim($per);
    $sz=iconv_strlen($per);
    $tz=iconv_strlen($per);
    echo "Per $per key $key sz $sz\n";
    
    while($sz>=2){ 
     $eb = substr($per,0,$sz-1);
    echo "eb $eb\n";
    $res=system("cat -b dict.txt | grep ' ".$eb." ' | head -n1 | gawk '{ print $1 }'");
    #echo "$res\n";
    if($res!=""){
    echo "res $res\n";
    $oc=substr($per,$sz-1,$tz); 
    $fd=fopen("$argv[1].txt","a");
    if($fd<0){echo "fopen";exit(0);};
      fseek($fd,SEEK_END,0);
    fwrite($fd,"$res$oc ");
    
    fclose($fd);
    $sz=1;
    sleep(3);
    };
      $sz--;
    };
    if($sz==1){
    
    $fd=fopen("$argv[1].txt","a");
    if($fd<0){echo "fopen";exit(0);};
      fseek($fd,SEEK_END,0);
      fwrite($fd,"$per ");
      fclose($fd);
    };
    
    };
    };
    $str="";
    $res="";
    }
    ?>

    Замена слова кодом из словаря, по номеру слова в словаре.

    AliceGoth, 25 Марта 2011

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

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

• 2111
• …
• 603
• 602
• 601
• 600
• 599
• 598
• 597
• 596
• 595
• …
• 1