• # Quelques remarques sur le script

    Posté par . En réponse au journal Hébergement dynamique et modem ADSL Comtrend. Évalué à 4.

    Voici quelques remarques:

    Tu fais des tests avec -s pour savoir si le fichier existe. En général on utilise plutot le -r.

    -s FILE True if file exists and is not empty.
    -r FILE True if file is readable by you.

    Si le fichier est vide, on s'en fiche un peu, ça n'empèche pas de le sourcer; par contre, s'il n'est pas lisible, ça va provoquer une erreur. Il est aussi possible de remonter une erreur si le fichier n'est pas lisible.

    Tu retournes l'aide en exécutant pratiquement vingt fois la commande echo. Dans bash, c'est une commande interne qui ne fait peut-être pas un fork, mais avec d'autres shell, cela peut faire appel à une commande externe. Pour cela, on préfère généralement faire un seul echo, ou un cat

    echo -e "Première ligne\\nSeconde ligne"

    echo "Première ligne
    Seconde ligne"

    ou encore

    cat << _VERSION_
    Première ligne
    Seconde ligne
    _VERSION_


    Si le wget retourne une erreur, tu ne testes rien, tu te contentes de regarder si quelque chose sort au bout de la chaine.
    L'adresse IP du routeur est codée en dur à 192.168.1.1
    wget utilise la version courte des paramêtres, rendant la relecture du script plus compliquée.
    Il y a aussi une dépendance sur logger, je ne sais pas s'il est présent sur toutes les distribs, sous les BSD...

    Voila, sinon c'est encourageant de voir qu'il y a encore des choses écrites proprement. Quand je vois ce pourquoi on paye au boulot...