Gibt es einen eleganteren Weg?

killernase · 21. Februar 2009

Hallo,

das ist so ziemlich mein erster Versuch mit der Shell-Programmierung. Ich lade per Skript XML-Dateien von einem Webserver und muss ein paar simple Prüfungen ausführen, um zu schauen, ob sie grob in Ordnung sind:

Code

XMLLINT=`which xmllint`


for xmlfile in /tmp/Alpen*xml
do
  if [ ! -s $xmlfile ]
  then
    echo "Fehler beim Download"
    continue
  fi


if [ -x $XMLLINT ]
then
  $XMLLINT --noout $xmlfile 2> /dev/null
  if [ $? -ne 0 ]
  then
    echo "Fehler beim Parsen von $xmlfile!"
    continue
  fi
fi


# Verarbeitung
done

Alles anzeigen

Nun zu den Fragen:
[list=1]
[*]Kann ich irgendwie rauskriegen wieviele Dateien in der Liste "/tmp/Alpen*xml" sind? (z.B. wenn ich die Liste vorher in eine Variable speichere)
[*]Lässt sich die Prüfung auf Wohlgeformtheit der XML-Dateien vereinfachen? Kann ich die Prüfung auf Existenz von xmllint und den Check des XMLs irgendwie zusammenfassen?
[/list=1]
Vielleicht hat ja jemand ein paar Tipps für mich.

MfG
Marco

wirbel · 21. Februar 2009

Die Anzahl könntest du über ls und wc herausfinden oder in der for loop eine variable inkrementieren.

Außerdem machts Sinn nur einmal direkt nach dem which zu prüfen, ob $XMLLINT gültig ist, nicht jedes mal im loop

Urig · 21. Februar 2009

Zitat

Originally posted by killernase
[list=1][*]Kann ich irgendwie rauskriegen wieviele Dateien in der Liste "/tmp/Alpen*xml" sind? (z.B. wenn ich die Liste vorher in eine Variable speichere)
[/list=1]

Genau so.

Code

xmlfiles=(/tmp/Alpen*xml)
echo ${#xmlfiles[*]}

Den Sonderfall, dass keine Datei vorhanden ist, musst du aber abfangen:

Code

if [ "${xmlfiles[0]}" = "/tmp/Alpen*xml" ] ; then echo 0 ; else echo ${#xmlfiles[*]} ; fi

Und wenn du schon die Dateiliste als Array hast, wirst du das auch benutzen wollen:

Code

for xmlfile in "${xmlfiles[@]}" ; do
  ...
done

Gruß,

Udo

Gibt es einen eleganteren Weg?

Jetzt mitmachen!

Teilen

Benutzer online in diesem Thema