Sed übergibt Dateinamen von Bash, aber friert ein

Question

Ich habe ein Bash-Skript, das versucht, eine durch Tabulatoren getrennte Datei zu konvertieren und in eine kommagetrennte Datei zu konvertieren. Das folgende Skript friert, wenn ich in ‚/tmp/filename.txt‘ passieren:

#!/bin/bash 
sed 's/  /,/g' $1 > /tmp/csvFile.csv 

jedoch verhält sich das Skript richtig, wenn ich die Dateinamen explizit erklären:

#!/bin/bash 
sed 's/  /,/g' /tmp/filename.txt > /tmp/csvFile.csv 

Ich brauche dieses Skript zu sein Der Dateiname kann als Befehlszeilenargument verwendet werden, sodass er wiederverwendbar ist. Kann mir jemand sagen, was ich falsch mache?

EDIT:

My $ 1 betrug leer, mein obiges Problem verursacht. Ich verstehe immer noch nicht, warum $ 1 leer war. Hier ist die vollständige Skript:

#!/bin/bash 

function main { 
    splitCSV 
    cat /tmp/output.txt 
} 

function splitCSV { 
    sed 's/  /,/g' $1 > /tmp/csvFile.csv 
    awk -F ',' 'NR > 1 !($2 in facilities) { facilities[$2] } END { for (x in facilities) print x }' /tmp/csvFile.csv > /tmp/output.txt 
} 

main 

Ich nehme an, das eine Art von Umfang Problem ist?

Answer 1

Mein $ 1 war leer, verursacht mein oben genanntes Problem. Ich verstehe immer noch nicht, warum $ 1 leer war.

Innerhalb einer Shell-Funktion, die Positionsparameter ($1, etc.) Beziehen sich auf die Funktionsargumente, nicht das Befehlszeilenargument des Skripts. In einer Shell-Funktion manipulieren die Shell-Einbauten, die die Positionsparameter manipulieren (z. B. shift), die lokale Parameterliste der Funktion, nicht die Liste des gesamten Skripts.

Wenn Sie möchten, dass eine Shell-Funktion die Positionsparameter des gesamten Skripts anzeigt, müssen Sie sie weiterleiten. Zum Beispiel

#!/bin/bash 

function main { 
    # forward the first two function arguments 
    splitCSV "$1" "$2" 
    cat /tmp/output.txt 
} 

function splitCSV { 
    sed 's/  /,/g' "$1" > /tmp/csvFile.csv 
    awk -F ',' 'NR > 1 !($2 in facilities) { facilities[$2] } END { for (x in facilities) print x }' /tmp/csvFile.csv > /tmp/output.txt 
} 

# forward all the positional parameters, each one as one argument 
main "$@" 

Sie brauchen ein bisschen vorsichtig mit zitiert, aber sein, falls die Parameter Leerzeichen oder andere Shell-Metazeichen enthalten.