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

    +96

    1. 1
    2. 2
    3. 3
    4. 4
    OdbcCommand dist_cmd = ifx_conn.CreateCommand();
                dist_cmd.CommandText = string.Format(@"select  -1 as districtid, '-=ВСІ=-' as districtname from systables where  tabname = 'systables'                               union all 
                                        select id as districtid , cpz_name as districtname from  post_cpz where directorate_id = {0} and id != 0 ORDER BY 2", (String.IsNullOrEmpty(DDL_obl.SelectedValue))?"-1":DDL_obl.SelectedValue);
                reader = dist_cmd.ExecuteReader();

    в процессе рефакторинга, убирая NHibernate код был заменен на это

    Запостил: bercerker, 17 Мая 2012

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

    • ava someone 17.05.2012 06:58 # 0

      > убирая NHibernate

      ЗАЧЕМ?

      В чём профит от замены ORM на голый SQL? Пусть автор тогда и с C# на ассемблер пересаживается.
      Ответить

      • ava bercerker 17.05.2012 12:17 # 0

        Автор говорит, что NHibernate очень тяжелый для понимания, а запросы можно контролировать
        Ответить

        • ava roman-kashitsyn 17.05.2012 12:21 # +3

          массивы слишком сложны для понимания, давайте использовать переменные с цифрой на конце, их можно контролировать
          циклы сложны для понимания, давайте использовать ^c ^v, их можно контролировать
          Ответить

          • ava 3.14159265 17.05.2012 16:43 # 0

            Возможно речь шла о том, что NHibernate генерирует говнозапросы, а чем заставить его поумнеть, то проще написать запрос руками.
            Но вообще не вижу смысла в объединении системной таблицы сервера и post_cpz, то есть это стопудов какаша:
            >from systables where tabname = 'systables'
            Ответить

        • ava someone 17.05.2012 12:34 # +2

          То, что автор ниасилил NHibernate - это проблемы автора, а никак не NHibernate.
          Ответить

    • ava DBdev 17.05.2012 10:46 # +1

      from systables where tabname = 'systables'
      зачем ЭТО?
      *негодуэ*
      Ответить

      • ava bercerker 17.05.2012 12:42 # 0

        это для красоты :-)
        Ответить

      • ava bercerker 17.05.2012 12:53 # 0

        а в общем то
        from systables where tabname = 'systables' - здесь очень глубокий смысл, это для того, чтобы добавить первую строчку в asp: DropDownList, типа "выберите ..."
        Ответить

        • ava DBdev 17.05.2012 14:16 # 0

          "select -1 as districtid, '-=ВСІ=-' as districtname
          этого достаточно.
          Зачем там конструкция FROM да еще и с WHERE?
          Ответить

    • ava guest 18.05.2012 09:36 # −1

      Отака штука "select -1 as districtid, '-=ВСІ=-' as districtname union all ..." в Sql Server працює, а в Informix кидає креш. Через те лисий рішив зробити так.
      Ответить

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