“已被另一个程序使用的文件”编写错误

Question

我不断收到此错误＆＃34;文件已被其他程序使用＆＃34;每当我尝试写入已打开的文件时，虽然我使用了＆＃34; close（）＆＃34;和＆＃34; flush（）＆＃34;命令，它仍然给我错误，这是我的代码：

private void Openbtn_Click(object sender, EventArgs e)
    {
        textBox1.Text = "";
        listView1.Items.Clear();
        OpenFileDialog ofd = new OpenFileDialog();
        ofd.Title = "Open GSMB File";
        ofd.Filter = "GSMB Files (*.gsmb)|*.gsmb|All Files (*.*)|*.*";
        if (ofd.ShowDialog() == System.Windows.Forms.DialogResult.OK)
        {
            MessageBox.Show("File opened Succesfully!", "Info", MessageBoxButtons.OK, MessageBoxIcon.Information);
            path = ofd.FileName;
            BinaryReader br = new BinaryReader(File.OpenRead(path), Encoding.Unicode);
            BinaryReader brs = new BinaryReader(File.OpenRead(path), Encoding.Unicode);
            brs.BaseStream.Position = 0x4;
            menuItem9.Text = brs.ReadInt32().ToString();
            if (menuItem9.Text == "3620")
            {
                num_pointers = 204;
                menuItem8.Text = num_pointers.ToString();
            }
            else if (menuItem9.Text == "54662")
            {
                num_pointers = 2372;
                menuItem8.Text = num_pointers.ToString();
            }
            else if (menuItem9.Text == "9560")
            {
                num_pointers = 88;
                menuItem8.Text = num_pointers.ToString();
            }
            else if (menuItem9.Text == "1126")
            {
                num_pointers = 130;
                menuItem8.Text = num_pointers.ToString();
            }
            else if (menuItem9.Text == "342")
            {
                num_pointers = 16;
                menuItem8.Text = num_pointers.ToString();
            }
            else if (menuItem9.Text == "6232")
            {
                num_pointers = 467;
                menuItem8.Text = num_pointers.ToString();
            }
            else if (menuItem9.Text == "75698")
            {
                num_pointers = 498;
                menuItem8.Text = num_pointers.ToString();
            }
            else if (menuItem9.Text == "9914")
            {
                num_pointers = 110;
                menuItem8.Text = num_pointers.ToString();
            }
            else if (menuItem9.Text == "128")
            {
                num_pointers = 4;
                menuItem8.Text = num_pointers.ToString();
            }
            else if (menuItem9.Text == "5394")
            {
                num_pointers = 156;
                menuItem8.Text = num_pointers.ToString();
            }
            else if (menuItem9.Text == "12000")
            {
                num_pointers = 580;
                menuItem8.Text = num_pointers.ToString();
            }
            else 
            {
                MessageBox.Show("This is not a Pokémon Typing Adventure string file !", "ERROR", MessageBoxButtons.OK, MessageBoxIcon.Error);
                return;
            }
            List<int> offsets = new List<int>();
            int startstr = 0x1C;
            br.BaseStream.Position = startstr;
            int startstrval = br.ReadInt32();
            for (int i = 4; i < (num_pointers * 4 + 1); i += 4)
            {
                br.BaseStream.Position = startstr + i;
                offsets.Add(br.ReadInt32() + startstrval);
            }
            Dictionary<int, string> values = new Dictionary<int, string>();
            for (int i = 0; i < offsets.Count; i++)
            {
                int currentOffset = offsets[i];

                int nextOffset = (i + 1) < offsets.Count ? offsets[i + 1] : (int)br.BaseStream.Length;

                int stringLength = (nextOffset - currentOffset - 1) / 2;

                br.BaseStream.Position = currentOffset;

                var chars = br.ReadChars(stringLength);
                values.Add(currentOffset, new String(chars));
            }
            foreach (int offset in offsets)
            {
                listView1.Items.Add(offset.ToString("X")).SubItems.Add(values[offset]);
             listView1.Items[offset].SubItems[1].Text.Replace(System.Environment.NewLine, "\n");
            }

            br.Close();
            br = null;
        }
        ofd.Dispose();
        ofd = null;
    }
private void menuItem10_Click(object sender, EventArgs e)
    {
        BinaryWriter bw = new BinaryWriter(File.OpenWrite(path));

        bw.BaseStream.Position = 0x20;
        int number_pointers = Convert.ToInt32(num_pointers);
        Encoding enc = Encoding.Unicode;

        bw.Write(number_pointers);

        int curr_pointer = 4 + number_pointers * 4;
        for (int i = 0; i < number_pointers; i++)
        {
            bw.Write(curr_pointer);
            curr_pointer += enc.GetByteCount(listView1.Items[i].SubItems[1].Text) + 2;
        }

        for (int i = 0; i < number_pointers; i++)
            bw.Write(enc.GetBytes(listView1.Items[i].SubItems[1].Text + '\0'));

        bw.Flush();
        bw.Close();
        bw = null;
    }

谢谢，我为任何干扰道歉。 PS：请不要发布无用的评论和答案，例如＆＃34; seogle in google＆＃34;，＆＃34;我们已经回答了这些类型的问题＆＃34;等... 我花了5个小时在谷歌寻找答案，但没有出现，但使用ofd.close（），我也在stackoverflow网站搜索类似的问题，同样的答案，谢谢。

Answer 1

如果可能，应在using - 声明

中使用所有一次性物品

