welinux.ru / Есть вопрос!

dedsy 24.07.2011 19:22

Есть вопрос! — bash script

Написал небольшой скрипт для того чтоб не париться с подставлением ключей к утилите tar, собственно в bash скриптинге я новенький , послушал бы советы по улучшению скрипта. Скрипт работает, но в глубине сознания мелькает мысль , что можно было обойтись без стольких вложений проверки условий. А ну да ещё туда добавил zip архивы, скрипт служит только для разорхивирования..

#!/bin/bash
SUC="echo Job is done"
file $1 | grep gzip
if [ $? = 0 ]
     then
          tar -zxvf $1
          $SUC
 else
          file $1 | grep bzip
          if [ $? = 0 ]
                   then
                         tar -jxvf $1
                         $SUC
                        else
                 file $1 | grep "POSIX tar archive"
                   if [ $? = 0 ]
                           then
                                 tar -xvf $1
                                  $SUC
                                else
                         file $1 | grep "Zip archive data"
                            if [ $? = 0 ]
                                   then
                                         unzip $1
                                             $SUC
                                else
                                         echo " That's not a tar/bzip/gzip/zip archive"
                           fi
                   fi
           fi
fi

Тэги:

+ 0 -

Похожие

mironov_orig 24.07.2011 19:25 #

+ 1 -

Лол. Все случаи применения tar для распаковки можно описать так

1	tar xavf file

опция «a» заставляет tar автоматически определять тип архива.

K-9 25.07.2011 16:04 # ↑

+ 0 -

А еще (ВНЕЗАПНО!!!) в новых версиях tar это вообще дефолт, можно просто xf.

dedsy 24.07.2011 19:29 #

+ 1 -

XD олол, вот слона то я и не замелил...

mironov_orig 24.07.2011 19:30 #

+ 1 -

Вместо стольких условий нужно юзать case, например так

var='some value'
case $var in
  'value')
    echo 'value'
    ;;
  'some value')
    echo 'some value')
    ;;
  'some other value')
    echo 'some other value'
    ;;
esac

Пример, конечно, искусственный, но что надо иллюстрирует.

dedsy 24.07.2011 19:34 # ↑

+ 1 -

Спасибо.. :) Скрипт конечно как оказалось ненужны, но всё равно на нём попробую такую структуру.. учиться то на чём-то надо..

ananas 25.07.2011 13:30 # ↑

+ 2 -

в данном случае для case лучше использовать выхлоп file -bi $1