Ошибка при двойном обращении к файлу

@vlad_kostashuk · Регистрация: 18.01.2015

Студворк — интернет-сервис помощи студентам

Здравствуйте, у меня была задача сделать программу которая при нажатии на button вставляет текст з textbox в txt файл и когда нажимаешь на button2 программа искала текст из textbox в файле, если нашла то удаляет строку. Но когда 2 программы использували 1 файл 1 из 2 программ просто вылетала з ошибкой. Помогите пожалуйста исправить код чтобы когда 2 программы использовали файл они работалы б без ошибок.

Код

C#
1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
17
18
19
20
21
22
23
24
25
        private void button1_Click_1(object sender, EventArgs e)
        {
            var text = paste.text;
            using (var writer = new StreamWriter(@"Whitelister/whitelist.txt", true))
            {
                writer.WriteLine(text);
            }
        }
 
        private void button2_Click(object sender, EventArgs e)
        {
            String findingString = String.Format(textBox1.text);
            List<String> newFile = new List<String>();
            String[] currentFile = File.ReadAllLines(@"Whitelister/whitelist.txt");
            String[] singleLine;
            foreach (String line in currentFile)
            {
                singleLine = line.Split(' ');
                if (singleLine[1] == findingString)
                    continue;
                newFile.Add(line);
            }
 
            File.WriteAllLines(@"Whitelister/whitelist.txt", newFile.ToArray());   
        }

Добавлено через 2 часа 4 минуты
Помогите пожалуйста.

Добавлено через 2 часа 24 минуты
Up!

Добавлено через 15 часов 51 минуту
Up!

@lKORENl · 01.07.2015, 15:11

Потоки закрывайте

C#
1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
17
18
19
20
21
22
23
24
25
26
27
    private void button1_Click_1(object sender, EventArgs e)
        {
            var text = paste.text;
            using (var writer = new StreamWriter(@"Whitelister/whitelist.txt", true))
            {
                writer.WriteLine(text);
                writer.Close();
            }
        }
 
        private void button2_Click(object sender, EventArgs e)
        {
            String findingString = String.Format(textBox1.text);
            List<String> newFile = new List<String>();
            String[] currentFile = File.ReadAllLines(@"Whitelister/whitelist.txt");
            String[] singleLine;
            foreach (String line in currentFile)
            {
                singleLine = line.Split(' ');
                if (singleLine[1] == findingString)
                    continue;
                newFile.Add(line);
            }
 
            File.WriteAllLines(@"Whitelister/whitelist.txt", newFile.ToArray());    
           
        }

@vlad_kostashuk · 02.07.2015, 16:08 **[ТС]**

lKORENl, Пробовал уже, если много запросов идет то опять ошибка.

Добавлено через 8 часов 3 минуты
lKORENl, Это можно исправить?

@Даценд · 03.07.2015, 17:51

Сообщение от vlad_kostashuk

при нажатии на button вставляет текст з textbox в txt файл и когда нажимаешь на button2 программа искала текст из textbox в файле

один и тот же текстбокс?

Сообщение от vlad_kostashuk

Но когда 2 программы использували 1 файл 1 из 2 программ просто вылетала з ошибкой

имеется в виду именно 2 разных программы или 2 метода одной программы?

@vlad_kostashuk · 03.07.2015, 23:50 **[ТС]**

Даценд, 2 разных программы

@vlad_kostashuk · 09.07.2015, 17:39 **[ТС]**

UP!

@vlad_kostashuk · 23.07.2015, 19:51 **[ТС]**

UP!

@OwenGlendower · 24.07.2015, 05:08

vlad_kostashuk, раз программы разные, то здесь необходима синхронизация между процессами. Например, с помощью Mutex. Добавь в оба приложения класс WhitelistFile который обеспечит поочередный доступ к файлу:

WhitelistFile

