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

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

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

1. PHP (5706)
2. C++ (2739)
3. Куча (2360)
4. JavaScript (2030)
5. C# (1928)
6. Java (1502)
7. Си (1115)
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. PHP / Говнокод #8271

   +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
   82. 82
   83. 83
   84. 84
   85. 85
   86. 86
   87. 87
   88. 88
   89. 89
   90. 90
   91. 91

   $searchCondition1 = ''; 
    $searchCondition2 = ''; 
    $searchCondition3 = ''; 
    $flag = false; 
    $sql = 'SELECT f.flight_id,
                  (SELECT city
                   FROM airports
                   WHERE airport_id = f.from) as _from,
                  (SELECT city
                   FROM airports
                   WHERE airport_id = f.to) as _to,
                   f.flight_date as fdate,
                   f.flight_time,
                   f.distance,
                   p.plain_name as plain
           FROM flight f JOIN planes p
               ON f.plain_id = p.plain_id
           WHERE '; 
    $fields = 0; 
    if($_POST['txtSearch']){ 
        if($_POST['txtFrom'] != ''){ 
            $searchCondition1 = " city = '".$_POST['txtFrom']."'"; 
            $flag = true; // set flag to TRUE
            $fields++; 
        }
        if($_POST['txtTo'] != ''){ // if To field is not empty
            $searchCondition2 = " city = '".$_POST['txtTo']."'"; 
            $flag = true;  // set flag to TRUE
            $fields++; 
        }
        if ($_POST['txtDate'] != ''){ // if Date field is not empty
            $searchCondition3 = " DATE(f.flight_date) = '".$_POST['txtDate']."'";
            $fields++; // and inc 
        }
        if($fields == 0){                             
            echo "<br/><div style=\"color:red\">Please, enter at least one filed to search</div>"; 
        }else{ 
            if($searchCondition1 != '' && $searchCondition2 == ''){ 
            $sql .= " f.from IN (SELECT airport_id
                                       FROM airports
                                       WHERE ".$searchCondition1.")"; 
            }
            if($searchCondition1 == '' && $searchCondition2 != ''){ 
                $sql .= " f.to IN (SELECT airport_id
                                           FROM airports
                                           WHERE ".$searchCondition2.")"; 
            }                                                              
            if($searchCondition1 != '' && $searchCondition2 != ''){ 
                $sql .= " f.from IN (SELECT airport_id
                                           FROM airports
                                           WHERE ".$searchCondition1.")
                                AND f.to IN (SELECT airport_id
                                           FROM airports
                                           WHERE ".$searchCondition2.")"; 
            }
            if($searchCondition3 != ''){ 
                if($flag){ 
                    $sql .= " AND ".$searchCondition3; 
                }else{
                    $sql .= $searchCondition3; 
                }
   
            }
            include_once('database.php'); 
            $db = createPDO();
            $result = $db->query($sql);
            echo '<table>
                 <tr>
                   <th>
                       From
                   </th>
                   <th>
                       To
                   </th>
                   <th>
                       Flight Date
                   </th>
                   <th>
                       Flight Time
                   </th>
                   <th>
                       Distance
                   </th>
                   <th>
                       Plane
                   </th>
                   <th>
                       Action
                   </th>
               </tr>'; // create an HTML table with headers
            while($row = $result->fetch()){...}

   Ищем в табличке в зависимости от заполненных полей в форме

   denis90, 22 Октября 2011

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

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

   +162

   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

   template <
   	typename PropertyOwner // Класс владельца
   >
   class properties {
   public:
   	// Получить указатель на владельца по указателю на свойство
   	static PropertyOwner * owner( void * property ) { 
   		int aai = (int)&(((PropertyOwner*)0)->__properties);
   		return (PropertyOwner *)((char*)property - aai);
   	}
   };
    
   /**
    * Шаблон класса свойства
    */
   template <
   	typename PropertyOwner, // Класс владельца
   	typename PropertyType, // Тип свойства
   	PropertyType (PropertyOwner::*getter)(), // Геттер
   	void (PropertyOwner::*setter)(PropertyType) > // Сеттер
   class property {
   public:
    
   	/**
   	 * Чтение свойства - вызов геттера
   	 */
   	operator PropertyType() {
   		return (properties<PropertyOwner>::owner( this )->*getter)();
   	}
    
   	/**
   	 * Запись в свойство - вызов сеттера
   	 */
   	void operator = ( const PropertyType & value ) {
   		(properties<PropertyOwner>::owner( this )->*setter)( value );
   	}
   };
    
   // Макросы для удобного определения свойств /////////
    
   /**
    * Начать объявления свойств в классе cls
    */
   #define properties_start(cls) union { properties<cls> __properties;
    
   /**
    * Закончить объявление свойств в классе cls
    */
   #define properties_end() };
    
   /**
    * Объявить свойство в классе cls типа type c геттером getter и сеттером setter
    */
   #define property(cls,type,getter,setter) property<cls,type,&cls::getter,&cls::setter>

   http://www.codenet.ru/progr/cpp/cpp-properties.php

   AnimeGovno-_-, 22 Октября 2011

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

