☰

拡張子に基づいてファイルを並べ替えるこのスクリプトをどのように改善できますか？

tag-icon • tag-icon shell-script find xargs
拡張子に基づいてファイルを並べ替えるこのスクリプトをどのように改善できますか？

私のダウンロードフォルダに移動し、拡張子に従ってファイルを並べ替える小さなスクリプトがあります。

私はこれをどうすることができますか？よりきれいで/より良い？拡張子を追加するたびに新しい行を追加する必要がないように、拡張子とそのディレクトリのリストを保持し、forループなどを使用してコマンドを実行したいと思います。

現在のスクリプト：

#!/bin/sh

LOCKFILE=/tmp/.hiddensync.lock

if [ -e $LOCKFILE ]
        then
        echo "Lockfile exists, process currently running."
        echo "If no processes exist, remove $LOCKFILE to clear."
        echo "Exiting..."

        exit
fi

touch $LOCKFILE
timestamp=`date +%Y-%m-%d::%H:%M:%s`
echo "Process started at: $timestamp" >> $LOCKFILE

## Move files to various subfolders based on extensions
find ~/Downloads -maxdepth 1 -name "*.pdf" -print0 | xargs -0 -I % mv % ~/Downloads/PDF/
find ~/Downloads -maxdepth 1 -name "*.opm" -print0 | xargs -0 -I % mv % ~/Downloads/OPM/
find ~/Downloads -maxdepth 1 -name "*.yml" -print0 | xargs -0 -I % mv % ~/Downloads/YML/
find ~/Downloads -maxdepth 1 -name "*.css" -print0 | xargs -0 -I % mv % ~/Downloads/CSS/
find ~/Downloads -maxdepth 1 -name "*.tar.gz" -print0 | xargs -0 -I % mv % ~/Downloads/archives/
find ~/Downloads -maxdepth 1 -name "*.zip" -print0 | xargs -0 -I % mv % ~/Downloads/archives/
find ~/Downloads -maxdepth 1 -name "*.jpg" -print0 | xargs -0 -I % mv % ~/Downloads/Pictures/
find ~/Downloads -maxdepth 1 -name "*.png" -print0 | xargs -0 -I % mv % ~/Downloads/Pictures/
find ~/Downloads -maxdepth 1 -name "*.tiff" -print0 | xargs -0 -I % mv % ~/Downloads/Pictures/
find ~/Downloads -maxdepth 1 -name "*.pm" -print0 | xargs -0 -I % mv % ~/Downloads/Perl/
find ~/Downloads -maxdepth 1 -name "*.xls*" -print0 | xargs -0 -I % mv % ~/Downloads/Excel/
find ~/Downloads -maxdepth 1 -name "*.doc*" -print0 | xargs -0 -I % mv % ~/Downloads/Word/

echo "Task Finished, removing lock file now at `date +%Y-%m-%d::%H:%M:%s`"
rm $LOCKFILE

ベストアンサー1

ターゲットに複数の拡張がある場合は、findディレクティブにさらにロジックを追加できます。

find ~/Downloads -maxdepth 1 \( -name "*.tar.gz" -o -name "*.zip" \) -print0 | xargs -0 -I % mv % ~/Downloads/archives/

そしてxargsにパイプする必要はありません。

find ~/Downloads -maxdepth 1 \( -name "*.tar.gz" -o -name "*.zip" \) -exec mv -t ~/Downloads/archives/ {} +

今それを持っていますが-maxdepth 1、本当にそれが必要ですかfind？

shopt -s nullglob
cd ~/Downloads
mv -t archives/ *.tar.gz *.zip
mv -t Pictures/ *.jpg *.png *.tiff
# etc

この方法では、移動するファイルが存在しない場合は、いくつかのエラーが発生します。次のようにこの問題を解決できます。

shopt -s nullglob
movefiles() {
    local dest=$1
    shift
    if (( $# > 0 )); then
        mkdir -p "$dest"
        mv -t "$dest" "$@"
    fi
}
cd ~/Downloads
movefiles PDF/      *.pdf
movefiles OPM/      *.opm
movefiles YML/      *.yml
movefiles CSS/      *.css
movefiles archives/ *.zip *.tar.gz
movefiles Pictures/ *.jpg *.png *.tiff
movefiles Perl/     *.pm
movefiles Excel/    *.xls*
movefiles Word/     *.doc*

メモ:

  • nullglob がない場合、パターンに一致するファイルがない場合、関数はパターンを文字列として受け取ります。
    • たとえば、pdfファイルがない場合、シェルが実行されます。movefiles PDF/ "*.pdf"
  • nullglob の場合、一致するものがない場合、シェルはコマンドからパターンを削除します。movefiles PDF/
  • これが引数の数を確認する理由です。一致するファイルがない場合、$# はシフト後 0 なので、何も移動できません。

おすすめ記事