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

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

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

1. PHP (5701)
2. C++ (2737)
3. Куча (2347)
4. JavaScript (2028)
5. C# (1928)
6. Java (1502)
7. Си (1103)
8. Pascal (647)
9. Python (583)
10. 1C (535)
11. SQL (432)
12. ActionScript (292)
13. Objective C (202)
14. bash (198)
15. Perl (194)
16. Assembler (146)
17. Ruby (143)
18. VisualBasic (138)
19. Lua (49)
20. Go (30)
21. Swift (27)
22. Haskell (26)
23. Kotlin (13)

1. PHP / Говнокод #19333

   +1

   1. 1
   2. 2
   3. 3

   if ($registered_register != "")
       if (substr($registered_register, strlen($registered_register)-1, strlen($registered_register)) == ".")
           $registered_register = substr($registered_register, 0, strlen($registered_register) - 1);

   rtrim()?
   Нет, не слышали!

   Arris, 20 Января 2016

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

2. C# / Говнокод #19332

   +3

   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

   protected override void mergeModels(IStuff Stuff, IStuff With)
           {
               var stuff = (MyObject)Stuff;
               var with = (MyObject)With;
   
               if (with.title != null)
                   stuff.title = with.title;
   
               if (with.description != null)
                   stuff.description = with.description;
               if (with.creator != null)
                   stuff.creator = with.creator;
               if (with.status != null)
                   stuff.status = with.status;
               if (with.client_name != null)
                   stuff.client_name = with.client_name;
               if (with.client_mail != null)
                   stuff.client_mail = with.client_mail;
               if (with.client_phone != null)
                   stuff.client_phone = with.client_phone;
               if (with.folder != null)
                   stuff.folder = with.folder;
   
               if (with.flag1 != null)
                   stuff.flag1 = with.flag1;
               if (with.flag2 != null)
                   stuff.flag2 = with.flag2;
               if (with.flag3 != null)
                   stuff.flag4 = with.flag3;
               if (with.flag4 != null)
                   stuff.flag4 = with.flag4;
               if (with.flag5 != null)
                   stuff.flag5 = with.flag5;
               if (with.flag6 != null)
                   stuff.flag6 = with.flag6;
               if (with.flag7 != null)
                   stuff.flag7 = with.flag7;
           }

   Копирование данных из одного объекта в другой.

   yakov_255, 19 Января 2016

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

