[Bash script] Следилка за изменением на СИ

[Finch]

Подсел я на несколько книг, которые авторы выкладывают в СамИздате (СИ) по главам. Все время мониторить вручную, что то стало лениво. По быстрому написал простенький Bash скрипт, может кому либо тоже пригодится.
В файле link.txt нужно записать URL всех страниц, которые нужно мониторить. Файл хранится в одном каталоге со скриптом. В процессе работы создается файлы index.txt и index.tmp в том же каталоге. Для корректной работы, index.txt не стирать.

Код: (Bash)

#!/bin/bash

ABSOLUTE_FILENAME=`readlink -e "$0"`
DIRECTORY=`dirname "$ABSOLUTE_FILENAME"`
cd $DIRECTORY

TEMP=`mktemp`
FILELINK="link.txt"
FILEINDEXTEMP="index.tmp"
FILEINDEX="index.txt"

FOREACH=`cat $FILELINK`

if [ -f $FILEINDEXTEMP ]; then
   rm $FILEINDEXTEMP
fi


for i in $FOREACH
   do
   wget -q $i -O $TEMP
   LENGHT=`stat --format=%s $TEMP`
   echo "$i $LENGHT">> $FILEINDEXTEMP
   STRING=`cat $FILEINDEX| grep "$i"`
   OLDLENNGHT=`echo $STRING|cut -d' ' -f2`
   if [ -z $OLDLENNGHT ]; then
      OLDLENNGHT=0;
   fi
   if (( $OLDLENNGHT!=$LENGHT )); then
       echo "$i is changed. Old size is $OLDLENNGHT. New size is $LENGHT"
   fi
done
mv $FILEINDEXTEMP $FILEINDEX
rm $TEMP
cd - >>/dev/null

[RXL]

Несколько замечаний/предложений:

* Переменная может содержать произвольную последовательность и при интерполяции лучше заключить ее в кавычки.
Тут тоже:

Код: (Bash)

if [ -z $OLDLENNGHT ]

* Вместо `` можно использовать $(), что немного лучше читается.

* Там где интерполяция не нужна, следует использовать одинарные кавычки.
Пример:

Код: (Bash)

FILELINK="link.txt"

* Надо «форсить»:

Код: (Bash)

rm $FILEINDEXTEMP

* Можно в одну строку:

Код: (Bash)

   if [ -z $OLDLENNGHT ]; then
      OLDLENNGHT=0;
   fi