shell script で コマンドライン引数 を C++ 実行ファイルに渡す方法

Question

C++の実行ファイル./a.outはコマンドライン引数としてrankをとります．rankを0から10まで変えて./a.outを実行したい場合どのようにシェルスクリプトを書けばよいか，という質問です．

私は，以下のようにしました．

int rank=0
for  ((rank=0 ; rank<10; rank++))
do
 ./a.out $rank&
done
wait

ただ，これでは正常にrankの値が渡されていないようです．プログラムではrankが全て0として処理されているようです．何が問題でしょうか．

なお，C++の中では，コマンドライン引数を次のように受け取っています．

#include <cstdlib>

int main(int argc, char* argv[]){
   int rank = atoi( argv[1] );
   hogehoge
}

環境

CentOS, Bash

追記

問題を簡潔にするために，shellscriptの一部だけを取り出していましたが，問題のshellscriptの全容は以下です．以下のコードでrank引数がうまくいかない理由を考えているのです．

int cnt=0;
int noise_level=0;
int dataset_number=0;
int sample_size=100;
int rank=5;
for ((i=0 ; i<sample_size; i++))
do
for ((j=0 ; j<i+1 ; j++))
do
    cnt=$((cnt+1));
    if [ $cnt = 75 ]; then
        wait
        cnt=0;
    fi
    echo $((rank));
    ./a.out 0 0 opt $i $j $rank&
done
done
wait

Comments

[maisumakun]

問題はシェルスクリプト側にある、ということは何かしらの方法で確認しましたか？

[physics303]

ありがとうございます．

はい，コマンドラインに

./a.out 3

のように入力すると正常に処理が進むことから，shellscirptの文法を間違えているのかなと認識しています．$rankは"$rank"にしないといけない...など？

また，C++プログラムの方で，atoiを使うと，不正な値は全て0になるので，rankが不正？文字列として認識されている？のかなとも思っています．

Answer 1

y_waiwaiさんのコメントのところでも書きましたが。
int rank=0は変ですよね。
（それともブラウザによってアンダースコアが消えるなんてことが？）

if [ $cnt = 75 ]; then
これも変ですよね。等号ではなく「-eq」を使用しないといけません。

あとなんか「a.out」を990回バックグラウンドで実行し、
75回毎にwaitで引き取ることを意図している様に見えますが・・
（違っていたらすみません）

もっとシンプルなスクリプトからテストして、
問題なければ少しずつスクリプトを複雑にしていく方法を
取った方が良いと思います。

Comments

[physics303]

ありがとうございます．int rank=0 は不要でした．$cnt = 75 はダメなのですね，-eq に修正しました．

意図としては，スレッドが88個あるマシンの上で，rank を固定して，iをsample_sizeまで動かして，jをiまで動かして

./a.out i j rank

を実行したい，ということでした．

[hana_yama_san]

他の言語のの常識が染み付いて、シェルスクリプトの書き方が
馴染めない。・・と言うのよくある話です。
そこは追々努力で補って下さい。
※参考
https://qiita.com/piroor/items/77233173707a0baa6360

しかし私が一番理解してほしいのは、

＞＞もっとシンプルなスクリプトからテストして、
＞＞問題なければ少しずつスクリプトを複雑にしていく方法を
＞＞取った方が良いと思います。

の、部分です。

[physics303]

ありがとうございます．
論文の締め切りに追われて，慌てて慣れないことをして，ドツボにはまっています．

>>これではとてものことに解決したとは・・・

この点についてご解説いただくことは可能ですか？

[hana_yama_san]

それを私が書いたときには、
int cnt=0;が、ゴミとして残っていました。
また私の書いた等号の問題も未反映でした。
※このタイミングの問題が最大の原因です。

[physics303]

コメントありがとうございます．
int cnt=0; と -eq を修正したので，解決した，といえる，ということで問題ないでしょうか？

[hana_yama_san]

プログラムの多重実行部分はもっと慎重にテストを重ねて、
例えば[ $cnt -eq 75 ]が適切かどうかなど、
シェルスクリプト側にも今後変更が必要になることがあるかもしれません。

しかし、ご質問の 「$rank」が順次インクリメントされて「a.out」に
渡されるようになった。と言う部分では一旦解決として良いのでは無いでしょうか。

[physics303]

ありがとうございました．

Answer 2

解決しました．intの型宣言を行う必要がありませんでした．

cnt=0;
noise_level=0
dataset_number=0
sample_size=100
rank=5
for ((i=0 ; i<sample_size; i++))
do
for ((j=0 ; j<i+1 ; j++))
do
    cnt=$((cnt+1));
    if [ $cnt -eq 75 ]; then
        wait
        cnt=0;
    fi
    echo $((rank));
    ./a.out 0 0 opt $i $j $rank&
done
done
wait

Comments

[hana_yama_san]

これではとてものことに解決したとは・・・

[physics303]

これで正しく 変数rankの値がプログラムにわたっているように思うのですが，どこが問題でしょうか．=と-eqは修正しました．

[physics303]

shell script を初めて書いているので，問題点があれば教えてほしいです．

Answer 3

まずは、

int rank=0
for  ((rank=0 ; rank<10; rank++))
 echo $rank&
done
wait

としてなにが表示されるのかを見てみたらどうでしょう
表示されるものが引数として渡ってます

Comments

[physics303]

ありがとうございます．
やってみました．

まず，文法エラーがありますね．doが足りてないです．その旨，修正しました．その結果，

for ((rank=0 ; rank<10; rank++))
do
echo $rank
done

は0から9までの数字が順番にプロットされますね．

[y_waiwai]

なら０が渡ってるんでしょうね

[hana_yama_san]

まあ多分
bash: int : command not found
で止まるでしょうね。

・・・というか、止まりました。