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

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

   +165

   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

   if (seismicDriver()->has3DSeismic() && !seismicDriver()->has2DSeismic() ) {
       enableSeismicTab(0, true);
       enableSeismicTab(1, false);
     }
     else if (!seismicDriver()->has3DSeismic() && seismicDriver()->has2DSeismic() ) {
       enableSeismicTab(0, false);
       enableSeismicTab(1, true);
     }
     else if (seismicDriver()->has3DSeismic() && seismicDriver()->has2DSeismic() ) {
       enableSeismicTab(0, true);
       enableSeismicTab(1, true);
     }
     else if (!seismicDriver()->has3DSeismic() && !seismicDriver()->has2DSeismic() ) {
       enableSeismicTab(0, false);
       enableSeismicTab(1, false);
     }

   Этим своим первым постом предворяю серию говнокода, который пишет мой коллега, китаец, имеющий PhD по физике.
   
   В этом сниппете советую обратить внимание на два компонента в условиях if и на расстановку true и false в соответствии с ними.

   kitaec, 08 Февраля 2011

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

2. C++ / Говнокод #5588

   +161

   1. 1

   throw &std::exception();

   Из того же проекта, что и http://www.govnokod.ru/5581 .
   Естественно это потом иногда удаляется по delete...

   Говногость, 08 Февраля 2011

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

