Skip to content

refactor(core): 收拢四化事实组装 - #304

Merged
lzm0x219 merged 1 commit into
mainfrom
codex/transformation-facts-module
Aug 7, 2026
Merged

refactor(core): 收拢四化事实组装#304
lzm0x219 merged 1 commit into
mainfrom
codex/transformation-facts-module

Conversation

@lzm0x219

@lzm0x219 lzm0x219 commented Aug 7, 2026

Copy link
Copy Markdown
Member

变更

  • 将四化关系、生年四化和星曜自化的组装收拢到私有 TransformationFacts 模块
  • Natal 构造路径只负责编排,通过 BranchStarName 读取四化事实
  • 收紧 PalacePlacement 字段可见性,并集中宫名坐标查询

原因

原来的 natal.rs 同时承担完整命盘编排与四化关系计算,并直接访问三组平行数组。此次重构将四化规则和存储细节集中到一个内部模块,使构造流程与关系计算各自保持单一职责。

影响

不修改公开接口、排盘算法、领域事实或结果物理表示。两个构造入口继续保持零堆分配。

验证

  • cargo fmt --all --check
  • cargo clippy --workspace --all-targets --all-features --locked -- -D warnings
  • cargo test --workspace --all-targets --locked:53 passed,1 ignored
  • cargo test --workspace --doc --locked:1 passed
  • cargo test -p ziwei --test natal_exhaustive --release --locked -- --ignored:1 passed
  • RUSTDOCFLAGS="-D warnings" cargo doc --workspace --no-deps --locked
  • cargo bench -p ziwei_core --bench natal_allocations --locked:两个入口均为 0 allocation / 0 byte

@lzm0x219 lzm0x219 added the type: refactor 不应改变外部行为的代码或结构调整 label Aug 7, 2026
@lzm0x219
lzm0x219 merged commit c6630ae into main Aug 7, 2026
2 checks passed
@lzm0x219
lzm0x219 deleted the codex/transformation-facts-module branch August 7, 2026 02:51
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

type: refactor 不应改变外部行为的代码或结构调整

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant