1. C# / Говнокод #13466

    +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
    29. 29
    30. 30
    31. 31
    32. 32
    33. 33
    34. 34
    35. 35
    36. 36
    37. 37
    if (IsSetPrinter)
                        {
                            if (Counter == 0)
                            {
                                printingIsOkay = PrintDocument(letterIDsString
                               , row.DocumentTemplateVersionID
                               , row.DocumentTemplate
                               , (row.IsHeader_DocumentTemplateVersionIDNull() ? (int?)null : row.Header_DocumentTemplateVersionID)
                               , (row.IsFooter_DocumentTemplateVersionIDNull() ? (int?)null : row.Footer_DocumentTemplateVersionID)
                               , true
                               , Counter
                               , Counter == documentTemplateVersions.Count - 1);
                            }
                            else
                            {
                                printingIsOkay = PrintDocument(letterIDsString
                              , row.DocumentTemplateVersionID
                              , row.DocumentTemplate
                              , (row.IsHeader_DocumentTemplateVersionIDNull() ? (int?)null : row.Header_DocumentTemplateVersionID)
                              , (row.IsFooter_DocumentTemplateVersionIDNull() ? (int?)null : row.Footer_DocumentTemplateVersionID)
                              , false
                              , Counter
                              , Counter == documentTemplateVersions.Count - 1);
                            }

                        }
                        else
                        {
                            printingIsOkay = PrintDocument(letterIDsString
                               , row.DocumentTemplateVersionID
                               , row.DocumentTemplate
                               , (row.IsHeader_DocumentTemplateVersionIDNull() ? (int?)null : row.Header_DocumentTemplateVersionID)
                               , (row.IsFooter_DocumentTemplateVersionIDNull() ? (int?)null : row.Footer_DocumentTemplateVersionID)
                               , false
                               , Counter
                               , Counter == documentTemplateVersions.Count - 1);
                        }

    Я бы выложил весь класс. Он весь такой. Но, думаю, вы поняли кто писал?

    Запостил: kore_sar, 23 Июля 2013

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

    • ava eth0 23.07.2013 06:31 # +1

      Человек с построчной оплатой?
      Ответить

      • ava 3Doomer 24.07.2013 11:37 # 0

        Если он пишет удобочитаемый код(несмотря на причины его нечитаемости в классическом виде) - сразу построчная оплата? Я не согласен.
        Впрочем, всё хорошо в меру...бывают и совсем хреновые случаи http://vk.com/wall7468765_12845
        Ответить

      • ava Elvenfighter 24.07.2013 23:38 # 0

        К слову, а она существует вообще? Есть ссылки на вакансии?
        Ответить

        • ava bormand 25.07.2013 05:33 # +1

          Видимо нет. Или все вакансии уже заняты лентяями с кодогенератором.
          Ответить

        • ava eth0 25.07.2013 19:24 # 0

          Надо же как-то оценивать эффективность программиста. В закрытых багах не очень информативно.
          Ответить

        • ava kegdan 26.07.2013 06:30 # 0

          Существуешь - нанимаешься дураку софт писать - он даже будет ходить вокруг. заглядывать через плечо и спрашивать "Хули так мало строк" А потом скажет, что хочет, что бы из экрана вылетали единороги и какали пиццей. Но обычно работа с такими людьми заканчивается раньше срока - либо он тебе плешь проест, либо "ты не такой хороший программист, как я думал"
          Ответить

    • ava guest 23.07.2013 07:04 # +1

      Такой код, вероятнее всего, рождается в компаниях с корпоративной культурой программирования. Ну, чтоб сосед и потомок могли всё понять :)
      Ответить

      • ava kore_sar 23.07.2013 07:08 # 0

        Наверное в это сложно поверить, но эти программисты тупо не знают другого способа достигнуть задуманное. Этот софт был написано в небольшой конторе несколькими людьми. Я тут работаю с недавна. :(
        Ответить

        • ava bormand 23.07.2013 07:42 # 0

          Кстати, а чем отличаются эти 3 блока? Я навскидку не вижу отличий...

          P.S. А, нашел, в шестом параметре стоит IsSetPrinter && Counter == 0.
          Ответить

          • ava kore_sar 23.07.2013 07:43 # 0

            Поздравляю. Вы только что нашли корень зла!
            Ответить

            • ava bormand 23.07.2013 07:44 # 0

              Да не, корень зла походу в том, что автор не знал булевых операций ;) Или не знал, что их можно юзать не только в if/while.
              Ответить

              • ava guest 23.07.2013 09:58 # 0

                Тернарки то есть, должен знать
                Ответить

                • ava blackhearted 23.07.2013 10:40 # −4

                  Уходи, крестоблять!
                  Ответить

                • ava bormand 23.07.2013 11:04 # +5

                  > Тернарки то есть
                  IsSetPrinter ? Counter == 0 ? true : false : false
                  Ответить

                  • ava inkanus-gray 24.07.2013 19:00 # 0

                    Сразу видно, что не пэхапэшник. Пэхапэшник наставил бы скобочек, чтобы подстраховаться.
                    Ответить

                    • ava bormand 24.07.2013 19:10 # 0

                      И правильно бы сделал. Потому что в PHP тернарник перекошенный. Но здесь тема о C#, и во всех языках кроме PHP так прокатит.

                      P.S. Кстати этот код должен правильно отработать даже в PHP. Там же конструкции вот такого вида фейлятся:
                      $a ? "a" : $b ? "b" : "other"
                      Ответить

              • ava kegdan 26.07.2013 06:34 # 0

                Самое интересное, что ?: он все таки использует. Может его шайтан попутал.
                Ответить

      • ava 1024-- 25.07.2013 01:51 # 0

        Где-то мне довелось прочитать про питониста, который писал по аргументу на строчке, чтобы с свном было легче работать. Изменения сигнатуры и добавление новых аргументов не портили уже созданные строки с аргументами. Уж не упомню, на ГК, или где ещё было.
        Ответить

        • ava wvxvw 25.07.2013 23:34 # +1

          Ему бы по-японски писать, там вообще все можно вертикально: сабвершн бы следил за каждой буквой - разионализация!
          Ответить

    • ava wvxvw 25.07.2013 18:30 # 0

      Oh it's not fair, but the printing is ok,
      The version is ok,
      The header is ok,
      Oh you're supposed to care, but all you do is take,
      All you do is take
      Ответить

    • ava wvxvw 25.07.2013 22:07 # 0

      Кстати, а разве нужно кастовать null в int? ?
      Ответить

      • ava bormand 25.07.2013 22:46 # 0

        Нет, без каста вполне прокатывает.
        Ответить

      • ava kegdan 26.07.2013 06:40 # 0

        В принципе не нужно int? - это и есть обвертка для инта, способная принимать значение null. А вообще это извращение использовать такие обвертки - лучше value-object самому сделать
        Ответить

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