From 29a4d01ff49c0bcea82b366c6c6a800b60cb2ef8 Mon Sep 17 00:00:00 2001
From: =?UTF-8?q?=D0=94=D0=BC=D0=B8=D1=82=D1=80=D0=B8=D0=B9?=
 <donnalee11681676@gmail.com>
Date: Sat, 16 May 2026 17:51:27 +0300
Subject: [PATCH] =?UTF-8?q?fix(import):=20Task=205=20code-review=20?=
 =?UTF-8?q?=E2=80=94=20final-=D0=BA=D0=BB=D0=B0=D1=81=D1=81=20CsvLeadsPars?=
 =?UTF-8?q?er=20+=20self::EXPECTED=5FCOLUMNS?=
MIME-Version: 1.0
Content-Type: text/plain; charset=UTF-8
Content-Transfer-Encoding: 8bit

Code-review Task 5 (non-blocking 🟡): CsvLeadsParser объявлен final (симметрия
с DTO ParsedLeadRow/CsvParseResult, утилитарный класс без наследования);
строка ошибки про число колонок использует self::EXPECTED_COLUMNS вместо
литерала 9.

Co-Authored-By: Claude Opus 4.7 (1M context) <noreply@anthropic.com>
---
 app/app/Services/Import/CsvLeadsParser.php | 4 ++--
 1 file changed, 2 insertions(+), 2 deletions(-)

diff --git a/app/app/Services/Import/CsvLeadsParser.php b/app/app/Services/Import/CsvLeadsParser.php
index 518695f5..6c297865 100644
--- a/app/app/Services/Import/CsvLeadsParser.php
+++ b/app/app/Services/Import/CsvLeadsParser.php
@@ -17,7 +17,7 @@ use Throwable;
  * Невалидные строки не роняют парсинг — собираются в errors[].
  * Файл целиком загружается в память (MVP: ожидаемый объём — единицы тысяч строк).
  */
-class CsvLeadsParser
+final class CsvLeadsParser
 {
     private const EXPECTED_COLUMNS = 9;
 
@@ -45,7 +45,7 @@ class CsvLeadsParser
             $cells = str_getcsv($rawLine);
 
             if (count($cells) < self::EXPECTED_COLUMNS) {
-                $errors[] = ['line' => $dataLine, 'message' => 'Ожидалось 9 колонок, получено '.count($cells)];
+                $errors[] = ['line' => $dataLine, 'message' => 'Ожидалось '.self::EXPECTED_COLUMNS.' колонок, получено '.count($cells)];
 
                 continue;
             }