Conversation
|
|
||
| def make_dir(dir_path): | ||
| if os.path.exists(dir_path): | ||
| result = 2 # Error: a directory with the same name is already exists |
There was a problem hiding this comment.
Это C-way (ну или С++) разработки. Возвращение кода. Лучше разнесите на функции, типа path_exists. И Exception если директория не создалась - лучше не замалчивать. Обычно это ведёт лишь к дальнейшим ошибкам в выполнении программы.
| dir_path = ' ' | ||
| def is_abs_filename(file_or_dir): | ||
| # возвращает 1 если это абсолютная директория, и 0 если относительная | ||
| pattern = re.compile(r'[A-Z]:\)', re.IGNORECASE) |
There was a problem hiding this comment.
os.path.isabs есть такой метод в python. Можно пользоваться им)
There was a problem hiding this comment.
Но круто что воспользовались regex. Это полезно, как показало ваше расследование)
| else: | ||
| print("Вы действительно хотите удалить директорию '{}'".format(file_dir_name)) | ||
| answer = input("Ведите 'Y', если 'да' или 'N', если нет:") | ||
| if answer =='N' or answer == 'N': |
There was a problem hiding this comment.
Лучше делать так, во избежании ошибок, как тут) Это обычная практика, рекомендую взять на вооружение
answer.lower() == 'N'.lower()
There was a problem hiding this comment.
Да, спасибо! Я совершенно забыл, сто у литералов и строк можно менять регистр с большого верхнего на нижний и наоборот...
No description provided.