3. JavaScript / Говнокод #19331

   +6

   1. 1
   2. 2

   if(~window.location.href.indexOf('#')){
           var el = window.location.href.slice(window.location.href.indexOf('#')+1,~window.location.href.indexOf('?')?window.location.href.indexOf('?'):window.location.href.length);

   location.hash? не, не слышал

   whiteapps, 19 Января 2016

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

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

   +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

   static public function createFromStr($string)
           {
               $arr = explode('|', trim($string));
               if (!empty($arr)) {
                   $object = new Fingerprint();
                   for($i = 0, $stop = count($arr); $i < $stop; $i++) {
                       $value = trim($arr[$i]);
                       switch($i) {
                           case 0:
                               $object->fingerprint = $value;
                               break;
                           case 1:
                               $object->ctrlV = (int)$value;
                               break;
                           case 2:
                               $object->printableKey = (int)$value;
                               break;
                       }
                   }
   
                   return $object;
               } else {
                   return null;
               }
           }

   создание объекта класса FingerPrint c помощью статического метода; facepalm

   bykovski, 19 Января 2016

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

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

   +3

   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

   function getBounds(curve) {
       var px;
       var py;
       var qx;
       var qy;
       var rx;
       var ry;
       var sx;
       var sy;
       var tx;
       var ty;
       var tobx;
       var toby;
       var tocx;
       var tocy;
       var todx;
       var tody;
       var toqx;
       var toqy;
       var torx;
       var tory;
       var totx;
       var toty;
       var step;
       var x;
       var y;
       var minX;
       var minY;
       var maxX;
       var maxY;
       var ax = curve[0].x;
       var ay = curve[0].y;
       var bx = curve[1].x;
       var by = curve[1].y;
       var cx = curve[2].x;
       var cy = curve[2].y;
       var dx = curve[3].x;
       var dy = curve[3].y;
       var ITERATIONS_COUNT = 250;
   
       minX = minY = Number.POSITIVE_INFINITY;
       maxX = maxY = Number.NEGATIVE_INFINITY;
   
       tobx = bx - ax;
       toby = by - ay;
       tocx = cx - bx;
       tocy = cy - by;
       todx = dx - cx;
       tody = dy - cy;
       step = 1 / ITERATIONS_COUNT;
   
       for (var i = 0; i < ITERATIONS_COUNT; i++) {
           var d = i * step;
           px = ax + d * tobx;
           py = ay + d * toby;
           qx = bx + d * tocx;
           qy = by + d * tocy;
           rx = cx + d * todx;
           ry = cy + d * tody;
           toqx = qx - px;
           toqy = qy - py;
           torx = rx - qx;
           tory = ry - qy;
   
           sx = px + d * toqx;
           sy = py + d * toqy;
           tx = qx + d * torx;
           ty = qy + d * tory;
           totx = tx - sx;
           toty = ty - sy;
   
           x = sx + d * totx;
           y = sy + d * toty;
   
           minX = Math.min(minX, x);
           minY = Math.min(minY, y);
           maxX = Math.max(maxX, x);
           maxY = Math.max(maxY, y);
       }
   
       return {
           x: minX,
           y: minY,
           width: maxX - minX,
           height: maxY - minY
       };
   }

   Хорошей функции должно быть много

   ivasilev, 19 Января 2016

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

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

   +12

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

   \Yii::$app->session->setFlash('success', 'Пользователь ' . $model->username . ' успешно добавлен');
   // TODO: какая-то херня с назначением прав(работой с фалами), поэтому и слип
   sleep(3);
   return $this->redirect(['index']);

   В любой непонятной ситуации ложись спать :)

   alex_matviichuk, 19 Января 2016

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

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

   +7

   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

   <?php
   require __DIR__ . '/google-api-php-client/src/Google/autoload.php';
   define('CLIENT_SECRET_PATH', __DIR__ . '/client_secret.json');
   define('SCOPES', implode(' ', array(Google_Service_Calendar::CALENDAR)));
   // ini_set('error_reporting', E_ALL & ~E_NOTICE);
   // ini_set('display_errors', 1);
   // ini_set('display_startup_errors', 1);
   $token = array(
       'access_token' => 'ya29.VgIK8cb-zWiGwy-cwE5yV4F1MiGzp4LAPogdBCxaqOs3lC8jLxeZL_xK6vEHr_HwrMcg',
       'token_type' => 'Bearer',
       'expires_in' => 3600,
       'created' => 1451157166,
       'refresh_token' => '1/oNh5t4RUGAMiYnNmyIH8iySPr0CdQpnKiXxR57Cj3EpIgOrJDtdun6zK6XiATCKT');
   $json_token = json_encode($token);
     $client = new Google_Client();
     $client->setScopes(SCOPES);
     $client->setRedirectUri('http://localhost/obscur');
     $client->setAuthConfigFile(CLIENT_SECRET_PATH);
     $client->setAccessType('offline');
     $client->refreshToken('1/oNh5t4RUGAMiYnNmyIH8iySPr0CdQpnKiXxR57Cj3EpIgOrJDtdun6zK6XiATCKT');
     $client->setAccessToken($json_token);
   $flname = $_POST['flname'];
   $comment = $_POST['comment'];
   $startTime = str_replace(' ','T',$_POST['start']);
   $endTime = str_replace(' ','T',$_POST['end']);
   $members = $_POST['members'];
   $phone = $_POST['phone'];
   $summary = $flname + ' ' + $members + ' ' + $comment + ' ' + $phone;
   if($flname && $comment && $startTime && $endTime && $members && $phone != ""){
     $service = new Google_Service_Calendar($client);
     $event_info = new Google_Service_Calendar_Event(array(
     'summary' => 'Занято',
     'description' => $summary,
     'start' => array(
       'dateTime' => $startTime,
       'timeZone' => 'Asia/Almaty',
     ),
     'end' => array(
       'dateTime' => $endTime,
       'timeZone' => 'Asia/Almaty',
     ), 
     ));
     $events = $service->events->listEvents('primary');
     
     while(true) {
       foreach ($events->getItems() as $event) {
         
         if(substr($event->start->dateTime, 0, -6) == $startTime){
           echo 'Извините, на данное время уже есть запись';
           $exist = true;
           break;
         }else{
           $exist = false;
         }
       }
       $pageToken = $events->getNextPageToken();
       if ($pageToken) {
         $optParams = array('pageToken' => $pageToken);
         $events = $service->events->listEvents('primary', $optParams);
       } else {
         break;
       }
     }
   if($exist == false){
     $calendarId = '[email protected]';
     $event = $service->events->insert($calendarId, $event_info);
     printf('Запись добавлена');
     header("Location: http://localhost/obscur/забронировать.php");
   }
           
     
   }else{
   	echo "Вы заполнили не все поля";
   }
   ?>

   Блядь

   HiNeX, 19 Января 2016

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

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

   0

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

   QGeoTiledMap *map = new QGeoTiledMap(this,nullptr);
   QGeoMapController *controller = map->mapController();
   qDebug() << rotation << controller->bearing(); // 30 0
   controller->setBearing(rotation);
   qDebug() << controller->bearing(); // 0
   return map;

   because fuck you that's why

   Antervis, 19 Января 2016

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

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

   0

   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

   public static function create($siteId)
   {
   	$basket = new static();
   	$basket->setSiteId($siteId);
   
   //		if ($fuserId !== null)
   //			$basket->setFUserId($fuserId);
   
   	return $basket;
   }
   
   public static function loadItemsForFUser($fUserId, $siteId)
   {
   	/** @var Basket $basket */
   	$basket = static::create($siteId);
   
   	$basket->setFUserId($fUserId);
   	$basket->setSiteId($siteId);
           /********/
   }

   Новый говнокод в новой версии популярного framework'а среди бизнесменов
   Вопрос, зачем надо было столько раз делать одно и тоже?
   Угадай framework по говнокоду, и в качестве подарка, ты ничего не получишь =)

   turbosnail, 19 Января 2016

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

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

    +3

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

    function getLastItemName( $items ) {
    
      $lastItemName = '';
    
      foreach( $items as $item ) {
        $lastItemName = $item['name'];
      }
    
      return $lastItemName;
    }

    Имеем в проекте кучу россыпей подобного вида.
    Собственно, существует ли эффективный способ с этим бороться?

    bot, 18 Января 2016

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

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

• 2101
• …
• 1625
• 1624
• 1623
• 1622
• 1621
• 1620
• 1619
• 1618
• 1617
• …
• 1