3. C++ / Говнокод #5587

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

   // говно-код - вступление
                           KEY_ACCELERATOR_DATA acceleratorData;
                           WORD id; int i, j, k;
   
                           acceleratorData.line = 255;
                           acceleratorData.address = 255;
                           acceleratorData.sensors = 255;
                           acceleratorData.action = 255;
                           KeyAcceleratorDataList tempList;
   
                           // говно-код - квинтэссенция
                           for(i = 1; i < listWholeString.ItemsCount(); i++) 
                           {
                                       k = 0;
                                       strCurrSensor = listWholeString[i];
                                       if((k = strCurrSensor.Find(_T('-'), k)) <= 0)
                                                   break;
                                       acceleratorData.line = (BYTE)_ttoi(strCurrSensor.Left(k));
                                       // ну кто блядь это поймет??? Кто???!!!
                                       j = k;                            
                                       if((k = strCurrSensor.Find(_T('-'), k + 1)) <= 0)
                                                   break;
                                       acceleratorData.address = (BYTE)_ttoi(strCurrSensor.Mid(j + 1, k - (j + 1)));
   
                                       j = k;                            
                                       if((k = strCurrSensor.Find(_T('-'), k + 1)) <= 0)
                                                   break;
                                       acceleratorData.sensors = (BYTE)_ttoi(strCurrSensor.Mid(j + 1, k - (j + 1)));
   
                                       if((k = strCurrSensor.Find(_T('-'), k)) <= 0)
                                                   break;
                                       acceleratorData.action = (BYTE)_ttoi(strCurrSensor.Right(strCurrSensor.GetLength() - (k + 1)));
                                       
   
                                       if((acceleratorData.line == 255) || (acceleratorData.address == 255) || 
                                         (acceleratorData.sensors == 255) || (acceleratorData.action == 255))
                                                   break;
                                       tempList.push_back(acceleratorData);
                           }           
                           mapKeyAccelerators.operator[](id) = tempList;
               }
   // Аут… Тушите свет и сидите, где сидели… А я нахуй сам пойду…

   парень "кончал на компьютер" и даже получил сертификат Microsoft по MFC
   лучше бы он кончал на дорожного рабочего...

   ShuraKotov, 08 Февраля 2011

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

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

   +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

   # Добавление пользователя
   	if ($_GET['action'] == "add_user" OR $_POST['action'] == "add_user"){
   		#Проверяем достаточно ли прав на добавление пользователя
   		$pal = 15;
   		if (access_level() < $pal) {
   			echo "Извените, у вас недостаточно прав для данной операции<br>";
   			echo "Ваш уровень ".access_level()." а доступ возможен только с ".$pal."<br>";
   		}
   		else{
   		echo "Уровень доступа - $pal<br>";
   		echo "<form action=\"index.php\" method=\"post\">\n";
   		echo "<select name=type><option value=1>support<option value=2>installer</select><br>";
   		echo "Логин ( первая буква имени + фамилия (en))  <input type=\"text\" name=\"add_username\"><Br> passwd <input type=\"password\" name=\"add_passwd\"><Br>\n access_level <input type=text name=access_level><br>\n";
   		echo "Полное имя (Фамилия, Имя, [Отчество]) <input type=\"text\" name=\"add_full_name\"><br>";
   		echo "Телефон (только 10 цифр, без +7, 8 и т.п.) <input type=\"text\" name=\"add_phone\"><Br>";
   		echo "<input type=\"hidden\" name=\"action\" value=\"add_user\">\n";
   		echo "<input type=\"submit\" name=\"submit\" value=\"add\"><br>\n</form>";
   
   		$add_name=mysql_real_escape_string($_POST['add_username']);
   		$add_pass=mysql_real_escape_string($_POST['add_passwd']);
   		$add_full_name=mysql_real_escape_string($_POST['add_full_name']);
   		$add_phone=mysql_real_escape_string($_POST['add_phone']);
   		$add_access_level=mysql_real_escape_string($_POST['access_level']);
   		$add_pass = md5($add_pass);
   		if($_POST['type'] == "1"){
   			echo $add_name."-".$add_pass."-".$add_access_level."<br>";
   			$sql = 'INSERT INTO `switches`.`creators` (`id`, `name`, `passwd`, `access_level`, `descr`, `creator`, `create_date`) VALUES (\'\', \''.$add_name.'\', \''.$add_pass.'\', \''.$add_access_level.'\', \'\', \''.$_SESSION['user_id'].'\', NOW());';
   			mysql_query($sql,$dbutm);
   			echo "<br>Пользователь добавлен.<Br>";
   		}
   		elseif($_POST['type'] == "2"){
   			$sql = 'INSERT INTO `switches`.`installers` (`id`, `name`, `full_name`, `passwd`, `access_level`, `descr`, `creator`, `create_date`, `phone`) VALUES (\'\', \''.$add_name.'\', \''.$add_full_name.'\', \''.$add_pass.'\', \''.$add_access_level.'\', \'\', \''.$_SESSION['user_id'].'\', NOW(), \''.$add_phone.'\');';
   			mysql_query($sql,$dbutm);
   			echo "<br>Монтажник добавлен.<Br>";
   		}
   	}
   }

   Система биллинга. Начинаю нервничать...

   Gogogo, 08 Февраля 2011

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

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

   +160

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

   if ($this->Session->check('_userJustRegistered')) {
   	switch (($tmpJR = $this->Session->read('_userJustRegistered'))) {
   		case 'confirmed':
   			$this->Session->delete('_userJustRegistered');
   			break;
   		case 'not_confirmed_first':
   			$this->Session->write('_userJustRegistered', 'not_confirmed');
   			break;
   	}
   }

   Кусочек многоступенчатой системы по определению статуса юзера.

   quall, 08 Февраля 2011

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

6. Java / Говнокод #5584

   +71

   1. 1
   2. 2

   //rapeint
   this.repaint();

   Выеби интегер!

   danilissimus, 08 Февраля 2011

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

