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

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

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

1. PHP (5704)
2. C++ (2739)
3. Куча (2355)
4. JavaScript (2029)
5. C# (1928)
6. Java (1502)
7. Си (1113)
8. Pascal (647)
9. Python (583)
10. 1C (538)
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. Swift (27)
22. Haskell (26)
23. Kotlin (14)

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

   В номинации:

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

   За время:

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

2. JavaScript / Говнокод #20037

   +1

   1. 1

   http://pastebin.com/xww1EKP1

   http://map.vmr.gov.ua/scripts/__RasPil.js - было тут

   j123123, 18 Мая 2016

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

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

   +1

   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

   <?php
     
     /*
       > 60 seconds - "s"
       > 60 minutes - "m"
       > 24 hours   - "h"
       > 30 days    - "d"
       < 30 days    - "5 sep 2010"
     */
     
     class Date {
       
       public static function DateAgo($DateTime) {
         
         $s = s; $m = m; $h = h; $d = d;
   
         $Now = date('Y-m-d H:i:s');
         $Now = time();
   
         $Year   = substr($DateTime, 0, 4);
         $Month  = substr($DateTime, 5, 2);
         $Day    = substr($DateTime, 8, 2);
         $Hour   = substr($DateTime, 11, 2);
         $Minute = substr($DateTime, 14, 2);
         $Second = substr($DateTime, 17, 2);
   
         $Time = mktime($Hour, $Minute, $Second, $Month, $Day, $Year);
         
         $Difference = $Now - $Time;
         
         if($Difference < 60) {
           $Ago = $Difference.$s;
         } elseif($Difference < (60 * 60)) {
           $Ago = floor($Difference / 60).$m;
         } elseif($Difference < (60 * 60 * 24)) {
           $Ago = floor($Difference / (60 * 60)).$h;
         } elseif($Difference < (60 * 60 * 24 * 30)) {
           $Ago = floor($Difference / (60 * 60 * 24)).$d;
         } else {
           $Ago = $Day.'.'.$Month.'.'.$Year;
         }
         
         return $Ago;      
       }
       
     } // End class
     
   ?>

   Забирайте целиком! Нашёл в CMS от http://fn85.ru/ охрененный класс для работы с датами!

   nik757, 18 Мая 2016

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

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

   +1

   1. 1
   2. 2
   3. 3

   if (round($this->shot, 2, PHP_ROUND_HALF_DOWN) > round($this->getSumBonus(), 2, PHP_ROUND_HALF_DOWN)) {
         return false;
   }

   Сравнение двух double числе

   govnokoderphp, 10 Мая 2016

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

5. C++ / Говнокод #19978

   +1

   1. 01
   2. 02
   3. 03
   4. 04
   5. 05
   6. 06
   7. 07
   8. 08
   9. 09
   10. 10
   11. 11
   12. 12

   void NewTorrent_PeersListViewCtrl::OnAttach()
   {	
   	WTL::CMenuHandle menu;
   	BOOL menu_created = menu.LoadMenu(HAL_GENERIC_ADD_LV_MENU);
   	InitialSetup(menu);	
   // ...
   	for (int i=0, e=3; i < e; ++i)
   	{
   		AddColumn(names[i].c_str(), i, visible[i], widths[i]);
   	}	
   // ...
   }

   Седьмая строка в этом огрызке вызывает сомнения.

   emanon, 10 Мая 2016

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

