Thanks to visit codestin.com
Credit goes to github.com

Skip to content

Commit 463924c

Browse files
committed
readme
1 parent 3f1d1d6 commit 463924c

File tree

1 file changed

+4
-2
lines changed

1 file changed

+4
-2
lines changed

README.md

Lines changed: 4 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -6,7 +6,7 @@ You can fork this repo, and clone it down onto your computer.
66
The `main()` function is in `class FileManager`.
77
You should be able to run that function.
88

9-
## What is does
9+
## What it does
1010

1111
This is a very simple file management _shell_, it shows how to manipulate files using java standard library calls.
1212
Available commands:
@@ -22,13 +22,15 @@ Each class a different reason for being.
2222
There is a simple file hierarchy for manual testing in the _test_ folder.
2323
You can use the _test_ folder for doing simple file manipulations.
2424

25+
## Things to consider
2526

2627
Why are the manipulation commands, the things that affect files and folders, in a different class than
2728
the _file manager_ itself?
2829

2930
Why is all the input and output factored out into a separate class?
3031

31-
Why is Copy/Move in the same method?
32+
Why is Copy/Move in the same method? What about the two operations are so much the same?
33+
How would you separate them? Would it make it more understandable or less to spearate them?
3234

3335
What would you have to do to:
3436

0 commit comments

Comments
 (0)