7. JavaScript / Говнокод #5583

   +163

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

   function onsub(text){
   	if (confirm(text)) { 
   		return true;
   	}
   	else {
   		return false;
   	}
   }

   Копаюсь в системе биллинга...
   Убило!

   Gogogo, 08 Февраля 2011

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

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

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

   // Функция для проверки входнях данных и присвоения входных данных переменым
   // $pattern задаёт регулярное выражение
   // $key ключ в массивах $_POST[] или $_GET[]
   // $firts и $second порядок их просмотра
   // $second не обязательный параметр
   // $die_if_not_match задаёт поведение при не соответствии входных данных регулярному выражению
   // $die_if_not_set задаёт поведение при отсутствии входных данных
   // по умолчания при не совпаденини или отсутствии входных данных функция не возвращаяет ничего
   function inchk($pattern, $key, $first, $second = '' ,$die_if_not_match = '0',$die_if_not_set = '0' ){
   	if(!$pattern){
   		die("Шаблон для inchk() не задан");
   	}elseif(!$key){
   		die("Ключ для inchk() не задан");
   	}
   	elseif(!$first){
   		die("Источник данных inchk() не задан");
   	}elseif($first and $second){
   		if($first == "post" or $first == "p"){
   			if($_POST[$key] != ''){
   				if(preg_match($pattern, $_POST[$key])){
   					return $_POST[$key];
   				}else{
   					if($die_if_not_match){ die('недопустимый формат $_POST[\''.$key.'\']'); }
   				}
   			}elseif($_GET[$key] != ''){
   				if(preg_match($pattern, $_GET[$key])){
   					return $_GET[$key];
   				}else{
   					if($die_if_not_match){ die('недопустимый формат $_GET[\''.$key.'\']'); }
   				}
   			}else{
   				if($die_if_not_set){ die('$_POST[\''.$key.'\'] или $_GET[\''.$key.'\'] не задан'); }
   			}
   		}
   		elseif($first == "get" or $first == "g"){
   			if($_GET[$key] != ''){
   				if(preg_match($pattern, $_GET[$key])){
   					return $_GET[$key];
   				}else{
   					if($die_if_not_match){ die('недопустимый формат $_GET[\''.$key.'\']'); }
   				}
   			}elseif($_POST[$key] != ''){
   				if(preg_match($pattern, $_POST[$key])){
   					return $_POST[$key];
   				}else{
   					if($die_if_not_match){ die('недопустимый формат $_POST[\''.$key.'\']'); }
   				}			
   			}else{
   				if($die_if_not_set){ die('$_GET[\''.$key.'\'] или $_POST[\''.$key.'\'] не задан'); }
   			}
   		}else{
   			die("Источник данных inchk() задан не верно");
   		}
   
   	}elseif($first == "get" or $first == "g"){
   		if($_GET[$key] != ''){
   			if(preg_match($pattern, $_GET[$key])){
   				return $_GET[$key];
   			}else{
   				if($die_if_not_match){ die('недопустимый формат $_GET[\''.$key.'\']'); }
   			}
   		}else{
   			if($die_if_not_set){ die('$_GET[\''.$key.'\'] не задан'); }		
   		}
   	}elseif($first == "post" or $first == "p"){
   		if($_POST[$key] != ''){
   			if(preg_match($pattern, $_POST[$key])){
   				return $_POST[$key];
   			}else{
   				if($die_if_not_match){ die('недопустимый формат $_POST[\''.$key.'\']'); }
   			}
   		}else{
   			if($die_if_not_set){ die('$_POST[\''.$key.'\'] не задан'); }		
   		}
   	}elseif($first){
   		die("Источник данных inchk() задан не верно");
   	}else{
   		die("Источник данных inchk() не задан");
   	}
   
   }

   OH, SH*T! Казалось бы, зачем?

   Gogogo, 08 Февраля 2011

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

9. C++ / Говнокод #5581

   +163

   1. 1

   throw new TSilentException("");

   Говногость, 08 Февраля 2011

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

10. JavaScript / Говнокод #5580

    +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

    var DatePicker = function () {
      ...
      return {
        init: function(options) {...},
        showPicker: function() {...},
        hidePicker: function() {...},
        setDate: function(date, shiftTo) {...},
        getDate: function(formated) {...},
        clear: function() {...},
        fixLayout: function() {...}
      };
    }();
    $.fn.extend({
      DatePicker: DatePicker.init,
      DatePickerHide: DatePicker.hidePicker,
      DatePickerShow: DatePicker.showPicker,
      DatePickerSetDate: DatePicker.setDate,
      DatePickerGetDate: DatePicker.getDate,
      DatePickerClear: DatePicker.clear,
      DatePickerLayout: DatePicker.fixLayout
    });

    Взято с http://www.eyecon.ro/datepicker/

    wmmorgun, 08 Февраля 2011

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

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

• 2107
• …
• 553
• 552
• 551
• 550
• 549
• 548
• 547
• 546
• 545
• …
• 1