シェルスクリプトを実行したらmkdirのエラーが出ました

Question

ファイル名のパターンからフォルダーに配置していくシェルスクリプトを作りたいと思い以下のようなものを書きました。
実行すればカレントディレクトリにあるmp3ファイルを検索し条件に応じて各フォルダに移動していくものです。移動先のフォルダがなければその都度作成します。

しかし、実行すると以下のようなエラーメッセージが出てしまい、原因がわかりません。対策として、mkdirの後ろを""で囲んでみたりしましたが同じエラーが出ました。もしよろしければ解決するために皆様の知恵をお貸しいただけないでしょうか。
#コード

find . -name "*.mp3" -type f | while read -r f;do
fname="${f%.*}"
fname="${fname##*@}"
falbum="${fname%_*}"
fartist="${fname#*_}"

if [ -e /Users/hoge/radio/file/tmp/${fartist} ]; then
	if [ -e /Users/hoge/radio/file/tmp/${fartist}/${falbum} ]; then
		mv $f /Users/hoge/radio/file/tmp/${fartist}/${falbum}/
	else
		mkdir /Users/hoge/radio/file/tmp/${fartist}
		mkdir /Users/hoge/radio/file/tmp/${fartist}/${falbum}
		mv $f /Users/hoge/radio/file/tmp/${fartist}/${falbum}/
	fi
else
	mkdir /Users/hoge/radio/file/tmp/${fartist}/${falbum}/
	mv $f /Users/hoge/radio/file/tmp/${fartist}/${falbum}/
fi
done

#エラーメッセージ

mkdir: /Users/hoge/radio/file/tmp/hoge: No such file or directory
mv: rename ./19900101@hoge_hoge.flv to /Users/hoge/radio/file/tmp/hoge/hoge/: No such file or directory

Answer 1

mkdir /Users/hoge/radio/file/tmp/${fartist}の位置がおかしいです。

find . -name "*.mp3" -type f | while read -r f;do
fname="${f%.*}"
fname="${fname##*@}"
falbum="${fname%_*}"
fartist="${fname#*_}"

if [ -e /Users/hoge/radio/file/tmp/${fartist} ]; then
    if [ -e /Users/hoge/radio/file/tmp/${fartist}/${falbum} ]; then
        mv $f /Users/hoge/radio/file/tmp/${fartist}/${falbum}/
    else
        mkdir /Users/hoge/radio/file/tmp/${fartist}/${falbum}
        mv $f /Users/hoge/radio/file/tmp/${fartist}/${falbum}/
    fi
else
    mkdir /Users/hoge/radio/file/tmp/${fartist}
    mkdir /Users/hoge/radio/file/tmp/${fartist}/${falbum}/
    mv $f /Users/hoge/radio/file/tmp/${fartist}/${falbum}/
fi
done

もっとシンプルには、これでいいです。

find . -name "*.mp3" -type f | while read -r f;do
fname="${f%.*}"
fname="${fname##*@}"
falbum="${fname%_*}"
fartist="${fname#*_}"

mkdir -p /Users/hoge/radio/file/tmp/${fartist}/${falbum}/
mv $f /Users/hoge/radio/file/tmp/${fartist}/${falbum}/
done

Comments

[McKee]

回答していただきどうもありがとうございます。
このようにpオプションを指定すると、サブディレクトリごと作成でき、さらに既にディレクトリが存在する場合は何もしない、と私がやりたかったことにぴったりの結果が得られました、ありがとうございました。

Answer 2

[ missing ]となっていますが、閉じる]の前にはスペースが必要です。まずはそれを調整して確認してみて下さい。

Comments

[McKee]

回答していただきどうもありがとうございます、[]のところにはスペースを入れる必要があることを知りませんでした。スペースを追加したところ一部のエラーが解決されました。ありがとうございます。