shellcheck đang khuyên không nên sử dụng basename: tại sao?

Tôi đang thử shellcheck .

Tôi có một cái gì đó như thế

basename "${OPENSSL}" 

và tôi nhận được gợi ý sau

Use parameter expansion instead, such as ${var##*/}.

Từ quan điểm thực tế tôi thấy không có sự khác biệt

$ export OPENSSL=/opt/local/bin/openssl
$ basename ${OPENSSL}
openssl
$ echo ${OPENSSL##*/}
openssl

Vì basenametrong thông số kỹ thuật POSIX , tôi không biết lý do tại sao nó nên được thực hành tốt nhất. Có gợi ý nào không?

Đó không phải là về hiệu quả - đó là về tính đúng đắn. basenamesử dụng các dòng mới để phân định tên tệp mà nó in ra. Trong trường hợp thông thường khi bạn chỉ truyền một tên tệp, nó sẽ thêm một dòng mới vào đầu ra của nó. Vì tên tệp có thể chứa dòng mới, điều này gây khó khăn cho việc xử lý chính xác các tên tệp này.

Nó phức tạp hơn nữa bởi thực tế là mọi người thường sử dụng basenamenhư thế này : "$(basename "$file")". Điều này làm cho mọi thứ trở nên khó khăn hơn, bởi vì $(command)dải tất cả các dòng mới từ command. Xem xét các trường hợp không thể $filekết thúc với một dòng mới. Sau đó basenamesẽ thêm một dòng mới, nhưng "$(basename "$file")"sẽ loại bỏ cả hai dòng mới, để lại cho bạn một tên tệp không chính xác.

Một vấn đề khác basenamelà nếu $filebắt đầu bằng một -dấu gạch ngang (dấu gạch ngang), nó sẽ được hiểu là một tùy chọn. Cái này rất dễ sửa:$(basename -- "$file")

Cách sử dụng mạnh mẽ basenamelà:

# A file with three trailing newlines.
file=$'/tmp/evil\n\n\n'

# Add an 'x' so we can tell where $file's newlines end and basename's begin.
file_x="$(basename -- "$file"; printf x)"

# Strip off two trailing characters: the 'x' added by us and the newline added by basename. 
base="${file_x%??}"

Một cách khác là sử dụng ${file##*/}, dễ hơn nhưng có lỗi của riêng nó. Đặc biệt, nó sai trong trường hợp $filelà /hay foo/.

Các dòng có liên quan trong shellcheckcủa mã nguồn là:

checkNeedlessCommands (T_SimpleCommand id _ (w:_)) | w `isCommand` "dirname" =
    style id "Use parameter expansion instead, such as ${var%/*}."
checkNeedlessCommands (T_SimpleCommand id _ (w:_)) | w `isCommand` "basename" =
    style id "Use parameter expansion instead, such as ${var##*/}."
checkNeedlessCommands _ = return ()

Không có lời giải thích nào được đưa ra một cách rõ ràng nhưng dựa trên tên của hàm ( checkNeedlessCommands) có vẻ như @jordanm hoàn toàn đúng và điều đó gợi ý bạn nên tránh tiến trình mới.

dirname, basename, readlinkVv (nhờ @Marco - điều này được khắc phục) có thể tạo ra vấn đề di khi an ninh trở nên quan trọng (yêu cầu an ninh của con đường). Nhiều hệ thống (như Fedora Linux) đặt nó ở /bintrong khi các hệ thống khác (như Mac OSX) đặt nó tại /usr/bin. Sau đó, có Bash trên Windows, ví dụ cygwin, msys và những người khác. Luôn luôn tốt hơn để giữ Bash tinh khiết, khi có thể. (mỗi bình luận @Marco)

BTW, cảm ơn con trỏ tới shellcheck, tôi chưa từng thấy điều đó trước đây.