【架构优化】从 businessExecutionDuration 中拆出区间并集求和 unionDuration

Đang mở
  • Có gì không ổn?
#436 0 bình luận 0 reaction 0 người được giao Xem trên GitHub

Maintainer thường phản hồi trong vòng 1 ngày

@fx-kurumi đang làm issue này rồi.

Từ ngày 13/11/2026.

  • #437 của @fx-kurumi — đang mở

Đánh giá

Độ khó
2/5
Thời gian dự kiến
1-3 giờ
Mức phù hợp với người mới
25/100
Loại issue
Tái cấu trúc
Độ rõ ràng
Đặc tả rõ ràng
Mức độ hoạt động
Đình trệ
Công nghệ
go
Lĩnh vực
backend

Hướng nghiên cứu

The logic lives in businessExecutionDuration in internal/repo/analytics_business_delivery.go, at the end of the function where intervals are sorted, merged and summed. Read that function, then extract the merge-and-sum loop into a package-private unionDuration helper. Done means the helper has table-driven tests for empty, disjoint, overlapping, nested, touching and unsorted input, and go test ./internal/repo/ passes. Linked PR #437 already covers this work, so check with maintainers before starting.

Do mô hình lập chỉ mục viết ra từ nội dung của issue.

Mô tả

现状问题

internal/repo/analytics_business_delivery.go 中 businessExecutionDuration 约 85 行，同时承担三件事：筛选可计入的 run、逐 run 收集并裁剪执行区间、最后对区间排序 → 合并重叠 → 求总时长。末尾的「区间并集求和」是一段与业务无关的纯算法（约 15 行双层循环），内联在长函数里，既不便阅读，也无法单独测试。

重构方案

在同文件内抽出包级纯函数：

// unionDuration 返回区间并集的总时长（重叠/相接的区间只计一次）。
func unionDuration(intervals []domain.RunExecutionInterval) time.Duration

businessExecutionDuration 末尾改为 value := unionDuration(intervals).Milliseconds()。排序、合并条件（!StartedAt.After(FinishedAt)，相接也合并）逐字保留。

影响范围

  • 仅 internal/repo/analytics_business_delivery.go，包内私有函数，无对外接口变化，无行为变化。
  • 不改 domain/run_timing.go 中同类逻辑（跨包，不在本次范围）。

验证方式

  • 新增 unionDuration 表驱动单测：空输入、单区间、不相交、重叠、包含、首尾相接、乱序输入。
  • 既有 TestBusinessExecutionDuration* 等用例保持通过；go vet ./internal/repo/、go test ./internal/repo/。
Ngôn ngữ chính
Go
Star
141
Fork
28
Merge trung bình
1 ngày 15 giờ
Pull request đã merge (30 ngày)
113

Chuẩn bị môi trường

Bắt đầu từ đâu

  1. Đọc hết issue, rồi đọc hướng dẫn đóng góp của dự án.
  2. Bình luận trên issue rằng bạn sẽ nhận — tránh hai người làm cùng một việc.
  3. Fork repository và làm thay đổi trên một nhánh.
  4. Mở pull request có tham chiếu số hiệu của issue.

Issue khác của Tencent/LoopForge

Tất cả issue của Tencent/LoopForge

Issue tương tự

Thêm issue về Go

Nhận issue mới trong hộp thư của bạn

Bản tóm tắt ngắn những issue GitHub phù hợp với người mới.