bashで未入力時の共通メソッドを作りたい

Question

多分初めてbashに触ってみたのですが戻り値がないのが違和感あってなかなかロジック組み立てられずにいます。
やりたいことは未入力があれば入力があるまでreadさせ続けるというものなのですが、、、

function check_blank ()
{
    if [ "$1" = "" ]; then
        echo "It is not yet input."
        # 入力させる
        ## 問題は入力させる変数名。メソッドを個別に分けるのはいやだ。
        read Clone_url
        # 入力されていなかったらcheck_blankを呼び出す。
        if [ "$Clone_url" = "" ]; then
            check_blank
        fi
    fi
}


## Input setting
echo -e "\e[5;36m* \e[0m\e[1;36mPlease enter the URL of the remote repository of the clone to be created.\e[0;39m"
read Clone_url
check_blank $Clone_url
return

echo -e "\e[5;36m* \e[0m\e[1;36mPlease enter an alias for managing the clone to be created.\e[0;39m"
read Project_name
check_blank $Project_name

echo -e "\e[5;36m* \e[0m\e[1;36mPlease enter the domain name to reflect the setting.\e[0;39m"
read Domain_name
check_blank $Domain_name

check_blank1
check_blank2
check_blank3
と作れば解説しますが、それは個人的に嫌で、引数を利用して共通メソッドを作れればと思っているのですがロジックが思い浮かばず、、、

 ## 問題は入力させる変数名。メソッドを個別に分けるのはいやだ。
        read Clone_url

の部分なのですが共通メソッドを作りたいので変数名が可変しなくてはならないはずです。

function check_blank ()
{
    if [ "$2" = "" ]; then
        echo "It is not yet input."
        # 入力させる
        read $1
        # 入力されていなかったらcheck_blankを呼び出す。
        if [ "${$1}" = "" ]; then
            check_blank "$1" 
        fi
    fi
}

check_blank "Clone_url" $Clone_url

みたいなことをやってもうまくいかず、、、
知恵をお貸しいただきたく、、、

＊execでの変数に代入されていないと思われる件の追記。

本題から離れていってる気がするので、、、教えていただいた記述を例にメソッドを切り分けたいのですが、入力時も未入力時も動作がおかしいのです、、、

function read_input ()
{
    read input
    rtn_check_blank=$(check_blank) $input

    # 関数の返却値を標準出力に出力
    echo $rtn_check_blank
}

function check_blank ()
{
    input=$1
    while [ "$input" = "" ];
    do
        echo "It is not yet input."
        read input
    done
    echo $input
}

## Input setting
echo -e "\e[5;36m* \e[0m\e[1;36mPlease enter the URL of the remote repository of the clone to be created.\e[0;39m"
Clone_url=$(read_input)

echo -e "\e[5;36m* \e[0m\e[1;36mPlease enter an alias for managing the clone to be created.\e[0;39m"
Project_name=$(read_input)

echo -e "\e[5;36m* \e[0m\e[1;36mPlease enter the domain name to reflect the setting.\e[0;39m"
Domain_name=$(read_input)

echo $Clone_url,$Project_name,$Domain_name

Answer 1

通常、シェルスクリプトの中で関数から返却値を得たい場合は、return ではなく、標準出力に文字列で出力し呼び側でこれを変数に代入します。単純な例ですと

function test()
{
  echo "ans"
}

result=$(test)

# 以下で "ans" と表示される
echo $result

のような感じです。$(command)で__command__が標準出力に出力したものを文字列として受け取ります。
それで質問者様の例では関数内で標準出力をコンソールへの出力として使っているので、返却値の受取とファイルディスクリプタが競合してしまい、簡単にはこの方法が使えません。そこで、シェルの開始時に標準出力（＝コンソール）のファイルディスクリプタを別のファイルディスクリプタに複製しておいて、関数の出力と使い分ける必要があります。
具体的には以下のような感じです。

# 標準出力のファイルディスクリプタを複製してその番号をconsole という変数に保持
exec {console}>&1
function read_ans ()
{
    # 入力させる
    read ans
    while [ "$ans" = "" ]; do
        # 入力されていなかったらもう一度入力してもらう
        # 次の入力を促すメッセージは標準出力ではなくコンソールに出力
        echo "It is not yet input." >& $console
        read ans
    done
    # 関数の返却値を標準出力に出力
    echo $ans
}

## Input setting
echo -e "\e[5;36m* \e[0m\e[1;36mPlease enter the URL of the remote repository of the clone to be created.\e[0;39m"
Clone_url=$(read_ans)

echo -e "\e[5;36m* \e[0m\e[1;36mPlease enter an alias for managing the clone to be created.\e[0;39m"
Project_name=$(read_ans)

echo -e "\e[5;36m* \e[0m\e[1;36mPlease enter the domain name to reflect the setting.\e[0;39m"
Domain_name=$(read_ans)

