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

    +103

    1. 1
    2. 2
    3. 3
    4. 4
    private ListBoxItem FindItem(string guid)
{
        return _userlist.Items.Cast<ListBoxItem>().FirstOrDefault(item => ((TextBlock) ((StackPanel) item.Content).Children[1]).Text == guid);
}

    Функция поиска элемента в ListBox по значению Guid в одну строку. Наговнокодил ReSharper.

    Запостил: botinko, 09 Мая 2010

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

    • ava Karasb 09.05.2010 21:03 # +4

      И где же здесь говнокод. ИМХО все понятно и красиво. А вот то что тут куча приведений типа, то это косяк автора, но никак не Resharper'а
      Ответить

    • ava cfdev 12.05.2010 22:14 # +2

      Два предположительно лишних каста - не повод сокрушаться
      Ответить

      • ava Dmitriy 31.05.2010 23:52 # 0

        Этот код явно из Silverlight. А там двойной каст обусловлен полиморфным типом. Код абсолютно верный. Можно, конечно, без кастов обойтись, а можно и исключение получить... тут как в анекдоте про Ржевского: можно и по морде, а можно и впиндюрить. Но если можно застраховаться, то почему это не сделать?
        Ответить

        • ava guest 01.06.2010 00:12 # 0

          > Этот код явно из Silverlight

          откуда видно?
          Ответить

          • ava Dmitriy 01.06.2010 11:41 # 0

            Может быть WPF. А видно из использования StackPanel.
            ((TextBlock) ((StackPanel) item.Content) - item приводится к StackPanel, его контент - к TextBlock. - все верно.
            Cast<ListBoxItem>() - используется, чтоб сделать выборку только из элементов ListBoxItem, так как в списке _userlist.Items могут содержаться не только ListBoxItem-сы.
            Если этого не сделать, то, может возникнуть исключение.
            Так что, никаких лишних кастов в коде нет, и решарпер сформировал абсолютно верную лямбда конструкцию. Если бы люди так писали, то исключения возникали бы реже.
            Ответить

    • ava botinko 01.06.2010 13:45 # 0

      Да, совершенно верно, это Silverlight
      Ответить

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