Merge pull request 'refactor(repo): переименование и реструктуризация классов репозитория (close #47)' (#56) from feature/inmemory-repository into dev

Reviewed-on: #56

.gitignore

@@ -1,2 +1,3 @@
 todo/target/
 .vscode/
+.lingma/

CHANGELOG.md

@@ -1,9 +1,19 @@
-# Журнал изменений
-
 Все заметные изменения проекта будут документироваться в этом файле.
 
 Формат основан на [Keep a Changelog](https://keepachangelog.com/ru/1.0.0/),
-и этот проект придерживается [Semantic Versioning](https://semver.org/spec/v2.0.0.html).
+и этот проект придерживается [Semantic Versioning](https://semver.org/spec/v2.0.0.html)
+
+## [2.1.0] - 2025-06-01
+
+### Добавлено
+- Команда `search <текст>` для поиска задач по тексту (реализован класс `SearchCommand`)
+- Метод `search(String query)` в `TaskService` для поддержки поиска задач
+- Рефакторинг: вынесение общего функционала отображения задач в отдельный метод `formatWithTable()` в классе `ListCommand`
+
+### Изменено
+- Обновлены внутренние зависимости между компонентами CLI (`SearchCommand`, `ListCommand`) и моделью данных (`Task`, `SimpleTask`)
+- Добавлена обработка минимальной длины поискового запроса (3 символа) в `SearchCommand`# Журнал изменений
+
 
 ## [2.0.0] - 2025-05-29
 

README.md

@@ -19,6 +19,7 @@ java -jar ./target/todo.jar
 - `list` - Показать список всех задач
 - `complete <id>` - Отметить задачу как выполненную
 - `delete <id>` - Удалить задачу
+- `search <текст>` - Найти задачи по тексту
 - `exit` - Выйти из приложения
 
 ## Структура проекта
@@ -28,7 +29,7 @@ java -jar ./target/todo.jar
 - `cli` - Компоненты интерфейса командной строки
 
 ## Версии
-Актуальная версия: 2.0
+Актуальная версия: 2.1
 
 ## Лицензия
 [MIT](LICENSE)

todo/pom.xml

@@ -6,7 +6,7 @@
 
     <groupId>ru.kamask.pet</groupId>
     <artifactId>todo</artifactId>
-    <version>2.0-SNAPSHOT</version>
+    <version>2.2.0-SNAPSHOT</version>
 
     <properties>
         <maven.compiler.release>24</maven.compiler.release>

todo/src/main/java/ru/kamask/pet/todo/TodoApp.java

@@ -3,12 +3,13 @@ package ru.kamask.pet.todo;
 import java.io.IOException;
 
 import ru.kamask.pet.todo.cli.CliEngine;
-import ru.kamask.pet.todo.repo.InMemoryTaskRepository;
+import ru.kamask.pet.todo.model.SimpleTask;
+import ru.kamask.pet.todo.repo.InMemoryRepository;
 import ru.kamask.pet.todo.service.TaskService;
 
 public class TodoApp {
     public static void main(String[] args) throws IOException {
-        var service = new TaskService(new InMemoryTaskRepository());
+        var service = new TaskService(new InMemoryRepository<SimpleTask>());
         var cli = new CliEngine(service);
 
         cli.start();

todo/src/main/java/ru/kamask/pet/todo/cli/CliEngine.java

@@ -21,7 +21,7 @@ public class CliEngine {
     }
 
     public void start() throws IOException {
-        System.out.println("\nДобро пожаловать в Список задач 2.0!");
+        System.out.println("\nДобро пожаловать в Список задач 2.1!");
         System.out.println("Введите help для просмтора доступных команд или exit для выхода.");
         while (true) {
             System.out.print("\ntodo> ");
@@ -46,6 +46,7 @@ public class CliEngine {
         registerCommand(new CreateCommand());
         registerCommand(new ListCommand());
         registerCommand(new CompleteCommand());
+        registerCommand(new SearchCommand());
         registerCommand(new DeleteCommand());
     }
 

todo/src/main/java/ru/kamask/pet/todo/cli/ListCommand.java

@@ -1,9 +1,9 @@
 package ru.kamask.pet.todo.cli;
 
+import java.util.List;
 import java.util.Optional;
 
 import ru.kamask.pet.todo.model.SimpleTask;
-import ru.kamask.pet.todo.model.Task;
 import ru.kamask.pet.todo.service.TaskService;
 
 public class ListCommand implements Command {
@@ -17,18 +17,8 @@ public class ListCommand implements Command {
         if (args.length > 0)
             return Optional.of(Command.errorMessage);
 
-        String template = "%-2s | %-30s | %s\n";
+        var res = formatWithTable(service.list(), "Список задач пуст.");
-        String res = "";
-        res += String.format(template, "ID", "Название задачи", "Статус");
-        res += "-".repeat(50) + "\n";
 
-        if (service.list().size() == 0)
-            return Optional.of(res + "\nСписок задач пуст.");
-
-        for (Task task : service.list()) {
-            SimpleTask.Data data = ((SimpleTask) task).data();
-            res += String.format(template, data.id(), data.title(), data.done() ? "выполнено" : "не выполнено");
-        }
         return Optional.of(res);
     }
 
@@ -36,4 +26,20 @@ public class ListCommand implements Command {
     public String usage() {
         return String.format(templateUsage, name(), "Список всех задач.");
     }
+
+    String formatWithTable(List<SimpleTask> tasks, String msgIfEmpty) {
+        String template = "%-2s | %-30s | %s\n";
+        var res = new StringBuilder(String.format(template, "ID", "Название задачи", "Статус"));
+        res.append("-".repeat(50) + "\n");
+
+        if (tasks.size() == 0)
+            return res.append("\n" + msgIfEmpty).toString();
+
+        for (SimpleTask task : tasks) {
+            SimpleTask.Data data = task.data();
+            res.append(String.format(template, data.id(), data.title(), data.done() ? "выполнено" : "не выполнено"));
+        }
+
+        return res.toString();
+    }
 }

todo/src/main/java/ru/kamask/pet/todo/cli/SearchCommand.java

@@ -0,0 +1,30 @@
+package ru.kamask.pet.todo.cli;
+
+import java.util.Optional;
+
+import ru.kamask.pet.todo.service.TaskService;
+
+public class SearchCommand implements Command {
+    @Override
+    public String name() {
+        return "search";
+    }
+
+    @Override
+    public String usage() {
+        return String.format(templateUsage, name() + " <текст>", "Поиск задач по тексту (минимум 3 символа)");
+    }
+
+    @Override
+    public Optional<String> handle(String[] args, TaskService service) {
+        if (args.length != 1)
+            return Optional.of(Command.errorMessage);
+        if (args[0].length() < 3)
+            return Optional.of("Длина запроса должна быть не менее 3 символов.");
+
+        var matchTask = service.search(args[0]);
+        var res = new ListCommand().formatWithTable(matchTask, "Не найдено задач, соответствующих запросу.");
+
+        return Optional.of(res);
+    }
+}

todo/src/main/java/ru/kamask/pet/todo/model/Identifiable.java

@@ -0,0 +1,5 @@
+package ru.kamask.pet.todo.model;
+
+public interface Identifiable {
+    int getId();
+}

todo/src/main/java/ru/kamask/pet/todo/model/Task.java

@@ -1,6 +1,6 @@
 package ru.kamask.pet.todo.model;
 
-public abstract class Task {
+public abstract class Task implements Identifiable{
     private static int nextId = 1;
 
     protected int id;
@@ -11,10 +11,15 @@ public abstract class Task {
         this.title = title;
     }
 
-    public int id() {
+    @Override
+    public int getId() {
         return id;
     }
 
+    public int id() {
+        return getId();
+    }
+
     @Override
     public String toString() {
         return String.format("Задача: id - %d, title: \"%s\"", id, title);

todo/src/main/java/ru/kamask/pet/todo/repo/InMemoryRepository.java

@@ -6,23 +6,23 @@ import java.util.List;
 import java.util.Map;
 import java.util.Optional;
 
-import ru.kamask.pet.todo.model.Task;
+import ru.kamask.pet.todo.model.Identifiable;
 
-public class InMemoryTaskRepository implements TaskRepository {
+public class InMemoryRepository<T extends Identifiable> implements Repository<T> {
-    private Map<Integer, Task> storage = new HashMap<>();
+    private Map<Integer, T> storage = new HashMap<>();
 
     @Override
-    public void save(Task task) {
+    public void save(T obj) {
-        storage.put(task.id(), task);
+        storage.put(obj.getId(), obj);
     }
 
     @Override
-    public Optional<Task> findById(int id) {
+    public Optional<T> findById(int id) {
         return Optional.ofNullable(storage.get(id));
     }
 
     @Override
-    public List<Task> findAll() {
+    public List<T> findAll() {
         return new ArrayList<>(storage.values());
     }
 

todo/src/main/java/ru/kamask/pet/todo/repo/Repository.java

@@ -0,0 +1,14 @@
+package ru.kamask.pet.todo.repo;
+
+import java.util.List;
+import java.util.Optional;
+
+import ru.kamask.pet.todo.model.Identifiable;
+
+public interface Repository<T extends Identifiable> {
+    void save(T obj);
+    Optional<T> findById(int id);
+    List<T> findAll();
+    void delete(int id);
+    boolean has(int id);
+}

todo/src/main/java/ru/kamask/pet/todo/repo/TaskRepository.java

@@ -1,18 +0,0 @@
-package ru.kamask.pet.todo.repo;
-
-import java.util.List;
-import java.util.Optional;
-
-import ru.kamask.pet.todo.model.Task;
-
-public interface TaskRepository {
-    void save(Task task);
-
-    Optional<Task> findById(int id);
-
-    List<Task> findAll();
-
-    void delete(int id);
-
-    boolean has(int id);
-}

todo/src/main/java/ru/kamask/pet/todo/service/TaskService.java

@@ -3,14 +3,13 @@ package ru.kamask.pet.todo.service;
 import java.util.List;
 import java.util.Optional;
 
-import ru.kamask.pet.todo.model.Task;
 import ru.kamask.pet.todo.model.SimpleTask;
-import ru.kamask.pet.todo.repo.TaskRepository;
+import ru.kamask.pet.todo.repo.InMemoryRepository;
 
 public class TaskService {
-    private final TaskRepository repo;
+    private final InMemoryRepository<SimpleTask> repo;
 
-    public TaskService(TaskRepository repo) {
+    public TaskService(InMemoryRepository<SimpleTask> repo) {
         this.repo = repo;
     }
 
@@ -18,7 +17,7 @@ public class TaskService {
         repo.save(new SimpleTask(title));
     }
 
-    public Optional<Task> getById(int id) {
+    public Optional<SimpleTask> getById(int id) {
         return repo.findById(id);
     }
 
@@ -31,7 +30,7 @@ public class TaskService {
         return false;
     }
 
-    public List<Task> list() {
+    public List<SimpleTask> list() {
         return repo.findAll();
     }
 
@@ -42,4 +41,10 @@ public class TaskService {
     public boolean has(int id) {
         return repo.has(id);
     }
+
+    public List<SimpleTask> search(String query) {
+        return repo.findAll().stream()
+                .filter(task -> task.data().title().contains(query))
+                .toList();
+    }
 }