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

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

    +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
    <?php
    //----------------------------Сначала оригинал который был запосчен--------------------------------
    $mas = array("odin", "dva", "tri");
    
    function obm($mas[$i]){
        if $mas[$i] = "odin" $mas[$i] = 1;
            return $mas[$i];
        if $mas[$i] = "dva" $mas[$i] = 2;
            return $mas[$i];    
        if $mas[$i] = "tri" $mas[$i] = 3;
            return $mas[$i];
    }
     
      $n = 0;
      $new = array();
        for ($i = 0; $i < 100; $i++) {
          $n++;
            $i = $n * $mas[$i];
        }
    //----------------------------- Конец оригинала ---------------------------------------------------------------
    
    
    //--------------- Потом мне всбрело в голову прокаментить каждую строчку, вот что с этого получилось
    $mas = array("odin", "dva", "tri");//Инициализация массива.
    
    //Объявление функции.
    function obm($mas[$i]){//Входящий параметр, массив с перемменной? Даже пхп такое не понимает. Ну бум считать что это просто переменная типа $mas_i
        if $mas[$i] = "odin" $mas[$i] = 1;//Сравнение входящей переменной со строкой "odin", зачем было инциализировать массив, если совпадает, то мы массиву который раньше инициализировали присваеваем 1. Но так как это функция, то видимости массива тут нет, то есть основной не иземеняется. Просто создается внутренний и ему присвается значение. При этом у нас нету значения $i. По умолчанию будет $mas[0]. Также сама конструкция if сама по себе не правильная.
            return $mas[$i];//Возвращение значения из массива, присвоенного ранее. Можно было бы в предедущей строчку сделать ретурн. При этом эта строчка будет выполнятся всегда. Так как если бы if был бы правильный, то он сработал бы только на первую операцию. И дальнейший код никогда бы не выполнялся.
        if $mas[$i] = "dva" $mas[$i] = 2; //Ну тут аналогично первым двум строкам.
            return $mas[$i];    
        if $mas[$i] = "tri" $mas[$i] = 3;
            return $mas[$i];
    }
    //Функция то написана, но блин она же нигде не вызывается!
     
      $n = 0;//Зачем-то инициализируем счетчик.
      $new = array();//Инициализация нового массива.
        for ($i = 0; $i < 100; $i++) {//Стар цикла от 0 до 99 с шагом 1 в переменную $i. Единственная правильная сктрока с осмысленным кодом во всем скрипте.
          $n++;//К счетчику сразу пуляем +1, то есть он у нас является дубликатом переменной $i + 1
            $i = $n * $mas[$i];//Тут я долго думал :-) Допустим первый проход в цикле. $i=0; $n=1; $mas[$i]='odin'. Имеем комбинацию $i = 1 * "odin"; Пхп пытается перевести "odin" в цисловую переменную, получает 1. Итого мы имеем 1 * 1 =1;  Дальше лучше, следующий цикл $i=1; $n=2; $mas[$i]="dva"; $i = 2 * "dva"; То есть получаем в $i значение 2. В следующем цикле аналогично. $i=2;$n=3;$mas[$i]='tri';$i=3*'tri'=3; А вот следующая итерация уже достойна высшей математики :-) $i=3;$n=4; $mas[$i] то есть индекс 3, блин а нет таког, то есть пустая строка, и мы имеем далее $i=4 * "". Пхп интрепретирует пустую как ноль и в переменной $i получаем 0. Что сбрасывает цикл в самое начала. И если бы не растущий счетчик $n, то выхода из цикла бы вообще не было. На следующем шаге получается примерно такое $i=1;$n=5;$mas[$i]='dva';$i=5*"dva"=5. Следующая $i будет 6, потом 0, потом 8, потом 0 и т.д.
        }
        //И в конце концов мы ничего не получаем, так как нигде не использовали массив $new;
    //-------------------------------------Конец каментов-----------------------------------------
    
    
    //-------------Ну и чисто для стеба, решение задачи от меня в три строки---------
    $Str = 'odin';
    
    $mas = array("odin", "dva", "tri");
    $num = array_search($Str, $mas)+1;
    $new = range(0, $num*99, $num);
    
    echo "new='".var_export($new, true)."'\n";
    //-------------------------------------DIE() :-)---------------------------------------------------

    Нашел на форуме.
    Задача. На входе есть строка с содержанием odin, dva или tri. На выходе должен был получится массив со ста числами с шагом указанным в строке, то есть для строки tri, должен быть результат 0, 3, 6, 9, 12 ......294,297
    И чел спрашивает, почему у меня код не работает :-) И как его пофиксить.

    mrsol, 11 Марта 2015

    Комментарии (4)
  3. JavaScript / Говнокод #17707

    +158

    1. 1
    2. 2
    3. 3
    4. 4
    5. 5
    6. 6
    7. 7
    8. 8
    9. 9
    <p><!-- Need to do some UI cleanup --><br>
                    <script type="text/javascript">// <![CDATA[
                        document.getElementById("badge-images").style.visibility = "hidden";
                        document.getElementById("about-contact").style.visibility = "hidden";
                        document.getElementById("header-menu").style.visibility = "hidden";
                        var hideContactDetails = true;
                        // ]]&gt;</script>
                    <br>
                </p>

    Недавно один Java-pазработчик не смог разобраться, как пофиксеть баг в WordPress и решил переписать наш корпоративный сайт на Java мотивируя это тем, что будет намного легче поддерживать и развивать новую ситсему. Что из этого получилось видно на наглядном примере наложение стилей на элементы страницы(верстка).

    etual, 27 Февраля 2015

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

    −119

    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
    Процедура ДействияФормыДействие5(Кнопка)
    	
    	ТекущиеДанные = ЭлементыФормы.СправочникСписок.ТекущиеДанные;
    	Если ТекущиеДанные = Неопределено Тогда 
    		Возврат;
    	КонецЕсли;
    	
    	ТекПометкаУдаления = ТекущиеДанные.Ссылка.ПометкаУдаления;
    	
    	КонтрагентОбъект = ТекущиеДанные.Ссылка.ПолучитьОбъект();
    	КонтрагентОбъект.ПометкаУдаления = Не ТекПометкаУдаления;
    	Попытка
    		КонтрагентОбъект.записать();
    	Исключение 
    		Сообщить("Не удалось " + ?(ТекПометкаУдаления, "снять", "установить") + " пометку удаления.");
    	КонецПопытки;	
    	
    КонецПроцедуры

    На форме списка справочника "Контрагенты" добавлена кнопка со значком "непосредственное удаление". А это её обработчик.
    НАХРЕНА???

    doom2good, 20 Февраля 2015

    Комментарии (4)
  5. Куча / Говнокод #17653

    +127

    1. 1
    http://i.gyazo.com/dd1b407b6ea528e59de2966e49d4cb82.png

    Юзабилити)))

    motobuster, 16 Февраля 2015

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

    +158

    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
    //Получим все пробеги за все периоды
    		$ps_res = array();
    		foreach ($crews as $v)
    		{
    			foreach ($v['ps'] as $ps)
    			{
    				/**
    				 * Отправим горку запросов в виалон на получение километража.
    				 * Можноо, конечно сделать по людски и генерировать массив и
    				 * получить это всё одним запросом, но мне лень,
    				 * буду лучше как мудак грузить систему виалона.
    				 */
    				$g = $api->call('gps/get_c_report',
    						array('car_id'	 => $v['WI'],
    					'begin_time' => $ps['begin'],
    					'end_time'	 => $ps['end']));
    				$cnt = (float) str_replace(' km', '', $g[1]->reportResult->stats[0][1]);
    				if ($cnt > 2)
    				{
    					/**
    					 * Если проехали больше 2 км, то упаковать данные на
    					 * выставление пиздюлей
    					 */
    					$ps_res[$v['ID']][] = array('id' => $v['ID'], 'cnt' => $cnt, 'ps' => $ps);
    				}
    			}
    			if (!empty($ps_res))
    			{
    
    				/**
    				 * Выполним операции на каждой итерации, ибо виалон - зло.
    				 *  отваливается при большом количестве запросов.
    				 * Нет, я не настолько мудак в любом случае у них ограничение на кол-во
    				 *  операций варьирующееся от количества нагрузки на сервер.
    				 */
    				set_op($ps_res);
    				unset($ps_res[$v['ID']]);
    			}
    		}

    Комменты доставили, да.

    Rijen, 11 Февраля 2015

    Комментарии (4)
  7. SQL / Говнокод #17623

    −165

    1. 1
    2. 2
    3. 3
    SELECT `score` FROM `score`
    WHERE `score` = (SELECT DISTINCT `score` FROM `score`
      ORDER BY `score` DESC LIMIT 1,1)

    Про именование таблиц и полей...

    smart, 11 Февраля 2015

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

    +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
    !function(e){"object"==typeof exports?module.exports=e():"function"==typeof define&&define.amd?define(e):"undefined"!=typeof window?window.JSZipUtils=e():"undefined"!=typeof global?global.JSZipUtils=e():"undefined"!=typeof self&&(self.JSZipUtils=e())}(function(){var define,module,exports;return (function e(t,n,r){function s(o,u){if(!n[o]){if(!t[o]){var a=typeof require=="function"&&require;if(!u&&a)return a(o,!0);if(i)return i(o,!0);throw new Error("Cannot find module '"+o+"'")}var f=n[o]={exports:{}};t[o][0].call(f.exports,function(e){var n=t[o][1][e];return s(n?n:e)},f,f.exports,e,t,n,r)}return n[o].exports}var i=typeof require=="function"&&require;for(var o=0;o<r.length;o++)s(r[o]);return s})({1:[function(require,module,exports){
    'use strict';
    
    var JSZipUtils = {};
    
    ...
    
    module.exports = JSZipUtils;
    
    // enforcing Stuk's coding style
    // vim: set shiftwidth=4 softtabstop=4:
    
    },{}]},{},[1])
    (1)
    });
    ;

    пох, что не читается без бьютифаера, зато модульность

    makc3d, 06 Февраля 2015

    Комментарии (4)
  9. C++ / Говнокод #17567

    +48

    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
    void Sender::exceptionHandler()
    {
    	try
    	{
    		throw;
    	}
    	catch (std::exception& ex)
    	{
    		Log::write(ex);
    	}
    	catch (...)
    	{
    		Log::write("I am CraZzY!");

    laMer007, 03 Февраля 2015

    Комментарии (4)
  10. Ruby / Говнокод #17552

    −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
    def append_card(cards, new_card)
          if cards.nil?
            new_card == GameEvent::YELLOW_CARD ? 'yellow' : 'red'
          elsif cards == 'yellow'
            new_card == GameEvent::YELLOW_CARD ? 'two_yellow' : 'yellow_red'
          elsif cards == 'red'
            new_card == GameEvent::YELLOW_CARD ? 'yellow_red' : 'red'
          elsif cards == 'two_yellow'
            'two_yellow'
          elsif cards == 'yellow_red'
            new_card == GameEvent::YELLOW_CARD ? 'two_yellow' : 'yellow_red'
          end 
        end

    ничего особенного, просто подсчитываем, какую карточку получил игрок футбольной команды.
    а я думал, что что-то понимаю в футболе... я никогда в жизни так не ошибался

    iHateBidlocod, 31 Января 2015

    Комментарии (4)
  11. 1C / Говнокод #17551

    −120

    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
    //определяем сумму начисленной амортизации+ремонтов на заказы документом "РаспределениеАмортизацииНаЗаказы"
    
    	Запрос = Новый Запрос;
    	Запрос.Текст = "ВЫБРАТЬ
    	               |	НезавершенноеПроизводствоБухгалтерскийУчетОбороты.СчетУчета КАК СчетУчета,
    	               |	НезавершенноеПроизводствоБухгалтерскийУчетОбороты.Заказ,
    	               |	НезавершенноеПроизводствоБухгалтерскийУчетОбороты.СтоимостьПриход КАК СтоимостьПриход,
    	               |	НАЧАЛОПЕРИОДА(НезавершенноеПроизводствоБухгалтерскийУчетОбороты.Период, МЕСЯЦ) КАК Период,
    	               |	НезавершенноеПроизводствоБухгалтерскийУчетОбороты.Регистратор КАК Регистратор
    	               |ИЗ
    	               |	РегистрНакопления.НезавершенноеПроизводствоБухгалтерскийУчет.Обороты(&ДатаНач, &ДатаКон, Регистратор, ) КАК НезавершенноеПроизводствоБухгалтерскийУчетОбороты
    	               |ГДЕ
    	               |	(НезавершенноеПроизводствоБухгалтерскийУчетОбороты.Регистратор ССЫЛКА Документ.РаспределениеАмортизацииНаЗаказы
    	               |			ИЛИ НезавершенноеПроизводствоБухгалтерскийУчетОбороты.Регистратор = &Корректировка)
    	               |	И НезавершенноеПроизводствоБухгалтерскийУчетОбороты.СчетУчета = &СчетУчета
    	               |ИТОГИ
    	               |	СУММА(СтоимостьПриход)
    	               |ПО
    	               |	ОБЩИЕ";
    				   
    	Запрос.УстановитьПараметр("ДатаНач",НачалоМесяца(ДатаНач));
    	Запрос.УстановитьПараметр("ДатаКон",КонецМесяца(ДатаКон));
    	Запрос.УстановитьПараметр("СчетУчета", ПланыСчетов.Хозрасчетный.НайтиПоКоду("231"));
    	Запрос.УстановитьПараметр("Корректировка", Документы.ОперацияБух.НайтиПоНомеру("00000056",Дата(2011,05,01)));
    
    	Выборка = Запрос.Выполнить().Выбрать();
    	Выборка.Следующий();
    	СуммаНачисления = Выборка.СтоимостьПриход;

    Есть специальная супер секретная бухгалтерская операция которую нужно исключить )))

    yuriybilous, 31 Января 2015

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