echo $Clone_url,$Project_name,$Domain_name

---

1行目の exec {console}>&1 はシェルスクリプトでも凝った書き方（あまり見ませんし、周りで多少シェルスクリプトに詳しい人でも知らないことが多い）です。ちょっと長くなりますが、解説してみます。
通常、exec は別のコマンドをプロセスをfork せずに実行する場合に使用します。例えば、exec lsと書けば ls コマンドが実行されます。しかし、コマンドを指定しなかった場合、シェルは何もしません。例えば、

exec

と一行だけ書いて bash で実行しても何もしません。それで、この何もしないことを利用して、自分自身のプロセスのファイルディスクリプタをオープンしたり、クローズしたりするときなどに使います。例えば、flock というコマンドはファイルディスクリプタの番号を引数に取ってファイルをロックしますが、このファイルディスクリプタの取得にも使えます。以下は /var/hoge.lock というファイルをロックする例です。

# ファイルを追加書き込みモードでオープンしてそのファイルディスクリプタ番号を $lockfd に保持
exec {lockfd}>>/var/hoge.lock
# ロックを取得
flock $lockfd
# 独占した処理
...
# ロック開放
flock -u $lockfd
# ファイルをクローズ
exec {lockfd}>&-

最後の exec もファイルをクローズするときにこのように書きます。回答のシェルスクリプトではクローズしてませんが、丁寧に書くのであれば、exec {console}>&-を書くと良いでしょう。
次に{console}についてですが、これはbash に空いているファイルディスクリプタ番号を選んでもらって変数に入れてもらうときの書き方です。実際にファイルディスクリプタ番号に何が入るかというと10以上で最初に開いている番号が割り当てられるので、回答に示したシェルスクリプトでは 10 が入ります。試しに

exec {console}>&1
echo $console

というのを実行すると 10 が印字されると思います。短いシェルスクリプトであれば、変数を使わずにほぼ開いてることが確定している9というような番号を使うこともあります。回答例のシェルスクリプトでも

# 標準出力のファイルディスクリプタを9に複製
exec 9>&1
# 中略
        echo "It is not yet input." >& 9
# 後略

と書いても動作します。

Comments

[earnest_gay]

回答ありがとうございます。
処理の流れで何をしているのか、ひとつだけわからないところがあります。

>ディスクリプタを複製してその番号をconsole という変数に保持
exec {console}>&1

exec で調べてはみたのですが、これがこの処理においてなぜ必要なのか？
ないとどうなるのかというところが分かりません。

試しにexec {console}>&1を抜いて実行してみたりしましたが、差異が分かりませんでした。

[mit0223]

exec の解説を追加しました。 exec がなければ空入力のときに echo "It is not yet input." >& $console を実行しようとして、エラーになります。試してみましたが、「test.sh: 行 10: $console: 曖昧なリダイレクトです」というエラーになりました。動きが変わらなかったのは空入力をテストしていないからではないでしょうか。

[earnest_gay]

回答ありがとうございます。
やはり、こちらとそちらでの動作に差異があるように思えます。

実行結果の画像を追加しましたのでお手隙の際に見ていただけるとありがたいです。

[mit0223]

いろいろとミスっておられます。まずは私の回答のスクリプトを動作させてみて、空入力をテストしてみてください。それから exec をコメントアウトしてどうなるかやってみてください。ミスの内容としては
rtn_check_blank=$(check_blank) $input
の行は $input が check_blank の引数になっていません。カッコの中に引数を書かないと。それから、 echo で $console の内容を確認されてますが、$console が #console になってます。

[earnest_gay]

ありがとうございます！
解決できました。

Answer 2

mit0223さんのお書きのように、

通常、シェルスクリプトの中で関数から返却値を得たい場合は、return ではなく、標準出力に文字列で出力し呼び側でこれを変数に代入します。

ですが、read命令の代替のように使いたい場合は、evalで代入することで、渡した変数名に代入することも出来ます。

function my_read()
{
    while read input
          [ "$input" = "" -o "${input//[^A-Za-z0-9]/}" != "$input" ]
    do
        echo "It is not yet input, or invalid char."
    done
    eval $1="$input"
}
echo -e "～～～～" >&2
my_read foo
declare -p foo

ただし、evalは気をつけないと危険（1;dateのように入力したコマンドを実行できてしまう）なので、「誰に」「どのユーザー権限で」実行させるのかによっては十分なテストが必要です。
上記では、英数字以外があれば再入力にすることでこれを防いでいますが、;なども入力が必要なら面倒ですね。

一般的には、小さい整数を返すときはreturnで、そうで無いときは「標準出力」で返すのがいいと思います。

foo(){
    echo 123 456
}
ans=($(foo))
declare -p ans

のように、複数の文字列を返して、配列で受けることも出来ます。