Conversation
yurii-litvinov
left a comment
yurii-litvinov
left a comment
There was a problem hiding this comment.
Папку .gradle не нужно выкладывать. В остальном всё очень круто, поправьте мелкие замечания по коду --- и будет зачтена.
|
|
||
| /** | ||
| * Used as interface to command object. Takes input stream and returns result as output stream. | ||
| * @param stream |
There was a problem hiding this comment.
Пустые тэги не нужны, лучше всё-таки написать комментарий (или удалить комментарии для параметров вообще, если они очевидны)
| @@ -0,0 +1,54 @@ | |||
| package commands; | |||
There was a problem hiding this comment.
Пакеты в Java обычно именуются с использованием доменного имени, записанного наоборот, типа для глобальной уникальности имен пакета: https://docs.oracle.com/javase/tutorial/java/package/namingpkgs.html. Желательно следовать этой традиции.
shell/src/main/java/shell/Shell.java
Outdated
| /** | ||
| * constructs Shell with empty environment | ||
| */ | ||
| Shell() { |
There was a problem hiding this comment.
У него специально пакетная видимость, или просто так получилось?
There was a problem hiding this comment.
Просто так получилось)
Пофиксил
shell/src/test/java/ParserTests.java
Outdated
| try { | ||
| commands.get(0).run(new Stream()); | ||
| } catch (CommandExecutionException e) { | ||
| e.printStackTrace(); |
There was a problem hiding this comment.
Тут лучше исключение не ловить --- команда же не должна упасть, и если упала, то всё плохо.
2 participants
No description provided.