เหตุใดจึงเลือกสาขา 'ถ้า [$ 1 =“ 1”]' เสมอแม้ว่าจะไม่ใช่ $ 1 ก็ตาม

ฉันมีเชลล์สคริปต์ชื่อ 'teleport.sh' เช่นนี้:

if [ $1="1" ];
    then
    shift
        mv "$@" ~/lab/Sun
elif [ $1="2" ];
    then
    shift
        mv "$@" ~/lab/Moon
elif [ $1="3" ];
    then
    shift
        mv "$@" ~/lab/Earth
fi

เมื่อฉันรัน:

sh teleport.sh 2 testfile

สิ่งนี้testfileถูกย้ายไปยัง~/lab/Sunไดเรกทอรีซึ่งทำให้ฉันสับสนมากเพราะฉันไม่ผ่าน 1 หรือ '1' ไปยังสคริปต์นั้น

เกิดอะไรขึ้นที่นี่?

การใช้ช่องว่างช่วยแก้ไขปัญหาของคุณ

if [ "$1" = 1 ];
    then
    shift
        mv "$@" ~/lab/Sun
elif [ "$1" = 2 ];
    then
    shift
        mv "$@" ~/lab/Moon
elif [ "$1" = 3 ];
    then
    shift
        mv "$@" ~/lab/Earth
fi

แม้ว่านี่จะเป็นตอนสุดท้าย:

#!/bin/bash

action=$1
shift
files=("$@")
case $action in  
  1) mv -- "${files[@]}" ~/lab/Sun     ;;
  2) mv -- "${files[@]}" ~/lab/Moon    ;;
  3) mv -- "${files[@]}" ~/lab/Earth   ;;
esac

สิ่งที่เห็นได้ชัดเป็นครั้งแรกที่คุณควรให้ช่องว่างระหว่างข้อโต้แย้งของ[, testหรือ[[:

if [ "$1" = 1 ];

เมื่ออยู่ใน Bash [[ ]]แนะนำให้ใช้เนื่องจากจะไม่ทำสิ่งที่ไม่จำเป็นสำหรับการแสดงออกตามเงื่อนไขเช่นการแยกคำและการขยายชื่อพา ธ ไม่จำเป็นต้องมีการเสนอราคา ผู้ประกอบการอ่านได้มากขึ้น==นอกจากนี้ยังสามารถนำมาใช้

if [[ $1 == 1 ]];

เพิ่มหมายเหตุ: หากถูกดำเนินการที่สองนอกจากนี้ยังมีตัวแปรที่อ้างเป็นสิ่งที่จำเป็นในขณะที่มันอาจจะมีการจับคู่รูปแบบถ้ามันมีตัวละครที่เป็นที่รู้จักเช่น*, ?, []ฯลฯ .. ถ้าขยาย globbing หรือจับคู่รูปแบบการใช้งานด้วยshopt -s extglobรูปแบบอื่น ๆ ชอบ@(), !()ฯลฯ จะได้รับการยอมรับว่าเป็นรูปแบบ ดูรูปแบบการจับคู่

ด้วยโอเปอเรเตอร์ที่ชอบ<และ>อาจยังจำเป็นเพราะฉันเคยพบข้อผิดพลาดที่ไม่ได้อ้างถึงข้อโต้แย้งที่สองทำให้เกิดผลลัพธ์ที่แตกต่างกัน

สำหรับตัวถูกดำเนินการแรกไม่มีสิ่งใดนำไปใช้

พิจารณารูปแบบที่ง่ายกว่านี้เช่นกัน:

case "$1" in
1)
    mv -- "${@:2}" ~/lab/Sun
    ;;
2)
    mv -- "${@:2}" ~/lab/Moon
    ;;
3)
    mv -- "${@:2}" ~/lab/Earth
    ;;
esac

หรือย่อ:

case "$1" in
1) mv -- "${@:2}" ~/lab/Sun ;;
2) mv -- "${@:2}" ~/lab/Moon ;;
3) mv -- "${@:2}" ~/lab/Earth ;;
esac

"${@:2}"เป็นรูปแบบของการขยายสตริงย่อยหรือการขยายสมาชิกอาร์เรย์โดยที่2ออฟเซ็ต สิ่งนี้ทำให้การขยายเริ่มต้นที่ค่าที่สอง shiftด้วยวิธีนี้เราอาจไม่จำเป็นต้องใช้

การ--ป้องกันที่เพิ่มเข้ามาmvจากการรู้จักชื่อไฟล์ที่ขึ้นต้นด้วยขีด ( -) เป็นตัวเลือกที่ไม่ถูกต้อง

เพื่อตอบคำถามว่าทำไมสิ่งนี้จึงเกิดขึ้นลักษณะการทำงานของ[aka testนี้ได้รับการบันทึกไว้ใน POSIX :

ในรายการต่อไปนี้ $ 1, $ 2, $ 3 และ $ 4 แสดงถึงข้อโต้แย้งที่นำเสนอเพื่อทดสอบ:

[ ... ]

1 อาร์กิวเมนต์:

ออกจริง (0) ถ้า $ 1 ไม่ใช่โมฆะ; มิฉะนั้นให้ออกจาก false

คุณกำลังผ่านการโต้แย้ง 1 ข้อ2=1ซึ่งไม่เป็นโมฆะดังนั้นจึงtestออกมาพร้อมกับความสำเร็จ

ในฐานะที่เป็นกระทู้อื่น ๆ (และshellcheck ) ชี้ให้เห็นว่าถ้าคุณต้องการที่จะเปรียบเทียบเพื่อความเท่าเทียมกันคุณจะแทนที่จะต้องผ่าน 3 ข้อโต้แย้ง2, และ=1

ฉันแค่อยากจะแนะนำทางเลือกแบบพกพาและตัวเลือกอื่น ๆ Bash ไม่ใช่ universal (และถ้าคุณไม่ต้องการ universal ทำไมคุณถึงเขียนเชลล์สคริปต์?)

#! /bin/sh
action="$1"
shift
case "$action" in
    1) dest=Sun   ;;
    2) dest=Moon  ;;
    3) dest=Earth ;;
    *) echo "Unrecognized action code '$action' (must be 1, 2, or 3)" >&2; exit 1 ;;
esac
mv -- "$@" ~/lab/"$dest"

(หมายเหตุ pedants: ใช่ฉันรู้ว่าคำพูดที่อยู่รอบ ๆ$actionบนcase "$action" inเส้นที่ไม่จำเป็น แต่ผมรู้สึกว่ามันเป็นดีที่สุดที่จะทำให้พวกเขามีอยู่แล้วเพื่อให้ผู้อ่านในอนาคตจะได้ไม่ต้องจำไว้ว่า.)