ディレクトリを繰り返すときは、エントリがファイルかディレクトリかをテストします。

Question 1

問題は、次の構成があることです。

for [stuff]; do
   if [test]; then
       [stuff]
   else
       if [test]; then
          stuff
   fi
done

2つの入れ子になった項目がありますif。大丈夫ですが、そのうちの1つだけを終了できますfi。次のいずれかを実行する必要があります。

# Alternative one:
for [stuff]; do
   if [test]; then
       [stuff]
   else
       if [test]; then
          stuff
       fi
   fi
done

# Alternative two
for [stuff]; do
   if [test]; then
       [stuff]
   elif [test]; then
       stuff
   fi
done

Answer

問題は、次の構成があることです。

for [stuff]; do
   if [test]; then
       [stuff]
   else
       if [test]; then
          stuff
   fi
done

2つの入れ子になった項目がありますif。大丈夫ですが、そのうちの1つだけを終了できますfi。次のいずれかを実行する必要があります。

# Alternative one:
for [stuff]; do
   if [test]; then
       [stuff]
   else
       if [test]; then
          stuff
       fi
   fi
done

# Alternative two
for [stuff]; do
   if [test]; then
       [stuff]
   elif [test]; then
       stuff
   fi
done

Question 2

もう1つの問題は、テスト変数を使用するときに変数を（正しく）参照しますが、ループでは参照できないこと-dです。for

変える:

for i in $chosen_dir/*;

    do

使用：

for f in "$chosen_dir"/*; do

（名前の変更とスペースの変更はスタイルポイントです。追加された二重引用符が重要です。）

カウンターを実際に使ったことがないので、カウンターで何をしたいのかわかりません。また、上記の$i構成$fに基づいてfor。"$chosen_dir"

さて、これを宿題に指定していただきありがとうございます。より短い解決策があるかもしれませんが、構文エラーを修正して問題ない場合は、一般的なアプローチは問題ありません。変数参照。

サブディレクトリのファイル数の場合、使用するコマンドで正確な数が得られないことがあります。（実際にはそうでしょう。いいえ正確な数を得るには、コマンドを使用します。ls "$chosen_dir" | wc -lこれは、サブディレクトリではなく、ユーザーが入力したディレクトリをテストするためです。）サブディレクトリ内の正確なファイル数を提供できるように、linecountのls代わりに使用を検討することができます。ls -1 -A "$i"（およびフラグman lsの詳細については、参考資料を参照してください。）-1-A

これはまだ改行文字を含むファイル名を正しく処理しませんが、宿題のスコアを失うことは非常に疑わしいです。（実際に本番で使用されるスクリプトの多くは、改行文字を含むファイル名でも失敗します。これらの名前は、ほぼ完全に悪意のあるユーザーからのものです。）

C部分は自分で把握できると思います。 ;)（このサイトでタグを検索して必要かどうかを確認するman sortこともできます。）sort

Answer