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

    +114

    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
    public IQueryable<Log> Select(Context context)
        {
            // Return list of entities
            return (from l in context.Logs select l);
        }

        /// <summary>
        /// Fetch page from Log table
        /// </summary>
        /// <param name="nStartRowIndex">Starting index of rows to fetch</param>
        /// <param name="nMaxRows">Max number of rows</param>
        /// <returns>IEnumerable of Log</returns>
        public List<Log> SelectPage(int nStartRowIndex, int nMaxRows, 
                                    string strProperty, string strKeyword,
                                    string strSort, string strSortDirection, 
                                    out int nTotalCount)
        {            
            using (Context context = new Context())
            {
                var q = Select(context).Take (1000);
             }
      }

    Коллега на работе наворотил...

    Запостил: Katsy, 04 Февраля 2011

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

    • ava Lure Of Chaos 04.02.2011 11:47 # 0

      обьясните тупому где говнокод
      Ответить

    • ava Katsy 04.02.2011 20:49 # 0

      Ок, объясните тогда, зачем воротить столько кода, когда можно просто написать:
      context.Log.Take(1000)
      Ответить

      • ava koodeer 04.02.2011 21:06 # 0

        Мы от вас ждём объяснения, почему такой код.


        Но раз уж просите, вот возможные варианты:
        - параметры использовались, но после рефакторинга стали не нужны;
        - это временная заглушка, потом код добавится.
        Не?
        Ответить

        • ava Katsy 04.02.2011 21:09 # 0

          Нет, нет не в параметрах дело, а в том, чтобы нагородить функцию, когда можно обойтись одной строкой. Параметры надо было убрать, чтобы они не отвлекали.
          Ответить

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