c# - 如何正确处理元素？为什么代码分析不断改变主意？

Question

我昨天加入了一个项目，我们决定将我们的两个项目合并为一个，并结合两者的特点(他们基本上吸收了我们)。

问题是，他们某处有内存泄漏，他们让我找到并解决它。

我在 Visual Studio 中运行代码分析并得出一长串警告，其中大部分我们可以忽略，我试图修复的是对象处理问题。我从来没有真正特别注意过的东西，我很惊讶地发现通过代码分析来确定某些东西是正确的并不容易。

代码是这样开始的:

StreamWriter SW = new StreamWriter(File.Create("folder/file.txt"));
SW.WriteLine("text");
SW.WriteLine("text");
SW.WriteLine("text");
SW.WriteLine();
SW.WriteLine("text");
SW.WriteLine("text");
SW.WriteLine("text");
SW.Close();
SW.Flush();

但是代码分析说 File.Create 并没有沿着所有代码路径被处理掉，并且 SW 可以被处理多次，我去看了一下发现你不应该同时使用 Close 和 Flush，事实上，您应该使用 using 语句，所以我将其更改为:

using(StreamWriter SW = new StreamWriter(File.Create("folder/file.txt")))
{
    SW.WriteLine("text");
    SW.WriteLine("text");
    SW.WriteLine("text");
    SW.WriteLine();
    SW.WriteLine("text");
    SW.WriteLine("text");
    SW.WriteLine("text");
}

被处置的 SW 消失了，但它仍然说 File.Create 并没有沿着所有代码路径被处置，只有一个代码路径，所以它没有意义，我尝试添加一个单独的 Using File.Create 的语句，并分别创建/写入文件，但代码分析仍然捕获它。

这似乎很容易弄清楚，除非 VS 出现误报或其他问题。

无论如何，感谢您给我时间阅读本文。

~编辑

我在原来的帖子中撒了谎，当我添加双重 using 语句时，包括当我使用“Stream”时，它告诉我可以多次处理该对象，从而导致 objectdisposedexception。我假设它会发生一些奇怪的怪异事件。因为我从不自己处理它。

当前代码:

using (Stream stream = File.Create("folder/file.txt"))
{
    using (StreamWriter SW = new StreamWriter(stream))
    {
        SW.WriteLine("");
        SW.WriteLine("");
        SW.WriteLine("");
        SW.WriteLine();
        SW.WriteLine("");
        SW.WriteLine("");
        SW.WriteLine("");
    }
}

这个成功了，VS 通过了，非常感谢。

using (StreamWriter SW = File.CreateText("text/awardsList.txt"))
{
    SW.WriteLine("");
    SW.WriteLine("");
    SW.WriteLine("");
    SW.WriteLine();
}

Answer 1

我怀疑它正在寻找这样的东西:

using (Stream stream = File.Create("folder/file.txt"))
{
    using (StreamWriter writer = new StreamWriter(stream))
    {
        ...
    }
}

假设如果 File.Create 成功，那么 StreamWriter 构造函数也将成功——但 VS 可能不知道这一点。它也可能不知道处理流的写入处理。

我个人会使用 File.CreateText相反，这避免了这个问题并且更简单:)

using (TextWriter writer = File.CreateText("folder/file.txt"))
{
    ...
}