处理 GUI 应用程序中的复杂规则（C++ 或 C#）

Question

我正在处理一个对话框，在启用“确定”按钮之前必须满足多个规则。

目前，页面上的任何操作（例如输入数据或从下拉列表中选择项目（除其他外））都会调用一个名为 ProcessEvent() 的函数 - 该函数处理所有逻辑并启用或禁用“确定”按钮。

我的问题是我发现很难使规则变得简洁易懂。

有些规则可以通过对话框中的另一个操作来否定，现在我已经得到了 if else 语句遍布各处，或者难以阅读和遵循的语句。延长。

下面的代码是问题的简化，但很好地演示了它。我如何更好地处理这个问题（如果可能的话）

bool CWorkstation::ProcessEvent(void)
    {   
        UpdateData();

        CharCount = GetDlgItemInt(IDC_CharCount, NULL, FALSE); //get latest

        if ( IsDlgButtonChecked(IDC_USEDBNAME))
            {   
                if (!IsDlgButtonChecked(IDC_MAXDBNAME))
                    {
                        EnableNext(TRUE);
                    }
            }

        if (IsDlgButtonChecked(IDC_MAXDBNAME) && CharCount)
            {                   
                if (IsDlgButtonChecked(IDC_USEXMLNAME))
                    {
                        if ( PrefixName.IsEmpty() ) 
                            {
                                EnableNext(FALSE);
                            }
                        else
                            {
                                EnableNext(TRUE);
                            }
            }



            }   


        if (IsDlgButtonChecked(IDC_USEXMLNAME) && PrefixName.GetLength() > 1)
            {
                EnableNext(TRUE);
            }



        if  ( IsDlgButtonChecked(IDC_WSAUTONAME) || IsDlgButtonChecked(IDC_RENAMEIFDUP))
            {

            // TRACE("IDC_WSAUTONAME is Checked\n");

            if ( IsDlgButtonChecked(IDC_USEXMLNAME) && PrefixName.GetLength() > 1 ) 

                {   


                if ( IsDlgButtonChecked(IDC_IDC_USESHORTNAME) ) 

                    {

                    EnableNext(TRUE);
                    }

                else if ( IsDlgButtonChecked(IDC_USELONGNAME) )

                    {

                    EnableNext(TRUE);

                    }

                else

                    {
                    EnableNext(FALSE);
                    }



                }


            if ( !IsDlgButtonChecked(IDC_USEPREFIX) )

                {


                if ( IsDlgButtonChecked(IDC_IDC_USESHORTNAME) ||  IsDlgButtonChecked(IDC_USELONGNAME) )

                    {
                    EnableNext(TRUE);
                    }

                }


            return false;


            }

        } 

Answer 1

我会将您的 if/else 语句拆分为多个函数，并对发送到 EnableNext 的参数执行 &= 操作。您应该只调用一次 EnableNext。

因此，例如：

// in CWorkStation::ProcessEvent
bool enableNext = true; // start with true

enableNext &= Condition1(); // of course pick better names than Condition1
enableNext &= Condition2(); // this is just for an example

EnableNext(enableNext);

其中 Condition1() 可能是：

bool Condition1()
{
    return (IsDlgButtonChecked(IDC_USEDBNAME) 
         && !IsDlgButtonChecked(IDC_MAXDBNAME));
}

等等。

这里发生的情况是，enableNext 变量以 true 开头。然后，您执行的每个 &= 都意味着如果任何 ConditionX() 函数返回 false，enableNext 将最终返回 false。只有当所有条件都为真时，它才会在最后为真。

Answer 2

这个问题可以通过听众的概念来解决。

您可以使每个 GUI 组件都有一个 isEnabled() 方法，该方法根据某些条件检查其条件。当调用任何更改任何组件状态的操作时，每个 GUI 组件都会调用 isEnabled()。

这样，您就可以进行以下声明：

bool CheckBoxComponent::isValid() {
   return isNameFilled() && isEmailChecked();
}

bool OkButton::canSend() {
   return checkBoxName->isValid() && isEmailChecked();
}

然后，在创建 GUI 组件时，您可以使每个组件通过侦听器相互连接。

这样，您就拥有了每个组件所属的规则，并且没有大量的 if 语句。

Answer 3

尝试将规则制定为状态机可能会有所帮助，但是否可行取决于其性质。在这种方法中，每当用户填写对话框中的某些字段，或选中复选框或其他任何内容时，您都会相应地更新状态机的状态。如果你有，你可以使用 Boost.Statechart< /a> 来实现它。

Answer 4

在这种情况下，我倾向于通过（例如）默认启用该按钮来使其尽可能简单，如果设置（或未设置）任何其他条件，则禁用它；这限制了“if”条件与“else”中的不同情况。

Answer 5

将您的条件重述为正确的布尔语句，正确缩进所有条件并添加一些注释。恕我直言，您不应该在一次性方法中隐藏真正的检查。如果您想注释代码，请对其进行注释，但不要为此目的创建方法，它只会混淆事物，并且您的条件不会变得更简单：

EnableNext( 
        // condition 1 
        IsDlgButtonChecked(IDC_USEDBNAME) && !IsDlgButtonChecked(IDC_MAXDBNAME)
        // condition 2
    ||  IsDlgButtonChecked(IDC_MAXDBNAME) && CharCount 
        && IsDlgButtonChecked(IDC_USEXMLNAME) && !PrefixName.IsEmpty()
        // condition 3
    ||  IsDlgButtonChecked(IDC_USEXMLNAME) && PrefixName.GetLength() > 1
        // and so on
)

这样，您似乎会检查相同的条件两次USEXMLNAME && !PrefixName().IsEmpty()。现在也很明显，EnableNext 总是被调用。

Answer 6

虽然它可能比您想要的解决方案“重”一些，但您可能想看看 Adob​​e 的 亚当和夏娃图书馆。 Eve 处理小部件布局，Adam 采用一组有关小部件逻辑的语句，并将它们组合到一个控制器中，该控制器根据该逻辑启用和禁用小部件，以及处理初始化并将结果放入适当的变量中（例如，当用户单击“确定”时）。