Нужна помощь с моим скриптом bash - PullRequest
Новая
       24

Нужна помощь с моим скриптом bash

1 голос
/ 04 апреля 2011

нужна помощь с моим скриптом. Я хочу конвертировать файлы ISO в UTF-8. Проблема в том, что я не знаю, как написать IF: 

if [ `file -b {}` = "$UTF8" ] \

правильно, а как можно сказать программе sed - что она игнорирует # комментарии?

Вот мой сценарий: 

#!/bin/bash

clear

echo -e '\E[37mThis script encodes recursively files from \E[31mISO-8859-1 \E[37mto         \E[31mUTF-8 \E[37musing iconv.'
echo "Files of the following coded character sets will be encode: "
echo -e '\E[32m'

a='*.java'
b='*.txt'
c='*.php'
d='*.html'
e='*.aj'
f='*.patch'
g='*.css'
h='*.js'
i='*.conf'
j='*.jsp'
k='*.sh'
l='*.py'
m='*.pl'
n='*.rb'

for x in "$a" "$b" "$c" "$d" "$e" "$f" "$g" "$h" "$i" "$j" "$k" "$l" "$m" "$n" 
do
   echo $x
done

echo
tput sgr0

#
# TODO: COMMENTS aren't ignored
# TOOD: IF-THEN aren't working right
#


for y in  "$a" "$b" "$c" "$d" "$e" "$f" "$g" "$h" "$i" "$j" "$k" "$l" "$m" "$n"  
  do
    echo -e "\E[37mencoding all <\E[32m$y\E[37m> files ..."
    find . -name "$y" -exec sh -c "( \
      UTF=".*UTF-8 Unicode.*" \
      FILE={} \
      if [ `file -b {}` = "$UTF8" ] \ 
        then  \   
          iconv -f latin1 -t UTF-8 {} -o {}.iconv ; \
          sed -n '
              { 
                s/^ *#/#/#.*//g;
                s/ä/0xE4;/g;
                s/Ä/0xC4;/g;
                s/ü/0xFC;/g;
                s/Ü/0xDC;/g;
                s/ö/0xF6;/g;
                s/Ö/0xD6;/g;
                s/ß/0xDF;/g;
                p; 
                }  {}.iconv > {}.iconv_sed \ '
       mv {}.iconv_sed {} && rm {}.iconv ; \
    else \
      echo "$FILE is a UTF8 file. " \ 
  fi \
)" \;
      echo -e '\E[33m*** done ***'
done

echo 
tput sgr0

exit 0

спасибо

1 Ответ

1 голос
/ 04 апреля 2011

Кажется, что в вашем скрипте не так много неправильных вещей (например, я не вижу нигде определенной переменной "UTF8"), но вы сильно усложнили ее с точки зрения отладки. Если бы это был я, я бы:

  1. поместите все дерьмо находки sh -c "... в отдельный скрипт, чтобы вы могли проверить его отдельно 
  2. if [ "`file -b $1`" = ...

  3. возможно, поместите sed в отдельную функцию и проверьте , что

  4. не использовать sed -n, а затем явно p; каждую строку, это глупо
  5. правильно процитировать скрипт sed; Я полагаю, что вы пытаетесь сделать перенаправление внутри него

... для начала вам достаточно пяти предложений. Предложение 0 - «напишите более конкретный заголовок для вашего вопроса»

...