修正 size 为文件大小，添加已下载大小 #1
Conversation
Walkthrough该拉取请求对 Changes
Sequence Diagram(s)sequenceDiagram
participant User
participant Core
participant Torrent
User->>Core: 请求活动种子信息
Core->>Torrent: 获取种子信息
Torrent-->>Core: 返回种子信息（包括completed_size）
Core-->>User: 返回活动种子信息
📜 Recent review detailsConfiguration used: CodeRabbit UI 📒 Files selected for processing (2)
🚧 Files skipped from review as they are similar to previous changes (2)
Thank you for using CodeRabbit. We offer it for free to the OSS community and would appreciate your support in helping us grow. If you find it useful, would you consider giving us a shout-out on your favorite social media? 🪧 TipsChatThere are 3 ways to chat with CodeRabbit:
Note: Be mindful of the bot's finite context window. It's strongly recommended to break down tasks such as reading entire modules into smaller chunks. For a focused discussion, use review comments to chat about specific files and their changes, instead of using the PR comments. CodeRabbit Commands (Invoked using PR comments)
Other keywords and placeholders
CodeRabbit Configuration File (
|
![coderabbitai[bot]](https://avatars.githubusercontent.com/in/347564?s=60&v=4){kind=small}
There was a problem hiding this comment.
Actionable comments posted: 0
🧹 Outside diff range and nitpick comments (2)
deluge_peerbanhelperadapter/model/torrent.py (1)
83-85: 建议添加字段说明注释
completed_size字段的实现正确，但建议补充更详细的注释说明，包括：
- 字段的具体含义和单位（例如：字节）
- 与
size字段的关系（已下载大小不应超过总大小）建议修改注释如下：
- # 已下载的大小 + # 已下载的大小（字节） + # 注意：completed_size 始终小于等于 size completed_size: int = 0deluge_peerbanhelperadapter/core.py (1)
136-142: 建议添加属性文档说明建议为
completed_size和size属性添加文档字符串，说明它们的具体含义和单位，这样可以帮助其他开发者更好地理解和使用这些属性。示例文档：
def get_active_torrents_info(self): - """返回活跃种子的列表""" + """返回活跃种子的列表 + + Returns: + list: 包含以下属性的种子信息列表： + - completed_size: 已下载的数据大小（字节） + - size: 种子总大小（字节） + ... + """Also applies to: 212-216
📜 Review details
Configuration used: CodeRabbit UI
Review profile: CHILL
📒 Files selected for processing (2)
deluge_peerbanhelperadapter/core.py(2 hunks)deluge_peerbanhelperadapter/model/torrent.py(1 hunks)
🔇 Additional comments (4)
deluge_peerbanhelperadapter/model/torrent.py (1)
Line range hint 77-84: 验证 size 和 completed_size 的一致性
需要确保在运行时 completed_size 不会超过 size。
✅ Verification successful
size 和 completed_size 的赋值逻辑是安全的
根据代码分析，completed_size 和 size 的赋值来源是安全的：
size通过torrent_info.total_size()获取，这是种子的总大小completed_size通过deluge_torrent.status.total_wanted_done获取，这是已下载的大小
由于这些值都是从 Deluge 客户端直接获取的，Deluge 本身会确保下载进度的正确性，因此不需要额外的验证逻辑。这是一个标准的 Deluge API 使用方式，可以保证 completed_size 不会超过 size。
🏁 Scripts executed
The following scripts were executed for the analysis:
Script:
#!/bin/bash
# 描述：检查是否有相关的验证逻辑
# 搜索与 size 和 completed_size 相关的验证代码
rg -A 5 "def.*size|completed_size.*size|size.*completed_size"
Length of output: 63
Script:
#!/bin/bash
# 搜索与 size 和 completed_size 相关的代码，使用更宽松的模式
rg -A 5 "size.*=|completed_size.*=|progress.*="
# 搜索可能的验证逻辑
ast-grep --pattern 'if $_ > $_: $$$'
Length of output: 6236
Script:
#!/bin/bash
# 搜索相关的方法和属性定义
ast-grep --pattern 'def set_$_($$$): $$$'
# 搜索可能的赋值操作
ast-grep --pattern '$_.size = $_'
ast-grep --pattern '$_.completed_size = $_'
Length of output: 545
deluge_peerbanhelperadapter/core.py (3)
136-142: 修改正确地区分了总大小和已完成大小！
代码变更符合以下要求：
- 使用
total_wanted_done正确表示已下载的数据大小 - 使用
total_size()准确反映种子的总大小
212-216: 实现与 get_active_torrents_info 保持一致
代码变更保持了与活动种子信息获取方法相同的实现逻辑，确保了整个代码库的一致性。
Line range hint 151-154: 优秀的错误处理实现！
通过添加 UnicodeDecodeError 异常处理，防止了客户端名称解码失败导致的程序崩溃，并提供了合理的默认值 "unknown"。这种防御性编程的做法值得肯定。
|
考虑用其他方法？ |
This reverts commit 1bf5036.
作为 PBH-BTN/PeerBanHelper#748 的一部分
关联：PBH-BTN/PeerBanHelper#700
修复：PBH-BTN/PeerBanHelper#746
Summary by CodeRabbit
新功能
completed_size属性，以更好地跟踪下载进度。错误修复