Discussion :

[annap]

C'est ce qu'on réclame, à cor et à cri. Que veux-tu ?
Là, tu ne fais que empiler du code. C'est tout ce qu'il ne faut pas faire.

je veux que :
*dans : ./pasDeTresorIci.sh ligne colone carteTresor :
si [ligne][colone] (dans carteTresor) = X alors on exit 1 sinon on exit 0 #sans faire de tableau si possible car je n'ai pas encore vu comment les faire

*dans ./chasseAuTresor.sh ligne colone
tant que ./pasDeTresorIci.sh ligne colone (exit 0) alors on demande une nouvelle ligne et nouvelle colonne (en affichant) puis on read la reponse (les 2 reponses) puis on affecte les 2 valeurs qu'on a lu dans ligne et colone et la boucle est fini
on affiche "tresor trouvé!!" (quand on est sorti de la boucle car on à trouvé un X donc ca a exit 1 et pu 0)

je crois que j'ai réussi pasDeTresorIci.sh !!

[N_BaH]

pourquoi exit 1, alors que le caractère a été trouvé ? il n'y a pas d'erreur, alors il faut indiquer le succès, donc exit 0 !

tu as un script principal :./chasseAuTresor.sh
appelé ainsi : ./chasseAuTresor.sh ligne colonepourquoi indiquer ligne et colonne, alors que le programme est interactif : il demande jusqu'au succès de la "quête" les coordonnées où chercher ?

Code pseudo-code :

Sélectionner tout - Visualiser dans une fenêtre à part

1
2
3
4
5
6
7
8
 
tant que ./pasDeTresorIci.sh ligne colone (exit 0)
alors
   on demande une nouvelle ligne et nouvelle colonne (en affichant)
   puis on read la reponse (les 2 reponses)
   puis on affecte les 2 valeurs qu'on a lu dans ligne et colone
et la boucle est fini
on affiche "tresor trouvé!!" (quand on est sorti de la boucle car on à trouvé un X donc ca a exit 1 et pu 0)

je dirais non.

Code pseudo-code :

Sélectionner tout - Visualiser dans une fenêtre à part

1
2
3
4
5
6
7
tant que lire ligne colonne
   si vérifier $ligne $colonne dans $fichier # "vérifier" n'a pas à être un programme annexe, ça peut, ça _doit_ être une fonction du script principal.
   alors
      afficher succès
      quitter
   finsi
finTantQue

[annap]

pourquoi exit 1, alors que le caractère a été trouvé ? il n'y a pas d'erreur, alors il faut indiquer le succès, donc exit 0 ! [/Code]

je voulais exit 1 quand on trouve le trésor car c'est lui qui est "l'exception" et je voulais que se soit lui qui nous fasse sortir du while pour qu'on renvoie le message de fin vous avez trouvé le trésor!!

[Flodelarab]

je veux que :
*dans : ./pasDeTresorIci.sh ligne colone carteTresor :
si [ligne][colone] (dans carteTresor) = X alors on exit 1 sinon on exit 0 #sans faire de tableau si possible car je n'ai pas encore vu comment les faire

Tu es repartie comme en 40. Tu tripatouilles du code. Ce n'est pas la bonne méthode. Tu t'épuises à régler des détails.

2 points :

