Файл используется другим процессом(И замечания по коду)

@Felix Felicis · Регистрация: 31.05.2015

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

Доброго времени суток!
Есть такой код

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
63
64
65
66
67
68
69
70
71
72
73
74
75
76
77
78
79
80
81
82
83
84
85
86
87
88
89
90
91
92
93
94
95
96
97
98
99
100
101
102
103
104
105
106
107
108
109
110
111
112
113
114
115
116
117
118
119
120
121
122
123
124
125
126
127
128
129
130
131
132
133
134
135
136
137
138
139
140
141
142
143
144
145
146
147
148
149
150
151
152
153
154
155
156
157
158
159
160
161
162
163
164
165
166
167
168
169
170
171
172
173
174
175
176
177
178
179
180
181
182
183
184
185
186
187
188
189
190
191
192
193
        private Thread work;
        
        public Form1()
        {
            InitializeComponent();
            this.KeyPreview = true;
            
 
        }
        
AutoItX3 autoIt = new AutoItX3();
        int aBitmap = 0; // счетчик атакующих действий
        public bool IndefMap = false; //Определена ли карта
        public bool cancel = false; // остановка потока
        public int iMap = 0; //счетчик карт
 
        string[] AttackBitmap = { "D:/Sprite/Меч.bmp",
            "D:/Sprite/Меч-Бой.bmp",
           // "D:/Sprite/МагияЩит.bmp",
            //"D:/Sprite/МагияАрес.bmp",
            //"D:/Sprite/ТочкаМагии.bmp",
            "D:/Sprite/ВыборГероя.bmp",
            "D:/Sprite/ТочкаотправкиStandart.bmp",
            "D:/Sprite/МалоХП.bmp",
            "D:/Sprite/Завершитьбой.bmp",
            "D:/Sprite/Выход.bmp",
            "D:/Sprite/Магия.bmp",
            "D:/Sprite/Пополнить.bmp",
            "D:/Sprite/Купить.bmp", "D:/Sprite/Exit.bmp"};
        string[] MapBitmap =
        {  "D:/Sprite/ТочкаотправкиStandart.bmp"
 
 
 
        };
        string BmpScreen = "D:/Boo/filename.bmp";
         void ScreenS()
        {
            
 
            this.Invoke((MethodInvoker)delegate
            {
                richTextBox1.AppendText("Делаю скриншот\n");
                richTextBox1.Focus();
            });
 
            Graphics graph = null;
            Bitmap bmp = new Bitmap(Screen.PrimaryScreen.Bounds.Width, Screen.PrimaryScreen.Bounds.Height);
            graph = Graphics.FromImage(bmp);
            graph.CopyFromScreen(0, 0, 0, 0, bmp.Size);
            
            File.Delete(BmpScreen);
 
            bmp.Save(BmpScreen);
            GC.Collect();
           
 
        }
      
        void Search()
        {
            while (!cancel)
            {
                //int aBitmap = 0 ;
            
                ScreenS();
               
                Bitmap bitmap1 = new Bitmap(BmpScreen);
                Bitmap bitmap2 = new Bitmap((AttackBitmap[aBitmap]));
                 Bitmap bitmap3 = new Bitmap((MapBitmap[iMap]));
                if (bitmap1.Width > bitmap2.Width || bitmap1.Height > bitmap2.Height)
                {
                    Bitmap aux = bitmap2;
                    bitmap2 = bitmap1;
                    bitmap1 = aux;
                }
                Rectangle location = Rectangle.Empty;
                location = autoSearchBitmap(bitmap1, bitmap2);
 
                if (location.Width != 0)
                {
                    
                    this.Invoke((MethodInvoker)delegate
                    {
                        richTextBox1.AppendText(aBitmap + " Find\n");
                    });
 
                    //if (aBitmap <6) aBitmap = aBitmap + 1;
                    //else aBitmap = 0;
                    switch(aBitmap)
                    {
                        case 0: autoIt.MouseClick("Left", location.X, location.Y, 1, 1);
                            aBitmap++;
                            Thread.Sleep(1000); break;
                        case 1: autoIt.MouseClick("Left", location.X, location.Y, 1, 2);
                            Thread.Sleep(6000);
                            autoIt.MouseMove(location.X + 400, location.Y - 100, 10);
                            autoIt.Send("{LCTRL down}");
                            autoIt.MouseWheel("down", 10);
                            Thread.Sleep(400);
                            autoIt.Send("{LCTRL 1}"); //?
                            autoIt.Send("{LCTRL up}");
                            this.Invoke((MethodInvoker)delegate
                            {
                                richTextBox1.AppendText("Определение карты\n");
 
                            });
                            while (!IndefMap)
                            {
                                ScreenS();
 
                                location = autoSearchBitmap(bitmap1, bitmap3);
                                if (location.Width != 0)
                                {
                                    IndefMap = true;
                                    this.Invoke((MethodInvoker)delegate
                                    {
                                        richTextBox1.AppendText("Карта стандартная\n");
                                    });
                                }
                                else this.Invoke((MethodInvoker)delegate
                                {
                                    richTextBox1.AppendText("Карта не найдена\n");
                                });
                            }
                            IndefMap = false;
                            aBitmap++;
                            break;
                        case 2:
 
                            autoIt.MouseClick("Left", location.X, location.Y, 6, 2); aBitmap++;
                            break;
                        
                        case 3:
                            autoIt.MouseClick("Left", location.X, location.Y, 6, 0); aBitmap++;
                            Thread.Sleep(0); break;
                        case 4:
                            autoIt.MouseClick("Left", location.X, location.Y, 1, 0); aBitmap++;
                            Thread.Sleep(0); break;
                        case 5:
                            autoIt.MouseClick("Left", location.X, location.Y, 1, 0); aBitmap++; break;
                        case 6:
                            autoIt.MouseClick("Left", location.X, location.Y, 1, 0); aBitmap = 0 ;
                            Thread.Sleep(0); break;
                            
 
                    }
                    //autoIt.MouseClick("Left", location.X, location.Y, 1, 0);
 
                      Thread.Sleep(0);
                }
 
                else
                {
                    this.Invoke((MethodInvoker)delegate
                    {
                        richTextBox1.AppendText("Miss\n");
                        
                            richTextBox1.AppendText("Пробую найти\n");
                            location = autoSearchBitmap(bitmap1, bitmap2);
                        
                    });
                    
 
 
 
                } 
               
 
 
                bitmap1.Dispose();
                bitmap2.Dispose();
                //this.Cursor = Cursors.Default;
                Thread.Sleep(0);
                
            }
           
        }
        
 
        private void button2_Click(object sender, EventArgs e)  
        {
            cancel = false;
            work = new Thread(Search);
            work.Start();
 
            this.Invoke((MethodInvoker)delegate
            {
                richTextBox1.AppendText("Поток запущен\n");
                
            });
 
        }

