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

    +157

    1. 1
    2. 2
    3. 3
    if (strlen($arResult["ERROR_MESSAGE"]) <= 0){
// любая ахинея
}

    2014 год. Битрикс все так же терпеливо ожидает патент на строки отрицательной длины.
    Полагаю, такой икспрешн используется не только в компоненте корзины, учитывая страсть авторов к копипасте.

    velosipedistorg, 30 Мая 2014

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

  2. PHP / Говнокод #16086

    +154

    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
    if ($_SERVER["REQUEST_METHOD"] == "POST" && strlen($_REQUEST["backButton"]) > 0 && ($arParams["DELIVERY_NO_SESSION"] == "N" || check_bitrix_sessid()))
{
	if($arResult["POST"]["CurrentStep"] == 6 && $arResult["SKIP_FORTH_STEP"] == "Y")
		$arResult["CurrentStepTmp"] = 3;

	if($arResult["POST"]["CurrentStepTmp"] <= 5 && $arResult["SKIP_THIRD_STEP"] == "Y")
		$arResult["CurrentStepTmp"] = 2;

	if($arResult["POST"]["CurrentStepTmp"] <= 3 && $arResult["SKIP_SECOND_STEP"] == "Y")
		$arResult["CurrentStepTmp"] = 1;

	if(IntVal($arResult["CurrentStepTmp"])>0)
		$arResult["CurrentStep"] = $arResult["CurrentStepTmp"];
	else
		$arResult["CurrentStep"] = $arResult["CurrentStep"] - 2;
	$arResult["BACK"] = "Y";
}
if ($arResult["CurrentStep"] <= 0)
	$arResult["CurrentStep"] = 1;
$arResult["ERROR_MESSAGE"] = "";

    Компонент страницы заказа в магазине Битрикса, 14я версия.
    Тут вам и отсутствие констант для осмысленных шагов, и отсутствие конфигурирования этих шагов, как они i18n-ятся, вообще не представляю.
    КАК ЭТО ВСЁ РАБОТАЕТ??!!!

    ЗЫ мало того, тут же в компонент вкопипащен (или переговнокоден заново, не сравнивал) процесс залогинивания/регистрации нового юзера, елси он не залогинен. Не смог перекопипастить эту копипасту лишь потому, что она содержит 310 строк. Неслабый DRY.

    velosipedistorg, 30 Мая 2014

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

  3. Java / Говнокод #16085

    +72

    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
    public class PLock {
    private Map<Thread, Integer> readLocks = new HashMap<Thread, Integer>();
    private Thread writeLock = null;
    private int writeLockCount = 0;
    
    public synchronized void getReadLock() {
        Thread currentThread = Thread.currentThread();
        long startTimeMillis = System.currentTimeMillis();
        boolean gotStuck = false;
        while (canClaimReadLock(currentThread) == false) {
            gotStuck = true;
            try {
                wait();
            } catch (InterruptedException ex) {
                Log.warn("Interrupted while attempting to get read lock.", ex);
            }
        }
        report(gotStuck, startTimeMillis, "read");
        if (readLocks.containsKey(currentThread)) {
            readLocks.put(currentThread, 1 + readLocks.get(currentThread));
        } else {
            readLocks.put(currentThread, 1);
        }
    }
    
    ...
    
    public synchronized void relinquishReadLock() {
        Thread currentThread = Thread.currentThread();
        if (readLocks.containsKey(currentThread) == false) {
            throw new RuntimeException("Cannot relinquish read lock on thread " + currentThread + " because it does not hold a lock.");
        }
        int newLockCount = readLocks.get(currentThread) - 1;
        if (newLockCount == 0) {
            readLocks.remove(currentThread);
            notifyAll();  // IMPORTANT: allow other threads to wake up and check if they can get locks now.
        } else {
            readLocks.put(currentThread, newLockCount);
        }
    }
    
    public synchronized void getWriteLock() {
        //Log.warn("getWriteLock() in thread " + Thread.currentThread());
        //dumpLocks();
        Thread currentThread = Thread.currentThread();
        long startTimeMillis = System.currentTimeMillis();
        boolean gotStuck = false;
        while (canClaimWriteLock(currentThread) == false) {
            gotStuck = true;
            try {
                wait();
            } catch (InterruptedException ex) {
                Log.warn("Interrupted while attempting to get write lock.", ex);
            }
        }
        report(gotStuck, startTimeMillis, "write");
        writeLock = currentThread;
        writeLockCount++;
    }
    
    ...
    
    public synchronized void relinquishWriteLock() {
        //Log.warn("relinquishWriteLock() in thread " + Thread.currentThread());
        Thread currentThread = Thread.currentThread();
        if (writeLock != currentThread) {
            throw new RuntimeException("Cannot relinquish write lock on thread " + currentThread + " because it does not hold the lock.");
        }
        if (writeLockCount <= 0) {
            throw new RuntimeException("Tried to relinquish write lock on thread " + currentThread + " while write lock count is " + writeLockCount);
        }
        writeLockCount--;
        if (writeLockCount == 0) {
            writeLock = null;
            notifyAll();  // IMPORTANT: allow other threads to wake up and check if they can get locks now.
        }
    }
    
    ...
}

    А у вас уже есть свой теплый ламповый ReentrantReadWriteLock?
    Нет? Тогда https://github.com/orph/jujitsu/blob/master/src/e/ptextarea/PLock.java идет к вам...

    kostoprav, 30 Мая 2014

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

  4. Куча / Говнокод #16084

    +131

    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
    Шесть месяцев назад я нашёл в Интернете книжку "Dive into Python". Странная книжка, толстая,
