Stream无法关闭C#

问题描述 投票:0回答:1

我正在创建一个程序,该程序使用密码并将密码应用到我已creative标记为PASSWORDFILE文件的文件上。我是一个自学成才的业余程序员,这是我第一次使用流=>对不起,我的代码不够干净。当我向文件添加密码时,该文件拒绝打开(给我一个“ System.IO.IOException:该进程无法访问文件'[此处的文件路径]',因为它正在被另一个进程使用。”)。我已确保关闭所有流,但此错误仍然存​​在。

进一步增加混乱:

namespace PasswordSaver
{
    [Serializable]
    class Password
    {
        public string ID;
        string baseWord;
        public Password(string password, string ID)
        {
            this.ID = ID;
            baseWord = password;
        }
        public virtual string GetPassword()
        {
            return baseWord;
        }
    }

    [Serializable]
    class EncodedPassword : Password
    {
        EncoderAndDecoder Encoder;

        public EncodedPassword(string decodedBasePassword, string ID) : base(decodedBasePassword, ID)
        {
            Encoder = new EncoderAndDecoder();
        }

        public override string GetPassword()
        {
            return Encoder.Encode(base.GetPassword(), out _);
        }
    }

    [Serializable]
    class EncodedPasswordWithAddendum : EncodedPassword
    {
        string addendum;
        public EncodedPasswordWithAddendum(string decodedBasePassword, string addendum, string ID) : base(decodedBasePassword, ID)
        {
            this.addendum = addendum;
        }
        public override string GetPassword()
        {
            return base.GetPassword() + addendum;
        }
    }
}

仅当我尝试添加EncodedPasswordEncodedPasswordWithAddendum实例而不是Password实例时,才会发生错误。我的写作代码是

namespace PasswordSaver
{
    class PasswordWriter
    {
        public readonly string saveFilePath;
        static string directory = Directory.GetCurrentDirectory();

        #region Constructors
        public PasswordWriter()
        {
            saveFilePath = directory + @"\PasswordSaver"
                + ".passwordfile";
        }
        public PasswordWriter(string saveFilePath)
        {
            this.saveFilePath = saveFilePath;
        }
        #endregion

        #region Individual Writing Functions
        private void WriteBinary(object objectToEncode)
        {
            WriteBinary(objectToEncode, out _);
        }
        private void WriteBinary(object objectToEncode, out Exception exception)
        {
            exception = null;
            try
            {
                IFormatter binaryFormatter = new BinaryFormatter();

                Stream fileStream = new FileStream(saveFilePath, FileMode.OpenOrCreate, FileAccess.ReadWrite);
                Stream memoryStream = new MemoryStream();

                memoryStream.Position = memoryStream.Length;
                binaryFormatter.Serialize(memoryStream, objectToEncode);

                EncodeFromStream(ref memoryStream, ref fileStream);


                fileStream.Close();
                memoryStream.Close();
            }
            catch (Exception e)
            {
                exception = e;
            }
        }
        #endregion

        #region File Read and Writing
        public void WriteFile(Password[] passwords)
        {
            if (File.Exists(saveFilePath))
            {
                Stream stream = new FileStream(saveFilePath, FileMode.Truncate, FileAccess.Write);
                stream.Close();
            }

            WriteBinary(passwords.Length);
            foreach (Password password in passwords)
            {
                WriteBinary(password);
            }
        }

        public void WriteToFile(Password password)
        {
            Password[] oldPasswords = ReadFile();
            Password[] passwords = new Password[oldPasswords.Length + 1];
            for (int i = 0; i < oldPasswords.Length; i++)
            {
                passwords[i] = oldPasswords[i];
            }
            passwords[oldPasswords.Length] = password;
            WriteFile(passwords);
        }

        public bool ReplacePassword(string oldPasswordID, Password newPassword)
        {
            Password[] passwords = ReadFile();
            for (int i = 0; i < passwords.Length; i++)
            {
                if (passwords[i].ID == oldPasswordID)
                {
                    passwords[i] = newPassword;
                    return true;
                }
            }
            return false;
        }

