Merge branch 'develop' into feature/csv-student-io

app/pom.xml

@@ -42,6 +42,12 @@
       <scope>test</scope>
     </dependency>
 
+    <dependency>
+      <groupId>org.apache.commons</groupId>
+      <artifactId>commons-csv</artifactId>
+      <version>1.14.1</version>
+    </dependency>
+
     <dependency>
       <groupId>com.googlecode.lanterna</groupId>
       <artifactId>lanterna</artifactId>

app/src/main/java/ru/project/App.java

@@ -1,45 +0,0 @@
-package ru.project;
-
-import com.googlecode.lanterna.gui2.MultiWindowTextGUI;
-import com.googlecode.lanterna.screen.Screen;
-import com.googlecode.lanterna.terminal.DefaultTerminalFactory;
-import ru.project.model.AppState;
-import ru.project.ui.main.MainMenu;
-
-public final class App {
-
-  private App() {}
-
-  public static void main(String[] args) {
-    Screen screen = null;
-
-    try {
-      DefaultTerminalFactory terminalFactory = new DefaultTerminalFactory();
-
-      screen = terminalFactory.createScreen();
-      screen.startScreen();
-
-      MultiWindowTextGUI gui = new MultiWindowTextGUI(screen);
-
-      AppState state = new AppState();
-
-      MainMenu mainMenu = new MainMenu(gui, state);
-
-      mainMenu.showModal();
-
-    } catch (Exception e) {
-      System.err.println("Произошла ошибка при выполнении программы:");
-      e.printStackTrace();
-
-    } finally {
-      if (screen != null) {
-        try {
-          screen.stopScreen();
-        } catch (Exception e) {
-          System.err.println("Не удалось корректно завершить работу терминала:");
-          e.printStackTrace();
-        }
-      }
-    }
-  }
-}

app/src/main/java/ru/project/MyCollectors.java

@@ -1,10 +0,0 @@
-package ru.test.maven.spring;
-import java.util.stream.Collector;
-public class MyCollectors
-{
-    private MyCollectors() {
-    }
-    public static <T> Collector<T, MyList<T>, MyList<T>> toMyList() {
-        return new MyListCollector<>();
-    }
-}

app/src/main/java/ru/project/MyListCollector.java

@@ -1,41 +0,0 @@
-package ru.test.maven.spring;
-import java.util.Collections;
-import java.util.Set;
-import java.util.function.BiConsumer;
-import java.util.function.BinaryOperator;
-import java.util.function.Function;
-import java.util.function.Supplier;
-import java.util.stream.Collector;
-public class MyListCollector <T> implements Collector<T, MyList<T>, MyList<T>>
-{
-    @Override
-    public Supplier<MyList<T>> supplier()
-    {
-        return () -> new MyLinkedList<>();
-    }
-    @Override
-    public BiConsumer<MyList<T>, T> accumulator() {
-
-        return (list, element) -> list.add(element);
-    }
-    @Override
-    public BinaryOperator<MyList<T>> combiner() {
-        return (list1, list2) -> {
-            for (int i = 0; i < list2.size(); i++) {
-                list1.add(list2.get(i));
-            }
-            return list1;
-        };
-    }
-    @Override
-    public Function<MyList<T>, MyList<T>> finisher()
-    {
-        return list -> list;
-    }
-    @Override
-    public Set<Collector.Characteristics> characteristics() {
-        return java.util.Set.of(
-                Collector.Characteristics.IDENTITY_FINISH
-        );
-    }
-}

app/src/main/java/ru/project/collectors/MyCollectors.java

@@ -1,15 +1,12 @@
 package ru.project.collectors;
+
 import java.util.stream.Collector;
+import ru.project.collection.MyList;
 
