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

    В номинации:
    За время:
  2. Java / Говнокод #12798

    +83

    1. 01
    2. 02
    3. 03
    4. 04
    5. 05
    6. 06
    7. 07
    8. 08
    9. 09
    10. 10
    11. 11
    private static boolean getIsDeclinedByRules(boolean oldIsSatisfy, boolean oldIsDeclined, boolean newIsSatisfy) {
        if (!oldIsSatisfy &&  oldIsDeclined && !newIsSatisfy) return true;
        if (!oldIsSatisfy &&  oldIsDeclined &&  newIsSatisfy) return false;
        if (!oldIsSatisfy && !oldIsDeclined && !newIsSatisfy) return true;
        if (!oldIsSatisfy && !oldIsDeclined &&  newIsSatisfy) return false;
        if ( oldIsSatisfy &&  oldIsDeclined && !newIsSatisfy) return true;
        if ( oldIsSatisfy &&  oldIsDeclined &&  newIsSatisfy) return true;
        if ( oldIsSatisfy && !oldIsDeclined && !newIsSatisfy) return true;
        if ( oldIsSatisfy && !oldIsDeclined &&  newIsSatisfy) return false;
        return true;
      }

    glprizes, 25 Марта 2013

    Комментарии (16)
  3. PHP / Говнокод #12787

    +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
    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
    if (anidub::$exucution)
    		{
    			//получаем страницу для парсинга
    			$page = anidub::getContent($torrent_id, anidub::$sess_cookie);
    
    			if ( ! empty($page))
    			{
    				//ищем на странице дату регистрации торрента
    				if (preg_match("/<td width=\"\" class=\"heading\" valign=\"top\" align=\"right\">Добавлен<\/td><td valign=\"top\" align=\"left\">(.*)<\/td>/", $page, $array))
    				{
    					//проверяем удалось ли получить дату со страницы
    					if (isset($array[1]))
    					{
    						//если дата не равна ничему
    						if ( ! empty($array[1]))
    						{
    							//сбрасываем варнинг
    							Database::clearWarnings($tracker);
    							//приводим дату к общему виду
    							$date = $array[1];
    							$date_str = anidub::dateNumToString($array[1]);
    							//если даты не совпадают, перекачиваем торрент
    							if ($date != $timestamp)
    							{
                                    preg_match('/<a href=\"download\.php\?id=(\d{2,6})&name=(.*)\">/U', $page, $array);
                                    $torrent_id = $array[1];
                                    $torrent_id_name = $array[2];
    								//сохраняем торрент в файл
    								$torrent = anidub::getTorrent($torrent_id, $torrent_id_name, anidub::$sess_cookie);
    								$client = ClientAdapterFactory::getStorage('file');
    								$client->store($torrent, $id, $tracker, $name, $torrent_id, $timestamp);
    								//обновляем время регистрации торрента в базе
    								Database::setNewDate($id, $date);
    								//отправляем уведомлении о новом торренте
    								$message = $name.' обновлён.';
    								Notification::sendNotification('notification', $date_str, $tracker, $message);
    							}
    						}
    						else
    						{
    							//устанавливаем варнинг
    							if (anidub::$warning == NULL)
    							{
    								anidub::$warning = TRUE;
    								Errors::setWarnings($tracker, 'not_available');
    							}
    							//останавливаем процесс выполнения, т.к. не может работать без кук
    							anidub::$exucution = FALSE;
    						}
    					}
    					else
    					{
    						//устанавливаем варнинг
    						if (anidub::$warning == NULL)
    						{
    							anidub::$warning = TRUE;
    							Errors::setWarnings($tracker, 'not_available');
    						}
    						//останавливаем процесс выполнения, т.к. не может работать без кук
    						anidub::$exucution = FALSE;
    					}
    				}
    				else
    				{
    					//устанавливаем варнинг
    					if (anidub::$warning == NULL)
    					{
    						anidub::$warning = TRUE;
    						Errors::setWarnings($tracker, 'not_available');
    					}
    					//останавливаем процесс выполнения, т.к. не может работать без кук
    					anidub::$exucution = FALSE;
    				}
    			}			
    			else
    			{
    				//устанавливаем варнинг
    				if (anidub::$warning == NULL)
    				{
    					anidub::$warning = TRUE;
    					Errors::setWarnings($tracker, 'not_available');
    				}
    				//останавливаем процесс выполнения, т.к. не может работать без кук
    				anidub::$exucution = FALSE;
    			}
    		}

    Классическая лесенка пыхомакаки.

    Stallman, 23 Марта 2013

    Комментарии (16)
  4. PHP / Говнокод #12724

    +139

    1. 01
    2. 02
    3. 03
    4. 04
    5. 05
    6. 06
    7. 07
    8. 08
    9. 09
    10. 10
    11. 11
    <?php
    function string_size($string){
    $temporary_file = md5(rand().rand()).".temporary";
    $a=fopen($temporary_file,"w+");
    fwrite($a,$string);
    $size = filesize($temporary_file);
    fclose($a);
    unset($temporary_file);
    return $size;
    }
    ?>

    Аналог функции filesize() для строк.

    angrybird, 11 Марта 2013

    Комментарии (16)
  5. JavaScript / Говнокод #12673

    +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
    function validateForm(form){
    	if (isNotEmpty(form.fa)){
    		if (isColvo(form.fa,3)){
    			if (isNotEmpty(form.im)){
    				if (isColvo(form.im,2)){
    					if (isNotEmpty(form.ot)){
    						if (isColvo(form.ot,3)){
    							if(isNotEmpty(form.day)){
    								if(isNumbr(form.day)){
    									if(isNotEmpty(form.month)){
    										if(isNumbr(form.month)){
    											if(isNotEmpty(form.year)){
    												if (isColvo(form.year,4)){
    													if(isNumbr(form.year)){
    														if(isNotEmpty(form.city)){
    															if(isNotEmpty(form.road)){
    																if(isNotEmpty(form.house)){
    																	if(isNumbr(form.house)){
    																		return true;
    																	}
    																}
    															}
    														}
    													}
    												}
    											}
    										}
    									}
    								}
    							}
    						}
    					}
    				}
    			}
    		}
    	}
     return false;
    };

    http://национальныйзакон.рф

    lads, 01 Марта 2013

    Комментарии (16)
  6. Куча / Говнокод #12662

    +119

    1. 1
    Dummy commit for the sake of documentation (previous commit was made without a message)

    someone, 28 Февраля 2013

    Комментарии (16)
  7. 1C / Говнокод #12643

    −117

    1. 1
    2. 2
    3. 3
    4. 4
    5. 5
    6. 6
    7. 7
    8. 8
    9. 9
    Функция НайтиТовар(Наименование)
    	Запрос = Новый Запрос("ВЫБРАТЬ Ссылка, Наименование ИЗ Справочник.Номенклатура ГДЕ НЕ ПометкаУдаления");
    	Выборка = Запрос.Выполнить().Выбрать();
    	Пока Выборка.Следующий() Цикл
    		Если СокрЛП(ВРег(Выборка.Наименование))=ВРег(Наименование) Тогда
    			Возврат Выборка.Ссылка;
    		КонецЕсли;
    	КонецЦикла;
    КонецФункции

    и все это вызывается в цикле

    BlackMamba, 25 Февраля 2013

    Комментарии (16)
  8. JavaScript / Говнокод #12634

    +150

    1. 1
    <a href="javascript: document.location.href = document.location.href;" onclick="s_objectID=&quot;javascript: document.location.href = document.location.href;_1&quot;;return this.s_oc?this.s_oc(e):true">Нажмите здесь, если сканирование не закончилось.</a>

    document.location.href = document.location.href

    TRANE73, 22 Февраля 2013

    Комментарии (16)
  9. PHP / Говнокод #12568

    +88

    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
    //До рефакторинга
    static function initConnectOnce(){
        $who = get_called_class();
        if(!$GLOBALS[$who][class_cllct]){
            $db = new DataBase();
            $GLOBALS[$who][class_cllct] = $db->selectCollection(get_called_class());
        }
        return $GLOBALS[$who][class_cllct];
    }
    //После
    public static function initCollectOnce(){
        $who = get_called_class();  
        $cllct = &self::$cacheCollect[$who];
        if(!$cllct){
            $db = new DataBase();
            $cllct = $db->selectCollection($who);
        }
        return $cllct;
    }

    Недавно начал рефакторить свой же код. Нашел такой вот незаметный гавнокод...
    Для тех кто в танке =)
    1. selectCollection(get_called_class()) - зачем вызывать по второму разу, если результат уже есть в переменной $who.
    2. $GLOBALS[$who][class_cllct] - немного глюкнуло наверное, когда писал... Логичнее так $GLOBALS[class_cllct][$who].
    3. initConnectOnce - тут даже наверное не Connect должно быть, а Collect.
    4. Вместо $GLOBALS[$who][class_cllct] лучше(имхо) заюзать статичное свойство для класса.
    5. static function initConnectOnce - забыл public описать....

    haker, 11 Февраля 2013

    Комментарии (16)
  10. Си / Говнокод #12360

    +111

    1. 1
    2. 2
    for(p=first; p!=NULL; p=p->next)
      free(p);

    Освобождаем память всех элементов списка.

    taburetka, 26 Декабря 2012

    Комментарии (16)
  11. C++ / Говнокод #12263

    +18

    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
    /*
     *      The use of singletons for globals makes globals not
     *      actually be initialized until it is first needed, this
     *      makes the library faster to load, and have a smaller
     *      memory footprint
     */
    
    #define json_global_decl(TYPE, NAME, VALUE)                              \
    class jsonSingleton ## NAME {                                            \
    public:                                                                  \
            inline static TYPE & getValue() json_nothrow {                   \
                    static jsonSingleton ## NAME single;                     \
                    return single.val;                                       \
            }                                                                \
    protected:                                                               \
            inline jsonSingleton ## NAME() json_nothrow : val(VALUE) {}      \
            TYPE val;                                                        \
    }
    
    #define json_global(NAME) jsonSingleton ## NAME::getValue()              \
    
    json_global_decl(json_string, CONST_TRUE, JSON_TEXT("true"));
    json_global_decl(json_string, CONST_FALSE, JSON_TEXT("false"));
    json_global_decl(json_string, CONST_NULL, JSON_TEXT("null"));
    
    /* Использование */
    json_global(ERROR_NULL_IN_CHILDREN)

    Наткнулся на утечку памяти, отловленную Valgrind'ом
    А внутри вот это. И главное, совершенно непонятно, зачем воротить эту свистопляску, если линкеры умеют распознавать "true" по всюду (читаем про ROMability тут: http://www.open-std.org/jtc1/sc22/wg21/docs/papers/2002/n1396.pdf )
    http://stackoverflow.com/questions/690176/c-c-optimization-of-pointers-to-string-constants

    Сам код отсюда: http://code.google.com/p/wot-replay-parser/source/browse/libjson/Source/JSONGlobals.h

    myaut, 10 Декабря 2012

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