Skip to content

Tech debt: ImportService.importStrictdoc() is 150+ lines with three phases in one method #325

@Brad-Edwards

Description

@Brad-Edwards

Description

ImportService.importStrictdoc() is a 150+ line method that performs three distinct phases in sequence:

  1. Upsert requirements
  2. Create relations
  3. Create traceability links

Each phase has its own loop, error handling, and data transformations. The method is difficult to read, test in isolation, and maintain.

Recommendation: Extract each phase into a private method (e.g., upsertRequirements(), createRelations(), createTraceabilityLinks()). The orchestrating method becomes a clear three-step pipeline.

Location: ImportService.java, importStrictdoc() method

Traced Requirements

  • GC-B006: StrictDoc Import

Impact

Maintainability — long methods are where bugs hide and where changes are most error-prone.

Metadata

Metadata

Assignees

No one assigned

    Labels

    tech-debtTechnical debt and code quality improvements

    Type

    No type

    Projects

    No projects

    Milestone

    No milestone

    Relationships

    None yet

    Development

    No branches or pull requests

    Issue actions