Ce site est optimisé pour être consulté depuis un navigateur moderne dans lequel JavaScript est activé.

Script Bash

Tomy-Gunn

C'est vrai que c'est plus lisible, merci.
Par contre toujours le problème avec les mois de 01 à 09.
Je comprends pas pourquoi il me met "données incorrectes" et "((: 08 : valeur trop grande pour la base (le symbole erroné est "08"))"
vu que le nombre est bien compris entre 01 et 12. Et que ça marche avec 10,11,12.

pingouinux

Tomy-Gunn #21 a écritJe comprends pas pourquoi il me met "données incorrectes" et "((: 08 : valeur trop grande pour la base (le symbole erroné est "08"))"

Quand une chaîne débutant par un zéro est traitée comme un entier, bash considère que c'est de l'octal, et 8 n'est pas un chiffre octal.

Édité : Tu peux préciser la base (ici cest 10)

$ echo $((08))
bash: 08: value too great for base (error token is "08")

$ echo $((10#08))
8

Watael

on convertit en base 10 :

mois=08
mois=$((10#$mois))

edit: grilled! 🙂

Tomy-Gunn

Merci pour l'info. J'ai préféré modifier la regex pour qu'elle prenne en compte qu'un chiffre (restons simple pour le moment...)
Encore une question ! est il possible d’arrêter le script à la fin du test "if" (ligne 37) si les données ne sont pas correctes ? (avant le i=$age)
Sinon ça me met : Données incorrectes.
Vous avez ans, vous êtes jeune.

#!/bin/bash

#VARIABLE DATE DU JOUR

read jourActu moisActu anneeActu < <(date +'%d %m %Y')

#INSERTION DATE DE NAISSANCE

read -p 'Entrez votre jour de naissance :' jourNaissance
read -p 'Entrez votre mois de naissance :' moisNaissance
read -p 'Entrez votre année de naissance :' anneeNaissance 

# TEST VARIABLES

if [[ $jourNaissance =~ ^[0-9]{1,2}$ ]] \
 && (($jourNaissance >=1 && $jourNaissance <=31)) \
 && [[ $moisNaissance =~ ^[0-9]{1,2}$ ]] \
 && (($moisNaissance >=1 && $moisNaissance <=12)) \
 && [[ $anneeNaissance =~ ^[0-9]{4}$ ]] \
 && (($anneeNaissance > 1900 && $anneeNaissance < $anneeActu)) \
 && (($moisActu < $moisNaissance))
then 
     age=$((anneeActu-anneeNaissance-1))
else
if [[ $jourNaissance =~ ^[0-9]{1,2}$ ]] \
 && (($jourNaissance >=1 && $jourNaissance <=31)) \
 && [[ $moisNaissance =~ ^[0-9]{1,2}$ ]] \
 && (($moisNaissance >=1 && $moisNaissance <=12)) \
 && [[ $anneeNaissance =~ ^[0-9]{4}$ ]] \
 && (($anneeNaissance > 1900 && $anneeNaissance < $anneeActu)) \
 && (($moisActu > $moisNaissance))
then
     age=$((anneeActu-anneeNaissance))
else
     echo "Données incorrectes">&2
fi
fi

i=$age
if ((i<=30)); then
   echo "Vous avez $age ans, vous êtes jeune."
elif ((i>30 && i<60)); then
   echo "Vous avez $age ans."
elif ((i>=60 && i<=100)); then
   echo "Vous avez $age ans, vous êtes vieux."
elif ((i>100 && i<120)); then
   echo "Vous avez plus de 100 ans, incroyable !"
elif ((i>120)); then
   echo "Vous ne pouvez pas avoir plus de 120 ans."
else
   echo "Données incorrectes"
fi

nany

nany a écrit J’ajouterais aussi un exit après echo "Données incorrectes".

Tomy-Gunn

Autant pour moi...
Merci à tous pour votre aide et vos conseils sans lesquels je serais encore bien loin du résultat...

pingouinux

Pour info, si on veut juste vérifier qu'une date est correcte :

$ date -d 2019-02-30 &>/dev/null && echo Bonne date || echo Mauvaise date
Mauvaise date

$ date -d 2019-02-25 &>/dev/null && echo Bonne date || echo Mauvaise date
Bonne date

sputnick

Ton script, un peu corrigé:

- pas besoin de $ sur les variables en contexte arithmétique ((i<=30)) comme dans le dernier if
- prends l'habitude de formater/indenter ton code proprement, ça t'aidera toi et aussi ceux qui doivent te lire
- si le script sor et erreur, il affiche sur la sortie d'erreur STDERR avec >&2 et une sortie en erreur: exit 1. On peut tester la valeur de retour des commandes avec 'false; echo $?'

#!/bin/bash

#VARIABLE DATE DU JOUR

read jourActu moisActu anneeActu < <(date +'%d %m %Y')

#INSERTION DATE DE NAISSANCE

read -p 'Entrez votre jour de naissance :' jourNaissance
read -p 'Entrez votre mois de naissance :' moisNaissance
read -p 'Entrez votre année de naissance :' anneeNaissance 

# TEST VARIABLES

if [[ $jourNaissance =~ ^[0-9]{1,2}$ ]] \
    && ((jourNaissance >=1 && jourNaissance <=31)) \
    && [[ $moisNaissance =~ ^[0-9]{1,2}$ ]] \
    && ((moisNaissance >=1 && moisNaissance <=12)) \
    && [[ $anneeNaissance =~ ^[0-9]{4}$ ]] \
    && ((anneeNaissance > 1900 && anneeNaissance < anneeActu)) \
    && ((moisActu < moisNaissance))
then 
    age=$((anneeActu-anneeNaissance-1))
else
    if [[ $jourNaissance =~ ^[0-9]{1,2} ]] \
        && ((jourNaissance >=1 && $jourNaissance <=31)) \
        && [[ $moisNaissance =~ ^[0-9]{1,2}$ ]] \
        && ((moisNaissance >=1 && moisNaissance <=12)) \
        && [[ $anneeNaissance =~ ^[0-9]{4}$ ]] \
        && ((anneeNaissance > 1900 && anneeNaissance < anneeActu)) \
        && ((moisActu > moisNaissance))
        then
            age=$((anneeActu-anneeNaissance))
        else
            echo "Données incorrectes">&2
    fi
fi

i=$age
if ((i<=30)); then
    echo "Vous avez $age ans, vous êtes jeune."
elif ((i>30 && i<60)); then
    echo "Vous avez $age ans."
elif ((i>=60 && i<=100)); then
    echo "Vous avez $age ans, vous êtes vieux."
elif ((i>100 && i<120)); then
    echo "Vous avez plus de 100 ans, incroyable !"
elif ((i>120)); then
    echo "Vous ne pouvez pas avoir plus de 120 ans."
else
    echo "Données incorrectes" >&2
    exit 1
fi

Watael

- vu qu'une partie du test est la même, on peut en faire une fonction.
- i=age n'est pas utile.

j'aurais tendance à ne pas aller plus loin tant qu'une donnée n'est pas entrée correctement :

until conditionsJourNaissance #une fonction pour faire propre*
do
   read -r 'Entrez le jour de naissance (ou Ctrl-C pour arrêter le script) : ' jourNaissance
done

pour l'affichage aussi, plutôt une fonction acceptant un paramètre, et qui quitte le script :

((age<=30)) && affichage jeune
((30 < age && age <60)) && affichage age
#...

il y aurait un truc horrible (mais que j'adore), c'est l'opérateur ternaire :

(( condition : vrai ? faux ))

on peut les enchaîner dans "une seule" évaluation arithmétique :

mentions=( jeune age vieux ...)
((age<=30 : mention=0 ?
    30<age && age<60 : mention=1 ?
        ...))
affichage "${mentions[mention]}"

*soit plusieurs fonctions (une par saisie), soit une seule acceptant un paramètre pour indiquer quelle partie exécuter

marcus68

bonjour,

@Sputnik je me permets de commenter votre code (en plus des remarques pertinentes de Watael),

#!/bin/bash

#VARIABLE DATE DU JOUR

read jourActu moisActu anneeActu < <(date +'%d %m %Y')

#INSERTION DATE DE NAISSANCE

read -p 'Entrez votre jour de naissance :' jourNaissance
read -p 'Entrez votre mois de naissance :' moisNaissance
read -p 'Entrez votre année de naissance :' anneeNaissance 

# TEST VARIABLES

if [[ $jourNaissance =~ ^[0-9]{1,2}$ ]] \
    && ((jourNaissance >=1 && jourNaissance <=31)) \
    && [[ $moisNaissance =~ ^[0-9]{1,2}$ ]] \
    && ((moisNaissance >=1 && moisNaissance <=12)) \
    && [[ $anneeNaissance =~ ^[0-9]{4}$ ]] \
    && ((anneeNaissance > 1900 && anneeNaissance < anneeActu)) \
    && ((moisActu < moisNaissance))
then 
    age=$((anneeActu-anneeNaissance-1))
else
    if [[ $jourNaissance =~ ^[0-9]{1,2} ]] \
        && ((jourNaissance >=1 && $jourNaissance <=31)) \
        && [[ $moisNaissance =~ ^[0-9]{1,2}$ ]] \
        && ((moisNaissance >=1 && moisNaissance <=12)) \
        && [[ $anneeNaissance =~ ^[0-9]{4}$ ]] \
        && ((anneeNaissance > 1900 && anneeNaissance < anneeActu)) \
        && ((moisActu > moisNaissance))
        then
            age=$((anneeActu-anneeNaissance))
        else
            echo "Données incorrectes">&2
    fi
fi

i=$age
if ((i<=30)); then
    echo "Vous avez $age ans, vous êtes jeune."
elif ((i>30 && i<60)); then
    echo "Vous avez $age ans."
elif ((i>=60 && i<=100)); then
    echo "Vous avez $age ans, vous êtes vieux."
elif ((i>100 && i<120)); then
    echo "Vous avez plus de 100 ans, incroyable !"
elif ((i>120)); then
    echo "Vous ne pouvez pas avoir plus de 120 ans."
else
    echo "Données incorrectes" >&2
    exit 1
fi

- cela ne tient pas compte de la longueur des mois, il n'y a pas 31 jours dans tout les mois. (on peut renseigner le 31 février par exemple)
- cela ne marche pas si la personne est née durant le début de l'année en cours (cela produit une erreur)
- si la personne a moins qu'un an, il faut peut-être adapter l'affichage
- la condition ((i>120)) est impossible à réaliser, la date la plus petite renseignable est 1 janvier 1901
- il manque un "exit 1" après le premier "Données incorrectes", sinon il va tenter de faire un calcul sur des données incorrects
- à quoi peut bien servir le deuxième "Données incorrectes" ?

Cordialement

LeoMajor

bonjour,

oui bien vu. année bissextile aussi.
il y a un problème de sémantique sur la date de naissance .
Vous ne vérifiez pas que la date est vraiment une date ( +1 pingouinux #27)

MicP

Bonjour

Pour pouvoir contrôler la saisie d'une date,
il faudrait faire d'abord entrer l'année, ce qui permettrait de savoir si elle est bissextile ou pas,
puis le mois, et enfin le jour du mois qui serait alors limité au nombre de jours possibles en fonction du mois et de l'année.

Les calculs permettant de déterminer l'âge et tester les conditions
pourraient se faire en utilisant de simples comparaisons et soustractions directement avec la commande date

« Page précédente