Question

在定义变量，为它们分配空间，初始化它们然后正确清理所有内容时，我想请您就c＃中的良好编程习惯提出建议。

我目前面临的问题是，我有一个使用非托管API函数的函数，因此也访问非托管内存（使用Marshaling）。我希望在退出之前使功能清洁并妥善处理所有内容。但事实是，所有实际工作都是在 try-catch 块中完成的。这意味着我无法清除 catch 或 finally 块中的所有内容。

我所做的是声明所有变量，为它们保留内存并在输入函数后立即对它们进行初始化，然后在中清除所有内容（关闭句柄，释放内存......）块。

一切都很好，但我也希望在 try 块中完成变量声明，初始化和内存分配（例如，在初始化数组或在内存或上帝中分配空间时，也会出错）知道在哪里）。想到的唯一事情是嵌套两个 try-catch 块。这样可以，还是你会提出别的建议？

这是我到目前为止所做的：

//Declare variables, allocate memory, initialize variables.
........
try
{
    //Do actual work - write to file and read from a file in my case
    .........
}
catch (Exception exc)
{
    //Exception handler for file write/read errors
}
finally
{
    //Clean up (release handles, free memory,...)
}

以下是我的想法：

try
{
   //Declare variables, allocate memory, initialize variables.
   ........
   try
   {
       //Do actual work - write to file and read from a file in my case
       .........
   }
   catch (Exception exc)
   {
       //Exception handler for file write/read errors
   }
}
catch (Exception exc_1)
{
    //Exception handler for variable declaration, initialization, memory allocation errors
}
finally
{
    //Clean up (release handles, free memory,...)
}

提前感谢您的帮助！

干杯！

Answer 1

您可以实现IDisposable接口以调用Dispose方法。

或者最佳做法是使用阻止using

using (var variable = .....)
{

 ...

}

使用块的特殊性在于治疗结束时调用Dispose Method。

例如，如果您使用SqlConnection

var(var connection = new SqlConnection("...."))
{
....

}

这个代码就足够了

链接：http://msdn.microsoft.com/fr-fr/library/vstudio/system.idisposable.aspx

链接：http://msdn.microsoft.com/fr-fr/library/yh598w02%28v=vs.80%29.aspx

Answer 2

该方法的问题是变量超出了finally（和catch）

的范围

    try
    {
        string testString;
    }
    catch (Exception ex)
    {
    }
    finally
    {
        // testString is not in scope
    }

您担心的是声明可能会产生运行时错误吗？

根据评论OP不知道初始化可以与声明分开。

    List<string>  testLString;        
    try
    {
        testLString = new List<string>();
    }
    catch (Exception ex)
    {
    }
    finally
    {
        testLString = null;
    }

我不同意您对声明可能会导致运行时错误的担忧它什么也没做，只能宣布。

Answer 3

您可以根据需要嵌套任意数量的try...catch结构。这是使代码对自己的清理负责的好方法。

考虑使用只有try和finally的结构来代替总是需要清理的代码，无论它是否正常：

try {

  // do something here

  // declare some variable
  try {
    // allocate space for variable
    // do something with that variable
  } fincally {
    // deallocate space for variable
  }

  // do something more here

} catch(Exception ex) {
  // handle the exception here
}

您应该尝试使用尽可能具体的异常类，并且可以在同一结构中使用不同的类型来捕获不同的异常：

try {
  // do some i/o
} catch (IOException ex) {
  // here you know that it was actually the i/o that failed
} catch (Exception ex) {
  // here is for catching anything else that might have failed
}

Answer 4

我建议你创建一个单独的类型，它包含与非托管API的所有通信，管理内存等。它实现了IDisposable接口，其实现负责清理所有非托管资源。如果您使用的是Windows，最简单的方法是在C ++ / CLI中实现此包装器。

在变量声明，初始化和非托管内存分配之后进行清理

4 个答案: