[Bash] Contacter

[RedSkull]

Boh, volevo imparare il bash, ho fatto questo piccolissimo script che permette la gestione di una piccola rubrica. (aggiungere/togliere/cercare contatti)
Salva il tutto su un file txt
Visto che sto imparando mi servono dei consigli per correggere eventuali imperfezioni, grazie.
http://sprunge.us/MCNL

 

[fojeaf]

nice, anche se il fatto che usi un file per ogni contatto è bruttino

 

[vheon]

nice, anche se il fatto che usi un file per ogni contatto è bruttino

perche' un file per ogni contatto? c'e' un solo file ed e' contatti.txt

 

[imported_Bronsa]

è vulnerabile

 

[orakool]

non l'ho ancora letto, ma intanto cosina: usa basename $0 per l'usage, o verrà printato tutto il percorso con cui è stato chiamato lo script

e, carino ^^

---

Quello che mi viene in mente: non mettere il file in home, ma nella stessa dir del programma (ha più senso), e metti un'opzione per farlo scegliere, eventualmente, il file

 

[vheon]

è vulnerabile

Dato che siamo in periodo di "crisi", perche' non concludi i tuoi commenti dandogli un senso, aggiungendo delle spiegazioni invece di essere cosi stitico?

 

[imported_Bronsa]

Non fa controlli, ne escapa variabili

 

[vheon]

[ot]

Non fa controlli, ne escapa variabili

ti ci voleva tanto farlo da prima?
[/ot]

 

[RedSkull]

Perfetto grazie per i consigli, ho già modificato il sorgente, bronsa potresti farmi vedere in che modo dovrei risolvere il problema della "vulnerabilità" ?
Adesso modificando il sorgente ho messo dei controlli sull'add (controlla che nel secondo parametro vengano inseriti solo numeri)
Grazie per le dritte.
EDIT: http://sprunge.us/cYCS

 

[fojeaf]

nice, anche se il fatto che usi un file per ogni contatto è bruttino

perche' un file per ogni contatto? c'e' un solo file ed e' contatti.txt

perché sono gay ed ho perso la linea di dichiarazione della var e pensavo fosse generata via input.
sorry