shellcheck ไม่แนะนำให้ใช้ basename: ทำไม

ฉันกำลังพยายามออกshellcheck

ฉันมีบางอย่างเช่นนั้น

basename "${OPENSSL}" 

และฉันได้รับคำแนะนำต่อไปนี้

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

จากมุมมองจริงฉันไม่เห็นความแตกต่าง

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

เนื่องจากbasenameเป็นข้อมูลจำเพาะของ POSIXฉันจึงไม่มีเหตุผลว่าทำไมจึงควรเป็นวิธีปฏิบัติที่ดีที่สุด คำใบ้ใด ๆ

มันไม่เกี่ยวกับประสิทธิภาพ - แต่เกี่ยวกับความถูกต้อง basenameใช้การขึ้นบรรทัดใหม่เพื่อกำหนดขอบเขตชื่อไฟล์ที่พิมพ์ออกมา ในกรณีปกติเมื่อคุณส่งชื่อไฟล์เพียงไฟล์เดียวมันจะเพิ่มบรรทัดใหม่ต่อท้ายไปยังเอาต์พุต เนื่องจากชื่อไฟล์อาจมีการขึ้นบรรทัดใหม่ด้วยตนเองจึงทำให้ยากต่อการจัดการชื่อไฟล์เหล่านี้อย่างถูกต้อง

มันซับซ้อนมากขึ้นโดยความจริงที่คนมักจะใช้เช่นนี้basename "$(basename "$file")"สิ่งนี้ทำให้ยิ่งยากมากขึ้นเพราะ$(command)แถบทุกcommandบรรทัดใหม่ต่อท้ายจาก พิจารณากรณีที่ไม่น่า$fileจะจบลงด้วยการขึ้นบรรทัดใหม่ จากนั้นbasenameจะเพิ่มบรรทัดใหม่พิเศษ แต่"$(basename "$file")"จะตัดการขึ้นบรรทัดใหม่ทั้งสองทำให้คุณมีชื่อไฟล์ไม่ถูกต้อง

อีกปัญหาหนึ่งbasenameคือถ้า$fileเริ่มต้นด้วย-(ขีดหรือลบด้วยเครื่องหมายลบ) ก็จะถูกตีความเป็นตัวเลือก อันนี้ง่ายต่อการแก้ไข:$(basename -- "$file")

วิธีที่แข็งแกร่งในการใช้basenameคือ:

# 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%??}"

อีกทางเลือกหนึ่งคือการใช้งาน${file##*/}ซึ่งง่ายกว่า แต่มีข้อบกพร่องของตนเอง โดยเฉพาะอย่างยิ่งมันผิดในกรณีที่$fileเป็นหรือ/foo/

เส้นที่เกี่ยวข้องในshellcheck's รหัสที่มาคือ:

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 ()

ไม่มีคำอธิบายที่ชัดเจน แต่ขึ้นอยู่กับชื่อของฟังก์ชั่น ( checkNeedlessCommands) ดูเหมือนว่า @jordanm ค่อนข้างถูกต้องและแนะนำให้คุณหลีกเลี่ยงการตีกระบวนการใหม่

dirname, basename, readlinkฯลฯ (ขอบคุณ @Marco - นี้ได้รับการแก้ไข) สามารถสร้างปัญหาการพกพาเมื่อการรักษาความปลอดภัยเป็นสำคัญ (การรักษาความปลอดภัยที่กำหนดของเส้นทาง) ระบบจำนวนมาก (เช่น Fedora Linux) สถานที่ที่มัน/binขณะที่คนอื่น (เช่น Mac OSX) /usr/binวางไว้ที่ จากนั้นจะมี Bash บน Windows เช่น cygwin, msys และอื่น ๆ มันจะดีกว่าเสมอเมื่ออยู่กับ Bash บริสุทธิ์ถ้าเป็นไปได้ (ต่อความคิดเห็น @Marco)

BTW ขอบคุณสำหรับตัวชี้ไปที่ shellcheck ฉันไม่เคยเห็นมาก่อน