Repository navigation
Очищается папка перед выполнением функции СоздатьФайловуюБазу() #73
Description
Здравствуйте. В процессе написания с помощью v8runner скрипта создания начального образа наткнулся на неприятное поведение. В порядке упрощения для себя указал каталог сборки текущий каталог скрипта. Вызываю СоздатьФайловуюБазу() и у меня удаляется обработка.
Мне не совсем ясно, это я затупил или нужно это учитывать всегда? Может быть такой случай обработать? Я так понимаю, что если КаталогБазы указать как какой-то диск, есть риск потерять все, что внутри и во вложенных папках. Сейчас там вызывается УдалитьФайлы(КаталогБазы, ".");.
Activity
EvilBeaver commentedon Mar 19, 2018
Я вам больше скажу, если указать в любом другом коде УдалитьФайлы("C:*.*") то буде то же самое.
Ridje commentedon Mar 19, 2018
Да, но разница в том, что я это сделаю намеренно, а тут нужно посмотреть код библиотеке и из названия процедуры это неясно.
EvilBeaver commentedon Mar 19, 2018
Да, но это, вероятно, недостаток документирования. Каталог базы очищается, и мне кажется, что так и должно быть. Допускаю, что я мог что-то позабыть и теперь не совсем в теме. Не могли бы вы ткнуть меня в кусок кода (ссылкой на github) - о каком месте кода мы сейчас говорим?
Ridje commentedon Mar 19, 2018
v8runner/src/v8runner.os
Line 872 in 007b80a
Я мог бы сам заняться и подумать над этим, но надо выработать, что правильно в данном случае. Меня это смутило просто.
Ridje commentedon Mar 19, 2018
v8runner/src/v8runner.os
Line 877 in 007b80a
EvilBeaver commentedon Mar 20, 2018
@Ridje я не вполне понимаю, а какое ожидается правильное поведение? Перед созданием базы каталог базы очищается. Разве это неправильно? Какое ожидается поведение?
240596448 commentedon Jan 20, 2019
Думаю, что правильно - это просто задокументировать.
Менее ожидаемо: генерировать ошибку при непустой папке. Потому что норма - это создание базы в пустой папке.
EvilBeaver commentedon Nov 3, 2022
Приму пулреквест, который будет ругаться при непустой папке + флаг "СоздатьПринудительно"