Erreur unexpected EOF

Bonjour, je suis débutante en shell et bash. je cherchais à faire un script pasDeTresorIci.sh.
Voici mon code

Code:

---

1 2 3 4 5 6 7 8 9 10

#! /bin/bash   #Usage : ./pasDeTresorIci i j testTresor   head -n$1 $3 | tail -n1 | while IFS=',' read carac; do #je le fait afficher juste la ligne que je veux étudier carac=cut -c$2 #je recupere le carac qui m'interesse if [ carac == 'X' ];then exit 1 # je fais le test else exit 0 fi done

---

il doit vérifier a la ligne i si le caractère j est une crois X ou un O
testTresor est de la forme :
O,O,O,O
O,O,O,O
O,O,X,O
O,O,O,O
Je me suis servie de quelques forums sur le site pour faire mon code j'espère qu'il fait déjà à peu près ce que j'espère qu'il fasse ...

Pour l'instant je sais juste que lors de l'appel dans la console il me dit unexpected end of file ligne 10 soit la dernière ligne
Merci d'avance!

Bonjour :coucou:

Tu copies-colles ton code sur https://www.shellcheck.net/, et tu auras automatiquement toutes les remarques que j'allais te faire.

• Il manque la substitution de commande.
• Il manque les guillemets autour des variables
• On accède à la R-value d'une variable en mettant un dollar à gauche : $carac

merci beaucoup Flodelarab !:lol:

On doit toujours mettre des "" autour des variables ?
Substitution de commande ca veut dire qu'on met le résultat d'une commande dans une variable avec $ ?

De plus : dans la suite de l'exo je ceux creer un script chasseAuTresor.sh qui tant que pasDeTresorIci demande de rentrer une nouvelle position et refait le test.
peut on écrire :

voici mon code en entier qui ne semble pas marcher

Code:

---

1 2 3 4 5 6 7

nope=$(./pasDeTresorIci "$1" "$2" "$3") while "$nope"; do #sous entendur while nope = 0 puisque c'est l'éxécution normal , considère-t-on nope comme une variable ici ? echo "entrer un nvlle position (ligne colone) : " read -r nvI nvJ "$1"=$(nvI) "$2"=$(nvJ) done echo "vous avez trouver le tresor !!"

---

j'aurai bien aimé utiliser $? pour le while je sais pas si c'est possible ?
Doit on exporter le premier script pasDeTresorIci dans chasseAuTresor ?
Un parametre est il une variable ? donc entre ""?
un read lit il des varaibles ? donc aussi enrtre "" ?
Merci d'avance

Code:

---

1 2	nope=$(./pasDeTresorIci "$1" "$2" "$3") while "$nope"; do

---

nope est exécuté une seule fois, avant la boucle; sa valeur ne changera plus.

Code:

---

1 2 3 4

while ./pasDeTresorIci "$1" "$2" "$3" do : ... done

---

et, oui, il faut toujours mettre des guillemets autour des variables. TOUJOURS !
à moins de savoir pourquoi. :)

merci, aussi je ne sais pas si tu as vu le code de pasDeTresorIci.sh mais je crois que finalement il ne fait pas ce que j'aimerai qu'il fasse car j'affiche la ligne que je veux et je fait

Code:

---

1 2 3 4 5

head -n"$1" | tail -n1 | while IFS=',' read carac; do carac=$(cut -n"$2") #mais pour moi ca du coup c'est sencer lire le 2eme caractère mais sauf que le while IFS=',' read carac je crois que ca decoupe chaque 0 ou X de ma carte et ca le met dans carac car ils sont separé par une ",", peut être faudrait il fait plutot : head -n"$1" | tail -n1 | while IFS read carac #mais du coup il faudrai cut le k-ieme carac qui ne serait pas "$2" mais "$2"+"$2"-1 car il y a $2 carac + ce meme nombre -1 de virgule mais je sais pas comment faire j'avais essayer ceci : nbCarac="$2"+"$2"-1 carac=$(cut -c"$nbCarac")

---

j'ai aussi essayer en recupérant le j-ième champs separé par une virgule avec :

Code:

---

carac=$(cut -d, -f"$2")

---

