Erreur unexpected EOF

**annap** · 10/11/2020, 13h17

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

Code :

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

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!

**Flodelarab** · 10/11/2020, 13h42

Bonjour

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

**annap** · 10/11/2020, 13h54

merci beaucoup Flodelarab !

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 :

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

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

**N_BaH** · 10/11/2020, 16h21

Code :

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

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 :

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

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.

**annap** · 10/11/2020, 17h09

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 :

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

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 :

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

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 :

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

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

mon code actuel de pasDeTresorIci.sh :

Code :

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

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 :

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

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 :
Nom : Capture.PNG
Affichages : 88
Taille : 17,0 Ko

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

J'espère que tu as compris ce que j'ai voulu dire

Merci encore

**N_BaH** · 10/11/2020, 17h24

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 !

**Flodelarab** · 10/11/2020, 17h48

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

**annap** · 11/11/2020, 15h08

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

**annap** · 11/11/2020, 15h08

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 !

**disedorgue** · 11/11/2020, 15h34

Envoyé par annap

mon code actuel de chasseAuTresor.sh :

Code :

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

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 :

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

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 ???

**N_BaH** · 11/11/2020, 15h42

dans ./pasDeTresorIci, la boucle ne sert à rien

Code :

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

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.

**Flodelarab** · 11/11/2020, 16h03

Code :

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

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

Code :

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

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

**annap** · 11/11/2020, 16h09

Envoyé par N_BaH

dans ./pasDeTresorIci, la boucle ne sert à rien

Code :

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

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 :

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

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 :

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

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

**N_BaH** · 11/11/2020, 16h12

ç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.

**N_BaH** · 11/11/2020, 16h15

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 ? ...

**Flodelarab** · 11/11/2020, 16h25

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é.

**N_BaH** · 11/11/2020, 16h38

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.

**annap** · 11/11/2020, 16h59

Envoyé par N_BaH

dans ./pasDeTresorIci, la boucle ne sert à rien

Code :

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

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 :

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

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 :

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

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 :

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

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

**N_BaH** · 11/11/2020, 17h08

Code :

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

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 :

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

"$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".