Skip to content

Commit f09e3a7

Browse files
committed
removed really deleting or renaming files
1 parent 43bbf78 commit f09e3a7

File tree

1 file changed

+12
-39
lines changed

1 file changed

+12
-39
lines changed

patterns/behavioral/command.py

Lines changed: 12 additions & 39 deletions
Original file line numberDiff line numberDiff line change
@@ -20,8 +20,6 @@
2020
https://docs.djangoproject.com/en/2.1/ref/request-response/#httprequest-objects
2121
"""
2222

23-
import os
24-
2523

2624
class HideFileCommand:
2725
"""
@@ -33,53 +31,30 @@ def __init__(self):
3331
self._hidden_files = []
3432

3533
def execute(self, filename):
36-
if os.path.isfile(filename):
37-
print(f'hiding {filename}')
38-
39-
os.rename(filename, f'.{filename}')
40-
self._hidden_files.append(filename)
41-
else:
42-
print(f'{filename} does not exists to hide')
34+
print(f'hiding {filename}')
35+
self._hidden_files.append(filename)
4336

4437
def undo(self):
45-
if len(self._hidden_files) > 0:
46-
filename = self._hidden_files.pop()
47-
48-
print(f'un-hiding {filename}')
49-
50-
os.rename(f'.{filename}', filename)
38+
filename = self._hidden_files.pop()
39+
print(f'un-hiding {filename}')
5140

5241

5342
class DeleteFileCommand:
5443
"""
5544
A command to delete a file given its name
5645
"""
57-
_deleted_files_path = 'trash'
5846

5947
def __init__(self):
6048
# an array of deleted files, to undo them as needed
6149
self._deleted_files = []
6250

63-
# create a directory to store deleted files
64-
if not os.path.exists(self._deleted_files_path):
65-
os.makedirs(self._deleted_files_path)
66-
6751
def execute(self, filename):
68-
if os.path.isfile(filename):
69-
print(f'deleting {filename}')
70-
71-
os.rename(filename, f'{self._deleted_files_path}/{filename}')
72-
self._deleted_files.append(filename)
73-
else:
74-
print(f'{filename} does not exists to delete')
52+
print(f'deleting {filename}')
53+
self._deleted_files.append(filename)
7554

7655
def undo(self):
77-
if len(self._deleted_files) > 0:
78-
filename = self._deleted_files.pop()
79-
80-
print(f'restoring {filename}')
81-
82-
os.rename(f'{self._deleted_files_path}/{filename}', filename)
56+
filename = self._deleted_files.pop()
57+
print(f'restoring {filename}')
8358

8459

8560
class MenuItem:
@@ -105,27 +80,25 @@ def main():
10580
10681
# create a file named `test-file` to work with
10782
>>> test_file_name = 'test-file'
108-
>>> open(test_file_name, 'w').close()
10983
11084
# deleting `test-file`
11185
>>> item1.on_do_press(test_file_name)
11286
deleting test-file
11387
114-
# hiding `test-file` but it does not exists
115-
>>> item2.on_do_press(test_file_name)
116-
test-file does not exists to hide
117-
11888
# restoring `test-file`
11989
>>> item1.on_undo_press()
12090
restoring test-file
12191
12292
# hiding `test-file`
12393
>>> item2.on_do_press(test_file_name)
12494
hiding test-file
95+
96+
# un-hiding `test-file`
97+
>>> item2.on_undo_press()
98+
un-hiding test-file
12599
"""
126100

127101

128102
if __name__ == "__main__":
129103
import doctest
130-
131104
doctest.testmod()

0 commit comments

Comments
 (0)