Проблема в ScreenS(). Когда доходит до

C#
1
2
3
4
5
case 1: ...
 
 ScreenS();
...
                            {

То ругается на File.Delete(BmpScreen);. Говорит, что файл занят другим процессом. Пробовал несколько варинтов, но ничего не вышло.

Суть кода в том, что создается скрин экрана, сохраняется, потом в нем ищется нужная часть изображения, делается с этими координатами некие манипуляции, потом делает снова скрин, удаляет старый и сохраняет на его место новый и все по кругу.

Так же, если есть какие-то замечания по коду - буду только рад его улучшить

@Кузнец кода · 24.02.2018, 09:48

Попробуйте вот так

C#
1
2
3
4
5
6
7
8
9
using (FileStream stream = new FileStream("test.jpg", FileMode.Open, FileAccess.Read))
{
    pictureBox1.Image = Image.FromStream(stream);
     stream.Dispose();
}
 
 // delete your file.
 
 File.Delete(delpath);

Или это может быть проблема с сборкой мусора

C#
1
2
3
System.GC.Collect(); 
System.GC.WaitForPendingFinalizers(); 
File.Delete(picturePath);

@Felix Felicis · 24.02.2018, 23:44 **[ТС]**

Сообщение от Кузнец кода

Попробуйте вот так

C#
1
2
3
4
5
6
7
8
9
using (FileStream stream = new FileStream("test.jpg", FileMode.Open, FileAccess.Read))
{
    pictureBox1.Image = Image.FromStream(stream);
     stream.Dispose();
}
 
 // delete your file.
 
 File.Delete(delpath);

Или это может быть проблема с сборкой мусора

C#
1
2
3
System.GC.Collect(); 
System.GC.WaitForPendingFinalizers(); 
File.Delete(picturePath);

У меня нет этого элемента в форме

C#
1
pictureBox1.Image = Image.FromStream(stream);

@Diamante · 25.02.2018, 01:09

Кузнец кода,

C#
1
.Dispose()

лишний, using (SomeThing) для того и существует, что все закрывает и диспозит
Felix Felicis,

@Кузнец кода · 25.02.2018, 08:12

Верно говоришь, привычка дурная.

@Usaga · 25.02.2018, 08:24

Кузнец кода, добавляя в код строки вида

C#
1
System.GC.Collect();

... не забудьте добавить и приставку "говно-" к слову "код". Ведь в 99% случаев, инициировать сборку мусора в ручную - признак того, что в коде происходит хрень.

@Кузнец кода · 25.02.2018, 08:27

Вот главный тролль форума появился. Вы предлагайте и помогайте людям, а не тролльте их наработки.

@Usaga · 25.02.2018, 08:30

Кузнец кода, молодой человек, я вам прямо указал на хреновость подхода вами предложенного. В каком месте вы троллинг увидали? Или у вас всё - троллинг?

Новые блоги и статьи Все статьи Все блоги /
Использование SDL3-callbacks вместо функции main() на Android, Desktop и WebAssembly 8Observer8 24.01.2026 Если вы откроете примеры для начинающих на официальном репозитории SDL3 в папке: examples, то вы увидите, что все примеры используют следующие четыре обязательные функции, а привычная функция main(). . .	моя боль 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 з-ны. . .	Восстановить юзерскрипты Greasemonkey из бэкапа браузера damix 15.01.2026 Если восстановить из бэкапа профиль Firefox после переустановки винды, то список юзерскриптов в Greasemonkey будет пустым. Но восстановить их можно так. Для этого понадобится консольная утилита. . .	Сукцессия микоризы: основная теория в виде двух уравнений. anaschu 11.01.2026 https:/ / rutube. ru/ video/ 7a537f578d808e67a3c6fd818a44a5c4/

@Кузнец кода 169 / 188 / 35 Регистрация: 21.01.2018 Сообщений: 230
	25.02.2018, 08:12
	Верно говоришь, привычка дурная. 0

@Кузнец кода 169 / 188 / 35 Регистрация: 21.01.2018 Сообщений: 230
	25.02.2018, 08:27
	Вот главный тролль форума появился. Вы предлагайте и помогайте людям, а не тролльте их наработки. 0

@Usaga 14127 / 9347 / 1350 Регистрация: 21.01.2016 Сообщений: 35,125
	25.02.2018, 08:30
	Кузнец кода, молодой человек, я вам прямо указал на хреновость подхода вами предложенного. В каком месте вы троллинг увидали? Или у вас всё - троллинг? 0