да ещё на не русском языке.
Усердно прочитал её, методично и вдумываясь, но таки ничего и не понял.
Но что-то не отпускало меня - я пошёл на python.org и стал изучать стандартную библиотеку,
старательно, скурпулёзно, от корки до корки. Гугля примеры кода, я наткнулся на пост в блоге одного гуру,
где было написано про PEP8.
То, что рассказывал этот умудрённый опытом кодер, настолько впечатлило меня, что я словно в припадке
безумия бросился зубрить эту конвенцию по написанию питоньего кода.
У меня тряслись руки, по телу прокатывали волны возбуждения, я, можно сказать, бился в экстазе всё
это время - и теперь могу процитировать наизусть любой пункт, даже если меня разбудить посреди 
глубокой ночи.
Новые знания окрыляли меня, я бросился писать (что бы вы подумали?) очередную имиджборду.
Я не писал раньше имиджборд, эта была моей первой. По пути пришлось освоить азы вёрстки на html и css,
но там всё оказалось совсем не сложно.
Я плавно двигался вперёд, наращивая функционал. И знаете что? Я упёрся в недостаток
производительности. Ну, так мне показалось. Я делал замеры, устранял места с тяжёлым кодом.
Тысячи раз запускал ab, но так и не сумел перейти порог в 300 запросов в секунду.
Я как-то ожидал большего и был немного разочарован.
Гугл, снова гугл. Десятки статей и тем на форуме… Довольно быстро я заметил, что люди часто пишут про
какие-то "асинхронные веб-сервера". Часто встречались названия Tornado и Gevent. Я прочитал про них
подробнее - и был просто ошеломлён. Как мне это раньше не приходило в голову?!
Это же, это просто гениально, чёрт меня побери!
Исходники в ведро, всё переписать! Меня переполняло новое знание, поток мыслей ровно ложился в
строчки кода. Я просто не мог остановиться. Чашка кофе… Ещё чашка…
Мой небольшой кусочек софта приобретал кристалльную чистоту,
я смертельно устал, но продолжал в умилении полировать его зудящими руками, нанося последние штрихи
Пять тысяч. Я получил производительность в пять тысяч запросов в секунду. В ту ночь я так и не смог заснуть.
Наиувлекательнейший мир хай-лоада открылся передо мной. Позже я изучил ещё много классных вещей
типа сверхбыстрых асинхронных key-value-хранилищ, или, например, такого необычного подхода к
обработке данных, как map-reduce.
Это уже был другой я, новый, обновлённый. С незашоренным взглядом, с жаждой новых знаний и
незнакомых концепций.
Я летел вперёд, оставляя за спиной ступень за ступенью.
За пол-года я сменил несколько мест работы, и моя зарплата взлетела до умопомрачительных сумм.
Сотни замечательных мест открыли передо мной свои двери. Любые, самые изысканные девушки
проявляли просто чудеса изобретательности, чтобы находиться рядом со мной.
А ещё, я никогда не забуду тот момент, когда на одной из конференций я впервые в жизни поймал на
себе завистливый взгляд.
Это был взгляд какого-то сливающегося с толпой неудачника, взгляд преисполненный ненависти,
презрения и желчного вожделения, направленного на мой Олимп. Наверное, он всю свою никчёмную
жизнь писал на каком-нибудь си-шарпе или там на джаве, проводя долгие дни в одиночестве.
И теперь начинал подсознательно догадываться, что долгие годы безуспешно пытался
построить замок из навоза.

    IKing, 30 Мая 2014

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

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

    +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
    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
    92. 92
    93. 93
    94. 94
    95. 95
    96. 96
    97. 97
    98. 98
    // weclome to main-menu-hover.js
