Conversation
Khmelov
left a comment
Khmelov
left a comment
There was a problem hiding this comment.
В целом обращение с кодом приемлемое, но я советовал бы потратить чуть больше времени на реализацию этого проекта. Поставил оценку C
| public class MainApp { | ||
| private static final String DEFAULT_TEXT_FOLDER = "text"; | ||
|
|
||
| public static void main(String[] args) { |
There was a problem hiding this comment.
Независимо от местоположения длина метода не может превышать 25 строк. Метод не должен требовать с скроллинга
| @@ -0,0 +1,117 @@ | |||
| package com.javarush.ushanov; | |||
|
|
|||
| // MainApp.java | |||
There was a problem hiding this comment.
Комментарии в коде должны быть для других разработчиков а не для того кто его написал
|
|
||
| // CaesarCipher.java | ||
|
|
||
| public class CaesarCipher implements Cipher { |
There was a problem hiding this comment.
Приложение лучше разбить на пакеты даже если классов мало
| for (char c : text.toCharArray()) { | ||
| if (isCyrillicLower(c)) { | ||
| int index = indexOfChar(LETTERS_LOWER, c); | ||
| int shiftedIndex = mod(index + shift, lettersLength); |
There was a problem hiding this comment.
Тут какая-то странная логика если алфавит оформить в виде карты то это большое разветвления превратится в одну строчку
|
|
||
| public String readFile(String filePath) throws IOException { | ||
| StringBuilder content = new StringBuilder(); | ||
| try (BufferedReader br = new BufferedReader( |
There was a problem hiding this comment.
Советую обратить внимание на новые способы обращения к файловой системе
3 participants
No description provided.