已开启
refactor: 重构媒体文件信息获取逻辑,消除代码重复 #1
忘语创建于 2025年10月29日
refactor: 重构媒体文件信息获取逻辑,消除代码重复 #1
已开启
忘语创建于 2025年10月29日
忘语
忘语
2025年10月29日

问题描述

src/main/handlers/file.js 中,getVideoInfogetAudioInfo 两个方法存在大量重复代码,仅在编解码器类型参数上有差异,违反了 DRY (Don't Repeat Yourself) 原则。

解决方案

  • 新增 _getMediaInfo 私有方法,封装媒体文件信息获取的通用逻辑
  • 通过 codecType 参数区分视频流 (video) 和音频流 (audio)
  • 重构 getVideoInfogetAudioInfo 方法,调用统一的 _getMediaInfo 方法

优化效果

✅ 消除了约 20 行重复代码
✅ 提高代码可维护性,未来修改只需改一处
✅ 增强代码可读性和一致性
✅ 保持原有功能不变,无破坏性更改

变更文件

  • src/main/handlers/file.js

测试

  • ✅ 无 Linter 错误
  • ✅ 代码逻辑保持一致,仅重构实现方式
likedislike
合并受阻