Phân tích luồng sau chuyển — warehouse/import-excel-validate — vòng 1

Viết tắt: SVC = easyposbase:src/main/java/sds/ep/baseservice/services/impl/BaseWarehouseServiceImpl.java. Đường dẫn mới tính từ C:/Projects/warehouse-service/src/main/java/com/softdreams/easypos/warehouse/. VS = application/warehouse/WarehouseImportValidationService.java, XR = infrastructure/excel/PoiExcelImportReader.java, CK = domain/warehouse/policy/WarehouseImportSheetChecker.java, EP = domain/warehouse/policy/WarehouseImportExistingPolicy.java, WC = presentation/web/warehouse/WarehouseController.java.

F-01 Đọc file Excel (adapter POI)

• Điểm vào: cũ easyposbackend1:src/main/java/vn/softdreams/easypos/web/rest/WarehouseResource.java:98-107 → mới WC:206-230
• Quy tắc: R-01, R-02, R-03, R-04, R-05, R-06

Sơ đồ luồng mới (theo code thực tế)

sequenceDiagram
    participant C as Client
    participant WC as WarehouseController.validateImport
    participant VS as WarehouseImportValidationService.validateImport
    participant XR as PoiExcelImportReader
    participant SH as PoiExcelSheet
    C->>WC: file, comId (@RequestParam), WarehouseImportSheetRequest(indexSheet, type)
    WC->>WC: indexSheet null → BadRequestException(INVALID_REQUEST)
    WC->>VS: WarehouseImportValidateCommand(comId, indexSheet, type, file::getInputStream)
    VS->>XR: open(file, sheetIndex)
    XR->>XR: load: try(stream; WorkbookFactory.create) → đóng ngay; IOException|RuntimeException → Optional.empty
    XR->>XR: sheetOf: getSheetAt; IllegalArgumentException → SystemException(EXCEPTION_ERROR)
    VS-->>WC: empty → failed(INVALID_FILE)
    VS->>SH: lastRowNum() == 0 → failed(LIST_EMPTY)
    VS->>SH: headerCellCount(): rỗng hoặc ≠ 7 → BadRequestException(WAREHOUSE_IMPORT_INVALID_FILE)
    loop rowIndex = 1..min(lastRowNum, 2000), khi reading
        VS->>SH: isMergedRow(i) → bỏ qua
        VS->>SH: readRow(i, 7): getRow null → missingRow; mỗi ô evaluate (lỗi → SystemException) + rawValue + present/numeric
    end

Đối chiếu từng bước

Dữ liệu và side effect

Transaction, khoá, idempotency, tenant

• Ranh giới transaction: cũ là transaction cấp class bao cả phần đọc POI → mới @Transactional ở VS:63, cũng bao cả phần đọc POI (D-14).
• Khoá / cập nhật nguyên tử: không có ở cả hai bên.
• Idempotency, event: không có.
• Lọc comId: F-01 không truy vấn DB.

Lệch so với luồng mới đã duyệt (spec mục 6)

indexSheet/type nhận qua WarehouseImportSheetRequest thay vì @RequestParam.

Dev cần chú ý khi review

F-02 Kiểm dòng — tạo mới (type=1)

• Điểm vào: cũ SVC:849 → mới VS:63-84
• Quy tắc: R-05, R-07, R-08, R-09, R-11, R-13, R-14, R-15

Sơ đồ luồng mới (theo code thực tế)

sequenceDiagram
    participant VS as WarehouseImportValidationService
    participant CK as WarehouseImportSheetChecker.check
    participant CC as CurrentCompanyService
    participant WR as WarehouseRepository
    participant EP as WarehouseImportExistingPolicy.checkCreate
    participant WC as WarehouseController
    loop mỗi dòng
        VS->>CK: check(row): checkRawValue (Mã: trùng Set; Tên: rỗng/trùng Set; Trạng thái/Bán: thô rỗng → true) rồi assign(cleaned) nếu present
    end
    VS->>VS: lines rỗng → failed(LIST_ERROR)
    VS->>VS: skus/names của dòng hợp lệ; cả hai rỗng → bỏ qua
    VS->>CC: requireCompanyOfCurrentUser(comId)
    VS->>WR: findAllByNameIn(comId, nonEmpty(names)) (rỗng → SystemException)
    VS->>WR: findAllBySkuIn(comId, nonEmpty(skus)) (rỗng → SystemException)
    VS->>EP: checkCreate: toMap (trùng khoá → SystemException); tên có → NAME_EXISTS; còn hợp lệ và SKU có → SKU_EXISTS
    VS-->>WC: success(lines) → count, countValid/countInvalid
    WC->>WC: toImportValidateResponse (status null → false, sellable giữ nguyên)

