c# - 声明变量以在 try block 之外使用的正确方法

Question

所以我对 C# 相当陌生(一般不是编程)，我编写了一些几乎零错误处理的代码，以证明仅在 VS 中运行的概念。现在我准备在应用程序中构建一些错误处理，以便可以部署应用程序。所以现在我在弄清楚如何/在哪里声明变量以便正确配置 Try Catch block 时遇到问题。

所以对于原始类型，我只是在 block 之外创建变量

前:

String TheString = "Some times I break";
SomeFunctionThatBreaks(TheString);

SomeFunctionThatDoesntBreak(TheString);

后:

String TheString ="";
Try
{
TheString="Some Times I break";
SomeFunctionThatBreaks(TheString);
}
Catch
{
MessageBox.Show("Error")
return false;
}
SomeFunctionThatDoesntBreak(TheString);

但是对于像 FileStream 这样的复杂类型，我不确定创建 EMPTY 变量以供以后使用的正确方法:

前:

        FileStream SourceFile = File.OpenRead(TheFile);
        StreamReader sr ; = new StreamReader(SourceFile);            
            char[] block = new char[3];
            byte[] header = new byte[6];
            SourceFile.Read(header, 0, 6);
            SourceFile.Seek(0, 0);
            encoding = (header[1] == 0 && header[3] == 0 && header[5] == 0) ? Encoding.Unicode : Encoding.UTF8;
            sr.ReadBlock(block, 0, 3);
            String sBlock = new String(block);
            SourceFile.Seek(0, 0);
    if(sBlock=="ABC")
    {
MyFunction(SourceFile);
    }

导致编译错误:

    FileStream SourceFile ;
    String sBlock ="";
    Encoding encoding;
    StreamReader sr;

    try
    {
        SourceFile = File.OpenRead(TheFile);
        sr = new StreamReader(SourceFile);
        char[] block = new char[3];
        byte[] header = new byte[6];
        SourceFile.Read(header, 0, 6);
        SourceFile.Seek(0, 0);
        encoding = (header[1] == 0 && header[3] == 0 && header[5] == 0) ? Encoding.Unicode : Encoding.UTF8;
        sr.ReadBlock(block, 0, 3);
        sBlock = new String(block);
        SourceFile.Seek(0, 0);

    }
    catch (Exception ex)
    {
        String Error = "Error Accessing: " + TheFile  + " System Message: " + ex.Message;
        EventLog.LogEvent(dtmLogging.LogEventType.Error, Error);
        MessageError(Error, "MyFunction()");
    }
    if(sBlock=="ABC")
    {
MyFunction(SourceFile);  //THIS LINE DOES NOT COMPILE: Use of unassigned variables
    }

提议的更改://如果我使此更改应用程序看起来工作正常，但我不确定这是否“正确”

    FileStream SourceFile =null;
    String sBlock ="";
    Encoding encoding = null;
    StreamReader sr = null;

感谢您的任何帮助

Answer 1

我建议不要在声明中将变量设置为 null，至少在您的示例代码之类的情况下。在这种情况下，将其设置为 null 实际上会掩盖所编写代码的另一个问题:catch block 允许该方法继续执行(并尝试使用 FileStream 对象)，即使出现问题并且 FileStream 对象可能处于未知状态并且不应使用。

您最初看到“红色错误”的原因是代码可能“通过”catch block 。如果您添加了 return或 throw在将控制权返回给调用者的 catch block 结束时，红色错误将消失，您无需将变量设置为 null。

要回答您的原始问题，像您一样声明变量可能是构建代码的合理方式。更好的方法是将您在 FileStream 上的所有操作合并到 try 中。 block ，以便可以在同一个 block 中定义和引用所有变量。如果 try block 开始变得太大，这可能表明你应该将它的一些内容重构为更小的实用方法，可以从 try 中调用。堵塞。

还有一个相关点:你应该有 using使用后需要关闭或以其他方式清理的对象的声明(即任何实现 IDisposable 接口(interface)的对象)。这确保文件被关闭并释放其他资源，而不管其余代码中发生什么。

例如:

try
{
    using (FileStream SourceFile = File.OpenRead(TheFile))
    using (StreamReader sr = new StreamReader(SourceFile))
    {
        char[] block = new char[3];
        byte[] header = new byte[6];
        SourceFile.Read(header, 0, 6);
        SourceFile.Seek(0, 0);
        Encoding encoding = (header[1] == 0 && header[3] == 0 && header[5] == 0) ? Encoding.Unicode : Encoding.UTF8;
        sr.ReadBlock(block, 0, 3);
        String sBlock = new String(block);
        SourceFile.Seek(0, 0);

        if (sBlock=="ABC")
        {
            MyFunction(SourceFile);
        }

        // And do anything else you want with SourceFile & sr here
    }
}
catch (Exception ex)
{
    String Error = "Error Accessing: " + TheFile  + " System Message: " + ex.Message;
    EventLog.LogEvent(dtmLogging.LogEventType.Error, Error);
    MessageError(Error, "MyFunction()");
}

希望这可以帮助。