-public class MyCollectors
+public class MyCollectors {
-{
+  private MyCollectors() {}
-    private MyCollectors() {
 
-    }
+  public static <T> Collector<T, MyList<T>, MyList<T>> toMyList() {
-
-    public static <T> Collector<T, MyList<T>, MyList<T>> toMyList()
-    {
     return new MyListCollector<>();
-
   }
 }

app/src/main/java/ru/project/collectors/MyListCollector.java

@@ -1,16 +1,17 @@
 package ru.project.collectors;
-import java.util.Collections;
+
 import java.util.Set;
 import java.util.function.BiConsumer;
 import java.util.function.BinaryOperator;
 import java.util.function.Function;
 import java.util.function.Supplier;
 import java.util.stream.Collector;
-public class MyListCollector <T> implements Collector<T, MyList<T>, MyList<T>>
+import ru.project.collection.MyLinkedList;
-{
+import ru.project.collection.MyList;
+
+public class MyListCollector<T> implements Collector<T, MyList<T>, MyList<T>> {
   @Override
-    public Supplier<MyList<T>> supplier()
+  public Supplier<MyList<T>> supplier() {
-    {
     return () -> new MyLinkedList<>();
   }
 
@@ -29,15 +30,12 @@ public class MyListCollector <T> implements Collector<T, MyList<T>, MyList<T>>
   }
 
   @Override
-    public Function<MyList<T>, MyList<T>> finisher()
+  public Function<MyList<T>, MyList<T>> finisher() {
-    {
     return list -> list;
   }
 
   @Override
   public Set<Collector.Characteristics> characteristics() {
-        return java.util.Set.of(
+    return java.util.Set.of(Collector.Characteristics.IDENTITY_FINISH);
-                Collector.Characteristics.IDENTITY_FINISH
-        );
   }
 }

app/src/main/java/ru/project/finder/StudentOccurrenceIndexFinder.java

@@ -1,9 +1,13 @@
 package ru.project.finder;
+
 import java.util.concurrent.ExecutorService;
 import java.util.concurrent.Executors;
 import java.util.concurrent.Future;
-public class StudentOccurrenceIndexFinder
+import ru.project.collection.MyLinkedList;
-{
+import ru.project.collection.MyList;
+import ru.project.student.Student;
+
+public class StudentOccurrenceIndexFinder {
   public int[] findOccurrences(MyList<Student> students, Student target) {
     if (students == null) {
       throw new IllegalArgumentException();
@@ -32,7 +36,9 @@ public class StudentOccurrenceIndexFinder
       final int start = fromIndex;
       final int end = toIndex;
 
-            Future<MyList<Integer>> future = executor.submit(() -> {
+      Future<MyList<Integer>> future =
+          executor.submit(
+              () -> {
                 return findInRange(students, target, start, end);
               });
       futures.add(future);
@@ -110,7 +116,8 @@ public class StudentOccurrenceIndexFinder
     return (size + threadCount - 1) / threadCount;
   }
 
-    private MyList<Integer> findInRange(MyList<Student> students, Student target, int fromIndex, int toIndex) {
+  private MyList<Integer> findInRange(
+      MyList<Student> students, Student target, int fromIndex, int toIndex) {
     MyList<Integer> foundIndices = new MyLinkedList<>();
 
     for (int i = fromIndex; i < toIndex; i++) {

app/src/main/java/ru/project/io/StudentCsvReader.java

@@ -0,0 +1,88 @@
+package ru.project.io;
+
+import java.io.IOException;
+import java.io.UncheckedIOException;
+import java.nio.charset.StandardCharsets;
+import java.nio.file.Path;
+import java.util.Arrays;
+import java.util.Iterator;
+import java.util.Map;
+import java.util.NoSuchElementException;
+import org.apache.commons.csv.CSVFormat;
+import org.apache.commons.csv.CSVParser;
+import org.apache.commons.csv.CSVRecord;
+import ru.project.student.Student;
+import ru.project.student.StudentBuilder;
+  public class StudentCsvReader implements Iterator<Student>, AutoCloseable {
+  private static final String GROUP_NUMBER = "groupNumber";
+  private static final String AVERAGE_GRADE = "averageGrade";
+  private static final String RECORD_BOOK_NUMBER = "recordBookNumber";
+  private final CSVParser parser;
+  private final Iterator<CSVRecord> records;
+  private int lineOfFile=0;
+    public StudentCsvReader(Path path) {
+      if (!path.isAbsolute()) {
+        throw new IllegalArgumentException("File path must be absolute");
+      }
+      try {
+        CSVFormat format = CSVFormat.DEFAULT.builder()
+                .setHeader()
+                .setSkipHeaderRecord(true)
+                .get();
+        this.parser = CSVParser.parse(path, StandardCharsets.UTF_8, format);
+        validateHeader(parser.getHeaderMap());
+        this.records = parser.iterator();
+      } catch (IOException e) {
+        throw new UncheckedIOException(e);
+      }
+    }
+    @Override
+    public boolean hasNext() {
+      return records.hasNext();
+    }
+    @Override
+    public Student next() {
+      if (!records.hasNext()) {
+        throw new NoSuchElementException("No more records at line " + (lineOfFile + 1));
+      }
+      lineOfFile++;
+      CSVRecord record = records.next();
+      if (record.size() != 3) {
+        throw new IllegalArgumentException(
+                "Problem at line " + lineOfFile + ". Expected 3 columns, but got " + record.size());
+      }
+      String groupNumber = record.get(GROUP_NUMBER);
+      String averageGradeStr = record.get(AVERAGE_GRADE);
+      String recordBookNumberStr = record.get(RECORD_BOOK_NUMBER);
+      double averageGrade;
+      int recordBookNumber;
+      try {
+        averageGrade = Double.parseDouble(averageGradeStr);
+      } catch (NumberFormatException e) {
+        throw new IllegalArgumentException(
+                "Problem at line " + lineOfFile + ": cannot parse averageGrade '" + averageGradeStr + "' as double", e);
+      }
+      try {
+        recordBookNumber = Integer.parseInt(recordBookNumberStr);
+      } catch (NumberFormatException e) {
+        throw new IllegalArgumentException(
+                "Problem at line " + lineOfFile + ": cannot parse recordBookNumber '" + recordBookNumberStr + "' as int", e);
+      }
+      return new StudentBuilder()
+              .setGroupNumber(groupNumber)
+              .setAverageGrade(averageGrade)
+              .setRecordBookNumber(recordBookNumber)
+              .build();
+    }
+    @Override
+    public void close() throws IOException {
+      parser.close();
+    }
+    private static void validateHeader(Map<String, Integer> headerMap) {
+      String[] expected = {GROUP_NUMBER, AVERAGE_GRADE, RECORD_BOOK_NUMBER};
+      if (headerMap.size() != expected.length
+              || !headerMap.keySet().containsAll(Arrays.asList(expected))) {
+        throw new IllegalArgumentException("Invalid CSV header");
+      }
+    }
+  }

app/src/main/java/ru/project/random/RandomStudentGenerator.java

@@ -0,0 +1,156 @@
+package ru.project.random;
+import org.junit.jupiter.params.ParameterizedTest;
+import org.junit.jupiter.params.provider.ValueSource;
+import ru.project.collection.MyLinkedList;
+import ru.project.collection.MyList;
+import ru.project.student.Student;
+import java.util.Random;
+import java.util.regex.Pattern;
+
+public class RandomStudentGenerator
+{
+    private static final Random RANDOM = new Random();
+    private String minGroupNumber;
+    private String maxGroupNumber;
+    private Double minAverageGrade;
+    private Double maxAverageGrade;
+    private Integer minRecordBookNumber;
+    private Integer maxRecordBookNumber;
+    private static final double DEFAULT_MIN_GRADE = 0.0;
+    private static final double DEFAULT_MAX_GRADE = 5.0;
+    private static final int DEFAULT_MIN_RECORD_BOOK = 1;
+    private static final String GROUP_PATTERN_REGEX = "^[A-Z][0-9]{2}$";
+    private static final Pattern GROUP_PATTERN = Pattern.compile(GROUP_PATTERN_REGEX);
+    public RandomStudentGenerator setMinGroupNumber(String minGroupNumber) {
+        if (minGroupNumber != null && !GROUP_PATTERN.matcher(minGroupNumber).matches()) {
+            throw new IllegalArgumentException("minGroupNumber does not match Student group pattern: " + GROUP_PATTERN_REGEX);
+        }
+        this.minGroupNumber = minGroupNumber;
+        return this;
+    }
+    public RandomStudentGenerator setMaxGroupNumber(String maxGroupNumber) {
+        if (maxGroupNumber != null && !GROUP_PATTERN.matcher(maxGroupNumber).matches()) {
+            throw new IllegalArgumentException("maxGroupNumber does not match Student group pattern: " + GROUP_PATTERN_REGEX);
+        }
+        this.maxGroupNumber = maxGroupNumber;
+        return this;
+    }
+
+    public RandomStudentGenerator setMinAverageGrade(double minAverageGrade) {
+        if (minAverageGrade < DEFAULT_MIN_GRADE || minAverageGrade > DEFAULT_MAX_GRADE) {
+            throw new IllegalArgumentException(
+                    "minAverageGrade out of Student allowed range [" + DEFAULT_MIN_GRADE + ", " + DEFAULT_MAX_GRADE + "]"
+            );
+        }
+        this.minAverageGrade = minAverageGrade;
+        return this;
+    }
+
+    public RandomStudentGenerator setMaxAverageGrade(double maxAverageGrade) {
+        if (maxAverageGrade < DEFAULT_MIN_GRADE || maxAverageGrade > DEFAULT_MAX_GRADE) {
+            throw new IllegalArgumentException(
+                    "maxAverageGrade out of Student allowed range [" + DEFAULT_MIN_GRADE + ", " + DEFAULT_MAX_GRADE + "]"
+            );
+        }
+        this.maxAverageGrade = maxAverageGrade;
+        return this;
+    }
+
+    public RandomStudentGenerator setMinRecordBookNumber(int minRecordBookNumber) {
+        if (minRecordBookNumber < DEFAULT_MIN_RECORD_BOOK) {
+            throw new IllegalArgumentException(
+                    "minRecordBookNumber must be >= " + DEFAULT_MIN_RECORD_BOOK
+            );
+        }
+        this.minRecordBookNumber = minRecordBookNumber;
+        return this;
+    }
+
+    public RandomStudentGenerator setMaxRecordBookNumber(int maxRecordBookNumber) {
+        if (maxRecordBookNumber < DEFAULT_MIN_RECORD_BOOK) {
+            throw new IllegalArgumentException(
+                    "maxRecordBookNumber must be >= " + DEFAULT_MIN_RECORD_BOOK
+            );
+        }
+        this.maxRecordBookNumber = maxRecordBookNumber;
+        return this;
+    }
+    public MyList<Student> generate(int size) {
+        if (size < 0) {
+            throw new IllegalArgumentException("size must be non-negative");
+        }
+        validateRanges();
+        MyList<Student> result = new MyLinkedList<>();
+        for (int i = 0; i < size; i++) {
+            String group = generateGroupNumber();
+            double grade = generateAverageGrade();
+            int recordBook = generateRecordBookNumber();
+
+            Student student = new StudentBuilder()
+                    .setGroupNumber(group)
+                    .setAverageGrade(grade)
+                    .setRecordBookNumber(recordBook)
+                    .build();
+
+            result.add(student);
+        }
+        return result;
+    }
+    private void validateRanges() {
+
+        double effectiveMinGrade = (minAverageGrade != null) ? minAverageGrade : DEFAULT_MIN_GRADE;
+        double effectiveMaxGrade = (maxAverageGrade != null) ? maxAverageGrade : DEFAULT_MAX_GRADE;
+        if (effectiveMinGrade > effectiveMaxGrade) {
+            throw new IllegalArgumentException("minAverageGrade cannot be greater than maxAverageGrade");
+        }
+        int effectiveMinRecord = (minRecordBookNumber != null) ? minRecordBookNumber : DEFAULT_MIN_RECORD_BOOK;
+        int effectiveMaxRecord = (maxRecordBookNumber != null) ? maxRecordBookNumber : Integer.MAX_VALUE;
+        if (effectiveMaxRecord == Integer.MAX_VALUE) {
+            effectiveMaxRecord = effectiveMinRecord + 999_999;
+        }
+        if (effectiveMinRecord > effectiveMaxRecord) {
+            throw new IllegalArgumentException("minRecordBookNumber cannot be greater than maxRecordBookNumber");
+        }
+        if (minGroupNumber != null && maxGroupNumber != null) {
+            if (minGroupNumber.compareTo(maxGroupNumber) > 0) {
+                throw new IllegalArgumentException("minGroupNumber cannot be greater than maxGroupNumber");
+            }
+        }
+    }
+    private String generateGroupNumber() {
+        String min = (minGroupNumber != null) ? minGroupNumber : "A00";
+        String max = (maxGroupNumber != null) ? maxGroupNumber : "Z99";
+        char minLetter = min.charAt(0);
+        char maxLetter = max.charAt(0);
+        int minDigit = Integer.parseInt(min.substring(1));
+        int maxDigit = Integer.parseInt(max.substring(1));
+        char letter = (char) (minLetter + RANDOM.nextInt((maxLetter - minLetter) + 1));
+        int digit;
+        if (letter == minLetter && letter == maxLetter) {
+            digit = minDigit + RANDOM.nextInt((maxDigit - minDigit) + 1);
+        } else if (letter == minLetter) {
+            digit = minDigit + RANDOM.nextInt(100 - minDigit);
+        } else if (letter == maxLetter) {
+            digit = RANDOM.nextInt(maxDigit + 1);
+        } else {
+            digit = RANDOM.nextInt(100);
+        }
+        return letter + String.format("%02d", digit);
+    }
+
+    private double generateAverageGrade() {
+        double min = (minAverageGrade != null) ? minAverageGrade : DEFAULT_MIN_GRADE;
+        double max = (maxAverageGrade != null) ? maxAverageGrade : DEFAULT_MAX_GRADE;
+
+        double range = max - min;
+        return min + (range * RANDOM.nextDouble());
+    }
+    private int generateRecordBookNumber() {
+        int min = (minRecordBookNumber != null) ? minRecordBookNumber : DEFAULT_MIN_RECORD_BOOK;
+        int max = (maxRecordBookNumber != null) ? maxRecordBookNumber : min + 999_999; // разумный максимум
+        if (max < min) {
+            max = min;
+        }
+        return min + RANDOM.nextInt((max - min) + 1);
+    }
+}

app/src/test/java/ru/project/AppTest.java

@@ -1,15 +0,0 @@
-package ru.project;
-
-import static org.junit.jupiter.api.Assertions.assertTrue;
-
-import org.junit.jupiter.api.Test;
-
-/** Unit test for simple App. */
-public class AppTest {
-
-  /** Rigorous Test :-) */
-  @Test
-  public void shouldAnswerWithTrue() {
-    assertTrue(true);
-  }
-}

app/src/test/java/ru/project/MyListCollectorTest.java

@@ -1,63 +0,0 @@
-package ru.test.maven.spring;
-import org.junit.jupiter.api.Test;
-import static org.junit.jupiter.api.Assertions.*;
-import java.util.Set;
-import java.util.stream.Collector;
-import java.util.stream.IntStream;
-import java.util.Collections;
-public class MyListCollectorTest
-{
-    private final MyListCollector<Integer> collector = new MyListCollector<>();
-
-    @Test
-    void shouldTestCollectInOrderSequential() {
-        MyList<Integer> result = IntStream.range(0, 10)
-                .boxed()
-                .collect(MyCollectors.toMyList());
-
-        assertEquals(10, result.size());
-        for (int i = 0; i < result.size(); i++) {
-            assertEquals(i, (int) result.get(i));
-        }
-
-    }
-
-    @Test
-    void shouldTestCollectInOrderParallel() {
-        MyList<Integer> result = IntStream.range(0, 100)
-                .parallel()
-                .boxed()
-                .collect(MyCollectors.toMyList());
-
-        assertEquals(100, result.size());
-        for (int i = 0; i < result.size(); i++) {
-            assertEquals(i, (int) result.get(i));
-        }
-    }
-
-    @Test
-    void shouldTestHandleEmptyStream() {
-        MyList<Integer> result = IntStream.empty()
-                .boxed()
-                .collect(MyCollectors.toMyList());
-
-        assertTrue(result.isEmpty());
-
-    }
-    @Test
-    void shouldTestNotUseStandardList() {
-        MyList<String> result = IntStream.range(0, 5)
-                .mapToObj(i -> "item" + i)
-                .collect(MyCollectors.toMyList());
-
-        assertInstanceOf(MyList.class, result);
-        assertFalse(result instanceof java.util.List);
-    }
-    @Test
-    void shouldTestHaveCorrectCharacteristics() {
-        Set<Collector.Characteristics> characteristics = collector.characteristics();
-        assertTrue(characteristics.contains(java.util.stream.Collector.Characteristics.IDENTITY_FINISH));
-        assertFalse(characteristics.contains(java.util.stream.Collector.Characteristics.UNORDERED));
-    }
-
-}

app/src/test/java/ru/project/collection/MyLinkedListTest.java

@@ -1,8 +1,10 @@
-package ru.test.maven.spring;
+package ru.project.collection;
-import org.junit.jupiter.api.Test;
+
 import static org.junit.jupiter.api.Assertions.*;
-public class MyLinkedListTest
+
-{
+import org.junit.jupiter.api.Test;
+
+public class MyLinkedListTest {
   @Test
   public void shouldTestAdd() {
     MyLinkedList<Integer> list = new MyLinkedList<>();
@@ -14,13 +16,14 @@ public class MyLinkedListTest
     assertEquals(20, list.get(1));
     assertEquals(30, list.get(2));
   }
+
   @Test
-    public void shouldTestGet()
+  public void shouldTestGet() {
-    {
     MyLinkedList<Integer> list = new MyLinkedList<>();
     list.add(10);
     assertEquals(10, list.get(0));
   }
+
   @Test
   public void shouldTestRemove() {
     MyLinkedList<Integer> list = new MyLinkedList<>();
@@ -33,6 +36,7 @@ public class MyLinkedListTest
     assertEquals(20, list.get(0));
     assertEquals(30, list.get(1));
   }
+
   @Test
   void shouldTestSet() {
     MyLinkedList<String> list = new MyLinkedList<>();
@@ -45,23 +49,23 @@ public class MyLinkedListTest
     assertEquals("B", oldValue);
     assertEquals("Z", list.get(1));
     assertEquals(3, list.size());
-
   }
+
   @Test
-    void shouldTestSize()
+  void shouldTestSize() {
-    {
     MyLinkedList<Integer> list = new MyLinkedList<>();
     list.add(1);
     list.add(2);
     list.add(3);
     assertEquals(3, list.size());
   }
+
   @Test
-    void shouldTestisEmpty()
+  void shouldTestisEmpty() {
-    {
     MyLinkedList<Integer> list = new MyLinkedList<>();
     assertTrue(list.isEmpty());
   }
+
   @Test
   void shouldTestGet_OutOfBounds_ThrowsException() {
     MyLinkedList<Integer> list = new MyLinkedList<>();
@@ -71,6 +75,7 @@ public class MyLinkedListTest
     assertThrows(IllegalArgumentException.class, () -> list.get(1)); // index == size
     assertThrows(IllegalArgumentException.class, () -> list.get(5)); // index > size
   }
+
   @Test
   void shouldTestSet_OutOfBounds_ThrowsException() {
     MyLinkedList<Integer> list = new MyLinkedList<>();
@@ -79,12 +84,14 @@ public class MyLinkedListTest
     assertThrows(IllegalArgumentException.class, () -> list.set(-1, 99));
     assertThrows(IllegalArgumentException.class, () -> list.set(1, 99));
   }
+
   @Test
   void shouldTestRemove_IndexGreaterThanSize_ThrowsException() {
     MyLinkedList<String> list = new MyLinkedList<>();
     list.add("man");
     assertThrows(IndexOutOfBoundsException.class, () -> list.remove(5));
   }
+
   @Test
   void shouldTestRemove_NegativeIndex_ThrowsException() {
     MyLinkedList<String> list = new MyLinkedList<>();
@@ -93,6 +100,7 @@ public class MyLinkedListTest
 
     assertThrows(IndexOutOfBoundsException.class, () -> list.remove(-1));
   }
+
   @Test
   void shouldTestaddAll_nonEmptySource_toEmptyList_addsAllElementsInOrder() {
     MyList<String> list = new MyLinkedList<>();

app/src/test/java/ru/project/collectors/MyListCollectorTest.java

@@ -1,34 +1,30 @@
-package ru.test.maven.spring;
+package ru.project.collectors;
-import org.junit.jupiter.api.Test;
+
 import static org.junit.jupiter.api.Assertions.*;
+
 import java.util.Set;
 import java.util.stream.Collector;
 import java.util.stream.IntStream;
-import java.util.Collections;
+import org.junit.jupiter.api.Test;
+import ru.project.collection.MyList;
 
-public class MyListCollectorTest
+public class MyListCollectorTest {
-{
   private final MyListCollector<Integer> collector = new MyListCollector<>();
 
   @Test
   void shouldTestCollectInOrderSequential() {
-        MyList<Integer> result = IntStream.range(0, 10)
+    MyList<Integer> result = IntStream.range(0, 10).boxed().collect(MyCollectors.toMyList());
-                .boxed()
-                .collect(MyCollectors.toMyList());
 
     assertEquals(10, result.size());
     for (int i = 0; i < result.size(); i++) {
       assertEquals(i, (int) result.get(i));
     }
-
   }
 
   @Test
   void shouldTestCollectInOrderParallel() {
-        MyList<Integer> result = IntStream.range(0, 100)
+    MyList<Integer> result =
-                .parallel()
+        IntStream.range(0, 100).parallel().boxed().collect(MyCollectors.toMyList());
-                .boxed()
-                .collect(MyCollectors.toMyList());
 
     assertEquals(100, result.size());
     for (int i = 0; i < result.size(); i++) {
@@ -38,27 +34,25 @@ public class MyListCollectorTest
 
   @Test
   void shouldTestHandleEmptyStream() {
-        MyList<Integer> result = IntStream.empty()
+    MyList<Integer> result = IntStream.empty().boxed().collect(MyCollectors.toMyList());
-                .boxed()
-                .collect(MyCollectors.toMyList());
 
     assertTrue(result.isEmpty());
-
   }
+
   @Test
   void shouldTestNotUseStandardList() {
-        MyList<String> result = IntStream.range(0, 5)
+    MyList<String> result =
-                .mapToObj(i -> "item" + i)
+        IntStream.range(0, 5).mapToObj(i -> "item" + i).collect(MyCollectors.toMyList());
-                .collect(MyCollectors.toMyList());
 
     assertInstanceOf(MyList.class, result);
     assertFalse(result instanceof java.util.List);
   }
+
   @Test
   void shouldTestHaveCorrectCharacteristics() {
     Set<Collector.Characteristics> characteristics = collector.characteristics();
-        assertTrue(characteristics.contains(java.util.stream.Collector.Characteristics.IDENTITY_FINISH));
+    assertTrue(
+        characteristics.contains(java.util.stream.Collector.Characteristics.IDENTITY_FINISH));
     assertFalse(characteristics.contains(java.util.stream.Collector.Characteristics.UNORDERED));
   }
-
 }

app/src/test/java/ru/project/finder/StudentOccurrenceIndexFinderTest.java

@@ -1,12 +1,14 @@
 package ru.project.finder;
 
-import org.junit.jupiter.api.BeforeEach;
-import org.junit.jupiter.api.Test;
-
 import static org.junit.jupiter.api.Assertions.*;
 
-public class StudentOccurrenceIndexFinderTest
+import org.junit.jupiter.api.BeforeEach;
-{
+import org.junit.jupiter.api.Test;
+import ru.project.collection.MyLinkedList;
+import ru.project.collection.MyList;
+import ru.project.student.Student;
+
+public class StudentOccurrenceIndexFinderTest {
   private StudentOccurrenceIndexFinder finder;
   private MyList<Student> students;
 
@@ -21,6 +23,7 @@ public class StudentOccurrenceIndexFinderTest
     int[] result = finder.findOccurrences(students, new Student("A12", 4.5, 1));
     assertEquals(0, result.length);
   }
+
   @Test
   void shouldTesfindOccurrences_noMatches_returnsEmptyArray() {
     students.add(new Student("A12", 4.0, 101));
@@ -30,6 +33,7 @@ public class StudentOccurrenceIndexFinderTest
     int[] result = finder.findOccurrences(students, target);
     assertEquals(0, result.length);
   }
+
   @Test
   void shouldTesfindOccurrences_singleMatch_returnsCorrectIndex() {
     students.add(new Student("A12", 4.0, 101));
@@ -42,6 +46,7 @@ public class StudentOccurrenceIndexFinderTest
     assertEquals(1, result.length);
     assertEquals(1, result[0]);
   }
+
   @Test
   void shouldTesfindOccurrences_multipleMatches_returnsAllIndicesSorted() {
     // индексы: 0, 2, 4 — должны вернуться в порядке возрастания
@@ -97,18 +102,17 @@ public class StudentOccurrenceIndexFinderTest
     assertEquals(0, result.length);
   }
 
-
   @Test
   void shouldTesfindOccurrences_nullList_throwsException() {
     Student target = new Student("A12", 4.5, 101);
     assertThrows(IllegalArgumentException.class, () -> finder.findOccurrences(null, target));
   }
 
-
   @Test
   void shouldTesfindOccurrences_nullTarget_throwsException() {
     assertThrows(IllegalArgumentException.class, () -> finder.findOccurrences(students, null));
   }
+
   @Test
   void shouldTesfindOccurrences_resultIsSorted() {
     String[] validGroups = {"A00", "A01", "B12", "C99", "X55"};
@@ -126,7 +130,6 @@ public class StudentOccurrenceIndexFinderTest
     Student target = new Student("A12", 4.5, 101);
     int[] result = finder.findOccurrences(students, target);
 
-
     for (int i = 1; i < result.length; i++) {
       assertTrue(result[i] > result[i - 1], "Результат должен быть отсортирован");
     }

app/src/test/java/ru/project/io/StudentCsvReaderTest.java

@@ -0,0 +1,118 @@
+package ru.project.io;
+import java.io.IOException;
+import java.nio.charset.StandardCharsets;
+import java.nio.file.Files;
+import java.nio.file.Path;
+import java.util.NoSuchElementException;
+import java.util.Spliterators;
+import java.util.stream.StreamSupport;
+import org.junit.jupiter.api.Test;
+import org.junit.jupiter.api.io.TempDir;
+import ru.project.student.Student;
+import ru.project.collection.MyList;
+import ru.project.collectors.MyCollectors;
+import static org.junit.jupiter.api.Assertions.*;
+class StudentCsvReaderTest {
+    @TempDir
+    Path tempDir;
+    @Test
+    void shouldTestreadValidCsvIntoMyList() throws IOException {
+        Path csv = tempDir.resolve("students.csv");
+        String content = String.join("\n",
+                "groupNumber,averageGrade,recordBookNumber",
+                "M10,4.5,12345",
+                "B05,3.8,12346"
+        );
+        Files.writeString(csv, content, StandardCharsets.UTF_8);
+        try (var reader = new StudentCsvReader(csv.toAbsolutePath())) {
+            var studentStream = StreamSupport.stream(
+                    Spliterators.spliteratorUnknownSize(reader, 0),
+                    false
+            );
+            MyList<Student> students = studentStream
+                    .collect(MyCollectors.toMyList());
+            assertEquals(2, students.size());
+            assertEquals("M10", students.get(0).getGroupNumber());
+            assertEquals(4.5, students.get(0).getAverageGrade(), 0.0);
+            assertEquals(12345, students.get(0).getRecordBookNumber());
+            assertEquals("B05", students.get(1).getGroupNumber());
+            assertEquals(3.8, students.get(1).getAverageGrade(), 0.0);
+            assertEquals(12346, students.get(1).getRecordBookNumber());
+        }
+    }
+    @Test
+    void shouldTestinvalidHeaderThrowsException() throws IOException {
+        Path csv = tempDir.resolve("bad-header.csv");
+        String content = """
+                badGroup,badGrade,badBook
+                M101,4.5,12345
+                """;
+        Files.writeString(csv, content, StandardCharsets.UTF_8);
+        IllegalArgumentException ex = assertThrows(
+                IllegalArgumentException.class,
+                () -> new StudentCsvReader(csv.toAbsolutePath())
+        );
+        assertTrue(ex.getMessage().contains("Invalid CSV header"));
+    }
+    @Test
+    void shouldTestwrongColumnCountThrowsException() throws IOException {
+        Path csv = tempDir.resolve("wrong-columns.csv");
+        String content = """
+                groupNumber,averageGrade,recordBookNumber
+                M101,4.5
+                """;
+        Files.writeString(csv, content, StandardCharsets.UTF_8);
+        try (var reader = new StudentCsvReader(csv.toAbsolutePath())) {
+            assertTrue(reader.hasNext());
+
+            IllegalArgumentException ex = assertThrows(
+                    IllegalArgumentException.class,
+                    reader::next
+            );
+            assertTrue(ex.getMessage().contains("Expected 3 columns"));
+        }
+    }
+    @Test
+    void shouldTestparseErrorThrowsExceptionWithDetails() throws IOException {
+        Path csv = tempDir.resolve("parse-error.csv");
+        String content = """
+                groupNumber,averageGrade,recordBookNumber
+                M101,not-a-number,12345
+                """;
+        Files.writeString(csv, content, StandardCharsets.UTF_8);
+        try (var reader = new StudentCsvReader(csv.toAbsolutePath())) {
+            assertTrue(reader.hasNext());
+
+            IllegalArgumentException ex = assertThrows(
+                    IllegalArgumentException.class,
+                    reader::next
+            );
+            assertTrue(ex.getMessage().contains("cannot parse averageGrade"));
+        }
+    }
+    @Test
+    void shouldTestemptyDataAfterHeaderThrowsNoSuchElement() throws IOException {
+        Path csv = tempDir.resolve("empty-data.csv");
+        String content = """
+                groupNumber,averageGrade,recordBookNumber
+                """;
+        Files.writeString(csv, content, StandardCharsets.UTF_8);
+
+        // ИСПРАВЛЕНО: добавлен try-with-resources
+        try (var reader = new StudentCsvReader(csv.toAbsolutePath())) {
+            assertFalse(reader.hasNext());
+
+            assertThrows(NoSuchElementException.class, reader::next);
+        }
+    }
+    @Test
+    void shouldTestrelativePathThrowsException() {
+        Path relative = Path.of("relative-path.csv");
+
+        IllegalArgumentException ex = assertThrows(
+                IllegalArgumentException.class,
+                () -> new StudentCsvReader(relative)
+        );
+        assertTrue(ex.getMessage().contains("absolute"));
+    }
+    }

app/src/test/java/ru/project/random/RandomStudentGeneratorTest.java

@@ -0,0 +1,157 @@
+package ru.project.random;
+import org.junit.jupiter.api.Test;
+import org.junit.jupiter.params.ParameterizedTest;
+import org.junit.jupiter.params.provider.ValueSource;
+import ru.project.collection.MyLinkedList;
+import ru.project.collection.MyList;
+import ru.project.student.Student;
+import java.util.regex.Pattern;
+import static org.junit.jupiter.api.Assertions.*;
+public class RandomStudentGeneratorTest
+{
+    private static final Pattern GROUP_PATTERN = Pattern.compile("^[A-Z][0-9]{2}$");
+    private static final double MIN_GRADE = 0.0;
+    private static final double MAX_GRADE = 5.0;
+    @Test
+    void shouldTestgenerate_withoutConstraints_returnsCorrectSizeAndValidStudents() {
+        int size = 100;
+        RandomStudentGenerator generator = new RandomStudentGenerator();
+        MyList<Student> students = generator.generate(size);
+        assertEquals(size, students.size(), "Размер списка должен совпадать с запрошенным");
+        for (int i = 0; i < students.size(); i++) {
+            Student s = students.get(i);
+            assertTrue(GROUP_PATTERN.matcher(s.getGroupNumber()).matches(),
+                    "groupNumber должен соответствовать шаблону A00–Z99");
+            assertTrue(s.getAverageGrade() >= MIN_GRADE && s.getAverageGrade() <= MAX_GRADE,
+                    "averageGrade должен быть в диапазоне [0.0, 5.0]");
+            assertTrue(s.getRecordBookNumber() >= 1,
+                    "recordBookNumber должен быть >= 1");
+        }
+    }
+    @Test
+    void shouldTestgenerate_withGroupRange_respectsLexicographicalBounds() {
+        RandomStudentGenerator generator = new RandomStudentGenerator()
+                .setMinGroupNumber("A00")
+                .setMaxGroupNumber("B99");
+        MyList<Student> students = generator.generate(200);
+        assertEquals(200, students.size());
+        for (int i = 0; i < students.size(); i++) {
+            Student s = students.get(i);
+            String g = s.getGroupNumber();
+            assertTrue(g.compareTo("A00") >= 0, "Группа не должна быть меньше A00");
+            assertTrue(g.compareTo("B99") <= 0, "Группа не должна быть больше B99");
+            assertTrue(GROUP_PATTERN.matcher(g).matches(), "Группа должна соответствовать шаблону");
+        }
+    }
+    @Test
+    void shouldTestgenerate_withGradeRange_respectsBounds() {
+        double minGrade = 3.0;
+        double maxGrade = 4.0;
+
+        RandomStudentGenerator generator = new RandomStudentGenerator()
+                .setMinAverageGrade(minGrade)
+                .setMaxAverageGrade(maxGrade);
+        MyList<Student> students = generator.generate(100);
+
+        for (int i = 0; i < students.size(); i++) {
+            Student s = students.get(i);
+            double g = s.getAverageGrade();
+            assertTrue(g >= minGrade && g <= maxGrade,
+                    "averageGrade должен быть в заданном диапазоне");
+        }
+    }
+    @Test
+    void shouldTestgenerate_withRecordBookRange_respectsBounds() {
+        int minRec = 10_000;
+        int maxRec = 20_000;
+
+        RandomStudentGenerator generator = new RandomStudentGenerator()
+                .setMinRecordBookNumber(minRec)
+                .setMaxRecordBookNumber(maxRec);
+        MyList<Student> students = generator.generate(100);
+
+        for (int i = 0; i < students.size(); i++) {
+            Student s = students.get(i);
+            int r = s.getRecordBookNumber();
+            assertTrue(r >= minRec && r <= maxRec,
+                    "recordBookNumber должен быть в заданном диапазоне");
+        }
+    }
+
+    @Test
+    void shouldTestgenerate_negativeSize_throwsException() {
+        RandomStudentGenerator generator = new RandomStudentGenerator();
+        assertThrows(IllegalArgumentException.class, () -> generator.generate(-1));
+    }
+    @Test
+    void shouldTestsetMinGroupNumber_invalidPattern_throwsException() {
+        RandomStudentGenerator generator = new RandomStudentGenerator();
+        assertThrows(IllegalArgumentException.class,
+                () -> generator.setMinGroupNumber("invalid"));
+    }
+    @Test
+    void shouldTestsetMaxGroupNumber_invalidPattern_throwsException() {
+        RandomStudentGenerator generator = new RandomStudentGenerator();
+        assertThrows(IllegalArgumentException.class,
+                () -> generator.setMaxGroupNumber("ZZ")); // не 2 цифры
+    }
+    @Test
+    void shouldTestsetMinMaxGroupNumber_minGreaterThanMax_throwsException() {
+        RandomStudentGenerator generator = new RandomStudentGenerator();
+        assertThrows(IllegalArgumentException.class,
+                () -> generator.setMinGroupNumber("C00").setMaxGroupNumber("A99").generate(1));
+    }
+    @Test
+    void shouldTestsetMinAverageGrade_outOfRange_throwsException() {
+        RandomStudentGenerator generator = new RandomStudentGenerator();
+        assertThrows(IllegalArgumentException.class,
+                () -> generator.setMinAverageGrade(-0.1));
+        assertThrows(IllegalArgumentException.class,
+                () -> generator.setMinAverageGrade(5.1));
+    }
+    @Test
+    void shouldTestsetMaxAverageGrade_outOfRange_throwsException() {
+        RandomStudentGenerator generator = new RandomStudentGenerator();
+        assertThrows(IllegalArgumentException.class,
+                () -> generator.setMaxAverageGrade(-0.1));
+        assertThrows(IllegalArgumentException.class,
+                () -> generator.setMaxAverageGrade(5.1));
+    }
+    @Test
+    void shouldTestsetMinMaxAverageGrade_minGreaterThanMax_throwsException() {
+        RandomStudentGenerator generator = new RandomStudentGenerator();
+        assertThrows(IllegalArgumentException.class,
+                () -> generator.setMinAverageGrade(4.0).setMaxAverageGrade(3.0).generate(1));
+    }
+    @Test
+    void shouldTestsetMinRecordBookNumber_lessThanAllowed_throwsException() {
+        RandomStudentGenerator generator = new RandomStudentGenerator();
+        assertThrows(IllegalArgumentException.class,
+                () -> generator.setMinRecordBookNumber(0));
+    }
+    @Test
+    void shouldTestsetMinMaxRecordBookNumber_minGreaterThanMax_throwsException() {
+        RandomStudentGenerator generator = new RandomStudentGenerator();
+        assertThrows(IllegalArgumentException.class,
+                () -> generator.setMinRecordBookNumber(20_000).setMaxRecordBookNumber(10_000).generate(1));
+    }
+    @Test
+    void shouldTestchainedBuilderStyle_worksAsExpected() {
+        RandomStudentGenerator generator = new RandomStudentGenerator()
+                .setMinGroupNumber("A00")
+                .setMaxGroupNumber("Z99")
+                .setMinAverageGrade(2.0)
+                .setMaxAverageGrade(4.0)
+                .setMinRecordBookNumber(1_000)
+                .setMaxRecordBookNumber(9_999);
+        MyList<Student> students = generator.generate(50);
+        assertEquals(50, students.size());
+        for (int i = 0; i < students.size(); i++) {
+            Student s = students.get(i);
+            assertTrue(s.getGroupNumber().compareTo("A00") >= 0
+                    && s.getGroupNumber().compareTo("Z99") <= 0);
+            assertTrue(s.getAverageGrade() >= 2.0 && s.getAverageGrade() <= 4.0);
+            assertTrue(s.getRecordBookNumber() >= 1_000 && s.getRecordBookNumber() <= 9_999);
+        }
+    }
+}

app/src/test/java/ru/project/search/StudentSearchTest.java

@@ -1,12 +1,12 @@
 package ru.project.search;
 
+import static org.junit.jupiter.api.Assertions.*;
+
 import org.junit.jupiter.api.Test;
 import ru.project.collection.MyList;
 import ru.project.collection.MyLinkedList;
 import ru.project.student.Student;
 
-import static org.junit.jupiter.api.Assertions.*;
-
 class StudentSearchTest {
 
   @Test
@@ -39,16 +39,13 @@ class StudentSearchTest {
 
   @Test
   void testGroupNumberRangeInvalidMinMaxThrowsException() {
-        assertThrows(IllegalArgumentException.class,
+    assertThrows(IllegalArgumentException.class, () -> new GroupNumberRangeCriteria("C30", "A10"));
-                () -> new GroupNumberRangeCriteria("C30", "A10"));
   }
 
   @Test
   void testGroupNumberRangeNullValuesThrowsException() {
-        assertThrows(IllegalArgumentException.class,
+    assertThrows(IllegalArgumentException.class, () -> new GroupNumberRangeCriteria(null, "A10"));
-                () -> new GroupNumberRangeCriteria(null, "A10"));
+    assertThrows(IllegalArgumentException.class, () -> new GroupNumberRangeCriteria("A10", null));
-        assertThrows(IllegalArgumentException.class,
-                () -> new GroupNumberRangeCriteria("A10", null));
   }
 
   @Test
@@ -88,8 +85,7 @@ class StudentSearchTest {
 
   @Test
   void testAverageGradeRangeInvalidMinMaxThrowsException() {
-        assertThrows(IllegalArgumentException.class,
+    assertThrows(IllegalArgumentException.class, () -> new AverageGradeRangeCriteria(5.0, 4.0));
-                () -> new AverageGradeRangeCriteria(5.0, 4.0));
   }
 
   @Test
@@ -129,27 +125,29 @@ class StudentSearchTest {
 
   @Test
   void testRecordBookNumberRangeInvalidMinMaxThrowsException() {
-        assertThrows(IllegalArgumentException.class,
+    assertThrows(IllegalArgumentException.class, () -> new RecordBookNumberRangeCriteria(200, 100));
-                () -> new RecordBookNumberRangeCriteria(200, 100));
   }
 
   @Test
   void testCompositeIntersectionEmptyReturnsTrue() {
-        CompositeStudentSearchCriteria composite = new CompositeStudentSearchCriteria(SearchOperation.INTERSECTION);
+    CompositeStudentSearchCriteria composite =
+        new CompositeStudentSearchCriteria(SearchOperation.INTERSECTION);
     Student student = new Student("A12", 4.5, 123);
     assertTrue(composite.matches(student));
   }
 
   @Test
   void testCompositeUnionEmptyReturnsFalse() {
-        CompositeStudentSearchCriteria composite = new CompositeStudentSearchCriteria(SearchOperation.UNION);
+    CompositeStudentSearchCriteria composite =
+        new CompositeStudentSearchCriteria(SearchOperation.UNION);
     Student student = new Student("A12", 4.5, 123);
     assertFalse(composite.matches(student));
   }
 
   @Test
   void testCompositeIntersectionAllTrue() {
-        CompositeStudentSearchCriteria composite = new CompositeStudentSearchCriteria(SearchOperation.INTERSECTION);
+    CompositeStudentSearchCriteria composite =
+        new CompositeStudentSearchCriteria(SearchOperation.INTERSECTION);
     composite.add(new GroupNumberRangeCriteria("A10", "C30"));
     composite.add(new AverageGradeRangeCriteria(4.0, 5.0));
     Student student = new Student("B20", 4.5, 123);
@@ -158,7 +156,8 @@ class StudentSearchTest {
 
   @Test
   void testCompositeIntersectionOneFalse() {
-        CompositeStudentSearchCriteria composite = new CompositeStudentSearchCriteria(SearchOperation.INTERSECTION);
+    CompositeStudentSearchCriteria composite =
+        new CompositeStudentSearchCriteria(SearchOperation.INTERSECTION);
     composite.add(new GroupNumberRangeCriteria("A10", "C30"));
     composite.add(new AverageGradeRangeCriteria(4.0, 5.0));
     Student student = new Student("B20", 3.5, 123);
@@ -167,7 +166,8 @@ class StudentSearchTest {
 
   @Test
   void testCompositeUnionOneTrue() {
-        CompositeStudentSearchCriteria composite = new CompositeStudentSearchCriteria(SearchOperation.UNION);
+    CompositeStudentSearchCriteria composite =
+        new CompositeStudentSearchCriteria(SearchOperation.UNION);
     composite.add(new GroupNumberRangeCriteria("A10", "C30"));
     composite.add(new AverageGradeRangeCriteria(4.0, 5.0));
     Student student = new Student("D00", 4.5, 123);
@@ -176,7 +176,8 @@ class StudentSearchTest {
 
   @Test
   void testCompositeUnionAllFalse() {
-        CompositeStudentSearchCriteria composite = new CompositeStudentSearchCriteria(SearchOperation.UNION);
+    CompositeStudentSearchCriteria composite =
+        new CompositeStudentSearchCriteria(SearchOperation.UNION);
     composite.add(new GroupNumberRangeCriteria("A10", "C30"));
     composite.add(new AverageGradeRangeCriteria(4.0, 5.0));
     Student student = new Student("D00", 3.5, 123);
@@ -185,10 +186,12 @@ class StudentSearchTest {
 
   @Test
   void testCompositeWithNestedComposite() {
-        CompositeStudentSearchCriteria outer = new CompositeStudentSearchCriteria(SearchOperation.INTERSECTION);
+    CompositeStudentSearchCriteria outer =
+        new CompositeStudentSearchCriteria(SearchOperation.INTERSECTION);
     outer.add(new GroupNumberRangeCriteria("A10", "C30"));
 
-        CompositeStudentSearchCriteria inner = new CompositeStudentSearchCriteria(SearchOperation.UNION);
+    CompositeStudentSearchCriteria inner =
+        new CompositeStudentSearchCriteria(SearchOperation.UNION);
     inner.add(new AverageGradeRangeCriteria(4.0, 5.0));
     inner.add(new RecordBookNumberRangeCriteria(100, 200));
 
@@ -280,8 +283,7 @@ class StudentSearchTest {
 
   @Test
   void testSearchServiceWithNullCriteriaThrowsException() {
-        assertThrows(IllegalArgumentException.class,
+    assertThrows(IllegalArgumentException.class, () -> new StudentSearchService(null));
-                () -> new StudentSearchService(null));
   }
 
   @Test
@@ -291,7 +293,8 @@ class StudentSearchTest {
     students.add(new Student("B20", 4.5, 2));
     students.add(new Student("C30", 3.5, 3));
 
-        CompositeStudentSearchCriteria composite = new CompositeStudentSearchCriteria(SearchOperation.INTERSECTION);
+    CompositeStudentSearchCriteria composite =
+        new CompositeStudentSearchCriteria(SearchOperation.INTERSECTION);
     composite.add(new GroupNumberRangeCriteria("A10", "B20"));
     composite.add(new AverageGradeRangeCriteria(4.0, 5.0));