使用SED处理标准输入时Bash脚本执行失败问题排查

我帮你排查了脚本里的几个关键问题，这些问题正是导致执行失败的原因，下面一步步给你修正和解释：

1. 正则表达式语法错误 #

你写的if [[ "$line" =~ .*log" ]]存在引号不闭合的问题，而且这个匹配逻辑也不够准确——我们要找的是导入块里的log包，不是任意包含log的文本。

2. While循环逻辑缺陷 #

当前代码会在读取第一行后直接break，不管后面的行有没有匹配log。正确的逻辑应该是遍历所有导入行，标记是否找到目标库，最后统一输出结果。

3. Sed命令的兼容性问题 #

如果Go文件里的import是单行格式（比如import "log"），你的sed -n '/import (/,/)/p'会完全匹配不到，我们需要调整sed命令同时兼容单行和多行import写法。

修复后的完整脚本 #

#!/bin/bash

function testing() {
    local file="$1"
    local found=0  # 用标记变量记录是否找到log导入

    # 同时匹配多行import块和单行import语句
    while IFS='' read -r line; do
        # 匹配多种导入格式：包括直接导入、带别名、带注释的情况
        if [[ "$line" =~ ^\s*(.*\s)?log(\s.*)?$ ]]; then
            found=1
            break  # 找到后直接退出循环，提升效率
        fi
    done < <(sed -n -e '/^import (/,/)/p' -e '/^import ".*/p' "$file")

    # 根据标记输出最终结果
    if [[ $found -eq 1 ]]; then
        echo "log is imported in the file: $file"
    else
        echo "log is not imported in the file: $file"
    fi
}

function main() {
    local search_dir="$1"
    # 先校验输入目录是否存在
    if [[ ! -d "$search_dir" ]]; then
        echo "Error: Directory $search_dir does not exist!"
        exit 1
    fi

    # 处理文件名含空格的情况，避免遍历出错
    find "$search_dir" -name "*.go" -print0 | while IFS= read -r -d '' file; do
        testing "$file"
    done
}

# 调用主函数，传入命令行第一个参数作为搜索目录
main "$1"

关键改进点说明 #

• 精准匹配导入：^\s*(.*\s)?log(\s.*)?$可以匹配多种合法的Go导入格式，比如log、l "log"、 log // 日志库等。
• 逻辑修正：用found标记变量替代提前break的逻辑，确保遍历完所有导入行再判断结果。
• 兼容多种import写法：sed命令同时处理多行import (...)块和单行import "..."语句。
• 健壮性提升：添加目录存在校验，用find -print0+read -d ''组合处理带空格的文件名。

使用方法 #

把脚本保存为check_log_import.sh，添加执行权限：

chmod +x check_log_import.sh

然后运行脚本并指定要搜索的Go项目目录：

./check_log_import.sh /path/to/your/go/project

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