シェルスクリプトで、すべてのオプションパラメータが空でないことを確認する方法

Question

前提・実現したいこと

下記ソースコードのようにシェルスクリプトで、
オプションから受けたパラメータの妥当性をチェックしたいと思っています。

●チェックしたいこと
１．パラメータ数は正しいか？
２．すべてのパラメータの中身は空でないか？(1つでも空文字の場合はエラー)

１で数をチェックしても、「-a 1 -a 2 -b 3 ...」 のように1つのオプションを複数回指定した場合、１のパラメータ数チェックは通ってしまいます。
ですから、２の処理が必要と考えています。

該当のソースコード

## パラメータチェック
# IFパラメータを受けているかチェック
while getopts "a:b:c:d:e:f:g:h:" OPT
  do
    case $OPT in
      a ) apple=$OPTARG;;
      b ) banana=$OPTARG;;
      c ) chicken=$OPTARG;;
      d ) donut=$OPTARG;;
      e ) egg=$OPTARG;;
      f ) fish=$OPTARG;;
      g ) grape=$OPTARG;;
      h ) hamburger=$OPTARG;;
    esac
  done
 
# １．パラメータ数は正しいか？
if [ $# != 16 ];then
  exit 1
fi

# ２．すべてのパラメータの中身は空でないか？(1つでも空文字の場合はエラー)
for
  do
    *******************************
  done

こちらについてよい書き方をご存じの方、ご教示願えませんでしょうか。
どうぞよろしくおねがいします。

Comments

[Y.H.]

https://teratail.com/questions/236260
これでダメな理由を質問に記載ください。

[umino]

すみません、長らくこねまわしているうちに忘れ&わからなくなってしまい、同じような質問をしてしまいました。
こちらで解決できますね。
修正依頼ありがとうございました。ごめんなさい。

[umino]

Y.Hさん
何度もすみません。以前教えていただいた件ですが、変数名などを変更して実行してみたところ、思うように動きませんでした。
わたしが勘違いをしていると思うのですが、どこが間違っているかおわかりになれば教えていただけませんでしょうか？
●ソースコード
## パラメータチェック
# IFパラメータを受けているかチェック
while getopts "a:b:c:d:e:f:g:h:" OPT
do
case $OPT in
a ) apple=$OPTARG;;
b ) banana=$OPTARG;;
c ) chicken=$OPTARG;;
d ) donut=$OPTARG;;
e ) egg=$OPTARG;;
f ) fish=$OPTARG;;
g ) grape=$OPTARG;;
h ) hamburger=$OPTARG;;
esac
done

# １．パラメータ数は正しいか？
if [ $# != 16 ];then
exit 1
fi

# ２．すべてのパラメータの中身は空でないか？(1つでも空文字の場合はエラー)
## IF空文字チェック
for v in $apple $banana $chicken $donut $egg $fish $grape $hamburger
do
if [ -z $v ]; then
echo "${v^^}が空文字です。"
exit 1
fi
done

●実行時のコマンド
sh hoge.sh -a 1 -b 2 -c 3 -d 4 -e 5 -f 6 -g 7 -g 8

↑8番目の-hオプションがない(-gを重複して投入した)。
-hオプションで渡す変数humburgerの中身は空なので、if分岐に入って終了(エラーにしたい)と思うのですが、入りませんでした。
どこが誤っているか、教えていただければ幸いです。
何度もすみません。

Answer 1

#!/bin/bash
while getopts "a:b:c:d:e:f:g:h:" OPT
  do
    case $OPT in
      a ) apple=$OPTARG;;
      b ) banana=$OPTARG;;
      c ) chicken=$OPTARG;;
      d ) donut=$OPTARG;;
      e ) egg=$OPTARG;;
      f ) fish=$OPTARG;;
      g ) grape=$OPTARG;;
      h ) hamburger=$OPTARG;;
    esac
  done

# １．パラメータ数は正しいか？
if [ $# != 16 ];then
  exit 1
fi

for v in "apple" "banana" "chicken" "donut" "egg" "fish" "grape" "hamburger"
do
        eval val=$${v};
        if [ -z $val ]; then
            echo "${v}が空文字です。"
            exit 1
        fi
done

$ ./test.sh -a 1 -b 2 -c 3 -d 4 -e 5 -f 6 -g 7 -g 8
hamburgerが空文字です。

Comments

[Y.H.]

回答内容は以下と同じです。 (変数名が変わっているのでその差はありますが・・)
https://teratail.com/questions/236260

[umino]

Y.Hさん
正しく実行できました。
evalで引数を変数として、その変数を判定する、の順ですね。
for文から変数を指定して実行しようとしてました。

まぬけな誤りを正していただいてありがとうございました。
大変助かります。

Answer 2

これでどうでしょうか(ファイル名=test2.bash)。なお、optsはa～dの4個に簡略化しました。

#!/usr/bin/bash
opts_counter=0
while getopts "a:b:c:d" OPT
  do
    case $OPT in
      a ) apple=$OPTARG
          let ++opts_counter
          ;;
      b ) banana=$OPTARG
          let ++opts_counter
          ;;
      c ) chicken=$OPTARG
          let ++opts_counter
          ;;
      d ) donut=$OPTARG
          let ++opts_counter
          ;;
    esac
  done
if [ "$opts_counter" = 4 ] ; then
    echo "all opts specified"
else
    echo "not enough opts"
fi

実行結果(シェル名=test.bashとしました)

$ bash test.bash -a 1
not enough opts

$ bash test.bash -a 1 -b 2 -c 3 -d 4
all opts specified

Comments

[Y.H.]

> １で数をチェックしても、「-a 1 -a 2 -b 3 ...」 のように1つのオプションを複数回指定した場合
この要件が満たされてない気がします。
$ ./test2.sh -a 1 -b 2 -c 3 -a 9
all opts specified

[umino]

ikapyさん

回答いただいてありがとうございます。
let を使う方法があるんですね。はじめて知りました。

実行してみたのですが、
この場合、例えばオプション-dを意図的に渡さないと、↓
$ bash test.bash -a 1 -b 2 -c 3 -c 4
all opts specified
となります。(-d,変数donutの中身は空だがエラーにならない)

２．すべてのパラメータの中身は空でないか？(1つでも空文字の場合はエラー)
を満たしたいのですが、よい方法はありませんでしょうか？
ご回答に感謝します。

[ikapy]

Y.H様の通り、不十分でしたね。申し訳ないです。
連想配列を使えばできるのかなと思いましたが、解決したとのことで・・・

Answer 3

https://teratail.com/questions/236260
↑こちらの方法でスマートに解決できます。
同じような質問を過去にしていましたが、修正を繰り返すうちに忘れ＆わからなくなってしまい、似たような投稿をしてしまいました。すみません。