6. C++ / Говнокод #19970

   +1

   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

   #include <iostream>
   #include <map>
   
   template <typename T1, typename T2, typename F>
   void operator ^= (std::map<T1,T2> &m, F func) {
       for (typename std::map<T1,T2>::iterator i = m.begin();  i != m.end(); ) {
           if (func(i))
               i = m.erase(i);
           else
               ++i;
       }
   }
   
   template <typename T1, typename T2>
   void operator ^= (std::map<T1,T2> &m, T2 value) {
       for (typename std::map<T1,T2>::iterator i = m.begin();  i != m.end(); ) {
           if (value == i->second)
               i = m.erase(i);
           else
               ++i;
       }
   }
   
   int main() {
       std::map<int,bool> m = {
           {0,false},
           {1,true},
           {2,false},
           {3,true},
           {4,false},
           {5,true},
           {6,false},
           {7,true},
           {8,false},
           {9,true},
           {10,false}
       };
   
       for (auto i = m.begin(); i != m.end(); ++i)
           std::cout << "{" << i->first << "," << i->second << "} ";
       std::cout << std::endl;
   
       m ^= false;
       m ^= [](auto &i) { return i->first == 5; };
   
       for (auto i = m.begin(); i != m.end(); ++i)
           std::cout << "{" << i->first << "," << i->second << "} ";
       std::cout << std::endl;
   }

   Для std::vector'a запись не особо-то кошерная:

   std::vector<int> v = {1,2,3,4,5};
   
   v.erase(std::remove_if(v.begin(),v.end(),[](int &i){ return i <= 3;}), v.end());

   А для std::map даже такого не придумали. Пора исправлять

   Antervis, 10 Мая 2016

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

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

   +1

   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

   // новый тестовый экшн в контроллере
           public function actionNew($alias)
       {
           $model=Partners::model()->model()->findByAttributes(array('alias'=>$alias));
                  
                   if($model==null)
               throw new CHttpException(404,'The requested page does not exist.');
                          
                   $this->render('view',array(
               'model'=>$this->loadModel($model->id),
           ));
          
       }
    
   // правило в конфиге
   // '<module:\w+>/<controller:\w+>/<alias:\w+>' => '<module>/<controller>/new',

   https://vk.com/echo_php?w=wall-175_189930%2Fall
   
   Уи1

   Keeper, 07 Мая 2016

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

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

   +1

   1. 01
   2. 02
   3. 03
   4. 04
   5. 05
   6. 06
   7. 07
   8. 08
   9. 09
   10. 10
   11. 11

   <ol ng-init="citationsLimit = 3" ng-model="citationsLimit">
   	<li class="citation citationList" ng-repeat="citation in answerFact.citations | limitTo: citationsLimit as citationsResult">
   		<i class="fa ic-marker fa-circle" aria-hidden="true"></i>
   		<div class="citation-text">
   			<span ng-bind-html="citation.highlightedSentenceString"></span>
   			<span ng-if="citation.source">
   				(<a  href="{{citation.source}}" target="_blank">{{citation.source}}</a>)
   			</span>
   		</div>
   	</li>
   </ol>

   ifmy, 06 Мая 2016

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

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

   +1

   1. 1
   2. 2
   3. 3

   // $r[2] = 1080;
   $timeStart = (strlen(floor($r[2] / 60)) == 1 ? "0" . floor($r[2] / 60) : floor($r[2] /
                                          60)) . ":" . (strlen($r[2] % 60) == 1 ? "0" . $r[2] % 60 : $r[2] % 60);

   Превращаем минуты во время в формате H:i by ©senior shaurma developer

   pahhan, 05 Мая 2016

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

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

    +1

    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

    <?
    	/*
    		Bitrix Cleaner v2.2 - https://github.com/creadome/bitrixcleaner
    		Быстрая очистка 1С-Битрикс
    		(c) 2015 Станислав Васильев - http://creado.me
    		[email protected]
    	*/
    	require $_SERVER['DOCUMENT_ROOT'].'/bitrix/modules/main/include/prolog_admin_before.php';
    	$trash = array(
    		'cache' => array(
    			'/bitrix/cache/' => 'Неуправляемый кеш',
    			'/bitrix/managed_cache/' => 'Управляемый кеш',
    			'/bitrix/html_pages/' => 'HTML кеш'
    		),
    		'files' => array(
    			'/upload/resize_cache/' => 'Кеш изображений'
    		)
    	);
    	if ($_GET['clean']) {
    		BXClearCache(true);
    		$GLOBALS['CACHE_MANAGER']->CleanAll();
    		$GLOBALS['stackCacheManager']->CleanAll();
    		$staticHtmlCache = \Bitrix\Main\Data\StaticHtmlCache::getInstance();
    		$staticHtmlCache->deleteAll();
    		foreach ($trash['files'] as $files => $name) DeleteDirFilesEx($files);
    	}
    	function countsize($directory) {
    		$count = array('file' => 0, 'size' => 0);
    		foreach (scandir($directory) as $file) {
    			if ($file != '.' && $file != '..') {
    				if (is_dir($directory.$file)) {
    					$inner = countsize($directory.$file.'/');
    					$count['file'] += $inner['file'];
    					$count['size'] += $inner['size'];
    				} else {
    					$count['file'] ++;
    					$count['size'] += filesize($directory.$file);
    				}
    			}
    		}
    		return $count;
    	}
    ?>
    
    <table>
    	<tr>
    		<th>Описание</th>
    		<th>Путь</th>
    		<th>Файлы</th>
    		<th>Размер</th>
    	</tr>
    
    	<?
    		foreach ($trash as $type) {
    			foreach ($type as $directory => $name) {
    				$count = countsize($_SERVER['DOCUMENT_ROOT'].$directory);
    	?>
    
    				<tr>
    					<td><?=$name?></td>
    					<td><a href="/bitrix/admin/fileman_admin.php?lang=ru&amp;path=<?=$directory?>"><?=$directory?></a></td>
    					<td><?=$count['file']?></td>
    					<td><?=round($count['size'] / 1048576, 2)?> Мб</td>
    				</tr>
    
    	<?
    			}
    		}
    	?>
    </table>
    
    <input type="button" value="Очистить" class="adm-btn-save" data-action="clean">

    Битрикс. порадовала строчка "'/bitrix/cache/' => 'Неуправляемый кеш'". Оказывается даже адептам битрикса не подвластно управление кешом самого битрикса :D
    
    https://github.com/creadome/bitrixcleaner/blob/master/cleaner/cleaner.php

    Keeper, 05 Мая 2016

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

11. PHP / Говнокод #19927

    +1

    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

    if(!strpos($amount,'.')) {
        $amount = $amount . '.00';
    } else {
    	$strlength = strlen(explode('.',$amount)[1]);
    	if($strlength != 2) {
    		if($strlength == 1) {
    			$amount = $amount . '0';
    		}
    		if($strlength > 2) {
    			$amount = round($amount,2);
    		}
    	}
    }

    Получение дробного до сотых числа

    ykpon, 05 Мая 2016

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

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

• 1571
• …
• 19
• 18
• 17
• 16
• 15
• 14
• 13
• 12
• 11
• …
• 1