function load(){
	var url = window.location;
	var m1 = document.getElementById("m1");
	var m2 = document.getElementById("m2");
	var m3 = document.getElementById("m3");
	var m4 = document.getElementById("m4");
	//var m5 = document.getElementById("m5");
	var mm1 = document.getElementById("mm1");
	var mm2 = document.getElementById("mm2");
	var mm3 = document.getElementById("mm3");
	var mm4 = document.getElementById("mm4");
	var mm5 = document.getElementById("mm5");
	var mm6 = document.getElementById("mm6");
	var mm7 = document.getElementById("mm7");
var mm8 = document.getElementById("mm8");
	if (m1.href == url){
	m1.style.color = "#ffc12b";
	//m1.style.background = "url(%absolute_url_path%/wp-content/themes/%pattern%/images/menu-h-bg1.png)no-repeat top center";
	m1.style.textDecoration = "none";
	}
	if (m2.href == url){
	m2.style.color = "#ffc12b";
	m2.style.background = "url(%absolute_url_path%/wp-content/themes/%pattern%/images/menu-h-bg1.png)no-repeat center center";
	m2.style.textDecoration = "none";
	}
	if (m3.href == url){
	m3.style.color = "#ffc12b";
	m3.style.background = "url(%absolute_url_path%/wp-content/themes/%pattern%/images/menu-h-bg1.png)no-repeat center center";
	m3.style.textDecoration = "none";
	}
	if (m4.href == url){
	m4.style.color = "#ffc12b";
	m4.style.background = "url(%absolute_url_path%/wp-content/themes/%pattern%/images/menu-h-bg1.png)no-repeat center center";
	m4.style.textDecoration = "none";
	}
	if (m5.href == url || url.href.indexOf(m5.href)!=-1){
	m5.style.color = "#ffc12b";
	m5.style.background = "url(%absolute_url_path%/wp-content/themes/%pattern%/images/menu-h-bg2.png)no-repeat center center";
	m5.style.textDecoration = "none";
	}
	if (url.href.indexOf(mm1.href)!=-1){
	mm1.style.color = "#741f0c";
	mm1.style.background = "url(%absolute_url_path%/wp-content/themes/%pattern%/images/bg-min-menu-h.png)no-repeat center center";
	m3.style.color = "#ffc12b";
	m3.style.background = "url(%absolute_url_path%/wp-content/themes/%pattern%/images/menu-h-bg1.png)no-repeat center center";
	m3.style.textDecoration = "none";
	}
	if (url.href.indexOf(mm2.href)!=-1){
	mm2.style.color = "#741f0c";
	mm2.style.background = "url(%absolute_url_path%/wp-content/themes/%pattern%/images/bg-min-menu-h.png)no-repeat center center";
	m3.style.color = "#ffc12b";
	m3.style.background = "url(%absolute_url_path%/wp-content/themes/%pattern%/images/menu-h-bg1.png)no-repeat center center";
	m3.style.textDecoration = "none";
	}
	if (url.href.indexOf(mm3.href)!=-1){
	mm3.style.color = "#741f0c";
	mm3.style.background = "url(%absolute_url_path%/wp-content/themes/%pattern%/images/bg-min-menu-h.png)no-repeat center center";
	m3.style.color = "#ffc12b";
	m3.style.background = "url(%absolute_url_path%/wp-content/themes/%pattern%/images/menu-h-bg1.png)no-repeat center center";
	m3.style.textDecoration = "none";
	}
	if (url.href.indexOf(mm4.href)!=-1){
	mm4.style.color = "#741f0c";
	mm4.style.background = "url(%absolute_url_path%/wp-content/themes/%pattern%/images/bg-min-menu-h.png)no-repeat center center";
	m3.style.color = "#ffc12b";
	m3.style.background = "url(%absolute_url_path%/wp-content/themes/%pattern%/images/menu-h-bg1.png)no-repeat center center";
	m3.style.textDecoration = "none";
	}
	if (url.href.indexOf(mm5.href)!=-1){
	mm5.style.color = "#741f0c";
	mm5.style.background = "url(%absolute_url_path%/wp-content/themes/%pattern%/images/bg-min-menu-h.png)no-repeat center center";
	m3.style.color = "#ffc12b";
	m3.style.background = "url(%absolute_url_path%/wp-content/themes/%pattern%/images/menu-h-bg1.png)no-repeat center center";
	m3.style.textDecoration = "none";
	}
	if (url.href.indexOf(mm6.href)!=-1){
	mm6.style.color = "#741f0c";
	mm6.style.background = "url(%absolute_url_path%/wp-content/themes/%pattern%/images/bg-min-menu-h.png)no-repeat center center";
	m3.style.color = "#ffc12b";
	m3.style.background = "url(%absolute_url_path%/wp-content/themes/%pattern%/images/menu-h-bg1.png)no-repeat center center";
	m3.style.textDecoration = "none";
	}
	if (url.href.indexOf(mm7.href)!=-1){
	mm7.style.color = "#741f0c";
	mm7.style.background = "url(%absolute_url_path%/wp-content/themes/%pattern%/images/bg-min-menu-h.png)no-repeat center center";
	m3.style.color = "#ffc12b";
	m3.style.background = "url(%absolute_url_path%/wp-content/themes/%pattern%/images/menu-h-bg1.png)no-repeat center center";
	m3.style.textDecoration = "none";
	}
    if (url.href.indexOf(mm8.href)!=-1){
	mm8.style.color = "#741f0c";
	mm8.style.background = "url(%absolute_url_path%/wp-content/themes/%pattern%/images/bg-min-menu-h.png)no-repeat center center";
	m3.style.color = "#ffc12b";
	m3.style.background = "url(%absolute_url_path%/wp-content/themes/%pattern%/images/menu-h-bg1.png)no-repeat center center";
	m3.style.textDecoration = "none";
	}
}

    Пришел проект на поддержку... Сразу отказался=)
    Естественно вкрапления %_some_text_% от меня.
    Файл назывался main-menu-hover.js

    iHateBidlocod, 30 Мая 2014

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

  6. JavaScript / Говнокод #16082

    +161

    1. 1
    2. 2
    3. 3
    4. 4
    5. 5
    6. 6
    7. 7
    var percent = 0;
