Поток не может закрыться C# - PullRequest
0 голосов
/ 16 марта 2020

Я создаю программу, которая принимает пароли и применяет к ним кодировку для файла, который я творчески помечал как файл 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;
        }
    }
}

ошибка возникает, только когда я пытаюсь добавить экземпляры EncodedPassword или EncodedPasswordWithAddendum, но не экземпляр 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();
        }
    }
}

Я удалил код, который кодировал и декодировал потоки в EncodeFromStream и DecodeFromStream. любое вхождение new FileStream(saveFilePath + "1", FileMode.OpenOrCreate, FileAccess.ReadWrite) - это то, где я записывал отдельный файл в декодированном формате. Чтобы различать guish два файла, я изменил тип файла с PASSWORDFILE на PASSWORDFILE1.

В заключение: я использую методы WriteFile или WriteToFile с Password[] который содержит EncodedPassword или EncodedPasswordWithAddendum. затем, когда я пытаюсь открыть файл с помощью FileStream (обычно с помощью метода ReadFile), я получаю исключение «System.IO.IOException: процесс не может получить доступ к файлу« [путь к файлу здесь] », потому что он используется другим процессом ".

Спасибо за помощь.

1 Ответ

1 голос
/ 16 марта 2020

Потоки обычно содержат неуправляемые ресурсы (дескрипторы файлов ОС), поэтому они реализуют IDisposeable.

Хотя вы всегда можете быть уверены, что G C очистит одноразовые вещи со временем ( последний на закрытие приложения), как правило, это поздно. Вы должны сделать это явно. И для этого у меня есть одно правило, касающееся IDisposeable вещи:

"Никогда не разделяйте создание и удаление одноразового ресурса. Create. Use. Dispose. Все в одном куске кода, в идеале, используя блок using. «. Единственное исключение, которое я когда-либо встречал в лог-файлах. Ничто иное не стоит удаленно , что стоит того, чтобы держать что-то одноразовое открытым. Особенно это касается производительности.

Поскольку , использующий блок , использует попытку ... наконец, вы можете быть уверены, что она запустится. Компилятор и среда выполнения гарантируют, что блоки finally всегда работают, даже при возврате функции, переходе через goto или в исключительных случаях.

Добро пожаловать на сайт PullRequest, где вы можете задавать вопросы и получать ответы от других членов сообщества.
...