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

    +137

    1. 1
    2. 2
    private static OfficerResponse ToPerson<TField, TTitle>(IEnumerable<TField> fields, IEnumerable<TTitle> titles, Func<TField, string> getName, Func<TField, string> getValue, Func<TTitle, string> getTitle, bool active, TextType1 bio, YMDType birth, ContactInformationType contactInformation, string tickers, DegreeType1[] educationHistory, CertificateType[] certifications, PositionInformationType positionInformation, AffilationType[] corporateAffiliations, string officerId, OfficerDirectorType info,string repNo)
{...}

    нашел вот...

    Запостил: fade, 08 Августа 2013

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

    • ava someone 08.08.2013 10:09 # 0

      Автор ниасилил struct?
      Ответить

      • ava kegdan 08.08.2013 10:24 # +1

        Автор даже разбить код на строки не осилил

        Мать моя, это все сигнатура....
        Ответить

        • ava fade 08.08.2013 10:32 # 0

          сигнатура, сигнатура :)
          Ответить

          • ava kegdan 08.08.2013 10:54 # 0

            И что этот монстрометод делает?
            Ответить

            • ava fade 08.08.2013 11:08 # 0

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

              • ava NeoN 08.08.2013 14:30 # +1

                спагетти в студию! я требую продолжения банкета)
                Ответить

        • ava krypt 08.08.2013 10:33 # 0

          Может автоматический рефакторинг? Решарпер образца 2009го года при неудачном извлечении функции такие же чудеса генерил.
          Ответить

          • ava fade 08.08.2013 10:38 # 0

            нет, это не решарпер. дёргается эта хрень вот так:
            result.AddRange(from info in backendResponse.Officers.OfficersInformation
                where request.IncludeInactiveOfficers || info.Status != OfficerDirectorTypeStatus.Inactive
                select ToPerson(info.PersonInformation.Name.Info, info.PositionInformation.Titles,
                    item => item.Type, item => item.Value, title => title.LongTitle, info.Active
                    , info.BiographicalInformation.Text, info.PersonInformation.Name.Birth
                    , info.ContactInformation, request.Tickers, info.PersonInformation.EducationHistory
                    , info.PersonInformation.Certifications, info.PositionInformation
                    , info.CorporateAffiliations, info.ID, info, backendResponse.Officers.RepNo));
            Ответить

            • ava krypt 08.08.2013 11:11 # 0

              Хм. Так ничего не мешало выдрать в функцию выражение из select'а
              Ответить

            • ava Lokich 08.08.2013 11:37 # 0

              http://static.diary.ru/userdir/2/1/1/0/2110825/74738320.jpg
              Ответить

            • ava roman-kashitsyn 08.08.2013 11:58 # −1

              linq во всей красе
              Ответить

              • ava kegdan 08.08.2013 12:02 # +1

                тут не линк, тут архитектор во всей красе
                Ответить

                • ava anonimb84a2f6fd141 08.08.2013 21:19 # 0

                  А как надо правильно?
                  Ответить

                  • ava kegdan 08.08.2013 22:21 # +1

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

                    Предаются в основном поля Info. Проще было передать info и сокрыть всю эту мерзость.
                    Ответить

                  • ava neeedle 09.08.2013 12:45 # 0

                    Слишком большое кол-во аргументов. Если книгу Стива МакКонела читали, то там куча объяснений о том, почему так лучше не делать и почему такое решение называется низкокачественным.
                    Ответить

      • ava neeedle 08.08.2013 17:54 # 0

        Причем тут struct?
        Ответить

    • ava neeedle 08.08.2013 17:55 # 0

      Стива МакКонела на него нет!
      Ответить

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