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

    +145

    1. 1
    2. 2
    3. 3
    4. 4
    public partial class TaskViewSearchParameter
{
    public int Top5 { get; set; }
}

    Часть класса одного из наших проектов

    Запостил: ZlobnyiSerg, 25 Ноября 2010

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

    • ava banderror 25.11.2010 21:05 # 0

      может, поясните, что тут такого особенного?
      Ответить

      • ava burdakovd 25.11.2010 21:45 # 0

        Не соответствие имени и типа: int Top5

        Логично было бы ожидать в переменной с именем Top5 массив/список из пяти каких-то элементов.

        Но #4705 всё равно круче))
        Ответить

        • ava ZlobnyiSerg 25.11.2010 22:06 # +1

          На самом деле в переменной указывалось количество элементов, которые нужно выбрать (превращалось в TOP N в SQL-запросе).
          Или еще логично было бы ожидать тип Bool, тогда было бы понятно, что переменная накладывает ограничение и определяет выбирать ли только первые 5 записей, или все.
          А когда я встретил такую конструкцию пришлось крепко задуматься =) Мне показалось забавным, вот и поделился
          Ответить

          • ava banderror 26.11.2010 09:18 # +2

            Ну ясно. Было бы еще "забавнее", если бы свойство сопровождалось дезориентирующим комментом типа
            // представляет семь последних параметров
            Ответить

    • ava Lure Of Chaos 25.11.2010 23:11 # +1

      говнокод в общем то должен быть в поле топика....
      Ответить

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