        public Password[] ReadFile()
        {
            Stream fileStream = new FileStream(saveFilePath, FileMode.OpenOrCreate, FileAccess.Read);
            IFormatter binaryFormatter = new BinaryFormatter();
            Stream memoryStream = new MemoryStream();

            DecodeFromStream(ref fileStream, ref memoryStream);

            fileStream.Close();

            memoryStream.Position = 0;

            int length = (int) binaryFormatter.Deserialize(memoryStream);
            //Console.WriteLine(length + " is the length");//debug
            Password[] passwords = new Password[length];
            for (int i = 0; i < length; i++)
            {
                //Console.WriteLine(memoryStream.Position + " " + memoryStream.Length);//debug
                //Console.WriteLine(i);//debug
                passwords[i] = (Password)binaryFormatter.Deserialize(memoryStream);
            }

            memoryStream.Close();
            return passwords;
        }
        #endregion

        #region Encode and Decode
        private void EncodeFromStream(ref Stream stream, ref Stream newStream)
        {

            stream.Position = 0;
            newStream.Position = newStream.Length;


            for (int i = 0; i < stream.Length; i++)
            {
                int integer = stream.ReadByte();
                byte originalByte = (byte)integer;// get a byte off of the line
                //Encode byte here
                newStream.WriteByte(setOfBits1);
                newStream.WriteByte(setOfBits2);
            }
        }

        private void DecodeFromStream(ref Stream stream, ref Stream newStream)
        {
            newStream.Position = newStream.Length;

            stream.Position = 0;
            for (int i = 0; i < (stream.Length / 2); i++)// stream.Length / 2 because the program reads two bytes per iteration of the for loop
            {
                //I decode the bytes here
                newStream.WriteByte(originalByte);
            }

        }
        #endregion
        public void WriteContentsToFile()
        {
            Stream stream = new FileStream(saveFilePath + "1", FileMode.OpenOrCreate, FileAccess.ReadWrite);
            Stream stream1 = new FileStream(saveFilePath, FileMode.OpenOrCreate, FileAccess.ReadWrite);
            this.DecodeFromStream(ref stream1, ref stream);
            stream.Close();
            stream1.Close();
        }
    }
}

我已经删除了对EncodeFromStreamDecodeFromStream中的流进行编码和解码的代码。new FileStream(saveFilePath + "1", FileMode.OpenOrCreate, FileAccess.ReadWrite)的任何出现都是我以解码格式写入单独文件的地方。为了区分这两个文件,我将文件类型从PASSWORDFILE更改为PASSWORDFILE1。

结论:我将WriteFileWriteToFile方法与包含Password[]EncodedPasswordEncodedPasswordWithAddendum结合使用。然后,当我尝试通过FileStream(通常通过方法ReadFile)打开文件时,出现异常“ System.IO.IOException:该进程无法访问文件'[此处的文件路径]',因为由另一个进程使用”。

谢谢您的帮助。

c# file filestream memorystream
1个回答
1
投票

流通常包含不需要的资源(操作系统文件句柄),因此它们实现IDisposeable。

虽然您始终可以确定GC将清理可处置的东西最终(在应用程序关闭时最新),通常这很晚。您必须明确地做到这一点。为此,我有一个关于IDisposeable的规则:

“永远不要拆分可处置资源的创建和处置。创建,使用,处置。所有都在同一段代码中,最好使用using块。”我遇到的唯一例外是日志文件。 remote别无其他值得将可处置的东西打开的麻烦和头痛。尤其不是性能。

由于using block使用try ...最后,您可以确定它会运行。编译器和运行时使某些finally块始终运行,即使在函数返回,通过goto跳转或Exception情况跳转时也是如此。

© www.soinside.com 2019 - 2024. All rights reserved.