修复 WxMaExpressOrderCargo 字段类型以支持小数值#3802
Merged
binarywang merged 2 commits intodevelopfrom Dec 10, 2025
Merged
Conversation
… 字段类型从 Integer 改为 Double 以支持小数值 Co-authored-by: binarywang <1343140+binarywang@users.noreply.github.com>
Copilot
AI
changed the title
[WIP] Fix variable type issues in WxMaExpressOrderCargo class
修复 WxMaExpressOrderCargo 字段类型以支持小数值
Dec 10, 2025
There was a problem hiding this comment.
Pull request overview
此 PR 修复了 WxMaExpressOrderCargo 类中的字段类型问题,使其符合微信官方 API 规范并支持小数值。原实现使用 Integer 类型存储重量和尺寸,无法处理诸如 1.2kg 或 20.5cm 等小数值,现已更正为 Double 类型。
核心变更
- 将 4 个物理测量字段从
Integer改为Double:weight、spaceLength、spaceWidth、spaceHeight - 更新测试用例使用小数值验证新类型(1.2kg、10.0cm、20.0cm、15.0cm)
Reviewed changes
Copilot reviewed 2 out of 2 changed files in this pull request and generated no comments.
| File | Description |
|---|---|
| WxMaExpressOrderCargo.java | 将 weight、spaceLength、spaceWidth、spaceHeight 字段类型从 Integer 改为 Double 以支持小数值 |
| WxMaExpressServiceImplTest.java | 更新测试用例使用小数值(1.2、10.0、20.0、15.0)验证字段类型变更 |
审查结论:代码变更正确且完整,符合项目规范。类型变更向后兼容(整数值可自动装箱为 Double),测试覆盖充分,注解使用正确(@SerializedName 配置 snake_case),且遵循了现有的 Lombok 注解模式(@DaTa、@NoArgsConstructor、@AllArgsConstructor)。无需额外修改。
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
微信官方 API 文档规定
cargo对象中的weight、space_x、space_y、space_z字段类型为number,支持小数值(如 1.2kg、20.0cm)。但原实现使用Integer类型,导致无法处理小数。变更内容
WxMaExpressOrderCargo.java: 将以下字段类型从
Integer改为Doubleweight- 货物总重量(单位:kg)spaceLength(space_x) - 货物长度(单位:cm)spaceWidth(space_y) - 货物宽度(单位:cm)spaceHeight(space_z) - 货物高度(单位:cm)WxMaExpressServiceImplTest.java: 更新测试用例使用小数值
示例
兼容性
向后兼容。整数值自动装箱为 Double(
5→5.0),现有代码无需修改。Original prompt
💬 We'd love your input! Share your thoughts on Copilot coding agent in our 2 minute survey.