C#
1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
17
18
19
20
21
22
23
24
25
26
27
28
29
30
31
32
33
34
35
36
37
38
39
40
41
42
43
44
45
46
47
48
49
50
51
52
53
54
55
56
57
58
59
60
61
62
static class WhitelistFile
{
    static Lazy<Mutex> _fileMutex;
    const string FILE_NAME = @"Whitelister\whitelist.txt";
    
    public static string[] ReadAllLines()
    {
        Mutex mutex = _fileMutex.Value;
        mutex.WaitOne();
        try
        {
            return File.ReadAllLines(FILE_NAME);
        }
        finally
        {
            mutex.ReleaseMutex();
        }
    }
 
    public static void WriteAllLines(string[] lines)
    {
        Mutex mutex = _fileMutex.Value;
        mutex.WaitOne();
        try
        {
            File.WriteAllLines(FILE_NAME, lines);
        }
        finally
        {
            mutex.ReleaseMutex();
        }
    }
    
    public static void AppendString(string s)
    {
        Mutex mutex = _fileMutex.Value;
        mutex.WaitOne();
        try
        {
            File.AppendAllText(FILE_NAME, s);
        }
        finally
        {
            mutex.ReleaseMutex();
        }
    }
 
    static WhitelistFile()
    {
        _fileMutex = new Lazy<Mutex>(() => new Mutex(false, "kgjfkjgdjklfjlk"));
        AppDomain.CurrentDomain.ProcessExit += OnProcessExit;
    }
    
    static void OnProcessExit(object sender, EventArgs e)
    {
        if (_fileMutex.IsValueCreated)
        {
            Mutex mutex = _fileMutex.Value;
            mutex.Dispose();
        }
    }
}

Далее замени код работы с файлами на вызовы методов WhitelistFile:

C#
1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
17
18
19
20
21
22
23
24
25
26
27
28
private void button1_Click_1(object sender, EventArgs e)
{
    WhitelistFile.AppendString(paste.text);
}
 
private void button2_Click(object sender, EventArgs e)
{
    string stringToFind = textBox1.text;
    string[] allLines = WhitelistFile.ReadAllLines();
    List<string> newLines = new List<String>(allLines.Length);
    bool skipped = false;
    foreach (string line in allLines)
    {
        if (line.Split(' ')[1] == stringToFind)
        {
            skipped = true;
        }
        else
        {
            newLines.Add(line);
        }
    }
 
    if (skipped)
    {
        WhitelistFile.WriteAllLines(newLines.ToArray());
    }
}

Я немного переделал button2_Click:
- вызов String.Format(textBox1.text) не нужен
- в конструктор List<string> передается текущее кол-во строк в файле чтобы уменьшить кол-во выделений памяти
- добавлена переменная skipped чтобы запись в файл делалась только если его содержимое изменилось

@vlad_kostashuk · 29.07.2015, 18:01 **[ТС]**

OwenGlendower, А в второй программе файл читается так:

C#
1
2
3
4
5
6
7
8
9
10
StreamReader reader = new FileInfo("whitelist.txt").OpenText();
                        while ((str = reader.ReadLine()) != null)
                        {
                            if (str.Substring(0, 0x20).Equals(guid))
                            {
                                flag = true;
                            }
                        }
                        reader.Close();
                        reader.Dispose();

@OwenGlendower · 30.07.2015, 15:24

vlad_kostashuk, если файл whitelist.txt небольшой, то можно использовать WhitelistFile.ReadAllLines. В паре с LINQ код можно записать в одну строку:

C#
1
bool flag = WhitelistFile.ReadAllLines().Any(line => line.Substring(0, 0x20).Equals(guid));

Если файл большой, то можно создать наследника от StreamReader и использовать Mutex с тем же именем как в классе WhitelistFile. И далее использовать его.

Кстати, забыл в прошлый раз сказать - не забудь поменять имя Mutex в классе WhitelistFile (строка №50). Следует выбрать длинное и как можно более уникальное имя. Например, WhitelistMutex плюс Guid для гарантированной уникальности.

Сообщение от vlad_kostashuk

StreamReader reader = new FileInfo("whitelist.txt").OpenText();

