又给腾讯贡献了一段代码:从 `%` 格式化到 f-string 的改进

大家好,这里是程序员晚枫,全网同名。

最近,我在优化自己的开源项目:poocr,这个项目底层调用的是腾讯的文字识别功能,帮助小白用户轻松实现1行代码,批量识别。

在调用腾讯云 SDK 的源码时,发现了一个可以优化的地方。这个优化虽然看似微小,但却能提升代码的可读性和性能。我决定动手修改,并提交了一个 PR,希望为腾讯云的开源项目贡献一份力量。

发现问题

在阅读腾讯云 SDK 的源码时,我注意到一段代码中使用了 % 格式化来构造报错信息。代码如下:

python 复制代码
# GitHub:tencentcloud-sdk-python/tencentcloud/common/exception/tencent_cloud_sdk_exception.py - Line 14
s = "[TencentCloudSDKException] code:%s message:%s requestId:%s" % (self.code, self.message, self.requestId)

这段代码的功能是生成一个包含错误代码、错误消息和请求 ID 的字符串。虽然代码本身没有问题,但我意识到,使用 % 格式化的方式在现代 Python 中已经显得有些过时。Python 3.6 引入了 f-string(格式化字符串字面量),它不仅更简洁,而且在性能上也更优。

提出优化

我决定将这段代码改为使用 f-string 的方式,以提高代码的可读性和性能。

修改后的代码如下:

python 复制代码
# GitHub:tencentcloud-sdk-python/tencentcloud/common/exception/tencent_cloud_sdk_exception.py - Line 14
s = f"[TencentCloudSDKException] code:{self.code} message:{self.message} requestId:{self.requestId}"

f-string 的优势在于:

  1. 简洁性:直接在字符串中嵌入变量,无需额外的格式化操作。
  2. 性能 :f-string 的性能优于 % 格式化和 str.format() 方法。
  3. 可读性:代码更直观,易于理解。

提交 PR

完成代码修改后,我仔细检查了改动的正确性,并确保没有引入新的问题。随后,我向腾讯云的开源项目提交了一个 PR(Pull Request),详细描述了我的改动内容和原因。

在 PR 中,我提到:

  • 修改的背景:发现 % 格式化可以替换为更现代的 f-string。
  • 修改的好处:提升代码的可读性和性能。
  • 修改的范围:仅涉及报错信息的构造部分,未影响其他功能。

提交 PR 后,我感到非常期待腾讯团队的反馈。希望我的小小改动能得到认可,并最终被合并到主分支中。

总结

这次的经历让我深刻体会到开源社区的力量。每个人都可以通过自己的努力为开源项目贡献价值。虽然这次的改动很小,但它代表了我对代码质量的关注和对开源精神的支持。

我获得了1k star的开源项目:github.com/CoderWanFen...

如果你也发现了一些可以改进的地方,不妨动手尝试一下。提交 PR 不仅能帮助项目变得更好,也能让你在过程中学到更多。

希望腾讯的小伙伴们能尽快回复我的 PR,让我们一起让腾讯云的 SDK 更加优秀!

相关推荐
onceco12 分钟前
领域LLM九讲——第5讲 为什么选择OpenManus而不是QwenAgent(附LLM免费api邀请码)
人工智能·python·深度学习·语言模型·自然语言处理·自动化
狐凄1 小时前
Python实例题:基于 Python 的简单聊天机器人
开发语言·python
Leinwin2 小时前
微软开源GitHub Copilot Chat,AI编程领域迎新突破
microsoft·github·copilot
悦悦子a啊2 小时前
Python之--基本知识
开发语言·前端·python
笑稀了的野生俊4 小时前
在服务器中下载 HuggingFace 模型:终极指南
linux·服务器·python·bash·gpu算力
Naiva4 小时前
【小技巧】Python+PyCharm IDE 配置解释器出错,环境配置不完整或不兼容。(小智AI、MCP、聚合数据、实时新闻查询、NBA赛事查询)
ide·python·pycharm
路来了4 小时前
Python小工具之PDF合并
开发语言·windows·python
蓝婷儿4 小时前
Python 机器学习核心入门与实战进阶 Day 3 - 决策树 & 随机森林模型实战
人工智能·python·机器学习
AntBlack5 小时前
拖了五个月 ,不当韭菜体验版算是正式发布了
前端·后端·python
.30-06Springfield5 小时前
决策树(Decision tree)算法详解(ID3、C4.5、CART)
人工智能·python·算法·决策树·机器学习