Bash: использовать переменную в цикле

Question

Мне нужна помощь (снова) после того, как вы, ребята, помогли мне чудесно в прошлый раз. Я пишу скрипт для тестирования своих жестких дисков с помощью SmartCTL. Так что я могу конечно скопировать команды на диск, но это было бы еще более удивительным, если я могу просто установить диски один раз и сценарий делает их один за другим:

#!/bin/bash 
date=`date +%d-%m-%Y-%T` 
touch /var/log/disk/Disk-health-check-$date 
disks="/dev/sda 
/dev/sdb" 
for disk in disks 
    do 
    wait=$(smartctl -t short $disk | awk '/Please wait/ {print $3}') 
     echo "waiting..." 
     sleep $((wait * 60 + 60)) 
     echo "done" 
     smartctl --log=selftest $disk 
     smartctl -a $disk 
    done 
exit 

Но, к сожалению, он использует только/Dev/sda, а не sdb и т. д. Итак, как я могу это сделать? Спасибо заранее, ребята!

Answer 1

Попробуйте это:

#!/bin/bash 
date=$(date +%d-%m-%Y-%T) 
touch /var/log/disk/Disk-health-check-"$date" 
disks=(/dev/sda /dev/sdb) 
for disk in "${disks[@]}" 
    do 
    wait=$(smartctl -t short "$disk" | awk '/Please wait/ {print $3}') 
    echo "waiting..." 
    sleep $((wait * 60 + 60)) 
    echo "done" 
    smartctl --log=selftest "$disk" 
    smartctl -a "$disk" 
    done 
exit 

На основе массивов переменных, описанных здесь: http://wiki.bash-hackers.org/syntax/arrays?s[]=arrays

Answer 2

Попробуйте это:

for disk in disks; do 
    echo "$disk" 
done 

Вы увидите, что единственное, что отголоски является disks. Это потому, что вы говорите ему перебирать ровно одно значение, и это значение равно disks.

---

Чтобы сделать тонко глючный, что вы пытались сделать, вы хотите вместо этого:

for disk in $disks; do 
    echo "$disk" 
done 

Однако, как я уже сказал, что это багги. Допустим, ваш disks переменные были назначены немного по-разному:

disks=' 
/dev/disks/by-label/My Drive 
/dev/disks/by-label/Other Drive 
/dev/disks/by-label/* TEENAGE DAUGHTER'S DRIVE * 
' 

Это будет иметь /dev/disks/by-label/My как одна записи, Drive как следующий - и в * с будет расширено с именами файлов в каталоге вы находитесь когда вы запускаете скрипт. Очевидно, не то, что вы хотите.

Вместо этого используйте массив:

disks=(
    "/dev/disks/by-label/My Drive" 
    "/dev/disks/by-label/Other Drive" 
    "/dev/disks/by-label/* TEENAGE DAUGHTER'S DRIVE *" 
) 

... и перебирать их, как:

for disk in "${disks[@]}"; do 
    echo "Processing: $disk" 
done