Melody音乐同步工具中的文件名替换Bug分析与修复方案

问题描述

在Melody音乐同步工具中，用户发现当使用{artist}/{songName}-{artist}格式命名下载歌曲时，第二个{artist}占位符未被正确替换。这导致下载的音乐文件保留了原始的占位符文本，而非预期的艺术家名称。

技术分析

该问题源于JavaScript字符串替换方法的局限性。在原始代码中，开发者使用了简单的replace()方法而没有添加全局替换标志/g。JavaScript的replace()方法默认只替换第一个匹配项，因此当模板字符串中包含多个相同的占位符时，只有第一个会被替换。

解决方案

修复方案需要对buildDestFilename函数进行修改，确保所有占位符都能被正确替换。具体实现如下：

1. 为每个占位符的替换操作添加全局标志/g
2. 确保每个占位符都被完整替换
3. 添加默认值处理，防止未定义的值导致错误

核心修复代码如下：

function buildDestFilename(globalConfig, songInfo, playlistName) {
    const downloadPath = globalConfig.downloadPath;
    let filename = (playlistName ? globalConfig.playlistSyncToLocal?.filenameFormat : globalConfig.filenameFormat)
        .replace(/{artist}/g, songInfo.artist ? songInfo.artist : 'Unknown')
        .replace(/{songName}/g, songInfo.songName ? songInfo.songName : 'Unknown')
        .replace(/{playlistName}/g, playlistName ? playlistName : 'UnknownPlayList')
        .replace(/{album}/g, songInfo.album ? songInfo.album : 'Unknown');
    // 移除文件名开头的斜杠
    filename = filename.replace(/^[\/\\]+/, '') + '.mp3';
    return `${downloadPath}${libPath.sep}${filename}`
}

临时解决方案

对于使用Docker部署的用户，可以按照以下步骤临时修复：

1. 进入运行中的Melody容器：

id=$(docker ps | grep melody | awk '{print $1}')
docker exec -it $id /bin/sh

2. 编辑问题文件：

vi backend/src/service/sync_music/download_to_local.js

3. 按照上述修复方案修改代码后，重启容器使更改生效。

最佳实践建议

1. 对于字符串模板替换操作，始终考虑使用全局替换标志
2. 为占位符提供合理的默认值，增强代码健壮性
3. 考虑使用更现代的模板字符串处理库，如Lodash的_.template()函数
4. 对用户输入的文件名模板进行验证，防止路径遍历等安全问题

总结

这个Bug展示了在字符串处理中细节的重要性。虽然问题本身不复杂，但它影响了核心功能的用户体验。通过添加全局替换标志，我们确保了文件名模板中的所有占位符都能被正确替换，从而提供了更可靠的文件同步功能。