参数扩展是否需加双引号？Shellcheck警告合理性及疑问

首先，咱们结合Shell的变量扩展规则和Shellcheck的检查逻辑来拆解你的问题：

1. Shellcheck的警告是否合理？ #

这得看警告具体指向的位置：

• 如果警告是针对echo -e参数内部的${tmpdir}/${filename}.jpdf未加引号：这个警告不合理。因为你已经给echo的整个参数套了双引号，Shell会把双引号内的内容作为一个整体处理，变量扩展时会保留其中的空格、特殊字符（比如$、空格、通配符），不会触发分词或glob匹配，完全符合安全规范。
• 如果警告是针对目标文件路径（也就是你要插入内容的${tmpdir}/${filename}.jpdf）未加引号：这个警告非常合理。要是tmpdir或filename里包含空格、*这类特殊字符，未加引号的话Shell会把路径拆成多个参数，或者匹配当前目录下的其他文件，直接导致命令执行失败（比如找不到目标文件、误操作其他文件）。

2. 已给echo参数加双引号，还需要给变量单独加引号吗？ #

完全不需要，反而可能画蛇添足：

• 在双引号内部，${tmpdir}和${filename}的扩展是安全的，外层双引号已经提供了完整的保护，没必要再给变量单独套双引号。
• 强行给变量单独加引号（比如写成echo -e "插入内容 "${tmpdir}"/"${filename}".jpdf"）不仅多余，还可能因为引号开闭顺序出错引发语法问题（比如不小心写成"内容 ${tmpdir}"${filename}"..."，Shell会把${filename}当成双引号外的内容，失去保护）。

给你举个符合规范、还能避免Shellcheck警告的写法例子：

target_file="${tmpdir}/${filename}.jpdf"
echo -e "要插入的开头内容" | sed -i '1r /dev/stdin' "$target_file"

这里把目标文件路径单独赋值并加引号，echo的参数也用双引号包裹，既安全又整洁。

内容的提问来源于stack exchange，提问作者Tim