Error in Bash script. Wat gaat er fout?

[bart85]

Het volgende script geeft onderstaande error aan:

Code: [Selecteer]

$ ./path.sh
./path.sh: line 8: [: too many arguments


Code: [Selecteer]

#!/bin/bash
#
# Programma dat alle directory's die in $PATH staan op een nieuwe regel weergeeft.

IFS=:
string=$PATH

while [ -n $string ]
do
echo "${string%%$IFS*}"
string="${string#*$IFS}"
done
Hoe los ik dit op?

[bart85]

Het script wil uitvoeren  als ik de regel 8 vervang als hieronder:

Code: [Selecteer]

while [[ -n $string ]]
Het probleem is dat het programma niet stopt. De output is goed totdat de laatste directory word weergeven. Het blijft deze laatste directory herhalen. Het probleem is dat dan de pattern matching operator deze laatste er niet uit haalt.

[edehont]

1: de variabele in een test moet je altijd tussen aanhalingstekens zetten, want als hij bijvoorbeeld leeg is wordt de variabele vervangen door helemaal NIETS en

Code: [Selecteer]

while [  -n  ] is iets waar bash zich in verslikt. In jouw geval ziet bash een heleboel waarden, omdat je de IFS op : hebt gezet. Hij splitst $string dus op in allemaal velden terwijl hij na de -n maar 1 ding verwacht.

2: IFS is een interne variabele. Door die een waarde te geven verander je de manier waarop bash met velden omgaat. Het is dus niet de bedoeling dat je zelf verder met die variabele gaat werken. Je kunt dan simpelweg zoiets doen:

Code: [Selecteer]

IFS=:

for i in $PATH
    do echo $i
done


[edehont]

PS: je oorspronkelijke scriptje blijft loopen op het laatste item in path, omdat je laatste assignment geen ':' meer kan vinden. Bash geeft dan gewoon de hele variabele terug:

Code: [Selecteer]

string=$string, en dan blijf je eeuwig (nou ja, eeuwig...) met de laatste entry zitten.

PS2: de while met dubbele haken is ietsje slimmer en snapt dat de test loopt tot hij de ]] tegenkomt.

[grizzler]

Waarom zou je IFS überhaupt wijzigen?

Het oorspronkelijke scriptje zou er ook zo uit kunnen zien:

Code: [Selecteer]

string=$PATH:

while [ $string ]
do
    echo ${string%%:*}
    string=${string#*:}
done
De extra dubbele punt achter $PATH bij de toewijzing aan string zorgt ervoor dat string uiteindelijk wel leeg wordt, zodat de lus wordt beëindigd.

Als je verwacht dat PATH spaties bevat (wat normaal gesproken niet zo zal zijn), moet je aanhalingstekens gebruiken in de toewijzing en de test.