Решение задачи - #1
Conversation
| @@ -0,0 +1,6 @@ | |||
| <?xml version="1.0" encoding="UTF-8"?> | |||
There was a problem hiding this comment.
Файлы из .idea не исчезли, посмотри пример в предыдущем репозитории
| str = reader.readLine(); | ||
|
|
||
| } catch (Exception e) { | ||
| e.printStackTrace(); |
There was a problem hiding this comment.
Исключения, которые не можешь обработать, надо пробрасывать выше
… и reader. Убрал из класса Cryptor пути. Пути принимаются в параметрах метода
…мо реализовать класс FileHandlerV2
| @@ -0,0 +1,60 @@ | |||
| public class CryptorV2 { | |||
There was a problem hiding this comment.
Очень много общего между Cryptor и CryptorV2, как можно выделить общую идею?
|
|
||
| String fileIn; | ||
| String fileOut; | ||
| CryptorViaString cryptorViaString; |
|
|
||
| public class TestClass implements Crypt { | ||
|
|
||
| public TestClass(CryptorViaString cryptorViaString, CryptorViaStringBuilder cryptorViaStringBuilder){ |
There was a problem hiding this comment.
Почему пустой конструктор? Ты должен обе этих переменных запомнить в поля класса
| @@ -0,0 +1,4 @@ | |||
| public interface Crypt { | |||
There was a problem hiding this comment.
Crypt - глагол если я не ошибаюсь, этот интерфейс у тебя описывает сервис. Поэтому здесь должно быть существительное - Cryptor
There was a problem hiding this comment.
Переименовал на Cryptor
| @@ -0,0 +1,40 @@ | |||
| import java.io.IOException; | |||
|
|
|||
| public class Cryptor implements Crypt { | |||
There was a problem hiding this comment.
Это сервис и для него правильно - Cryptor. Только вот он же не просто шифровальщик а с некоторой изюминкой, надо отразить это в названии.
There was a problem hiding this comment.
Переименовал на CryptorWithTimeAssessment
| CryptorViaStringBuilder cryptorViaStringBuilder; | ||
|
|
||
| public Cryptor(CryptorViaString cryptorViaString, CryptorViaStringBuilder cryptorViaStringBuilder) { | ||
| this.cryptorViaString = cryptorViaString; |
There was a problem hiding this comment.
Не вижу чтобы ты его использовал и сравнивал время выполнения. Сейчас ты используешь только второй
| import java.nio.file.Path; | ||
|
|
||
| public class FileHandler { | ||
| Cryptor cryptor; |
There was a problem hiding this comment.
Этому классу должно быть без разницы, с какой реализацией интерфейса работать. Здесь нужно принимать максимально абстрактную сущность
2 participants
No description provided.