如何在c＃中减少这个IF-Else梯形图

Question

这是我创建的IF -Else梯形图，用于将第一个可视控件集中在我的窗体上。根据要求，任何控件都可以隐藏在窗体上。所以我必须找到第一个可见的控件并将其聚焦。

 if (ddlTranscriptionMethod.Visible)
    {
        ddlTranscriptionMethod.Focus();
    }
    else if (ddlSpeechRecognition.Visible)
    {
        ddlSpeechRecognition.Focus();
    }
    else if (!SliderControl1.SliderDisable)
    {
        SliderControl1.Focus();
    }
    else if (ddlESignature.Visible)
    {
        ddlESignature.Focus();
    }
    else
    {
        if (tblDistributionMethods.Visible)
        {
            if (chkViaFax.Visible)
            {
                chkViaFax.Focus();
            }
            else if (chkViaInterface.Visible)
            {
                chkViaInterface.Focus();
            }
            else if (chkViaPrint.Visible)
            {
                chkViaPrint.Focus();
            }
            else
            {
                chkViaSelfService.Focus();
            }
        }
    }

有没有其他方法可以做到这一点。 我认为使用LINQ会占用性能，因为我必须遍历整个页面集合。 我在页面很深，有主页。请建议。

Answer 1

我觉得你的树很好。 这当然看起来像一个可以简化的逻辑树，你有一个良好的嗅觉怀疑它。 但是，似乎逻辑树反映了您的需求。 这个逻辑确实很复杂 ，这是C＃为您提供处理这种情况的条件框架。 我不认为它可以改进。

如果您有一个应该具有焦点的简单控件列表，并且您希望将焦点放在列表中的第一个可见控件上，则可以执行以下操作：

(From c in ListOfControls
Where c.visible = true
Select c).First.Focus();

但是，看起来你有一些额外的标准，所以这是行不通的。

Answer 2

两种方法：

1. 如果可见，迭代控件并设置焦点
2. 使用TabIndex并将焦点设置为第一个。 然后焦点应该落到第一个可见的控制

Answer 3

如果您只是尝试将第一个可见控件聚焦在窗体上，那么我将用一个循环替换整个梯形图：

foreach (Control c in Controls)
{
    if (c.Visible)
    {
        c.Focus();
        break;
    }
}

如果需要聚焦内部控件，请使用递归方法：

bool FocusFirst(ControlCollection controls)
{
    foreach (Control c in controls)
    {
        if (c.Visible)
        {
            c.Focus();
            FocusFirst(c.Controls);
            break;
        }
    }
}

Answer 4

您可以在符合标准后返回，例如：

   if (ddlTranscriptionMethod.Visible) 
    { 
        ddlTranscriptionMethod.Focus(); 
        return;
    }

    if (ddlSpeechRecognition.Visible) 
    { 
        ddlSpeechRecognition.Focus(); 
        return;
    } 

等等..

Answer 5

您可以迭代控件并设置焦点（如果可见）。 但我建议您使用状态模式以获得更好的代码可读性。

Answer 6

您所做的就是设置焦点，即客户端功能。 我个人会在javascript（使用jQuery）中这样做。 未设置为可见的ASP.NET控件不会在HTML中呈现，因此您可以查找这些元素的存在，或者如果您只是在寻找第一个可见的启用控件，则可能会有更简单的方法。

Answer 7

怎么样跳转似乎你可以在这里使用它。

Answer 8

当要评估的项目列表很大（有时它非常大）时，我尝试将评估顺序与条件逻辑分开，如下所示：

List<WebControl> wcBasics = new List<WebControl>();
wcBasics.Add(ddlTranscriptionMethod);
wcBasics.Add(ddlSpeechRecognition);
wcBasics.Add(ddlESignature);

List<CheckBox> checks = new List<CheckBox>();
checks.Add(chkViaFax);
checks.Add(chkViaInterface);
checks.Add(chkViaPrint);

private void Focus()
{
    foreach (WebControl c in wcBasics)
        if (c.Visible) {
            c.Focus();
            return;
        }

    if (!tblDistributionMethods.Visible) return;

    foreach (CheckBox chk in checks)
        if (chk.Visible) {
            chk.Focus();
            return;
        }
    }

    chkViaSelfService.Focus();
}

Answer 9

这是一款经典的状态机。 通过设置机器，它可以使代码更容易理解和维护， 尽管它可能会增加总线数 。

我不会详细介绍您的代码。 通过确定用户正在操作的状态，您可以以可理解的特定状态方式以编程方式更改不同的值。 （下面的伪代码）

enum FormStates
{
    Initial_View
    Working_View
    Edit_View
    Shutdown_View
};

{ // Somewhere in code

switch (theCurrentState)
{

    case Initial_View :
        Control1.Enabled = true;
        Control2.Enabled = true;
        theCurrentState = Working_View;
    break;

   case Working_View
      if (string.empty(Contro1.Text) == false)
      {
          Control2.Enabled = false;
          Speachcontrol.Focus();
          theCurrentState = Edit_view;
      }
      else // Control 2 is operational
      {
         Control1.Enabled = false;
         SliderControl.Focus();
      }

    case Edit_View:
       ...
    break;  

   break;
}

通过按逻辑步骤组织代码，可以更轻松地添加更多状态，而不会危及巨大的if / else。

Answer 10

这是对问题略有不同的看法。 首先，定义一个接口来表示可能会或可能不会聚焦的控件：

public interface IFormControl
{
    bool Focus();
}

然后创建一个处理简单案例的实现：

public class FormControl : IFormControl
{
    private readonly Control _control;

    public FormControl(Control control)
    {
        _control = control;
    }

    public bool Focus()
    {
        if(_control.Visible)
        {
            _control.Focus();
        }

        return _control.Visible;
    }
}

并创建另一个处理更困难的案例：

public class DependentFormControl : IFormControl
{
    private readonly Control _control;
    private readonly Func<bool> _prerequisite;

    public DependentFormControl(Control control, Func<bool> prerequisite)
    {
        _control = control;
        _prerequisite = prerequisite;
    }

    public bool Focus()
    {
        var focused = _prerequisite() && _control.Visible;

        if(focused)
        {
            _control.Focus();
        }

        return focused;
    }
}

然后，创建一个扩展方法，将焦点设置在序列中的第一个控件上：

public static void FocusFirst(this IEnumerable<IFormControl> formControls)
{
    var focused = false;

    foreach(var formControl in formControls)
    {
        if(formControl.Focus())
        {
            break;
        }
    }
}

最后，创建要集中的控件集：

var controls = new FormControl[]
{
    new FormControl(ddlTranscriptionMethod),
    new FormControl(ddlSpeechRecognition),
    new DependentFormControl(SliderControl1, () => !SliderControl1.SliderDisable),
    new FormControl(ddlESignature),
    new DependentFormControl(chkViaFax, () => tblDistributionMethods.Visible),
    new DependentFormControl(chkViaInterface, () => tblDistributionMethods.Visible),
    new DependentFormControl(chkViaPrint, () => tblDistributionMethods.Visible),
    new DependentFormControl(chkViaSelfService, () => tblDistributionMethods.Visible)
};

controls.FocusFirst();

您可以在页面加载时创建控件集，并在必要时调用.FocusFirst() 。