3. Ruby / Говнокод #8269

   −99

   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

   # Work fine only on call with has_many association
   def self.new_from_params options
     new_rule = Rule.new
     return nil if new_rule.project_id.nil?
   
     rule = Rule.find_by_id options[:rule_id]
     if rule.nil? or (new_rule.project.id != rule.project.id)
       rule = Rule.new
     end
   
     new_rule.rule_id = options[:rule_id]
     new_rule.users = (rule.users.to_a + options[:users].to_a).uniq
     new_rule.statuses = (rule.statuses.to_a + options[:statuses].to_a).uniq
     new_rule.tags = (rule.tags.to_a + options[:tags].to_a).uniq
     new_rule.grouped_by = options[:grouped_by].presence || rule.grouped_by.presence
     new_rule.sorted_by = options[:sorted_by].presence || rule.sorted_by.presence
     new_rule.search = options[:search].presence || rule.search.presence
     new_rule.period_start = options[:period_start].presence || rule.period_start.presence
     new_rule.period_end = options[:period_end].presence || rule.period_end.presence
   
     new_rule
    end

   emerald, 21 Октября 2011

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

4. ActionScript / Говнокод #8268

   −111

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

   public function cloneString(str:String):String {
   			if(str == null) return null;
   			_byteArray.position = 0;
   			_byteArray.writeUTF(str);
   			_byteArray.position = 0;
   			var res:String =  _byteArray.readUTF();
   			// fuck yeah!
   			return res;
   		}

   суровые сибирские программисты клонируют строки like a pro

   makc3d, 21 Октября 2011

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

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

   +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

   ...
   
     public function getLocations($fresh = 0)
       {
           $tld =& $this->varGet('tld');
           $domain =& $this->varGet('domain');
           
           // if website is viewed via IP
           $noWWWButIP =& $this->varGet('noWWWButIP');
   
           $memCacheKey = 'getLocations';
   
           if (!$fresh) {
               // already fetched and stored in vars?
               if (is_array($this->varGet('arrLocations')))
                   $arrLocations = $this->varGet('arrLocations');
               // lets try fetching from memcache
               else
                   $arrLocations = kd()->lib('kdCache')->get($memCacheKey);
           }
           // regenerate
           unset($arrLocations);
   
           if (!is_array($arrLocations)) {
   ...

   Кэшируем))

   kovel, 21 Октября 2011

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

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

   +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

   /*
            * Для операций UPDATE идентификаторы объектов должны быть уникальными.
            */
           HashMap<Long,Boolean> filterUniqueChangeObjects = new HashMap<Long, Boolean>();
           for (RepositoryObject repositoryObject : resultRepositiry) {
   			if(repositoryObject.getOperationType()  == OperationType.UPDATE) {
   				if(filterUniqueChangeObjects.containsKey(repositoryObject.getObjectId())) {
   					log.warn("Объект с идентификатором уже присутствует в обработке UPDATE " + repositoryObject.getObjectId());
   				} else {
   					resultUniqueRepositiry.add(repositoryObject);
   					filterUniqueChangeObjects.put(repositoryObject.getObjectId(), true);
   				}
   			}
   		}

   al_open, 21 Октября 2011

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

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

   +157

   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

   $(function () {
       $("#tree_cat").jstree({ 
         
         
       
         "html_data" : {
           "data" : "<ul><li id='cat_41'><a href='/cat/41'>Продажа автомобилей, спецтехники, вело, мототехники, водной техники</a><ul><li id='cat_72'><a href='/cat/72'>Автомобили легковые  новые</a></li></ul><ul><li id='cat_73'><a href='/cat/73'>Автомобили легковые  подержанные</a></li>
   ....и т.д. ....
   d='cat_146'><a href='/cat/146'>Автокредитование, лизинг</a><ul><li id='cat_147'><a href='/cat/147'>Автокредитование</a></li></ul><ul><li id='cat_148'><a href='/cat/148'>Лизинг автомобилей, техники</a></li></ul><ul><li id='cat_203'><a href='/cat/203'>Автоломбарды</a></li></ul></li></ul><ul><li id='cat_199'><a href='/cat/199'>Автострахование</a><ul><li id='cat_200'><a href='/cat/200'>Автострахование</a></li></ul></li></ul>"
         },
   
   ......
   
   if((node.attr("class") !== "jstree-open") && (node.attr("class") !== "jstree-closed") && (node.attr("class") !== "jstree-last jstree-closed")&& (node.attr("class") !== "jstree-last jstree-open")){
           window.location = this+"#cat";
         }

   Ещё один эллитный питон программист....

   DrFreez, 21 Октября 2011

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

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

   +152

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

   if (asdfadfsdf != 1) {
   
   // всякий код
   
   define ('asdfadfsdf',1);
   }

   Типа проверка чтобы "всякий код" исполнялся только 1 раз...

   Tairesh, 21 Октября 2011

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

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

   +118

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

   if (mainWareHouseId.HasValue && mainWareHouseId.Value.ToString() == this._locationList.SelectedValue)
   {
       return false;
   }
   else
   {
       return location == null ? true : !location.RegionalFulfillment;
   }

   abatishchev, 21 Октября 2011

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

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

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

    echo "<center><br><br><form method=\"POST\" action=\"\">
      <select name=\"Data\">";
    $query="SELECT dtDate FROM Energo ORDER BY dtDate Desc"; 
    $res=mssql_query($query) or die("Ошибка запроса!"); 
    while($row = mssql_fetch_array($res))
    {
    echo "
      <option>".$row['dtDate']."</option>";
    }
    echo "</select>
      <input value=\"OK\" type=\"submit\">
    </form><center>";
    
    if(!empty($Data))
    	{
    	echo "<br><br><center><span style=\"font-weight: bold;\">Справка</span><br style=\"font-weight: bold;\">
            <span style=\"font-weight: bold;\">по ..... по состоянию на $Data г.<br><br>";
    		require_once ("tablebuh.lpd");
    		echo "<center><br><br>
    Начальник %Отдел% <img style=\"width: 50%;\"
     alt=\"\" src=\"podpis.gif\" align=\"middle\">
    К***в Н.А.<br><br>
    </center>"; 
    	}
    	
    else
    	{
    	echo "<form method=\"POST\" action=\"\">";
    	echo "<br><br><center><span style=\"font-weight: bold;\">Справка</span><br style=\"font-weight: bold;\">
            <span style=\"font-weight: bold;\">по .... по состоянию на ";
    	$query="SELECT dtDate FROM Energo WHERE id='1'"; 
    	$res=mssql_query($query) or die("Ошибка запроса!"); 
    	while($row = mssql_fetch_array($res)) 
    		{ 
    			$Data=$row['dtDate'];
    			echo $row['dtDate']."<br><br>";
    			require_once ("tablebuh.lpd");
    			echo "</center>"; 
    		}
    	echo "<center><br>
    Начальник %Отдел% <img style=\"width: 50%;\"
     alt=\"\" src=\"podpis.gif\" align=\"middle\">
    К***в Н.А.<br><br>
    ";
    
    //А вот и сам tablebuh.php
    
    <?php
    if(empty($Data))
    {
    require_once ("t2.php");
    }
    else 
    {
    require_once ("t1.php");
    }

    Хах, первый вопрос который задал себе, не пьян ли я был тогда=)
    Самое забавное, в течении 4 месяцев директор не имел никаких претензий по поводу одной и той же информации ежедневно=)

    LazyPiG, 21 Октября 2011

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

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

• 2105
• …
• 814
• 813
• 812
• 811
• 810
• 809
• 808
• 807
• 806
• …
• 1