シェルスクリプトで、パラメータの空文字""を検出したい

Question

前提・実現したいこと

シェルスクリプトtest.shにパラメータを4つ渡し、
パラメータが1つでも空文字"" の場合は不正として終了したいです。

正常時の実行イメージ
sh test.sh 10 15 2 30

例えばパラメータを下記に設定した場合、エラーにしたい。
sh test.sh 10 15 "" 30

なお「パラメータの個数が4つかどうか」は他の処理でチェックしているので、
確認したいのは「パラメータに空文字""が渡っていないか」だけです。

試したこと

#!/bin/bash -
## IFパラメータ空文字チェック
for v in  $1 $2 $3 $4
do
  eval val=$${v};
  if [ -z ${val} ]; then
    echo"パラメータに空文字を含んでいるため、登録できません" 
    exit 1
  fi
done

echo "テストは成功です"

-xvオプションを付与して実行した場合、下記のような結果になりました。

# sh -xv UnitSyncWebSettingCron.sh 10 15 "" 30
## IFパラメータ空文字チェック
for v in  $1 $2 $3 $4
do
  eval val=$${v};
  if [ -z ${val} ]; then
    echo"パラメータに空文字を含んでいるため、登録できません" 
    exit 1 
  fi
done
+ for v in '$1' '$2' '$3' '$4'
+ eval 'val=$10'
val=$10
++ val=100
+ '[' -z 100 ']'
+ for v in '$1' '$2' '$3' '$4'
+ eval 'val=$15'
val=$15
++ val=105
+ '[' -z 105 ']'
+ for v in '$1' '$2' '$3' '$4'
+ eval 'val=$30'
val=$30
++ val=0
+ '[' -z 0 ']'

変数が思うように展開されていないように見えるのですが、どのように書けば正しく展開できるか、解決方法がわかりません。
お詳しい方、よい書き方をご教示願えませんでしょうか。
よろしくおねがいします。

補足情報（FW/ツールのバージョンなど）

GNU bash, version 4.2.46(2)-release (x86_64-redhat-linux-gnu)

Answer 1

count=0
while [ $count -lt $(($#)) ]; do
    n=$count
    eval val=$$count
    if [ -z ${val} ]; then
        echo "パラメータに空文字を含んでいるため、登録できません"
        exit 1
    fi
    ((count++))
done
echo $*

興味深かったので書いてみました。
このままだと引数の数が不定っぽかったので、 比較のため $(($#)) を引数の数にしてループさせています。

Comments

[umino]

tetsunosukeさん
ご回答ありがとうございます。
頂いたコードで試してみたところ、期待した結果を得ることができました。
普段わたしが知らないような書き方で、大変勉強になりました。
丁寧なご回答、ありがとうございます。

count=0
+ count=0
while [ $count -lt $(($#)) ]; do
n=$count
eval val=$$count
if [ -z ${val} ]; then
echo "パラメータに空文字を含んでいるため、登録できません"
exit 1
fi
((count++))
done
+ '[' 0 -lt 4 ']'
+ n=0
+ eval 'val=$0'
val=$0
++ val=test.sh
+ '[' -z test.sh ']'
+ (( count++ ))
+ '[' 1 -lt 4 ']'
+ n=1
+ eval 'val=$1'
val=$1
++ val=10
+ '[' -z 10 ']'
+ (( count++ ))
+ '[' 2 -lt 4 ']'
+ n=2
+ eval 'val=$2'
val=$2
++ val=15
+ '[' -z 15 ']'
+ (( count++ ))
+ '[' 3 -lt 4 ']'
+ n=3
+ eval 'val=$3'
val=$3
++ val=
+ '[' -z ']'
+ echo パラメータに空文字を含んでいるため、登録できません

[tetsunosuke]

あ、n=$count はいらんかった。試しながら書いていたので名残ですｗ

自分も $(($#)) は知らなかったのでためになりました。

Answer 2

""で囲めば良いです。

#!/bin/bash -
## IFパラメータ空文字チェック
for v in  "$1" "$2" "$3" "$4"
do
  if [ -z "$v" ]
  then
    echo "パラメータに空文字を含んでいるため、登録できません" 
    exit 1
  fi
done

echo "テストは成功です"

Comments

[otn]

個数が4であることを別途確認しているということなので、
for v in "$@"
でもいいです。

[umino]

otnさん
ありがとうございます。いま頂いたコードで試したところ、無事期待する結果が得られました。
わたしは""をつけていないうえ、不要な処理を入れていました。
丁寧に示していただいて助かりました。ありがとうございます。

[umino]

なるほどですね。
プラスアルファまでありがとうございます。改修しやすいよう、"$@"でやろうとおもいます。

[otn]

「空かも知れない」「空白文字を含んでいるかも知れない」という場合は "" で囲みます。

希に、「空なら、（空文字列で無く）空（＝何も無いこと）になって欲しい」「空白文字を含んでいれば複数文字列に分かれて欲しい」というケースがあり、その場合だけ囲みません。

[umino]

なるほどですね、いままでは「展開後もひとつの文字列として扱いたい場合は""で囲うんだ」と勝手に思っていたのですが、「空かも知れない」「空白文字を含んでいるかも知れない」という場合は ""で囲う、というお話を伺って、正しいイメージができたと思います。
メモしておいて、今後コーディングの際に見返します。
わかりやすく教えていただき、ありがとうございました。

[otn]

> いままでは「展開後もひとつの文字列として扱いたい場合は""で囲うんだ」

文章としては、それで合ってます。
A="" の $Aを、「空でなく、1つの空文字列として扱いたい」
A="1 2" の $Aを、「2つでなく、1つの文字列として扱いたい」
ということなので。

Answer 3

こうする必要があるのではないでしょうか。

for v in  "$1" "$2" "$3" "$4"
...

Comments

[umino]

KojiDoiさん
ご回答ありがとうございます。
変数を囲うのみだと、期待する結果が得られませんでした。
↓の部分にも修正を加え、
for v in $1 $2 $3 $4
do
eval val=$${v};
if [ -z ${val} ]; then
-----------------
for v in "$1" "$2" "$3" "$4"
do
if [ -z "$v" ]
...

とすることで、期待する結果を得ることができました。
ご回答、ありがとうございました。

Answer 4

こういう感じになります。

#!/bin/sh

i=1
while [ $i -lt 5 ]; do
        eval "v=${$i}"
        if [ -z $v ]; then
                echo "void argument"
        else
                echo $v
        fi
        i=`expr 1 + ${i}`
done

for 文で回そうとすると、

sh test.sh 10 15 "" 30

はスクリプト中で

for 10 15 30

と展開されてしまいます。

Comments

[umino]

なるほどですね、空文字はスクリプト内ではない(というと正しいのかはわかりませんが)扱いになるのですね。ありがとうございます。