if 语句中的变量声明

Question

在 C# 中，对于我正在编写的一个相对简单的程序，我试图创建一个事件处理函数来处理多个源，如下所示：

private void fooHandler(object sender, RoutedEventArgs e)
{
    fooObject objectFoo = (fooObject)sender;
    if (objectFoo.name == "bla1"){
        bla1Window bla = new bla1Window();
    }
    if (objectFoo.name == "bla2"){
        bla2Window bla = new bla2Window();
    }
    .
    .
    .
    else{
        //default stuff happens
    }
bla.Left = this.Left
bla.Top = this.Top
bla.Show();
this.Close();
}

该函数用于窗口切换。 问题是，一旦我退出 if 语句，变量就会超出范围。我这样做是因为，查看我定义的一系列函数来单独处理每个事件，除了一个变量声明之外，它们都是相同的。有没有办法做到这一点，或者我是否必须坚持为每个事件处理程序使用一个函数？

Answer 1

如果 bla1Window 和 bla2Window 都共享基类或接口，您可以通过这种方式引用它们。在这种情况下，看起来您只是访问 Window 的属性，因此您可以执行以下操作：

Window window = null;
fooObject objectFoo = (fooObject)sender;
if (objectFoo.name == "bla1"){
    window = new bla1Window();
}
else if (objectFoo.name == "bla2"){
    window = new bla2Window();
}
.
.
.
else{
    //default stuff happens
}

window.Left = this.Left
window.Top = this.Top
window.Show();
this.Close();

Answer 2

考虑一下：

private void fooHandler(object sender, RoutedEventArgs e)
{
    fooObject objectFoo = (fooObject)sender;
    Window bla; // a super-type or interface, don't assign a value here
                // so there will be a compile error if it was
                // forgotten below
    if (objectFoo.name == "bla1"){
        bla = new bla1Window();
    } else if (objectFoo.name == "bla2"){
        bla = new bla2Window();
    } else {
        // just make sure to assign to bla
        // or there will a compiler error later
    }
    bla.Left = this.Left
    bla.Top = this.Top
    bla.Show();
    this.Close();
}

我通常会这样写，但是：

Window CreateFromName(string name) {
    if (name == "bla1"){
        return new bla1Window();
    } else if (name == "bla2"){
        return new bla2Window();
    } else {
        // just make sure to return a value
        // or there will a compiler error later
    }
}

private void fooHandler(object sender, RoutedEventArgs e)
{
    fooObject objectFoo = (fooObject)sender;
    Window bla = CreateFromName(objectFoo.name);
    bla.Left = this.Left
    bla.Top = this.Top
    bla.Show();
    this.Close();
}

快乐编码。

Answer 3

解决方案是简单地将变量提升到 if 语句之后使用所需的范围。真的就是这么简单。不过，我建议您尝试重构它，或者至少发布您的真实代码，以便我们可以尝试一下。当你有一堆代码在多个 if 语句中一个接一个地重复时，通常可以将其简化为一个或两个方法。

private void fooHandler(object sender, RoutedEventArgs e)
{
    fooObject objectFoo = (fooObject)sender;

    // use the base class and work with that.
    // all windows have the properties you use 
    // below, so there is no need to declare it
    // as a more specific type.
    blahWindow bla = null; 
    if (objectFoo.name == "bla1"){
        bla = new bla1Window();
    }
    if (objectFoo.name == "bla2"){
        bla = new bla2Window();
    }
    .
    .
    .
    else{
        //default stuff happens
        bla = new BlahDefault();
    }

    // 'bla' cannot be nbull here if each branch above assigns it
    bla.Left = this.Left
    bla.Top = this.Top
    bla.Show();
    this.Close();
}

Answer 4

您的所有窗口都应该有一个共享的父窗口。使用子构造函数并将其分配给可以在“if”语句之外单独声明的父对象。

private void fooHandler(object sender, RoutedEventArgs e)
{
    Window bla = null;
    fooObject objectFoo = (fooObject)sender;
    if (objectFoo.name == "bla1"){
        bla = new bla1Window();
    }
    if (objectFoo.name == "bla2"){
        bla = new bla2Window();
    }
    .
    .
    .
    else{
        //default stuff happens
    }
    if(bla != null)
    {
        bla.Left = this.Left
        bla.Top = this.Top
        bla.Show();
        this.Close();
    }
}

Answer 5

您确实希望将其设为一个接口，并在 if 代码之前声明该接口。看起来下面的 bla 上调用的所有方法都是通用的，这是接口（或抽象类，如果更合适的话）的一个很好的候选者。

事实上，如果您的代码根本不在这个文件中切换，并且您在工厂或其他东西内部通过它，那就最好了。如果您决定走这条路，互联网上有很多相关信息。

Answer 6

我认为你应该确保在 if 语句之前声明变量。
那应该可以解决你的问题。
例子

public string IfStatement()
{   
    string myValue = null;

    bool condition = true;
    if (condition)
    {
        myValue  = "something";
    }
    else
    {
        myValue  = "something else";
    }
    return myValue;
}