编写Bash脚本移除指定扩展名文件后缀报错，求助修改方案

我想要编写一个程序，移除目录中指定扩展名文件的后缀。例如目录d1内有a.jpg、b.jpg、c.txt三个文件，指定处理的扩展名为.jpg，执行程序后文件应变为a、b、c.txt。

以下是我目前编写的脚本：

#!/bin/bash
echo "Enter an extension"
read extension
echo "Enter a directory"
read directory
allfiles=$( ls -l $directory)
for x in $allfiles
do
        ext=$( echo $x | sed 's:.*.::')
        if [ $ext -eq $extension]
        then
                echo $( $x | cut -f 2 -d '.')
        else
                echo $x
        fi

done

运行该脚本时出现错误：

'-f' is not defined
'-f' is not defined

请问我应该如何修改这段代码？

核心错误点 #

• 错误使用ls -l：ls -l输出的是文件详细信息（权限、所有者、大小等），不是单纯的文件名，遍历这些内容会把非文件名字段当成处理对象，引发报错。
• 字符串比较用了数值运算符：-eq是整数比较运算符，字符串比较应该用=。
• 语法空格缺失：[ $ext -eq $extension]中]前必须加空格，否则会被解析成变量的一部分，导致语法错误。
• 命令执行逻辑错误：$( $x | cut -f 2 -d '.')会把$x当成命令执行，而非作为文本输入传递给cut，且要获取文件名前缀应该用-f 1而非2。
• 未处理带空格的文件名：直接遍历ls输出会拆分带空格的文件名，引发异常。

修改后的脚本 #

#!/bin/bash
echo "Enter an extension (without leading dot, e.g., jpg):"
read extension
echo "Enter a directory:"
read directory

# 切换到目标目录，失败则退出程序
cd "$directory" || { echo "Failed to access directory $directory"; exit 1; }

# 遍历所有匹配指定扩展名的文件
for file in *."$extension"; do
    # 跳过不存在的文件（当无匹配文件时，*.$extension会作为字面量存在）
    [ -f "$file" ] || continue
    # 使用Bash参数扩展直接移除后缀，高效可靠
    mv "$file" "${file%.$extension}"
done

echo "Suffix removal completed."

关键改进说明 #

• 用*."$extension"直接匹配目标文件，规避ls的陷阱，同时支持带空格的文件名。
• 使用Bash内置的参数扩展${file%.$extension}生成去掉后缀的文件名，比sed/cut更高效且不易出错。
• 添加目录访问失败的错误处理，避免后续无效操作。
• 明确提示扩展名输入格式（不带点），减少用户混淆。

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