http://msdn.microsoft.com/en-US/library/yh598w02(v=vs.80).aspx

如果你把所有的一次性对象（BinaryReader，FileStream等）都放到这样的语句中，它就会起作用

Answer 2

可能的罪魁祸首是你自己的计划！

正如user287107所说，如果您处理文件的句柄，您应该可以再次打开它以便稍后写。

此外，sircapsalot有一个很好的建议：使用try {} finally {}确保一切都处理妥当。

Answer 3

由于您在此函数中没有异常处理，如果发生异常，它将不会继续，因此永远不会执行br.Close();函数。

您应该使用try { } finally { }来确保它已关闭。

private void Openbtn_Click(object sender, EventArgs e)
{
    textBox1.Text = "";
    listView1.Items.Clear();
    OpenFileDialog ofd = new OpenFileDialog();
    ofd.Title = "Open GSMB File";
    ofd.Filter = "GSMB Files (*.gsmb)|*.gsmb|All Files (*.*)|*.*";
    if (ofd.ShowDialog() == System.Windows.Forms.DialogResult.OK)
    {
        try {
            MessageBox.Show("File opened Succesfully!", "Info", MessageBoxButtons.OK, MessageBoxIcon.Information);
            path = ofd.FileName;
            BinaryReader br = new BinaryReader(File.OpenRead(path), Encoding.Unicode);
            BinaryReader brs = new BinaryReader(File.OpenRead(path), Encoding.Unicode);
            brs.BaseStream.Position = 0x4;
            menuItem9.Text = brs.ReadInt32().ToString();
            if (menuItem9.Text == "3620")
            {
                num_pointers = 204;
                menuItem8.Text = num_pointers.ToString();
            }
            else if (menuItem9.Text == "54662")
            {
                num_pointers = 2372;
                menuItem8.Text = num_pointers.ToString();
            }
            else if (menuItem9.Text == "9560")
            {
                num_pointers = 88;
                menuItem8.Text = num_pointers.ToString();
            }
            else if (menuItem9.Text == "1126")
            {
                num_pointers = 130;
                menuItem8.Text = num_pointers.ToString();
            }
            else if (menuItem9.Text == "342")
            {
                num_pointers = 16;
                menuItem8.Text = num_pointers.ToString();
            }
            else if (menuItem9.Text == "6232")
            {
                num_pointers = 467;
                menuItem8.Text = num_pointers.ToString();
            }
            else if (menuItem9.Text == "75698")
            {
                num_pointers = 498;
                menuItem8.Text = num_pointers.ToString();
            }
            else if (menuItem9.Text == "9914")
            {
                num_pointers = 110;
                menuItem8.Text = num_pointers.ToString();
            }
            else if (menuItem9.Text == "128")
            {
                num_pointers = 4;
                menuItem8.Text = num_pointers.ToString();
            }
            else if (menuItem9.Text == "5394")
            {
                num_pointers = 156;
                menuItem8.Text = num_pointers.ToString();
            }
            else if (menuItem9.Text == "12000")
            {
                num_pointers = 580;
                menuItem8.Text = num_pointers.ToString();
            }
            else 
            {
                MessageBox.Show("This is not a Pokémon Typing Adventure string file !", "ERROR", MessageBoxButtons.OK, MessageBoxIcon.Error);
                return;
            }
            List<int> offsets = new List<int>();
            int startstr = 0x1C;
            br.BaseStream.Position = startstr;
            int startstrval = br.ReadInt32();
            for (int i = 4; i < (num_pointers * 4 + 1); i += 4)
            {
                br.BaseStream.Position = startstr + i;
                offsets.Add(br.ReadInt32() + startstrval);
            }
            Dictionary<int, string> values = new Dictionary<int, string>();
            for (int i = 0; i < offsets.Count; i++)
            {
                int currentOffset = offsets[i];

                int nextOffset = (i + 1) < offsets.Count ? offsets[i + 1] : (int)br.BaseStream.Length;

                int stringLength = (nextOffset - currentOffset - 1) / 2;

                br.BaseStream.Position = currentOffset;

                var chars = br.ReadChars(stringLength);
                values.Add(currentOffset, new String(chars));
            }
            foreach (int offset in offsets)
            {
                listView1.Items.Add(offset.ToString("X")).SubItems.Add(values[offset]);
             listView1.Items[offset].SubItems[1].Text.Replace(System.Environment.NewLine, "\n");
            }               
        }
        finally {
            br.Close();
            br = null;
        }
    }
    ofd.Dispose();
    ofd = null;
}
private void menuItem10_Click(object sender, EventArgs e)
    {
    BinaryWriter bw;
    try {
        bw = new BinaryWriter(File.OpenWrite(path));

        bw.BaseStream.Position = 0x20;
        int number_pointers = Convert.ToInt32(num_pointers);
        Encoding enc = Encoding.Unicode;

        bw.Write(number_pointers);

        int curr_pointer = 4 + number_pointers * 4;
        for (int i = 0; i < number_pointers; i++)
        {
            bw.Write(curr_pointer);
            curr_pointer += enc.GetByteCount(listView1.Items[i].SubItems[1].Text) + 2;
        }

        for (int i = 0; i < number_pointers; i++)
            bw.Write(enc.GetBytes(listView1.Items[i].SubItems[1].Text + '\0'));
    }
    finally {
        bw.Flush();
        bw.Close();
        bw = null;
    }
}

我也不确定你是否应该在逻辑开头显示一个显示“文件成功打开”的msgbox。