Нет никакого смысла тратить память на экземпляр FileInfo:

C#
1
StreamReader reader = new StreamReader("whitelist.txt");

Сообщение от vlad_kostashuk

flag = true;

После этой строки есть смысл поставить break чтобы не делать лишних итераций.

Сообщение от vlad_kostashuk

reader.Close(); reader.Dispose();

Вызывать одновременно Close() и Dispose() нет смысла. Они делают одно и тоже в данном случае. Также советую использовать блок using который сам вызывает Dispose().

C#
1
2
3
4
5
6
7
8
9
10
using (StreamReader reader = new StreamReader("whitelist.txt"))
{
    while ((str = reader.ReadLine()) != null)
    {
        if (str.Substring(0, 0x20).Equals(guid))
        {
            flag = true; break;
        }
    }
}

Новые блоги и статьи Все статьи Все блоги /
Установка Android SDK, NDK, JDK, CMake и т.д. 8Observer8 25.01.2026 Содержание блога Перейдите по ссылке: https:/ / developer. android. com/ studio и в самом низу страницы кликните по архиву "commandlinetools-win-xxxxxx_latest. zip" Извлеките архив и вы увидите. . .	Вывод текста со шрифтом TTF на Android с помощью библиотеки SDL3_ttf 8Observer8 25.01.2026 Содержание блога Если у вас не установлены Android SDK, NDK, JDK, и т. д. то сделайте это по следующей инструкции: Установка Android SDK, NDK, JDK, CMake и т. д. Сборка примера Скачайте. . .	Использование SDL3-callbacks вместо функции main() на Android, Desktop и WebAssembly 8Observer8 24.01.2026 Содержание блога Если вы откроете примеры для начинающих на официальном репозитории SDL3 в папке: examples, то вы увидите, что все примеры используют следующие четыре обязательные функции, а. . .	моя боль iceja 24.01.2026 Выложила интерполяцию кубическими сплайнами www. iceja. net REST сервисы временно не работают, только через Web. Написала за 56 рабочих часов этот сайт с нуля. При помощи perplexity. ai PRO , при. . .
Модель сукцессии микоризы anaschu 24.01.2026 Решили писать научную статью с неким РОманом	http://iceja.net/ математические сервисы iceja 20.01.2026 Обновила свой сайт http:/ / iceja. net/ , приделала Fast Fourier Transform экстраполяцию сигналов. Однако предсказывает далеко не каждый сигнал (см ограничения http:/ / iceja. net/ fourier/ docs ). Также. . .	http://iceja.net/ сервер решения полиномов iceja 18.01.2026 Выкатила http:/ / iceja. net/ сервер решения полиномов (находит действительные корни полиномов методом Штурма). На сайте документация по API, но скажу прямо VPS слабенький и 200 000 полиномов. . .	Расчёт переходных процессов в цепи постоянного тока igorrr37 16.01.2026 / * Дана цепь(не выше 3-го порядка) постоянного тока с элементами R, L, C, k(ключ), U, E, J. Программа находит переходные токи и напряжения на элементах схемы классическим методом(1 и 2 з-ны. . .

@vlad_kostashuk 0 / 0 / 0 Регистрация: 18.01.2015 Сообщений: 58
	02.07.2015, 16:08 [ТС]
	lKORENl, Пробовал уже, если много запросов идет то опять ошибка. Добавлено через 8 часов 3 минуты lKORENl, Это можно исправить? 0

@vlad_kostashuk 0 / 0 / 0 Регистрация: 18.01.2015 Сообщений: 58
	03.07.2015, 23:50 [ТС]
	Даценд, 2 разных программы 0

@vlad_kostashuk 0 / 0 / 0 Регистрация: 18.01.2015 Сообщений: 58
	09.07.2015, 17:39 [ТС]
	UP! 0

@vlad_kostashuk 0 / 0 / 0 Регистрация: 18.01.2015 Сообщений: 58
	23.07.2015, 19:51 [ТС]
	UP! 0