シェルスクリプトの引数のファイルが存在するかどうか

Question

前提・実現したいこと

引数を複数持ち、その引数の名前のファイルが存在すれば、そのファイルの詳細情報をls -lを用いて表し、存在しなければ、「存在しないと表現するシェルスクリプトをさくせいしたいです。

このような機能を実装中に以下のエラーメッセージが発生しました。

発生している問題・エラーメッセージ

引数が一つのときは、

if[-e "$1"]; then
ls -s
else
echo "存在しない"
fi
で実行できたのですが、

複数の引数に対応するように書き直した結果、全ての存在しないと表示するものが出来上がりました。

しかし、なぜ存在しないが表示されているのか理解できません。

sh test ex1.c ex2.cの場合、

存在しない
存在しない

該当のソースコード

for((i=1; i<="$#";i++))
do
  if[ -e "$i" ]; then
    ls -l "$i"
  else
    echo "存在しない"
  fi
doen

試したこと

インターネットで調べること

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

コンパイラ　Tera Tarm
エディタ　vi

Comments

[KojiDoi]

> doen

明らかにスペルミスですね。質問には実際に書いたものをそのままコピペするようにしてください。見た人がそのまま自分のPCで実行できるように。

Answer 1

引数全てに対して同じ処理をしたいときは、

for file
do
  if[ -e "$file" ]
  then
    ls -l "$file"
  else
    echo "存在しない"
  fi
done

のようにin無しのforを使います。

添え字を使うのであれば、

ARGV=("$@")
for((i=0; i<"$#"; i++))
do
  echo $i
  if [ -e "${ARGV[i]}" ]
  then
    ls -l "${ARGV[i]}"
  else
    echo "存在しない"
  fi
done

元のコードはスペルミスその他文法がおかしいですね。

Answer 2

おそらく

for x in $* ; do
  if [ -e "$x" ]; then
...

を使うべきではないでしょうか。

Comments

[takumi-33]

ご回答ありがとうございます。
その形式の書き方を始めて拝見しました。
詳しく調べたいのですが、何と検索すれば出てくるでしょうか？

[KojiDoi]

bash 引数 ループ
とか。

[takumi-33]

ありがとうございました！お陰様で、解決することが出来ました！

[otn]

これは、ファイル名が空白を含んでいると駄目ですね。汎用に使うのでなければ良いのでしょうが。
sh test foo.c "bar bar.c"

[KojiDoi]

> ファイル名が空白を含んでいると駄目ですね。
その可能性は考えていませんでしたね。まだまだ修行が足りない。

Answer 3

if[ -e 1 ]; thenが実行されることになりますよね？

Comments

[takumi-33]

ご回答ありがとうございます！

ご指摘いただいた内容が理解しきれていないかもしれませんが、ご指摘の箇所が実行されて、ファイルが存在するにもかかわらず、存在しないが表示されるので、困っています。

[itagagaki]

何が起こっているかを解説しますね。
for で、まず最初に変数 $i に 1 がセットされますね。
そして [ -e "$i" ] を実行しています。
変数 $i の値は 1 なので、[ -e 1 ] を実行していることになります。
それはつまり、1 という名前のファイルが存在するかどうかをテストしていることになります。
1 という名前のファイルは存在していないのでしょうから、
echo "存在しない" が実行されているわけです。

[KojiDoi]

実際には「1」とか「2」とかいう名前のファイルが存在するのですか？ そうではないのでは？

[KojiDoi]

被っちゃった。失礼。

[takumi-33]

理解できました！
ご丁寧に解説頂いてありがとうございます。
凄く分かり安かったです。