Tại sao nhánh 'if [$ 1 = Ngày 1]] luôn được chọn ngay cả khi $ 1 không phải là 1?

Tôi có một tập lệnh shell có tên 'teleport.sh' như thế này:

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

Khi tôi thực thi:

sh teleport.sh 2 testfile

Điều này testfileđược chuyển đến ~/lab/Sunthư mục, điều này làm tôi bối rối rất nhiều vì tôi đã không chuyển 1 hoặc '1' cho tập lệnh đó.

Có chuyện gì ở đây vậy?

Sử dụng không gian khắc phục vấn đề của bạn.

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

Mặc dù điều này gọn gàng hơn:

#!/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

Điều rõ ràng đầu tiên là bạn nên cung cấp khoảng trắng giữa các đối số của [, testhoặc [[:

if [ "$1" = 1 ];

Khi ở Bash, việc sử dụng [[ ]]được khuyến nghị vì nó không làm những việc không cần thiết cho biểu thức có điều kiện như tách từ và mở rộng tên đường dẫn. Trích dẫn xung quanh dấu ngoặc kép cũng không cần thiết. Một toán tử dễ đọc hơn ==cũng có thể được sử dụng.

if [[ $1 == 1 ]];

Thêm lưu ý: Nếu toán hạng thứ hai cũng chứa biến, trích dẫn là cần thiết vì nó có thể bị phù hợp mô hình nếu nó chứa các ký tự dễ nhận biết như *, ?, [], vv .. Nếu mở rộng globbing hoặc phù hợp với mô hình được kích hoạt với shopt -s extglob, các hình thức khác thích @(), !()vv cũng sẽ được công nhận là mẫu. Xem Khớp mẫu .

Với các toán tử thích <và >nó vẫn có thể cần thiết vì tôi đã từng gặp một lỗi trong đó không trích dẫn đối số thứ hai gây ra kết quả khác nhau.

Đối với toán hạng đầu tiên, không có gì áp dụng.

Cũng xem xét biến thể đơn giản hơn này:

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

Hoặc cô đặc:

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

"${@:2}"là một hình thức mở rộng chuỗi con hoặc mở rộng thành viên mảng trong đó 2là phần bù. Điều này làm cho việc mở rộng bắt đầu ở giá trị thứ hai. Với điều này, chúng tôi có thể không cần sử dụng shift.

Việc thêm vào --ngăn không cho mvnhận ra tên tệp bắt đầu bằng dấu gạch ngang ( -) là tùy chọn không hợp lệ.

Để trả lời câu hỏi tại sao điều này xảy ra, hành vi này của [aka testđược ghi lại trong POSIX :

Trong danh sách sau đây, $ 1, $ 2, $ 3 và $ 4 đại diện cho các đối số được trình bày để kiểm tra:

[...]

1 đối số:

Thoát đúng (0) nếu $ 1 không rỗng; nếu không, thoát sai.

Bạn đang truyền cho nó 1 đối số, 2=1không phải là null, và do đó testthoát khỏi thành công.

Như bài viết khác (và shellcheck ) chỉ ra, nếu bạn muốn so sánh bình đẳng, bạn sẽ thay phải vượt qua 3 đối số 2, =và 1.

Tôi chỉ muốn giới thiệu một thay thế di động nhưng cũng gọn gàng hơn. Bash không phải là phổ quát (và nếu bạn không cần phổ quát, tại sao bạn lại viết một kịch bản shell?)

#! /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"

(Lưu ý với các giáo viên: có, tôi biết các trích dẫn $actiontrên case "$action" indòng là không cần thiết, nhưng tôi cảm thấy tốt nhất là đặt chúng ở đó bằng mọi cách, để người đọc trong tương lai không phải nhớ điều đó.)