Как в WP заблокировать в цикле кнопки на форме?

Question

[nicklayk]

Имеется форма, есть переменная, нужно в цикле блокировать кнопки в зависимости от значения переменной.

Примерное представление:

for(int i=0;i<value;i++) {
    btn_[i].IsEnabled = false;
}

Как реализовать?

Answer 1

[Вадим Мартынов]

Ох. Начнём.
1. Правильно ли я понимаю, что кнопки генерируются динамически? Или коллекция btn_[i] заполнена заранее и все кнопки в ней есть?
2. Можно более подробно узнать о сценарии, в котором возникла такая задача? Сейчас неясно в какой момент времени должна происходить смена доступности кнопок.
3. ViewFirst или ViewModelFirst вы используете?

В любом случае, если возникла такая ситуация, то шаблон MVVM уже не соблюдается - представление управляет само собой, а не данные управляют представлением. Тут есть несколько вариантов.
Если использование команд вам не подходит, то в любом случае где-то должна быть та самая переменная со значением, на которую можно забиндиться.
Если переменная не булевого типа, то сделать это можно через конвертер.
Если кнопки генерируются в контейнере элементов в зависимости от наполнения коллекции в VM, то можно прибиндиться к элементу коллекции. Если же для коллекции нужно прибиндиться к внешнему свойству, то подойдёт использование RelativeSource.
Если всё-таки есть переменная в View, которая влияет на доступность кнопок - стоит подумать, как изменить код так, чтобы подобный сценарий не возник. Соблюдать mvvm полностью не всегда просто, но стараться это делать крайне рекомендуется. Что бы там не писали на хабре, но это удобный шаблон, который позволяет не запутаться в коде и вести разработку максимально быстро.

Я бы всё-таки попросил более подробно описать проблему, сейчас я не понимаю даже, в чём именно возникла загвоздка.

Comments

[nicklayk]

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

[nicklayk]

Можно более подробно узнать о сценарии, в котором возникла такая задача?
Во время загрузки формы должен происходить цикл.

[Вадим Мартынов]

Ок.
Если кнопки определены заранее и их ограниченное количество, то почему не назначить каждой из них имя (в xaml - Name="btn1" и т.д.), а в конструкторе или инициализаторе формы сделать следующее:

private List<Button> _buttons;
public Initialize()
{
    base.Initialize();
    buttons = new List<Button>() { this.btn1, this.btn2, ... , this.btnN };
    // здесь ваш цикл
}

Похоже, я всё ещё не понимаю загвоздки проблемы :/ Поле, которое будет влиять на видимость кнопок находится во View?

[nicklayk]

Я так и делал, только ваш код мне не совсем подходит. Мне нужно заниматься с Loaded'ом, с загрузчиком формы.
Мне бы всё по полочкам...

[Вадим Мартынов]

Давай я попробую привести исчерпывающий пример, а ты скажешь, чем именно он отличается от задачи, которую необходимо решить.

MainWindow.xaml

<Window x:Class="NicklaykTestApp.MainWindow"
        xmlns="http://schemas.microsoft.com/winfx/2006/xaml/presentation"
        xmlns:x="http://schemas.microsoft.com/winfx/2006/xaml"
        Title="MainWindow" Loaded="WindowLoaded">
    <Grid>
        <Grid.RowDefinitions>
            <RowDefinition />
            <RowDefinition />
        </Grid.RowDefinitions>
        <StackPanel>
            <Button Name="button1" Content="Кнопка 1" />
            <Button Name="button2" Content="Кнопка 2" />
        </StackPanel>

        <Button Grid.Row="1" Name="button3" Content="Кнопка 3"></Button>
    </Grid>
</Window>

MainWindow.xaml.cs

using System.Collections.Generic;
using System.Windows;
using System.Windows.Controls;

namespace NicklaykTestApp
{
    /// <summary>
    /// Форма
    /// </summary>
    public partial class MainWindow : Window
    {
        /// <summary>
        /// Коллекция кнопок, которые должны быть изменены в зависимости от значения поля _masterField.
        /// </summary>
        private readonly IEnumerable<Button> _slavedButtons;

        /// <summary>
        /// Поле, от значения которого зависит изменение доступности кнопок на форме.
        /// </summary>
        private readonly bool _masterField;


        public MainWindow()
        {
            InitializeComponent();

            var buttons = new List<Button>
                          {
                              this.button1,
                              this.button2,
                              this.button3
                          };

            _slavedButtons = buttons;

            _masterField = false; // Здесь кастомная логика вычисления поля
        }

        private void WindowLoaded(object sender, RoutedEventArgs e)
        {
            foreach (var button in _slavedButtons)
            {
                button.IsEnabled = _masterField;
            }
        }

    }
}

Тестовое приложение: zalil.ru/34900493

Всё просто - при назначении в xaml для кнопки атрибута Name к ней можно обращаться, как к обычному члену класса окна.

Я всё еще хочу сказать о том, что сам сценарий, что переменная в behindcode, которая влияет на доступность кнопки - плохо. Но если очень хочется, то можно :/

[nicklayk]

Замечательно, это действительно то что мне нужно. Только мне нужно поработать с этим списком в примерно таком цикле, как я написал выше.
А подобное обращение не позволяет - btn_[i].IsEnabled = false;

[Вадим Мартынов]

Ты имеешь в виду, что тебе нужно именно обращение по индексу? Ну дык тут всё просто - меняем IEnumerable на IList:

using System.Collections.Generic;
using System.Windows;
using System.Windows.Controls;

namespace NicklaykTestApp
{
    /// <summary>
    /// Форма
    /// </summary>
    public partial class MainWindow : Window
    {
        /// <summary>
        /// Коллекция кнопок, которые должны быть изменены в зависимости от значения поля _masterField.
        /// </summary>
        private readonly List<Button> _slavedButtons;

        /// <summary>
        /// Поле, от значения которого зависит изменение доступности кнопок на форме.
        /// </summary>
        private readonly bool _masterField;


        public MainWindow()
        {
            InitializeComponent();

            _slavedButtons = new List<Button>
                          {
                              this.button1,
                              this.button2,
                              this.button3
                          };

            _masterField = false; // Здесь кастомная логика вычисления поля
        }

        private void WindowLoaded(object sender, RoutedEventArgs e)
        {

            for (int i = 0; i < _slavedButtons.Count ; i++)
            {
                _slavedButtons[i].Content = string.Format("Пример использования индекса {0}", i);
                _slavedButtons[i].IsEnabled = _masterField;
            }
        }

    }
}

[nicklayk]

Идеально, огромное спасибо.

Answer 2

[Вячеслав Золотов]

Не самая лучшая идея. Делайте лучше binding в XAML

Comments

[nicklayk]

В моей ситуации нужно именно так, как я описал.

Answer 3

[gleb_kudr]

Ну если не хотите через баиндинги делать, то засуньте все кнопки в list и блокируйте их по индексу. Чуток псевдокода.

//Наполняем массив
List<Button> myButtons=new List<Button>();
for(int i=1;i<10;i++){
 var newButton= new Button();
 form1.add(newButton);
 myButtons.add(newButton);
}
//вырубаем кнопки
foreach(Button btn in myButtons){
 if(btn!=null){
  btn.IsEnabled=false;
 }
}