c# - 类型转换是代码设计不当的标志吗？

Question

我会是第一个告诉别人我的代码设计需要改进的人。我忍不住觉得，当我的代码中有类型转换时，这表明需要重新设计某些内容以删除类型转换。这个问题有两个部分，第一个只是发布的那个:类型转换是代码设计不当的标志吗？

第二个问题是在第一个问题的基础上，如果类型转换真如我所想的那样邪恶，那么如何避免下面的情况呢？

class InputPanel : Control
{
  public event EventHandler InputEvent;
}

class OutputPanel : Control
{
}

class MainWindow : Form
{
  public MainWindow()
  {
    var loadButton = new Button();
    loadButton.Click += new EventHandler(HandleButtonClick);

    var inputPanel = new InputPanel();
    inputPanel.InputEvent += new EventHandler(HandleInputEvent);
    bodyControl = inputPanel;
  }

  private void HandleButtonClick(object sender, EventArgs args)
  {
    OpenFileDialog dialog = new OpenFileDialog();
    if (dialog.ShowDialog(this) == DialogResult.OK)
    {
      var data = LoadDataFromFile(dialog.FileName);
      var inputPanel = bodyControl as InputPanel; // Ugly typecast...
      if (inputPanel != null)
      {
        inputPanel.PopulateFromData(data);
      }
    }
  }

  private void HandleInputEvent(object sender, EventArgs args) 
  {
    var outputPanel = new OutputPanel();
    bodyControl = outputPanel;
  }

  Control BodyControl;
}

上述代码背后的原因是 MainWindow 窗体包含一个 MenuStrip(在此示例中简化为一个按钮)和一个控件(BodyControl)。由于需要将显示的控件从输入面板更改为输出面板，通过单击按钮，您可以简单地重新分配 BodyControl 字段(调整父项等)。这意味着一次只加载一个面板，布局逻辑变得简化，因为在 MainWindow 中只有一个面板(如果包含 MenuStrip，则为两个)，而不是根据哪个状态有条件地布置多个“主体”控件程序在(输入与输出)中。

Answer 1

使用 interface 可以使代码更简洁(并避免类型转换)或 common base class对于两个控件(inputPanel 和 outputPanel)。只需将 BodyControl 存储为基类或接口(interface)(而不是 Control)。假设接口(interface)或基类实现了 PopulateFromData 方法，您根本不需要强制转换。

还要确保您知道 where clause在 C# 中。在处理类似场景时可以派上用场。