Удаление файло :o

@Flyer · Регистрация: 21.05.2010

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

Печальная проблема у меня.
Удаляю 3 файла на каждой итерации цикла

C++
1
2
3
4
5
6
        UnicodeString temp2 = path + "Temp.264";
        UnicodeString temp3 = path + "Temp.aac";
        UnicodeString temp1 = path + "Temp.avi";
        DeleteFile(temp1.w_str());
        DeleteFile(temp2.w_str()) ;
        DeleteFile(temp3.w_str());

Проблема в том, что какого то неизвестного мне фака, иногда допустим в удаление файла на 5 итерации, попадает переменная path с 6 итерации -> файл не удаляется, так как его там еще попросту нету. Собственно вопрос, как подождать пока файл будет удален и тогда аж переходить на следующую итерацию?

cpp_developer · 10.10.2011, 23:16

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

это только намёк на твой цикл.

там могло быть просто

C++
1
for(/*...*/)

не придирайся

и такой цикл тебе никогда не попадался ?

@Flyer · 10.10.2011, 23:27 **[ТС]**

с 2 ;; попадался, а вот с 3 ; с моим небольшим/отсутствующим опытом нет

@Programmer. · 11.10.2011, 03:04

Flyer,

Да отстан ты от

C++
1
for(;;){}

ето бил толко пример, там дожен быть твой цикл, на пример

C++
1
for(int Start=0;Start<=10;Start++){}

@Flyer · 12.10.2011, 13:29 **[ТС]**

