不好的 MVVM 实践？ （命令参数）

Question

在我的代码中，我有 3 个按钮。他们每个人都执行不同的事情。我让它们使用相同的命令执行，但我给了它们不同的 CommandParameter 来指定差异。

这是我所说的一个例子

XAML：

            <Button Command="{Binding UpdateCommand}" CommandParameter="Add">Add Client</Button> 
            <Button Command="{Binding UpdateCommand}" CommandParameter="Change">Change Client</Button>
            <Button Command="{Binding UpdateCommand}" CommandParameter="Remove">Remove Client</Button>

视图模型：

        public MainWindowViewModel()
        {
            clients.Add(new Client() { Name = "Client 1" });
            clients.Add(new Client() { Name = "Client 2" });

            //UpdateCommand = new ClientUpdateCommand(this);
            UpdateCommand = new DelegateCommand(param => ClientExecuteCommand((string)param), param => ClientCanExecuteCommand((string)param));
        }

        public void ClientExecuteCommand(string param)
        {
            ClientDialog cd;
            switch(param)
            {
                case "Add":
                    cd = new ClientDialog("Add Client", "Add Client", "Random User");
                    cd.ShowDialog();
                    clients.Add(new Client() { Name = cd.nameTxtBox.Text });
                    break;
                case "Change":
                    cd = new ClientDialog("Change Client", "Change Client", SelectedClient.Name);
                    cd.ShowDialog();
                    SelectedClient.Name = cd.nameTxtBox.Text;
                    break; 
                case "Remove":
                    clients.Remove(SelectedClient);
                    break;
            }
        }

        public bool ClientCanExecuteCommand(string param)
        {
            if (param == "Add")
                return true;
            else
                return !(SelectedClient == null);
        }

我只是想知道这是否是不好的编程习惯，或者我是否应该为每个按钮创建不同的命令？如果这没问题；我什么时候应该不创建 CommandParameter？

提前感谢您的回答:)

Answer 1

这充其量是值得怀疑的做法。问问自己：如果您要创建方法而不是命令，您会将它们命名为“更新”并将要更新的内容作为“字符串参数”传递，还是更愿意使用具有有意义名称的单独方法？

您的公共界面变得不那么清晰（您需要记录作为“参数”传递的内容），您使您的设计更加脆弱（错字不会导致任何错误或警告），您引入了魔术常量。总的来说，你让你的代码变得更糟。但要短一点，如果这是你所追求的。

如果您想避免为每个操作创建命令（这可能会变得冗长并因此很烦人），您可以使用 MVVM 库，它不会强迫您这样做。例如，使用 Caliburn.Micro，您可以只使用三个方法和三个按钮，而无需弄乱命令和参数。