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

    +138

    1. 1
    2. 2
    3. 3
    4. 4
    5. 5
    Control authorizationPanel= iddbd
      .Controls.Cast<Control>().Where(x => x.Name == "panelMaxim").FirstOrDefault()
      .Controls.Cast<Control>().Where(x => x.Name == "panelInternetB").FirstOrDefault()
      .Controls.Cast<Control>().Where(x => x.Name == "panelPassiveIB").FirstOrDefault()
      .Controls.Cast<Control>().Where(x => x.Name == "panelPhone").FirstOrDefault();

    не, ну а как его еще достать? У нас это квест такой на работе - "Найди контрол" называется

    Запостил: taburetka, 18 Декабря 2013

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

    • ava anonimb84a2f6fd141 18.12.2013 13:56 # −4

      .AssParallel?
      Ответить

    • ava roman-kashitsyn 18.12.2013 14:00 # +4

      #Query
      Ответить

    • ava krypt 18.12.2013 17:08 # +9

      > iddbd
      Даже iddqd написать правильно не смог :)
      Ответить

    • ava bormand 18.12.2013 20:44 # 0

      Интересно, какой Control подразумевается под default в FirstOrDefault? :) null?
      Ответить

      • ava kegdan 18.12.2013 23:17 # 0

        Ну так а что же еще? Не будет же оно возвращать панельку не привязанную к форме и править ее. Вылет с эксепшеном гораздо удачнее
        Ответить

        • ava bormand 18.12.2013 23:19 # 0

          А зачем пачка кастов? Если внутри контролов один хрен могут быть только контролы. Или не только?
          Ответить

          • ava kegdan 18.12.2013 23:22 # 0

            >Если внутри контролов один хрен могут быть только контролы

            Капитан не дремлет)

            >> А зачем пачка кастов?

            видимо кому то было скучно и одиноко
            Ответить

          • ava taburetka 19.12.2013 12:43 # 0

            без этих кастов вроде LINQ не будет работать
            Ответить

            • ava kegdan 19.12.2013 13:03 # −1

              Ну тут и linq то не нужен. И даже он юзается тут неправильно

              "Правильно" будет так

              .Controls.Cast<Control>().FirstOrDefault (x => x.Name == "panelMaxim");
              Ответить

              • ava taburetka 19.12.2013 13:26 # 0

                а как бы можно было это реализовать без LINQ?
                Ответить

              • ava taburetka 19.12.2013 13:38 # 0

                можно было бы теоретически написать extension метод для поиска контрола рекурсивно типа:
                public static Control FindControl(this Control root, string name) {
    foreach(Control child in root.Controls) {
        if(child.Name == name) return child;
        Control found = FindControl(child, name);
        if(found != null) return found;
    }
    return null;
}
                Ответить

                • ava kegdan 19.12.2013 17:55 # 0

                  ну или самому писать.

                  Хотя ответ более чем очевиден - нормально хранить данные нужно
                  Ответить

          • ava taburetka 19.12.2013 14:21 # +1

            а если конкретно, то ControlCollection коллекция имплементует интерфейс IEnumerable но не имплементует IEnumerable<T>. Поэтому нужен каст чтобы коллекция стала строго типизированной и LINQ мог создать свои методы.
            Ответить

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