【架构优化】用 slices.Contains 简化 runeval/facts.go 中的 appendUnique 并补充单元测试

Đang mở
  • Có gì không ổn?
#416 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.

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

Đánh giá

Độ khó
1/5
Thời gian dự kiến
1-3 giờ
Mức phù hợp với người mới
30/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

Read internal/runeval/facts.go and inspect the existing appendUnique function and its BuildFacts caller. An open pull request #417 is already linked, so check its status before taking up the work; the issue’s stated completion criteria are the table-driven tests for the listed cases and passing gofmt, go vet ./internal/runeval/, and go test -cover ./internal/runeval/.

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

Mô tả

现状问题

internal/runeval/facts.go 中的 appendUnique 用手写 for 循环判断切片是否已包含某值，再决定是否追加。Go 1.21+ 标准库 slices.Contains 已提供等价能力（仓库 go.mod 为 1.26），手写循环增加了阅读负担，与仓库内其他已统一到 slices.Contains 的写法不一致。

重构方案

  • appendUnique 改为 if slices.Contains(list, value) { return list }; return append(list, value)，导入 slices。
  • 函数签名与语义（已存在则原样返回、不存在追加到末尾、保持顺序）不变，唯一调用方 BuildFacts 无需改动。
  • 新增表驱动单元测试覆盖：空切片/nil、已存在、不存在、追加后顺序、空字符串值。

影响范围

仅 internal/runeval/facts.go 一个函数及新增测试文件；不改对外接口与行为。

验证方式

gofmt、go vet ./internal/runeval/、go test -cover ./internal/runeval/。

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.