• Dans Windows, les exécutables renvoient 1 (vrai) quand ça marche, et 0 (faux) quand ça ne marche pas.
  Dans Linux, un exécutable renvoie 0 quand tout a fonctionné (il n'y a rien à ajouter), et un code différent de 0 en cas d'erreur (ce code erreur permet de savoir ce qui s'est mal passé).
  

  Code :

  Sélectionner tout - Visualiser dans une fenêtre à part

  1
  2
  3
  4
  $ ls Prout
  ls: impossible d'accéder à 'Prout': Aucun fichier ou dossier de ce type
  $ echo $?
  2

  Le code retour est 2 car ls n'a pas trouvé le fichier "Prout".
  .
• Il ne faut pas confondre le code retour et la sortie. Un exécutable écrit sur la sortie standard, sur la sortie d'erreur, et renvoie un code retour qui indique si le logiciel a fonctionné (cf plus haut).
  

  Code :

  Sélectionner tout - Visualiser dans une fenêtre à part

  1
  2
  3
  4
  5
  $ ls Prout afac_tresor.txt
  ls: impossible d'accéder à 'Prout': Aucun fichier ou dossier de ce type
  afac_tresor.txt
  $ echo $?
  2

  Dans cet exemple, le message d'erreur est la sortie d'erreur de ls, le nom du fichier est la sortie standard, et 2 est le code retour. Ici, on a les 3.

[annap]

Tu es repartie comme en 40. Tu tripatouilles du code. Ce n'est pas la bonne méthode. Tu t'épuises à régler des détails.

je ne peux pas faire à ma guise si cela m'arrange pour la suite ?
ceci marche bien :

Code :

Sélectionner tout - Visualiser dans une fenêtre à part

1
2
3
4
5
6
7
carac=$(head -n"$1" "$3" | tail -n1 | tr -d "," | cut -c"$2")
#head -n$ligne $3 | tail -n1 | tr -d "," | cut -c"$2"
if [ "$carac" == 'X' ]; then echo "bravo : "$carac""
        exit 1
else echo "dommage : "$carac""
        exit 0
fi

alors logiquement dans mon while ./pasDETresorIci.sh ligne colone fichier; do ... ca devrait marcher nan ?! puisque quand le code de sortie sera 1 ca devra écrire "tresor trouvé!" ?
comment différencier les 2 cas (pas de trésor ici ou un trésor ici) sinon ?

[annap]

j'ai réussi je crois !! Après je ne sais pas si c'est très bien en bash mais ca marche ne tout cas :

Code :

Sélectionner tout - Visualiser dans une fenêtre à part

1
2
3
4
5
6
7
8
9
10
11
12
ligne=$1
colone=$2
fichier=$3
while ./pasDeTresorIci.sh "$ligne" "$colone" "$fichier"; do
        echo "entrer un nvlle position (ligne colone) : "
        read nvligne nvcolone
                ligne=$nvligne
                colone=$nvcolone
export ligne
export colone
done
echo "vous avez trouver le  tresor !!"

il manque de tester si les coordonnées entrées sont bien sur la carte

[Flodelarab]

Si ta grille a un nombre variable de caractères par champ, ton script ne marchera pas.

Code :

Sélectionner tout - Visualiser dans une fenêtre à part

head -n"$1" "$3" | tail -n1 | cut -d ',' -f "$2"

"Colonne" prend 2 "n".

[jack-ft]

ceci marche bien :

Code :

Sélectionner tout - Visualiser dans une fenêtre à part

1
2
3
4
5
6
7
carac=$(head -n"$1" "$3" | tail -n1 | tr -d "," | cut -c"$2")
#head -n$ligne $3 | tail -n1 | tr -d "," | cut -c"$2"
if [ "$carac" == 'X' ]; then echo "bravo : "$carac""
        exit 1
else echo "dommage : "$carac""
        exit 0
fi

Bonjour, plutôt que de faire 2 scripts (le 2ème ayant très peu de chances d'être appelé en-dehors du contexte du 1er), je te propose de conserver le script principal et de remplacer le "sous-script" par une fonction:

Code :

Sélectionner tout - Visualiser dans une fenêtre à part

1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
17
18
19
20
21
22
23
24
ligne=$1
colonne=$2
fichier=$3
 
function pasDeTresorIci() {
    carac=$(head -n"$1" "$3" | tail -n1 | tr -d ',' | cut -c"$2")
    #head -n$ligne $3 | tail -n1 | tr -d ',' | cut -c"$2"
    if test "$carac" = 'X'
    then echo "Bravo : $carac"
         return 1
    else echo "Dommage : $carac"
         return 0
    fi
}
 
while pasDeTresorIci "$ligne" "$colonne" "$fichier"
do
    echo "Entrez une nouvelle position (ligne colonne) : "
    read nvligne nvcolonne
    ligne=$nvligne
    colonne=$nvcolonne
done
 
echo "Vous avez trouvé le trésor !!"

Grosso modo, j'ai juste remplacé les "exit" par des "return" (et corrigé quelques petites erreurs typographiques...)

Et, effectivement, il est "logique" que la fonction "pasDeTresorIci", dont le nom commence malencontreusement par "pas" (avec la sémantique volontairement inversée), retourne 0 lorsqu'il est vrai qu'il n'y a PAS de trésor aux coordonnées fournies.

Mais il serait encore plus logique àmha d'utiliser la fonction inverse "tresorPresent", qui retournerait 0 dans le cas où le trésor est présent et serait utilisé dans un while ! tresorPresent "$ligne" "$colonne" "$fichier" (sous réserve que le shell accepte le '!'):

Code :

Sélectionner tout - Visualiser dans une fenêtre à part

1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
17
18
19
20
21
22
23
24
ligne=$1
colonne=$2
fichier=$3
 
function tresorPresent() {
    carac=$(head -n"$1" "$3" | tail -n1 | tr -d ',' | cut -c"$2")
    #head -n$ligne $3 | tail -n1 | tr -d ',' | cut -c"$2"
    if test "$carac" = 'X'
    then echo "Bravo : $carac"
         return 0
    else echo "Dommage : $carac"
         return 1
    fi
}
 
while ! tresorPresent "$ligne" "$colonne" "$fichier"
do
    echo "Entrez une nouvelle position (ligne colonne) : "
    read nvligne nvcolonne
    ligne=$nvligne
    colonne=$nvcolonne
done
 
echo "Vous avez trouvé le trésor !!"

[N_BaH]

while ! false = until true