1. Список говнокодов пользователя MrFranke

    Всего: 3

  2. JavaScript / Говнокод #23601

    0

    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
    if (!App.detection.browser_mobile && !App.detection.browser_tablet) {
  $('.hint', service_list_element)
  .on('mouseover', function() {

    var item = $(this),
    text = $('.text', item).text();

    if (text !== '' && !tooltip.visible) {
      tooltip.setTarget(item);

      tooltip.setData({
        'content' : text
      });

      tooltip.show();
    }
  })
  .on('mouseleave', function() {
    if (tooltip.visible) {
      tooltip.hide();
    }
  });
}
else {
  $('.hint', service_list_element)
  .on('pep_tap', function() {
    var item = $(this),
    text = $('.text', item).text();

    if (text !== '' && !tooltip.visible) {
      tooltip.setTarget(item);

      tooltip.setData({
        'content' : text
      });

      tooltip.show();
    }
  });
}

    Если это десктоп, то при клике на иконку всплывает подсказка, но если это мобильник или планшет, то копипастим код с той же логикой, только с другим событием, которое эмулирует клик.

    Кажется парню платили за количество строк в коде =/

    MrFranke, 19 Декабря 2017

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

  3. JavaScript / Говнокод #17337

    +153

    1. 1
    2. 2
    3. 3
    4. 4
    5. 5
    6. 6
    // В чем смысл переменной knc?
var knc = 0;
var k_fr = parent.document.getElementById(kph);
if (k_fr) {
    knc = 1;
}

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

    MrFranke, 18 Декабря 2014

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

  4. JavaScript / Говнокод #17238

    +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
    $('.price_input', this.getDummy()).on('keydown', function (e) {
    if (e.keyCode == 13) {
        e.preventDefault();
    }

    var charCode = e.keyCode, char = e.originalEvent.keyIdentifier;
    if ((((charCode == 188) && (!char || char == 'U+002C')) || (charCode == 190) && (!char || char == 'U+002E')) && ($(this).val().indexOf('.') < 0) && ($(this).val().indexOf(',') < 0)) {
        return true;
    }

    if ((charCode >= 37 && charCode <= 40) || (charCode == 46) || (charCode == 8) || (charCode >= 48 && charCode <= 57) || (charCode >= 96 && charCode <= 105)) {
        return true;
    }

    e.preventDefault();
});

    Код ведущего front-end разработчика.
    Это далеко не самый длинный if в проекте.
    После месячного рефакторинга подобного шедевра я готов открыть компанию по сбору денег для программистов инвалидов.

    MrFranke, 02 Декабря 2014

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