Си диез / Говнокод #16205 Ссылка на оригинал

0

  1. 1
  2. 2
  3. 3
  4. 4
  5. 5
  6. 6
  7. 7
  8. 8
  9. 9
  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
return string.Format(templateStr, "",
                _p1_Tb.Text,
                _p2_Tb1.Text + ", " + _p2_Tb2.Text,
                _p3_Tb.Text,
                "",//_p4_Tb.Text,
                _p5_Tb1.Text + " / " + _p5_Tb2.Text,
                _p6_Tb.Text,
               RblTxt(_p7_Rbl) + " " + _p7_Tb.Text,
                b("Построен в ") + _p8_Tb1.Text + " году, в " + _p8_Tb2.Text + " году произведен "
                + _p8_Tb3.Text + " ремонт. " + _p8_Tb4.Text + ", Количество корпусов " + _p8_Tb4.Text + ", Количество этажей в каждом корпусе " + _p8_Tb5.Text
                + (_p8_Ch1.Checked ? (", " + _p8_Ch1.Text) : "")
                + (_p8_Ch2.Checked ? (", " + _p8_Ch2.Text) : "")
                + (_p8_Ch3.Checked ? (", " + _p8_Ch3.Text) : "") ,
                FillTbl9() ,
                b("Уровень средства размещения: ") + _p10_Tb.Text,
                b("Объект находится рядом с ") + _p11_Tb.Text,
                "принимаются  с " + _p12_Tb1.Text + " лет, " + _p12_Tb2.Text,
                _p13_Tb.Text,
                b("Период работы: ") + (_p14_Rb1.Checked ? ("С " + _p14_Tb1.Text + " По " + _p14_Tb2.Text) : "Круглогодично"),
                "к " + _p15_Tb1.Text + " до " + _p15_Tb2.Text,
                _p16_Tb.Text
                + (_p16_Ch1.Checked ? ("<br/> От аэропорта " + _p16_Tb1.Text + " автобусом № " + _p16_Tb2.Text + " до остановки" + _p16_Tb3.Text) : "")
                + (_p16_Ch2.Checked ? ("<br/> От жд вокзала " + _p16_Tb4.Text + " автобусом № " + _p16_Tb5.Text + " до остановки" + _p16_Tb6.Text) : "")
                + (_p16_Ch3.Checked ? ("<br/> Другое " + _p16_Tb7.Text) : "")
                ,
                b("Типы номеров: ") + _p17_Tb.Text,
                b("В номере: ")
                 ((_p23_Ch1.Checked ? (" " + _p23_Ch1.Text + ",") : "")
                + (_p23_Ch2.Checked ? (" " + _p23_Ch2.Text + ",") : "")
                + (_p23_Ch3.Checked ? (" " + _p23_Ch3.Text + ",") : "")
                + (_p23_Ch4.Checked ? (" " + _p23_Ch4.Text + ",") : "")
                + (_p23_Ch5.Checked ? (" " + _p23_Ch5.Text + ",") : "")
                + (_p23_Ch6.Checked ? (" " + _p23_Ch6.Text + ",") : "")
                + (_p23_Ch7.Checked ? (" " + _p23_Ch7.Text + ",") : "")
                + (_p23_Ch8.Checked ? (" " + _p23_Ch8.Text + ",") : "")
                + (_p23_Ch9.Checked ? (" " + _p23_Ch9.Text + ",") : "")
                + (_p23_Ch10.Checked ? (" " + _p23_Ch10.Text + ",") : "")
                + (_p23_Ch11.Checked ? (" " + _p23_Ch11.Text + ",") : "")
                + (_p23_Ch12.Checked ? (" " + _p23_Ch12.Text + ",") : "")
                + (_p23_Ch13.Checked ? (" " + _p23_Ch13.Text + ",") : "")
                + (_p23_Ch14.Checked ? (" " + _p23_Ch14.Text + ",") : "")
                + (_p23_Ch15.Checked ? (" " + _p23_Ch15.Text + ",") : "")
                + (_p23_Ch16.Checked ? (" " + _p23_Ch16.Text + ",") : "")
                + (_p23_Ch17.Checked ? (" " + _p23_Ch17.Text + ",") : "")
                + (_p23_Ch18.Checked ? (" " + _p23_Ch18.Text + ",") : "")
                + (_p23_Ch19.Checked ? (" " + _p23_Ch19.Text + ",") : "")
                + (_p23_Ch20.Checked ? (" " + _p23_Ch20.Text + ",") : "")
                + (_p23_Ch21.Checked ? (" " + _p23_Ch21.Text + ",") : "")
                + (_p23_Ch22.Checked ? (" " + _p23_Ch22.Text + ",") : "")
                + (_p23_Ch23.Checked ? (" " + _p23_Ch23.Text + ",") : "")
                + (_p23_Ch24.Checked ? (" " + _p23_Ch24.Text + ",") : "")
                + (_p23_Ch25.Checked ? (" " + _p23_Ch25.Text + ",") : "")
                + (_p23_Ch26.Checked ? (" " + _p23_Ch26.Text + ",") : "")
                + (_p23_Ch27.Checked ? (" " + _p23_Ch27.Text + ",") : "")
                + (_p23_Ch28.Checked ? (" " + _p23_Ch28.Text + ",") : "")
                + (_p23_Ch29.Checked ? (" " + _p23_Ch29.Text + ",") : "")
                + (_p23_Ch20.Checked ? (" " + _p23_Ch30.Text + ",") : "")
                + (_p23_Ch31.Checked ? (" " + _p23_Ch31.Text + ",") : "")
                + (_p23_Ch32.Checked ? (" " + _p23_Ch32.Text + ",") : "")
                + (_p23_Ch33.Checked ? (" " + _p23_Tb.Text) : "")).TrimEnd(new char[] { ' ', ',' }),
                ((_p24_Ch1.Checked ? (" " + _p24_Ch1.Text + ",") : "")
                + (_p24_Ch2.Checked ? (" " + _p24_Ch2.Text + ",") : "")
                + (_p24_Ch3.Checked ? (" " + _p24_Tb.Text) : "")).TrimEnd(new char[] { ' ', ',' }),
                FillTbl25(),
                RblTxt(_p26_Rbl1) + " " + _p26_Tb.Text + ". " + RblTxt(_p26_Rbl2),
                ((_p27_Ch1.Checked ? (" " + _p27_Ch1.Text + ",") : "")
                + (_p27_Ch2.Checked ? (" " + _p27_Ch2.Text + ",") : "")
                + (_p27_Ch3.Checked ? (" " + _p27_Ch3.Text + ",") : "")
                + (_p27_Ch4.Checked ? (" " + _p27_Ch4.Text + ",") : "")
                + (_p27_Ch5.Checked ? (" " + _p27_Ch5.Text + ",") : "")
                + (_p27_Ch6.Checked ? (" " + _p27_Ch6.Text + ",") : "")
                + (_p27_Ch7.Checked ? (" " + _p27_Tb.Text) : "")).TrimEnd(new char[] { ' ', ',' }),
                _p28_Tb.Text,
                b("Минимальная продолжительность заезда ") + _p29_Tb.Text + " дней",
                "от " + _p30_Tb1.Text + " до " + _p30_Tb2.Text + " лет " + _p30_Tb3.Text,
                ((_p31_Ch1.Checked ? (" " + _p31_Ch1.Text + " " + _p31_Tb1.Text) : "")
                + "<br/>" + (_p31_Ch2.Checked ? (" " + _p31_Ch2.Text + " " + _p31_Tb2.Text) : "")
                + "<br/>" + (_p31_Ch3.Checked ? (" " + _p31_Ch3.Text + " " + _p31_Tb3.Text) : "")

Бэкенды крупнейшего российского туроператора. На отдел разработки уходит 1500000 рб/мес.

Запостил: tablecell tablecell, (Updated )

Комментарии (30) RSS

  • Ну пусть теперь заводят отдел рефакторинга.
    Ответить
  • Код выглядет, как будто html-верстальщику дали в руки c#
    Ответить
  • >российского туроператора.
    >1500000 рб/мес.
    Древняя Российская традиция пилить любую сумму у которой больше 2 нулей.
    Ответить
    • 1,5 ляма - это с налогами, арендой, обслуживанием и т.д.?
      тогда это далеко не самый большой отдельчик
      Ответить
        • тогда сотрудники видят только 150 000. На 10-15 человек. Остальное уходят на эффективный менеджмент
          p.s. Аффтар кода - норкоман. Мне это что-то напоминает. Такое знакомое и неуловимое, не могу понять что.
          Ответить
            • ну в принципе да. Что-то вроде однострочника, типа патча Брамина. НЁХ в чистом виде.
              Ответить
              • Они просто не освоили циклы или поклейку прямо в скуеле.
                Ответить
          • >Остальное уходят на эффективный менеджмент
            Во-во, от туда и растут корни.

            Видимо задача ставилась так:
            - Сделай форму с 2мя инпутами, которые будут форматироваться в строку с неким шаблоном.
            - Сделано
            - А теперь добавь туда ещё два инпута...
            Ответить
            • мало! у меня начинается с одного инпута, а потом еще с десяток контроллов.
              но поражает то, что изначально и я, и начальство знают о том, что быстро сделать простой интерфейс - можно. докрутить туда еще десяток контроллов - можно. еще десяток - можно.
              поддерживать тот фарш, и избежать ошибок - очень сложно.
              избежать этого вначале, дав больше времени, и сразу делая все то, что нужно - можно.
              и сука все это знают, и все понимают, но все-равно "сделай два инпута, а потом добавь еще два..."
              Ответить
              • Я тут столкнулся с новым направлением развития бизнеса: "Continuous alteration"
                Делается примерно так:
                - Нужна супермегаформа, которая выглядит так
                ... идёт работа в течении n-ч.часов
                - Сделано
                - О, здорово. Меня тут посетила одна мысь, давай её всю переделаем сяк...

                Ну и процесс переделки стремится в бесконечность...
                >и сука все это знают, и все понимают, но все-равно "сделай два инпута, а потом добавь еще два..."
                Если уже есть некий авторитет в коллективе, то начинай продавливать удобный для комманды вариант работы.
                Можешь посоветовать некие продукты для проектирования интерфейсов, типа NinjaMock.
                Ответить
      • да сколько бы там не было, за этот фарш я бы больше 100 рублей не платил бы
        Ответить
  • > На отдел разработки уходит 1500000 рб/мес.
    Интересно, какой процент от этой суммы достаётся, кхм, отделу разработки?
    Ответить
  • А я еще переживаю, что я говно пишу. Ну нахуй.
    Ответить

Добавить комментарий

Я, guest, находясь в здравом уме и твердой памяти, торжественно заявляю:

    А не использовать ли нам bbcode?


    8