lanbinshijie / bili2text

Bilibili视频转文字,一步到位,输入链接即可使用
MIT License
590 stars 58 forks source link

1. 模块导入修正: #21

Closed fzlzjerry closed 4 days ago

fzlzjerry commented 6 days ago
1.  模块导入修正:
•   将 import urllib 替换为 import urllib.parse,以适应 Python 3 中的 URL 编码和解析。
2.  优化 get_signa 函数:
•   使用 signa.decode('utf-8') 代替 str(signa, 'utf-8'),更规范地处理编码。
3.  改进 upload 函数:
•   使用 with open(upload_file_path, 'rb') as f: 安全地打开文件,确保文件正确关闭。
•   将 fileSize 从整数转换为字符串,避免类型错误。
•   使用 urllib.parse.urlencode(param_dict) 正确编码 URL 参数。
•   直接使用 response.json() 获取响应数据,简化代码。
4.  完善 get_result 函数:
•   增加对上传响应 code 的检查,确保上传成功;如果失败,打印错误信息并返回 None。
•   在轮询获取结果时,检查响应 code,确保请求成功;若失败,打印错误信息并返回 None。
•   使用 response.json() 直接解析响应数据。
5.  错误处理和日志输出:
•   添加了对上传和获取结果过程中可能出现的错误的处理。
lanbinleo commented 4 days ago

@fzlzjerry

您好,感谢您对本项目的关注和贡献,并提交了PR。然而,经过仔细审查,我们认为,当前的PR存在一些问题,导致我们无法接受这次提交。

首先,虽然PR中的代码改进看似有助于提高代码的可读性和安全性,但这些改动并未与现有代码库很好地融合。当前模块的开发并不在我们的近期计划中,而且我们已经在进行该模块的重构。这意味着您的改动可能会与我们未来的更新产生冲突。

其次,PR中的一些改动虽然在技术上是合理的,但并没有为项目带来实质性的功能提升。例如,使用with open语句来确保文件关闭确实是个好选择,但这并不会显著影响模块的功能。此外,许多改动只是将现有代码替换为等效的代码,并没有解决任何实际问题或增加新的功能。这个模块的代码只是起到了临时占位作用,其代码是直接摘抄自讯飞语音转写官方文档提供的SDK,详情参考API文档链接

最后,从项目发展的角度来看,我们需要尽量确保每个提交都能为项目带来实际的价值和提升,且不会对现有功能造成不必要的影响。当前的PR虽然包含了一些代码技术上的优化,但并未解决任何现有的bug或引入新的功能,鉴于以上原因,我们认为这不是一个优先级很高的改动。

综上所述,我们建议您在提交PR之前,在社区Discussion或者Issue进行更多的沟通。这样可以确保您的贡献能够更好地融入项目,并为项目的发展带来真正的价值。

再次感谢您的理解和支持。如果您有任何疑问或需要进一步讨论,请随时与我们联系。

Lanbin