Решение задачи by robotdorog · Pull Request #1 · robotdorog/Projects

robotdorog · 2021-10-01T11:28:07Z

No description provided.

rs-pluss · 2021-10-03T17:28:07Z

.idea/misc.xml

@@ -0,0 +1,6 @@
+<?xml version="1.0" encoding="UTF-8"?>


Файлы из .idea не исчезли, посмотри пример в предыдущем репозитории

src/main.java

src/Cryptor.java

rs-pluss · 2021-10-03T17:35:39Z

src/Cryptor.java

+            str = reader.readLine();
+
+        } catch (Exception e) {
+            e.printStackTrace();


Исключения, которые не можешь обработать, надо пробрасывать выше

src/Cryptor.java

… и reader. Убрал из класса Cryptor пути. Пути принимаются в параметрах метода

src/Cryptor.java

…ия и записи в файл.

src/Cryptor.java

…мо реализовать класс FileHandlerV2

…ileHandlerV2

src/Cryptor.java

src/CryptorTest.java

src/FileHandlerV2.java

rs-pluss · 2021-11-29T13:55:26Z

src/CryptorV2.java

@@ -0,0 +1,60 @@
+public class CryptorV2 {


Очень много общего между Cryptor и CryptorV2, как можно выделить общую идею?

src/CryptorViaString.java

rs-pluss · 2021-12-08T05:12:14Z

src/FileHandler.java

+
+    String fileIn;
+    String fileOut;
+    CryptorViaString cryptorViaString;


Этих ребят здесь не должно быть

rs-pluss · 2021-12-08T05:14:03Z

src/TestClass.java

+
+public class TestClass implements Crypt {
+
+    public TestClass(CryptorViaString cryptorViaString, CryptorViaStringBuilder cryptorViaStringBuilder){


Почему пустой конструктор? Ты должен обе этих переменных запомнить в поля класса

Исправил

rs-pluss · 2021-12-13T06:49:25Z

src/Crypt.java

@@ -0,0 +1,4 @@
+public interface Crypt {


Crypt - глагол если я не ошибаюсь, этот интерфейс у тебя описывает сервис. Поэтому здесь должно быть существительное - Cryptor

Переименовал на Cryptor

rs-pluss · 2021-12-13T06:50:05Z

src/Cryptor.java

@@ -0,0 +1,40 @@
+import java.io.IOException;
+
+public class Cryptor implements Crypt {


Это сервис и для него правильно - Cryptor. Только вот он же не просто шифровальщик а с некоторой изюминкой, надо отразить это в названии.

Переименовал на CryptorWithTimeAssessment

rs-pluss · 2021-12-13T06:50:30Z

src/Cryptor.java

+    CryptorViaStringBuilder cryptorViaStringBuilder;
+
+    public Cryptor(CryptorViaString cryptorViaString, CryptorViaStringBuilder cryptorViaStringBuilder) {
+        this.cryptorViaString = cryptorViaString;


Не вижу чтобы ты его использовал и сравнивал время выполнения. Сейчас ты используешь только второй

rs-pluss · 2021-12-13T06:51:14Z

src/FileHandler.java

+import java.nio.file.Path;
+
+public class FileHandler {
+    Cryptor cryptor;


Этому классу должно быть без разницы, с какой реализацией интерфейса работать. Здесь нужно принимать максимально абстрактную сущность

robotdorog added 2 commits October 1, 2021 19:26

Решение задачи

2815d52

Добавлен .idea в .gitignore

7a8bb66

rs-pluss reviewed Oct 3, 2021

View reviewed changes

src/main.java Outdated Show resolved Hide resolved

rs-pluss reviewed Oct 3, 2021

View reviewed changes