setInterval(function() {
    if(percent < 100) {
        percent += 10;
        showprogress(percent);
    }
}, 50);

    Прогресс-бар асинхронной загрузки картинки

    kissarat, 30 Мая 2014

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

  7. C++ / Говнокод #16081

    +5

    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
    #include <iostream>
using namespace std;
struct ko{
	int r;
} f;
const ko& taras=f;
int main() {
	cout<<
	is_same<decltype(taras.r), int>::value
	<<" "<<
	is_same<decltype((taras.r)), const int &>::value
	<<endl;
	return 0;
}

    What is output you expect? You should answer without looking at ideone link and without using compiler.
    http://ideone.com/5O9vtZ

    LispGovno, 30 Мая 2014

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

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

    +6

    1. 1
    2. 2
    3. 3
    4. 4
    5. 5
    const int cx = 5;
int main() {
	auto lam = [cx]() mutable {cx=40;};
	return 0;
}

    Will it compile or not? Why?
    http://ideone.com/gtlXKb

    LispGovno, 30 Мая 2014

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

  9. Куча / Говнокод #16078

    +142

    1. 01
    2. 02
    3. 03
    4. 04
    5. 05
    6. 06
    7. 07
    8. 08
    9. 09
    10. 10
    11. 11
    Начните зарабатывать деньги с Bitcoin прямо сейчас!

Как заработать на биткойнах? Очень просто!

1. Зарегистрируйте кошелёк Bitcoin: https://blockchain.info/ru/wallet
2. После регистрации введите свой Bitcoin-адрес здесь: http://eb.by/freebitcoins
3. Нажмите кнопку «ROLL!», чтобы получить бесплатные биткойны
4. Спустя час вы можете получить ещё, поэтому рекомендую периодически возвращаться на этот сайт.
5. Как только вы наберёте такую сумму, которую вы хотели бы вывести, зарегистрируйтесь на бирже btc-e.com, и получите рубли, доллары или евро.

Или оставьте деньги в биткойнах, и наблюдайте, как вы становитесь всё богаче за счёт роста стоимости Bitcoin!

    ifkelvofep34lf, 29 Мая 2014

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

  10. C++ / Говнокод #16077

    +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
    #define BOOST_INTRUSIVE_INTERNAL_STATIC_BOOL_IS_TRUE(TRAITS_PREFIX, TYPEDEF_TO_FIND) \
template <class T>\
struct TRAITS_PREFIX##_bool\
{\
   template<bool Add>\
   struct two_or_three {one _[2 + Add];};\
   template <class U> static one test(...);\
   template <class U> static two_or_three<U::TYPEDEF_TO_FIND> test (int);\
   static const std::size_t value = sizeof(test<T>(0));\
};\
\
template <class T>\
struct TRAITS_PREFIX##_bool_is_true\
{\
   static const bool value = TRAITS_PREFIX##_bool<T>::value > sizeof(one)*2;\
};\
//

    Ну тут как бы все просто и понятно. Но зачем заканчивают макрос кодом

    \
    
//
    ?

    laMer007, 29 Мая 2014

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