mais ca ne marche pas non plus et je n'arrive pas a afficher dans echo mon carac aussi

je rappel que ma carte a cette forme
O,O,O,O
O,X,O,O

de plus dans chasseAuTresor.sh j'ai des erreurs au niveau du read pour les nouvelles valeurs nvI et nvJ je ne suis pas sur qu'on les récupère comme cela (il me dit qu'elle ne sont pas utilisées) :

Code:

---

1 2	read -r nvI nvJ "$1"=$("nvI") "$2"=$("nvJ") #sans les "" aussi ca ne marche pas

---

mon code actuel de pasDeTresorIci.sh :

Code:

---

1 2 3 4 5 6 7 8 9 10

#! /bin/bash   #Usage : ./pasDeTresorIci i j testTresor   head -n"$1" "$3" | tail -n1 | while read carac; do carac=$(cut -d, -f"$2") if [ "$carac" == 'X' ];then echo "1 car $carac" exit 1 else echo "0 car "$carac"" exit 0 fi done

---

mon code actuel de chasseAuTresor.sh :

Code:

---

1 2 3 4 5 6 7 8

while ./pasDeTresorIci.sh "$1" "$2" "$3"; do echo "entrer un nvlle position (ligne colone) : " read -r nvI nvJ "$1"="$nvI" "$2"="$nvJ" done done echo "vous avez trouver le tresor !!"

---

quand j'exécute j'obtient cela :
Pièce jointe 583573

l7 et 8 sont "$1"="$nvI" et "$2"="$nvJ"

J'espère que tu as compris ce que j'ai voulu dire :roll:
Merci encore

tu devrais construire un tableau associatif à partir de ton fichier.
en utilisant le numéro de ligne et l'emplacement sur la ligne en tant qu'index (séparés par une virgule, par exemple), ça te permettrait de retrouver directement la valeur de la cellule indiquée.

ce serait plus propre !

Quel est le but de ce script ?
Quel est le contexte ? C'est un travail pour l'école, dans lequel bash est obligatoire ?

Citation:

---

Envoyé par Flodelarab

Quel est le but de ce script ?
Quel est le contexte ? C'est un travail pour l'école, dans lequel bash est obligatoire ?

---

oui c'est un exercice d'école et on utilise effectivement que du bash

Citation:

---

Envoyé par N_BaH

tu devrais construire un tableau associatif à partir de ton fichier.
en utilisant le numéro de ligne et l'emplacement sur la ligne en tant qu'index (séparés par une virgule, par exemple), ça te permettrait de retrouver directement la valeur de la cellule indiquée.

ce serait plus propre !

---

je ne sais pas encore faire mais merci je regarderai !

Citation:

---

Envoyé par annap

mon code actuel de chasseAuTresor.sh :

Code:

---

1 2 3 4 5 6 7 8

while ./pasDeTresorIci.sh "$1" "$2" "$3"; do echo "entrer un nvlle position (ligne colone) : " read -r nvI nvJ "$1"="$nvI" "$2"="$nvJ" done done echo "vous avez trouver le tresor !!"

---

---

Tu confonds variables et arguments positionnels...
La bonne syntaxe pour valoriser des arguments positionnels serait plus du genre (dans ton cas) :

Code:

---

1 2 3 4 5 6 7

while ./pasDeTresorIci.sh "$1" "$2" "$3"; do echo "entrer un nvlle position (ligne colone) : " read -r nvI nvJ set "$nvI" "$nvJ" "$3" done done echo "vous avez trouver le tresor !!"

---

Le repositionnement de "$3" n'est pas facultatif, sinon set ne déclarera que 2 arguments et non 3.

PS: ton script n'est pas du pure bash puisqu'il utilise des commandes externes tel que head,tail,cut dont tu pourrais te passer si tu utilisais un tableau comme le disait si bien N_BaH.

PS2: A quoi sert le 2ème done dans ta boucle ???

dans ./pasDeTresorIci, la boucle ne sert à rien

Code:

---

head -$((nbLines-(nbLines-userLine))) "$fichierCarte" | tail -1 | tr -d ',' | cut -c $userCol

---

. et ça ne devrait pas être un script, mais une fonction (et encore, vu que ce nest qu'une ligne, est-ce vraiment nécessaire ?).

j'ai l'impression que c'est un peu le chantier.
tu écris le script directement, ou existe-t-il un algorithme ? parce qu'il n'apparaît pas dans les commentaires...
c'est indispensable : écrire un algorithme AVANT d'écrire la moindre ligne de code, ET afficher les commentaires (avant la ligne de code) qui explique la raison d'être du code.

en ce qui me concerne, en général, ça reprend simplement l'algorithme.

Code:

---

awk -F',' "NR==$userline{print $ $usercol;}" fichier.txt

---

Code:

---

1 2 3 4 5 6 7 8 9 10

$ cat <<eof >afac_tresor.txt > 1,2,3,4 > 5,6,7,8 > 9,a,b,c > d,e,f,0 > eof $ userline=2 $ usercol=3 $ awk -F',' "NR==$userline{print $ $usercol;}" afac_tresor.txt 7

---

Sans objectif, on peut en empiler des lignes de code inutiles...

Citation:

---

Envoyé par N_BaH

dans ./pasDeTresorIci, la boucle ne sert à rien

Code:

---

head -$((nbLines-(nbLines-userLine))) "$fichierCarte" | tail -1 | tr -d ',' | cut -c $userCol

---

. et ça ne devrait pas être un script, mais une fonction (et encore, vu que ce nest qu'une ligne, est-ce vraiment nécessaire ?).

j'ai l'impression que c'est un peu le chantier.
tu écris le script directement, ou existe-t-il un algorithme ? parce qu'il n'apparaît pas dans les commentaires...
c'est indispensable : écrire un algorithme AVANT d'écrire la moindre ligne de code, ET afficher les commentaires (avant la ligne de code) qui explique la raison d'être du code.

en ce qui me concerne, en général, ça reprend simplement l'algorithme.

---

il n'y a pas d'algorithme je n'en fait jamais on nous a jamais appris à en faire...

mais après la ligne avec les pipe j'aurait bien aimé garder le carac qu'il garde donc le :

Code:

---

cut -c"$2"

---

pour ensuite faire le test et renvoyer un echo mais je ne sais pas comment faire,
peut on mettre des variable et des substitutions de commande dans des pipes ? comme ceci :

Code:

---

1 2 3 4

head -n"$1" "$3" | tail -n1 | tr -d "," | carac=$(cut -c"$2") if [ "$carac" == X ];then echo "1 car $carac" exit 1 else echo "0 car "$carac"" exit 0 fi

---

merci encore pour vos réponses

ça n'a pas besoin d'être très compliqué, il faut surtout prévoir chaque opération, et d'envisager les optimisations avant de coder.

j'ai un peu de mal à préconiser awk (ou sed), quand il est demandé de réaliser un exercice en shell, parce qu'alors jusqu'où on va ? python ? perl ? PHP ? java ? un petit bout de C ? ...

J'entends bien. Mais comme le dit disedorgue, head tail, cut sont tous aussi externes. Comme l'objectif n'est pas fixé, c'est un peu la foire à neuneu. Tout le monde peut proposer ce qui lui passe par la tète. Et un fichier CSV fera souvent passer awk par la tête. La première posteuse fera son marché.

head, tail... sont en effet des commandes externes, mais ce ne sont pas de langages de programmation distincts.
Elles sont bien des outils du shell, et ne peuvent pas se suffire pas à elles-mêmes.

mais on pourrait aussi s'en passer, et parcourir le fichier (ligne par ligne et caractère par caractère) tout en shell. Tu sais que je suis prêt à le faire. Tu le sais, hein !? alors me cherche pas. :ptdr: ;)

Citation:

---

Envoyé par N_BaH

dans ./pasDeTresorIci, la boucle ne sert à rien

Code:

---

head -$((nbLines-(nbLines-userLine))) "$fichierCarte" | tail -1 | tr -d ',' | cut -c $userCol

---

. et ça ne devrait pas être un script, mais une fonction (et encore, vu que ce nest qu'une ligne, est-ce vraiment nécessaire ?).

j'ai l'impression que c'est un peu le chantier.
tu écris le script directement, ou existe-t-il un algorithme ? parce qu'il n'apparaît pas dans les commentaires...
c'est indispensable : écrire un algorithme AVANT d'écrire la moindre ligne de code, ET afficher les commentaires (avant la ligne de code) qui explique la raison d'être du code.

en ce qui me concerne, en général, ça reprend simplement l'algorithme.

---

et puis je crois que j'ai forcément besoin de faire le test après puisque dans chasseAuTresor.sh je fait

Code:

---

while ./pasDeTresorIci.sh "$1" "$2" "$3" #sous entendu que ca exit 0 ?!

---

ou alors je veux bien faire quelque chose avec un test if mais je sais pas si c'est possible de faire ca :

Code:

---

1 2 3 4 5 6 7

if [ $(./pasDeTresorIci.sh "$1" "$2" "$3") == 'X' ];then echo "entrer un nvlle position (ligne colone) : " read [ nvI nvJ ? ] #je suis en train de faire le cours sur le site et j'ai vu cette forme écrite c'est mieux je pense non ? "$1"="$nvI" "$2"="$nvJ" fi echo "vous avez trouver le tresor !!"

---

(dans chasseAuTresor.sh)

et ceci :

Code:

---

1 2 3 4

head -n"$1" "$3" | tail -n1 | tr -d "," | carac=$(cut -c"$2") #if [ "$carac" == 'X' ]; then echo "1 car $carac" exit 1 #else echo "0 car "$carac"" exit 0 #fi

---

ne marche pas ca me renvoie toujours 0 car exit 0

je sais aussi il me semble qu'on peut utiliser $? pour faire des tests ca prend le retour du dernier script exécuter je crois, est ce que ca pourrai nous aider peut être ?

Aussi, oui effectivement je ne pense pas qu'on va utiliser (dans mes cours) tout ce que vous mettez, par exemple les awk mais c'est bien pour moi et pour les autres je pense ça permet de voir différentes façons de faire ! même si déjaàmaîtriser toute les base pour arriver à faire ce que je veux est assez compliqué comme ca je trouve:?

8O

Code:

---

read [ nvI nvJ ? ]

---

tu peux nous indiquer la page où tu as vu ça, s'il te plaît ?
autant que je sache, ça n'existe pas en shell.

Code:

---

"$1"="$nvI"

---

ça, c'est une mauvaise pratique. et je ne suis pas certain que ta aies compris ce qu ça signifie.

àmha, tu devrais arrêter de coder, et rédiger un algorithme.

et, en tant que modérateur, je ne vais pas tarder à clore la discussion, parce qu'on dépassé le cadre du sujet (Erreur unexpected EOF), et on est passé à "aidez-moi à faire mon exercice en entier".

Citation:

---

Envoyé par annap

faire ce que je veux

---

:furieux: 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.

j'essaye de comprendre un petit exercice et vos réponses m'aident beaucoup. Même si je suis toujours bloquée..., j'ai appris des choses ! Je veux bien créer une autre page de forum alors?! Merci encore en tout cas

:lahola: :lol:

Dans la documentation des commandes, ici comme ailleurs, les crochets indiquent le caractère optionnel. Les variables qui suivent la commande read ne sont pas obligatoires.

Les commandes suivantes sont toutes valides :

Code:

---

read

---

Code:

---

read var1

---

Code:

---

read var1 var2

---

Code:

---

read var1 var2 var3

---

Code:

---

read var1 var2 var3 var4

---

Code:

---

read var1 var2 var3 var4 var5

---

Code:

---

read var1 var2 var3 var4 var5 var6

---

Il ne faut pas écrire les crochets.

haaaa, ok je ne savais pas, merci

il s'agit de la description de la syntaxe, pas de la syntaxe telle qu'elle s'applique dans un script.
cela signifie que read accepte un nombre variable de noms de variables.
elles seront assignées en fonction de l'IFS tant qu'il y a des données :

Code:

---

1 2 3 4 5 6

$ read var1 var2 var3 a b c d $ for i in "$var1" "$var2" "$var3"; do echo "$i"; done aa bb cc dd

---

comme tu le vois, s'il n'y a pas assez de variables pour les données, la dernière variable prend tout le reste.