Đối chiếu từng bước

Dữ liệu và side effect

Transaction, khoá, idempotency, tenant

• Ranh giới transaction: cũ cấp class → mới @Transactional (không readOnly) ở VS:63, D-14.
• Khoá: không có.
• Idempotency, event: không có.
• Lọc comId: cả hai truy vấn đều có com_id; mới thêm bước kiểm công ty của người đăng nhập (D-01).

Lệch so với luồng mới đã duyệt (spec mục 6)

Phần đánh dấu lỗi DB tách sang WarehouseImportExistingPolicy (domain), không nằm trong application service.

Dev cần chú ý khi review

F-03 Kiểm dòng — cập nhật (type=2)

• Điểm vào: cũ SVC:849 → mới VS:63-84
• Quy tắc: R-05, R-07, R-08, R-09, R-10, R-13, R-14, R-15

Sơ đồ luồng mới (theo code thực tế)

sequenceDiagram
    participant VS as WarehouseImportValidationService
    participant CK as WarehouseImportSheetChecker.check
    participant CC as CurrentCompanyService
    participant WR as WarehouseRepository
    participant EP as WarehouseImportExistingPolicy.checkUpdate
    loop mỗi dòng
        VS->>CK: check(row): Mã thô rỗng → SKU_EMPTY_ON_UPDATE; Tên rỗng; không kiểm trùng trong file
    end
    VS->>VS: lines rỗng → LIST_ERROR; skus/names rỗng cả hai → bỏ qua
    VS->>CC: requireCompanyOfCurrentUser(comId)
    VS->>WR: findAllBySkuIn(comId, nonEmpty(skus)) (rỗng → SystemException)
    VS->>EP: checkUpdate: toMap; hợp lệ và không có SKU → SKU_NOT_FOUND; có SKU → code null? SystemException; WH1 && hợp lệ && !status && sellable null? SystemException : !sellable → DEFAULT_WAREHOUSE_LOCKED

Đối chiếu từng bước

Dữ liệu và side effect

Transaction, khoá, idempotency, tenant

• Ranh giới transaction: như F-02 (D-14).
• Khoá: không có.
• Idempotency, event: không có.
• Lọc comId: có; thêm kiểm công ty (D-01).

Lệch so với luồng mới đã duyệt (spec mục 6)

Như F-02. Ngoài ra nhánh UPDATE cũng dùng nonEmpty(skus), spec không mô tả (F-logic-1).

Dev cần chú ý khi review

F-04 Kiểm dòng — type khác / không gửi

• Điểm vào: cũ SVC:849 → mới VS:63-84
• Quy tắc: R-05, R-07, R-08, R-12, R-13, R-15

Sơ đồ luồng mới (theo code thực tế)

sequenceDiagram
    participant VS as WarehouseImportValidationService
    participant CK as WarehouseImportSheetChecker.check
    participant WC as WarehouseController
    loop mỗi dòng
        VS->>CK: check(row): chỉ kiểm Tên rỗng và SĐT
    end
    VS->>VS: lines rỗng → LIST_ERROR
    VS->>VS: checkExisting: !update && !create → return (không kiểm công ty, không truy vấn)
    VS-->>WC: success(lines)

Đối chiếu từng bước

Dữ liệu và side effect

Transaction, khoá, idempotency, tenant

• Ranh giới transaction: @Transactional mở nhưng không có truy vấn.
• Khoá: không có.
• Idempotency, event: không có.
• Lọc comId: không truy vấn nên không kiểm công ty, đúng D-01.

Lệch so với luồng mới đã duyệt (spec mục 6)

không

Dev cần chú ý khi review