C++
1
2
        while (FileExists(path + "Temp.avi")) {
            DeleteFile((path + "Temp.avi").w_str());

сделал так, но чувствую это не самый хороший и правильный вариант

cpp_developer · 12.10.2011, 13:34

проверка существования файла - это хороший и правильный вариант.

@Flyer · 12.10.2011, 13:35 **[ТС]**

я имею ввиду вайл

хотя хз, может мне просто кажется, что не стоит так делать

@Programmer. · 12.10.2011, 14:00

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

while (FileExists(path + "Temp.avi")) {
DeleteFile((path + "Temp.avi").w_str());

Файл может быть ReadOnly, и в етом случай он канечно не удалится...

@Flyer · 12.10.2011, 15:39 **[ТС]**

не будет этот файл рид онли

@Flyer · 14.10.2011, 09:50 **[ТС]**

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

cpp_developer · 14.10.2011, 09:51

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
AnsiString temp[124];
temp[0] = "Y:\\wtf\\temp.avi";
temp[1] = "Y:\\wtf\\temp.aac";
temp[2] = "Y:\\wtf\\temp.264";
//
temp[3]="Y:\\wtf2\\temp.avi";
temp[4]="Y:\\wtf2\\temp.aac";
temp[5]="Y:\\wtf2\\temp.264";
//
temp[6]="Y:\\wtf3\\temp.avi";
temp[7]="Y:\\wtf3\\temp.aac";
temp[8]="Y=:\wtf3\\temp.264";
//
temp[9]="Y:\\wtf4\\temp.avi";
temp[10]="Y:\\wtf4\\temp.aac";
temp[11]="Y:\\wtf4\\temp.264";
 
int i,cnt
i = 0;
cnt = 12;
for (i;i<=cnt;i++) {
  if (FileExists(temp[i])) {
    DeteFile(temp[i]);
  }
  else {
    ShowMessage("Файл \""+temp[i]+"\" отсутствует.")
  }
}

или

C++
1
2
  else
    continue;

схема, не для копипейста, не проверял
консультации и техническая поддержка не предоставляются

mimicria · 14.10.2011, 09:58

Может имеет смысл сделать наоборот?

C++
1
2
3
DeleteFile((path + "Temp.avi").w_str());
while (FileExists(path + "Temp.avi")) 
{sleep 10; } // слип к примеру =)

Добавлено через 5 минут
А не, прочитал тему ещё раз от начала до конца повнимательней, ошибка то в получении path а не удалении. Может выложите лучше этот кусок?

@Flyer · 14.10.2011, 10:02 **[ТС]**

ужас, я знаю. распихание всего этого счастья по функция в ближайших планах, но пока как есть. это весь обработчик батона, от начала до конца

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
194
195
    working = true;
    Button3->Enabled = true;
    Button2->Enabled = false;
    for (i = offset; i < StringGrid1->RowCount; i++) {
 
        ext = ExtractFileExt(StringGrid1->Cells[0][i]);
        if (ext == ".mkv") {
            demuxer = " -demuxer mkv";
        }
        else {
            demuxer = "";
        }
        first_cmd = (" " + StringGrid1->Cells[3][i] + demuxer + " -o \"" +
            StringGrid1->Cells[2][i] +
            "Temp.avi\" -vf scale=480:272,harddup -af channels=2 -ofps 25 -srate 44100 -oac faac -faacopts br=128:mpeg=4:object=2 -ovc x264 -ffourcc avc1 -x264encopts bitrate=410:me=umh:level_idc=13:keyint=250:frameref=1:bframes=0:threads=auto:profile=main:rc_lookahead=60:crf=21 -aspect 16:9 -mc 3 -subfont C://Windows//Fonts//arialbd.ttf -subpos 100 -subcp cpUnicode -subfont-text-scale 3");
 
        path = StringGrid1->Cells[2][i];
        cmd2 = " \"" + path +
            "Temp.avi\" -ovc copy -nosound  -of rawvideo -o \"" + path +
            "Temp.264\"";
        cmd3 = " \"" + path +
            "Temp.avi\" -ovc frameno -oac copy -of rawaudio -o \"" + path +
            "Temp.aac\"";
        cmd4 = " -fps 25.000 -add \"" + path + "Temp.264\" -add \"" + path +
            "Temp.aac\" -new \"" + StringGrid1->Cells[4][i] + "\"";
 
        if (DirectoryExists(path.w_str()) == false) {
            CreateDirectory(path.w_str(), NULL);
        }
 
        unsigned long exit = 0;
 
        enc(first_cmd, i);
        ///////////////////////////////////////
        if (stop) {
            TerminateProcess(pi.hProcess, 0);
            WaitForSingleObject(pi.hProcess, INFINITE);
            CloseHandle(pi.hThread);
            CloseHandle(pi.hProcess);
            CloseHandle(newstdout);
            CloseHandle(read_stdout);
            offset = i;
            temp_file = path + "Temp.avi";
            DeleteFile(temp_file.w_str());
            break;
        }
        ///////////////////////////////////////
        ((TGauge*)StringGrid1->Objects[1][i])->Progress = 100;
        WaitForSingleObject(pi.hProcess, INFINITE);
        CloseHandle(pi.hThread);
        CloseHandle(pi.hProcess);
        CloseHandle(newstdout);
        CloseHandle(read_stdout);
        // -----------------------------------------------------------------------
 
        GetStartupInfo(&si);
        si.dwFlags = STARTF_USESHOWWINDOW;
        si.wShowWindow = SW_HIDE;
 
        CreateProcess(L"mencoder.exe", cmd2.w_str(), 0, 0, 1,
            CREATE_NEW_CONSOLE, 0, 0, &si, &pi);
 
        for (; ;) {
 
            GetExitCodeProcess(pi.hProcess, &exit);
            Application->ProcessMessages();
            if (exit != STILL_ACTIVE)
                break;
 
            ///////////////////////////////////
            if (stop) {
                ((TGauge*)StringGrid1->Objects[1][i])->Progress = 0;
                break;
            }
            ///////////////////////////////////
        }
 
        ///////////////////////////////////////
        if (stop) {
            TerminateProcess(pi.hProcess, 0);
            WaitForSingleObject(pi.hProcess, INFINITE);
            CloseHandle(pi.hThread);
            CloseHandle(pi.hProcess);
            temp_file = path + "Temp.h264";
            DeleteFile(temp_file.w_str());
            break;
        }
        ///////////////////////////////////////
        ((TGauge*)StringGrid1->Objects[1][i])->Progress = 102;
        WaitForSingleObject(pi.hProcess, INFINITE);
        CloseHandle(pi.hThread);
        CloseHandle(pi.hProcess);
 
        // ----------------------------------------------------------
 
        GetStartupInfo(&si);
        si.dwFlags = STARTF_USESTDHANDLES | STARTF_USESHOWWINDOW;
        si.wShowWindow = SW_HIDE;
 
        CreateProcess(L"mencoder.exe", cmd3.w_str(), 0, 0, 1,
            CREATE_NEW_CONSOLE, 0, 0, &si, &pi);
 
        for (; ;) {
            GetExitCodeProcess(pi.hProcess, &exit);
            Application->ProcessMessages();
            if (exit != STILL_ACTIVE)
                break;
 
            ///////////////////////////////////
            if (stop) {
                ((TGauge*)StringGrid1->Objects[1][i])->Progress = 0;
                break;
            }
            ///////////////////////////////////
        }
 
        ///////////////////////////////////////
        if (stop) {
            TerminateProcess(pi.hProcess, 0);
            WaitForSingleObject(pi.hProcess, INFINITE);
            CloseHandle(pi.hThread);
            CloseHandle(pi.hProcess);
            stop = false;
            working = false;
            offset = i;
            temp_file = path + "Temp.aac";
            DeleteFile(temp_file.w_str());
            break;
        }
        ///////////////////////////////////////
        ((TGauge*)StringGrid1->Objects[1][i])->Progress = 104;
        WaitForSingleObject(pi.hProcess, INFINITE);
        CloseHandle(pi.hThread);
        CloseHandle(pi.hProcess);
 
        // ----------------------------------------------------------
 
        GetStartupInfo(&si);
        si.dwFlags = STARTF_USESTDHANDLES | STARTF_USESHOWWINDOW;
        si.wShowWindow = SW_HIDE;
        CreateProcess(L"MP4Box.exe", cmd4.w_str(), 0, 0, 1, CREATE_NEW_CONSOLE,
            0, 0, &si, &pi);
 
        for (; ;) {
            GetExitCodeProcess(pi.hProcess, &exit);
            Application->ProcessMessages();
            if (exit != STILL_ACTIVE)
                break;
            ///////////////////////////////////
            if (stop) {
                ((TGauge*)StringGrid1->Objects[1][i])->Progress = 0;
                break;
            }
            ///////////////////////////////////
        }
        ///////////////////////////////////////
        if (stop) {
            TerminateProcess(pi.hProcess, 0);
 
            CloseHandle(pi.hThread);
            CloseHandle(pi.hProcess);
            WaitForSingleObject(pi.hProcess, INFINITE);
 
            break;
        }
        ///////////////////////////////////////
 
        WaitForSingleObject(pi.hProcess, INFINITE);
        CloseHandle(pi.hThread);
        CloseHandle(pi.hProcess);
 
        while (FileExists(path + "Temp.avi")) {
            DeleteFile((path + "Temp.avi").w_str());
        }
        while (FileExists(path + "Temp.aac")) {
            DeleteFile((path + "Temp.aac").w_str());
        }
        while (FileExists(path + "Temp.264")) {
            DeleteFile((path + "Temp.264").w_str());
        }
        ((TGauge*)StringGrid1->Objects[1][i])->Progress = 105;
 
    }
    if (!stop) {
        offset = StringGrid1->RowCount;
        Button2->Enabled = true;
        Button3->Enabled = false;
 
    }
    else {
        stop = false;
        working = false;
        offset = i;
    }
    Complete_notification_form->Show();

mimicria · 14.10.2011, 10:25

Ну как предложение вывести path (или трассировкой) сразу после чтения из StringGrid и перед удалением, так можно выяснить где сбой происходит.

Новые блоги и статьи Все статьи Все блоги /
Модульная разработка через nuget packages DevAlt 07.03.2026 Сложившийся в . Net-среде способ разработки чаще всего предполагает монорепозиторий в котором находятся все исходники. При создании нового решения, мы просто добавляем нужные проекты и имеем. . .	Модульный подход на примере F# DevAlt 06.03.2026 В блоге дяди Боба наткнулся на такое определение: В этой книге («Подход, основанный на вариантах использования») Ивар утверждает, что архитектура программного обеспечения — это структуры,. . .	Управление камерой с помощью скрипта OrbitControls.js на Three.js: Вращение, зум и панорамирование 8Observer8 05.03.2026 Содержание блога Финальная демка в браузере работает на Desktop и мобильных браузерах. Итоговый код: orbit-controls-threejs-js. zip. Сканируйте QR-код на мобильном. Вращайте камеру одним пальцем,. . .	SDL3 для Web (WebAssembly): Синхронизация спрайтов SDL3 и тел Box2D 8Observer8 04.03.2026 Содержание блога Финальная демка в браузере. Итоговый код: finish-sync-physics-sprites-sdl3-c. zip На первой гифке отладочные линии отключены, а на второй включены:. . .
SDL3 для Web (WebAssembly): Идентификация объектов на Box2D v3 - использование userData и событий коллизий 8Observer8 02.03.2026 Содержание блога Финальная демка в браузере. Итоговый код: finish-collision-events-sdl3-c. zip Сканируйте QR-код на мобильном и вы увидите, что появится джойстик для управления главным героем. . . .	Реалии Hrethgir 01.03.2026 Нет, я не закончил до сих пор симулятор. Эта задача сложнее. Не получилось уйти в плавсостав, но оно и к лучшему, возможно. Точнее получалось - но сварщиком в палубную команду, а это значит, в моём. . .	Ритм жизни kumehtar 27.02.2026 Иногда приходится жить в ритме, где дел становится всё больше, а вовлечения в происходящее — всё меньше. Плотный график не даёт вниманию закрепиться ни на одном событии. Утро начинается с быстрых,. . .	SDL3 для Web (WebAssembly): Сборка библиотек: SDL3, Box2D, FreeType, SDL3_ttf, SDL3_mixer и SDL3_image из исходников с помощью CMake и Emscripten 8Observer8 27.02.2026 Недавно вышла версия 3. 4. 2 библиотеки SDL3. На странице официальной релиза доступны исходники, готовые DLL (для x86, x64, arm64), а также библиотеки для разработки под Android, MinGW и Visual Studio. . . .

@Flyer wtf!? 750 / 281 / 25 Регистрация: 21.05.2010 Сообщений: 732
	10.10.2011, 23:27 [ТС]
	с 2 ;; попадался, а вот с 3 ; с моим небольшим/отсутствующим опытом нет 0

cpp_developer 20124 / 5691 / 417 Регистрация: 09.04.2010 Сообщений: 22,546 Записей в блоге: 1
	12.10.2011, 13:34
	проверка существования файла - это хороший и правильный вариант. 0

@Flyer wtf!? 750 / 281 / 25 Регистрация: 21.05.2010 Сообщений: 732
	12.10.2011, 13:35 [ТС]
	я имею ввиду вайл хотя хз, может мне просто кажется, что не стоит так делать 0

@Flyer wtf!? 750 / 281 / 25 Регистрация: 21.05.2010 Сообщений: 732
	12.10.2011, 15:39 [ТС]
	не будет этот файл рид онли 0

@Flyer wtf!? 750 / 281 / 25 Регистрация: 21.05.2010 Сообщений: 732
	14.10.2011, 09:50 [ТС]
	тю, а все равно иногда не удаляется, т.к. проскакивает path со следующей итерации как вариант думаю сделать, чтобы файлы хранились в одной и той же папке и юзать функцию gettempname. Тогда путь будет один и тот же да и конфликтов в именах не должно бы быть 0

mimicria return (true); 1977 / 1112 / 221 Регистрация: 19.04.2011 Сообщений: 2,346
	14.10.2011, 10:25
	Ну как предложение вывести path (или трассировкой) сразу после чтения из StringGrid и перед удалением, так можно выяснить где сбой происходит. 0