GeezerVison is showing github.com — 修复 WxMaExpressOrderCargo 字段类型以支持小数值 by Copilot · Pull Request #3802 · binarywang/WxJava
Conversation
Copilot
AI
commented
Dec 10, 2025
•
edited
Loading
Uh oh!
There was an error while loading. Please reload this page.
微信官方 API 文档规定 cargo 对象中的 weight、space_x、space_y、space_z 字段类型为 number,支持小数值(如 1.2kg、20.0cm)。但原实现使用 Integer 类型,导致无法处理小数。
变更内容
示例
WxMaExpressOrderCargo cargo = new WxMaExpressOrderCargo();
cargo.setWeight(1.2); // 支持小数,符合官方文档示例
cargo.setSpaceLength(20.0);
cargo.setSpaceWidth(15.0);
cargo.setSpaceHeight(10.0);
兼容性
向后兼容。整数值自动装箱为 Double(5 → 5.0),现有代码无需修改。
Original prompt
This section details on the original issue you should resolve
<issue_title>WxMaExpressOrderCargo 类中变量类型问题</issue_title>
<issue_description>WxMaExpressOrderCargo类中的weight,spaceLength,,spaceWidth 不支持小数
</issue_description>
Comments on the Issue (you are @copilot in this section)
@binarywang
原始问题来自:https://gitee.com/binary/weixin-java-tools/issues/IDBUKJ
Custom agent used: 全部用中文
需要用中文,包括PR标题和分析总结过程
💬 We'd love your input! Share your thoughts on Copilot coding agent in our 2 minute survey.
Sorry, something went wrong.
Uh oh!
There was an error while loading. Please reload this page.
Copilot
AI
changed the title
[WIP] Fix variable type issues in WxMaExpressOrderCargo class
修复 WxMaExpressOrderCargo 字段类型以支持小数值
Dec 10, 2025
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
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)。无需额外修改。
Sorry, something went wrong.
Uh oh!
There was an error while loading. Please reload this page.
Uh oh!
There was an error while loading. Please reload this page.
Reviewers
binarywang
Awaiting requested review from binarywang
Uh oh!
There was an error while loading. Please reload this page.
Development
Successfully merging this pull request may close these issues.
WxMaExpressOrderCargo 类中变量类型问题
Uh oh!
There was an error while loading. Please reload this page.
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.