Khramov 2tasks complete, Change pom.xml#39
Conversation
Khmelov
left a comment
Khmelov
left a comment
There was a problem hiding this comment.
Даже если приложение очень маленькое все равно его нужно разбивать на понятные пакеты потому что в дальнейшем когда классы будут добавляться должно быть понятно где они должны находиться. Выполнено две задачи есть некоторые замечания поставил оценку Ц
| public final int hashEnAlphabet(String letter) { | ||
|
|
||
| HashMap<String, Integer> enAlphabet = new HashMap<>(); | ||
| enAlphabet.put("a", 1); enAlphabet.put("b", 2); enAlphabet.put("c", 3); |
There was a problem hiding this comment.
Тут лучше сделать строчку а потом циклом из неё заполнить карту
| public final String letterEnAlphabet(int hash){ | ||
|
|
||
| HashMap<Integer, String> enAlphabet = new HashMap<>(); | ||
| enAlphabet.put(1, "a"); enAlphabet.put(2, "b"); enAlphabet.put(3, "c"); |
|
|
||
| if (alphabet.booleanEnAlphabet(x) == true) { | ||
| int a = alphabet.hashEnAlphabet(x); | ||
| if ((a + key) > 26) { |
There was a problem hiding this comment.
Числам и строкам лучше присваивать понятные константные имена
|
|
||
| public class FileManager { | ||
|
|
||
| public List<String> readingFile(String fileAddress) { |
There was a problem hiding this comment.
Тут всё хорошо но все же не очень понятно почему именно лист
| public void writeFile(List<String> list, String encryptedFileAddress) { | ||
|
|
||
| try (BufferedWriter writer = new BufferedWriter(new FileWriter(encryptedFileAddress))) { | ||
| for (String line : list) { |
There was a problem hiding this comment.
Ага вот понятно почему лист. Лучше всё-таки было оставить его обычным стрингом
| import java.util.Scanner; | ||
|
|
||
| public class MainApp { | ||
| public static void main(String[] args) { |
There was a problem hiding this comment.
Чаще всего в методе main создают все компоненты приложения и/или собирают это приложение, а его запуск выполняется в каком-то понятном методе например старт
|
|
||
| public static String fileAddress() { | ||
| InputStream stream = System.in; | ||
| Scanner console = new Scanner(stream); |
There was a problem hiding this comment.
Не очень хорошая идея создавать сканер при каждом обращении к методу
| public class PathAndKey { | ||
|
|
||
|
|
||
| public static String fileAddress() { |
There was a problem hiding this comment.
Когда методы статические их трудно тестировать
| import java.io.InputStream; | ||
| import java.util.Scanner; | ||
|
|
||
| public class PathAndKey { |
There was a problem hiding this comment.
Если в классе все методы статические то ему делают приватный конструктор
2 participants
No description provided.