私は次のスクリプトを書いた。
#!/bin/bash
#files list
file1=/tmp/1wall_long.txt
file2=/tmp/1wall_test1.txt
file3=/tmp/1wall_test2.txt
file4=/tmp/1wall_test3.txt
file5=/tmp/3mt_long.txt
file6=/tmp/3mt_OpenSpace_test1.txt
file7=/tmp/3mt_OpenSpace_test2.txt
file8=/tmp/3mt_OpenSpace_test3.txt
file9=/tmp/3rooms_test1.txt
file10=/tmp/3rooms_test2.txt
file11=/tmp/3rooms_test3.txt
file12=/tmp/20mt_OpenSpace_test1.txt
file13=/tmp/20mt_OpenSpace_test2.txt
file14=/tmp/20mt_OpenSpace_test3.txt
#script for 1wall_long file
if [ ! -e "$file1" ]; then #check if the file exist
echo "File 1wall_long.txt does not exist" #if not exist print echo output
else
sed -i -e 's/- /-/g' $file1 #remove space on the first 10 values
awk '{print $7}' $file1 > /tmp/1wall_long_S.txt #print the column number 7 and copy the output in a file
rm $file1 #remove old file
fi
変数で説明されているすべてのファイルに対してスクリプトを繰り返す(基本的に別の変数を使用して同じスクリプトを14回繰り返しました)これを行うより良い方法がありますが、その場合のベストプラクティスは何ですか?
ベストアンサー1
非循環的
まず、関数を試してください。
function sevenc
{
if [ ! -e "$1" ]; then #check if the file exist
echo "File $1 does not exist" #if not exist print echo output
else
sed -i -e 's/- /-/g' "$1" #remove space on the first 10 values
awk '{print $7}' "$1" > /tmp/$(basename $1.txt)_S.txt #print the column number 7 and copy the output in a file
rm "$1" #remove old file
fi
}
- シェルが関数を認識すると、引数(存在する場合は$1 $2...など)が渡されます。
- ちなみに
's/- /-/g' "$1" #最初の10個の値からスペースを削除
いいえ、1、4、10、255などはすべてspace-1行に移動します。-
これでvarsはもう必要ありません。
sevenc /tmp/1wall_long.txt
sevenc /tmp/1wall_test1.txt
sevenc /tmp/1wall_test2.txt
sevenc /tmp/1wall_test3.txt
sevenc /tmp/3mt_long.txt
sevenc /tmp/3mt_OpenSpace_test1.txt
sevenc /tmp/3mt_OpenSpace_test2.txt
sevenc /tmp/3mt_OpenSpace_test3.txt
sevenc /tmp/3rooms_test1.txt
sevenc /tmp/3rooms_test2.txt
sevenc /tmp/3rooms_test3.txt
sevenc /tmp/20mt_OpenSpace_test1.txt
sevenc /tmp/20mt_OpenSpace_test2.txt
sevenc /tmp/20mt_OpenSpace_test3.txt
(もはやfileXX varを使用しない場合)
ループなし(解決策2)
より多くのパラメータを渡し、Terdonの最適化を使用するには、次のことを試してください。
function eight
{
file=$1
destdir=${2-/tmp} # use second arg if defined, else /tmp
exten=${3-S}
if [ ! -e "$file" ]; then #check if the file exist
echo "File $file does not exist" #if not exist print echo output
else
sed -e 's/- /-/g' "$file" \
awk '{print $7}' "$1" > /"$destdir"/$(basename $1.txt)_"$exten".txt #print the column number 7 and copy the output in a file
rm "$file" #remove old file
fi
}
として呼び出されます
eight /tmp/1wall_test3.txt /my/projec/dir T ## will use /my/project/dir as dit, T as extension
eight /tmp/1wall_test1.txt /my/project ## will use /my/project as dir
eignt /tmp/1wall_test2.txt ## will use default value
これらの関数は .bashrc で定義され、インタラクティブに使用できます。
ループを含む
while read f
do
if [ ! -e "$f" ]; then #check if the file exist
echo "File $1 does not exist" #if not exist print echo output
else
sed -i -e 's/- /-/g' "$f" #remove space on the first 10 values
awk '{print $7}' "$f" > "/tmp/$(basename $f .txt)_S.txt" #print the column number 7 and copy the output in a file
rm "$f" #remove old file
fi
done <<EOF
/tmp/1wall_long.txt
/tmp/1wall_test1.txt
/tmp/1wall_test2.txt
/tmp/1wall_test3.txt
/tmp/3mt_long.txt
/tmp/3mt_OpenSpace_test1.txt
/tmp/3mt_OpenSpace_test2.txt
/tmp/3mt_OpenSpace_test3.txt
/tmp/3rooms_test1.txt
/tmp/3rooms_test2.txt
/tmp/3rooms_test3.txt
/tmp/20mt_OpenSpace_test1.txt
/tmp/20mt_OpenSpace_test2.txt
/tmp/20mt_OpenSpace_test3.txt
EOF