erreur en assemblage

Question

bonjour,j'ai écrit un code source et en assemblant j'ai eu une erreur dont j'ignore la façon de corriger qui est la suivante:
 unexpected end of file encoutered 
si quelqu'un peut me donner les raisons de son apparition et les manières de la corriger...
merci

cs_ghuysmans99 · Answer

On dirait que tu utilises TASM. As-tu mis un end à la fin du fichier ?

VB.NET is good ... VB6 is better
Utilise Réponse acceptée quand un post répond à ta question

marooh · Answer

oui bien sur j'ai o debut main proc et à la fin main endp et main end

cs_ghuysmans99 · Answer

Tu peux nous poster le code source complet ?

VB.NET is good ... VB6 is better
Utilise Réponse acceptée quand un post répond à ta question

marooh · Answer

.model small
.stack 100h
.data
commande db 10,13,"saisir une phrase$"
msg1 db 10,13,"la chaine set vide$"
msg2 db 10,13,"la taille de la chaine saisie est$"
msg5 db 10,13, "le nombre de separateur est:$"
msg4 db 10,13, "la longueur du  mot le plus long est:$"
msg6 db 10,13, "le mot le plus long dans votre chaine est$"
taille dw 0
chaine dw ?
.code
saisir_chaine_et_taille macro chaine,taille,msg2,msg1
local repeter,vide
mov si,0 
repeter: 
inc si 
inc taille
mov ah,1 
int 21h 
mov chaine[si],al 

cmp chaine[si],"#" 
je vide
jne repeter
mov ah,9 
mov dx,offset msg 2
int 21h
mov dx,taille
mov ah,2
int 21h
mov ah,4ch 
mov al,0 
int 21h 
vide:
mov ah,9
mov dx,offset msg1
int 21h
endm

nombre_mot macro chaine,nbm
local boucle_pour_chaque_caractere,mot_trouve
 mov nbm,0
   mov bx, OFFSET chaine 
boucle_pour_chaque_caractere: 
    mov al, BYTE PTR [bx] 
    cmp al,20h
    je mot_trouve
    inc bx               
    cmp al, '$'           
    jnz boucle_pour_chaque_caractere
mov ax,nbm
mov ah,9
int21h
mot_trouve:
inc nbm
endm

longueur macro chaine,msg6,t_max
local calcul_longueur_mot,oui_separateur,re_initialisation,incrementation, stockage
mov ah,9
mov dx,offset msg4
int 21h                

mov t_max,0
mov cx,n
mov si,0
mov p,0
calcul_longueur_mot:
test_si_separateur:
cmp chaine[si],' '
je oui_separateur
cmp chaine[si],';'
je oui_separateur
cmp chaine[si],','
je oui_separateur
cmp chaine[si],'.'
je oui_separateur
cmp chiane[si],':'
je oui_separateur
cmp chaine[si],'-'
je oui_separateur
jne non_separateur

oui_separateur:
mov bx,p
cmp bx,t_max
ja stockage
jbe re_initialisation
stockage:
mov bx,p
mov t_max,ax
mov indice_mot,si
mov p,0
mov ah,9
mov dx,offset msg6
int 21h                   
sub indice_mot,t_max
mov cx,t_max
mov ah,2
mov si,indice_mot
le_mot_plus_long:
mov dl,chaine[si]
int 21h
inc si
loop le_mot_plus_long
jmp incrementation
re_initialisation:
mov p,0
incrementation:
inc si
loop calcul_longueur_mot

nombre_separateur macro  chaine,taille,sep,msg5, 
local repete13,condition13,incrementation 
mov si,offset chaine
mov sep,0 
mov cx,taille
repete13: 
mov al,byte ptr[si] 
cmp al,';' 
je separateur
cmp al,' ' 
je separateur
cmp al,',' 
je separateur
cmp al,'.' 
je separateur
jmp incrementation 
separateur: 
inc sep
incrementation:
inc si 
loop repete13 
mov dx,offset msg5 
affiche_msg 
mov ax,sep 
affiche_entier 
mov dx,offset entre 
affiche_msg 
endm
main proc 
mov ax,@data
mov ds, ax
saisir_chaine_et_taille chaine,taille,msg2,msg1
nombre_mot chaine,nbm
longueur  chaine,msg6,t_max
nombre_separateur chaine,sep,msg5,taille
main endp
end main

BruNews · Answer

Peut-etre une derniere ligne avec seulement:
END

ciao...
BruNews, MVP VC++

cs_patatalo · Answer

salut,


oui ou ce genre de truc collé dans les macros: affiche_msg

l'erreur étant dans la macro, tasm est incapable de donner un numéro de ligne. Une macro n'est pas une fonction !!!


@++

marooh · Answer

j'ai enlever affiche_msg et je l'ai remplacé par son equivalent mais rien n'ai changé

cs_patatalo · Answer

re,

(tutorial) tu peux essayer aussi "start PROC" au lieu de main PROC


@++

marooh · Answer

euuuh j'ai essayé le truc de start et end start mais le resultat est le meme.ça va me rendre fouuuuuuu 
ya-t-il pas un probleme avec les macros??
personellement,j'ai verifié plusieurs fois et je ne trouve pas d'erreurs

cs_patatalo · Answer

re,

main semble ok, ici un gentil qui met plein de commentaires


Attention, tes macros sont définies dans le segment de code. Je ne sais pas si tasm est d'accord. En général, on met ça dans des fichiers d'entête car ce sont des déclarations.

@++

marooh · Answer

j'ai lu des doc concernant les macros et ils disent qu'il faut ecrire les macros au debut du code puis les appelées en temps voulu

marooh · Answer

vous avez essayé de l'assembler avec votre assembleur???

cs_patatalo · Answer

re,

de toute manière, ton code ne s'assemblera avec rien étant donné que tu utilise "affiche_msg" mais que nul part il est déclaré.

Je te conseille de le faire sans macros et ensuite tu placeras les différentes parties souhaitées dans des macros.

@++

marooh · Answer

re,
affiche_msg je l'ai enlevé just aprés avoir envoyé le code mais ce n'est pas elle le probléme.
en tt cas je vais essayé sans macro

ToutEnMasm · Answer

Peut être qu'il manque un ret dans le code.
Il me semble que tasm fait appel a la crt dans ce genre d'écriture(point d'entrée dans lib).


main proc 
mov ax,@data
mov ds, ax
saisir_chaine_et_taille chaine,taille,msg2,msg1
nombre_mot chaine,nbm
longueur  chaine,msg6,t_max
nombre_separateur chaine,sep,msg5,taille
RET          ;<<<<<<<<<
main endp

marooh · Answer

il 'y a un probleme avec le code qui affiche le mot le plus long dans une chaine de caractère.si vous pouvez me corriger

cs_patatalo · Answer

chaine dw ?

donc, chaine est un mot soit 2 octets.
chaine[si] sera en débordement avec si > 1 et peut-être également avec si = 1 si chaine[si] comprends que chaine est un type WORD.

Pour finir, je ne corrigerais aucun code qui n'est pas commenté.

@++

Guilou34 · Answer

Le problème vient du fait d'une mauvaise utilisation des macros. Ainsi le mot "taille" est à la fois le nom d'un variable, d'un argument formel et d'un argument d'appel d'une macro. 
L'assembleur est incapable de gérer ce type d'erreur complètement imprévisible par ceux qui l'ont écrit.
Plus grave encore : chacune des macros n'est jamais appelée qu'une seule fois.
Quand tu sauras à quoi elles servent, ton problème sera résolu.
Il y a aussi beaucoup de maladresses qui témoignent  d'une mauvaise connaissance des
processeurs Intel: l'instruction Stosb, par exemple, remplace avantageusement les deux  instructions Mov [si],al Inc si  etc.
Mais que dire de ces lignes de code :
...
 je oui_separateur
 jne non_separateur
oui_separateur:
...
La première et la troisième ligne ne servent strictement à rien.
Bon courage quand même.

BruNews · Answer

stosb comme tant d'autres instructions sont obsolètes. Elles ne sont plus présentes que pour preserver la compatibilité des vieux codes.

ciao...
BruNews, MVP VC++

marooh · Answer

re,
une remarque: je suis debutant en ce genre de langage donc j'essais de faire ce que je peux et c'est la raison pour laquelle je demande de l'aide.
En tout cas j'ai ecrit un nouveau code pour la fonction qui cherche le mot le plus long et je demande des rectifications:
je n'ai pas su comment faire pour afficher le mot le plus long

mov indice_mot,0 \indice du mot le plus long
mov bx,chaine[si] \l'indice du mot courant
mov t_max,0    \ longueur du mot le plus long 
mov cx,0     \ la longueur du mot courant
mov si,offset chaine
scrute:
cmp byte ptr[ds:si],35  \comparaison avec # qui marque la fin de la chaine
jz findescrutation
cmp byte ptr[ds:si],20h \comp avec les espaces
je separateur
inc si 
inc cx
separateur:
cmp t_max,cx
jge stockage
stockage:
mov t_max,cx \ stckage de la nouvelle longueur
mov indice_mot,bx \stockage du nouveau indice
mov bx,chaine[si+2]
mov cx,0
jmp scrute
findescrutation: \traitement du dernier mot et affichage
mov bx,chaine[si+2] 
cmp t_max,cx
jge stock
mov ah,9
mov dx,offset msg3
int 21h
mov cx,t_max
mov ah,2
int 21h
mov ah,9
mov dx,offset msg6
int 21h
mov  dx,bx
mov ah,2
int 21h

cs_patatalo · Answer

chaine est un pointeur vers une chaine ou un tableau de caractères ? Ca ne se déclare pas de la même manière.

mov bx,chaine[si+2] ; sert à quoi dans ton code ?

@++

marooh · Answer

Re,
la chaine est saisie au debut par l'utilisateur,je veux la parcourir.
Mov bx,chaine[si+2]: en fait j'avais l'intention de choisir bx comme etant le registre qui donne l'indice du mot courant mais je n'ai pas su vraiment comment faire

ToutEnMasm · Answer

un truc du style
mov esi,offset chaine ou
lea esi,chaine
puis
mov bx,0
scrute:
cmp byte ptr [ds:si+bx],"$"
jz fini
inc bx
jmp scrute
fini:
;en plus on doit avoir la longueur de la chaine
;dans bx,peut être à 1 près (vérifier)

cs_patatalo · Answer

re,


trouves toi un debugger 16 bits (il doit en exister des gratuits style turbo debugger), il faut que tu arrives a t'imager ce que donnent les données et les instructions qui y sont associées dans la memoire.

@++

cs_patatalo · Answer

au fait,

pour ton problème de compilation, tu ne serais pas sous linux par hasard ?

marooh · Answer

re,
@ toutEnAsm : j'ai pensé à cette solution aussi mais le probleme c'est l'affichage du mot pointé par bx 
@ patatalo: j'utilise emu8086,j'ai réussi à compiler mais l'exécution ne donne rien,juste une fenetre noire (j'ai essayé d'executer cette fonction seulement pour voir si elle fonctionne)et non je ne suis pas sous linux,je suis sous windows xp

cs_ghuysmans99 · Answer

Pour débugger, utilise Insight(open-source)

VB.NET is good ... VB6 is better
Utilise Réponse acceptée quand un post répond à ta question

Guilou34 · Answer

Quand on lit dans ton code:
...
cmp t_max,cx
jge stockage
stockage:
...
 
on se pose la question: que se passe-t-il si t_max est plus petit que cx? 

En réalité, il est très difficile de trouver une erreur d'inattention dans la lecture d'une source.
Il faut en tracer l'exécution pas à pas à l'aide de Debug, par exemple et tout devient évident.
Amicalement.

marooh · Answer

j'ai changé le registre bx par dl parce qu'il y avait un probleme de bits mais  rien 
une petite question en plus : est ce que l'instruction cmp al,35 qui permet de comparer al avec # est correcte? je pose cette question car en essayant de faire une saisie qui se termine par # j'ai mis cette instruction et ça ne marche pas 
et cmp al,20h est elle correcte pour comparer avec les separateurs???

ToutEnMasm · Answer

Pour tester un caractere
cmp al," "  ;et on a un espace
cmp al,"$"
et ainsi de suite

BruNews · Answer

cmp al, '$'

ciao...
BruNews, MVP VC++

marooh · Answer

Re
donc pour comparer avec # je fais cmp al,"$" ??
mais mon probleme n'est pas encore resolu celui de trouver le mot le plus long et de l'afficher,j'ai essaye avec un debugguer mais je n'ai pas trouver la faille,je passe des heures à essayer de nouveaux trucs mais en vain ... :(((

cs_patatalo · Answer

re,

non, tu compares bien avec cmp al,'#'. C'est que Toutenmasm pense que tu veux chercher une fin de phrase pour D.O.S.


il est normal que tu ais du mal, l'assembleur n'est pas un code que l'on apprends en baclant vite fait. Reprends cela a 0, commence par un hello world simple ou tu pourras verifier le bon fonctionnement de l'emulateur et te redonneras de l'espoir avec l'affichage d'un truc.

ensuite, avance pas à pas en comprenant bien chaque ligne de code que tu ajoutes. Commentes ton code réellement. Teste avec un debugger des petits bout de code s'il le faut.

poste tes problemes 1 à 1 en donnant bien le code complet à chaque fois. On ne risque pas de gaspiller du papier ici ;-)

@++

ToutEnMasm · Answer

Pour arriver quelque part,il faut savoir ou on va non ?
Les fins de phrases peuvent être 0 (valeur décimale) ou 13 ou "$" (ascii caractère) peut être dans ton cas.
Question:Quel est le caractere de terminaison de ligne ?. Un debugger peut aider içi.

marooh · Answer

re,
je suis bien conscient que l'assembleur ne s'apprend pas en quelques secondes mais je vous assure que je fais ce que je peux pour apprendre,déja à la fac on a étudié l'assembleur en 3h seulement et on nous demande de faire des tps compliqués,ils veulent qu'on apprend tout seul donc j'essaie de faire le max et c'est pourquoi je demande votre aide.
j'ai essayé  cmp al,"#" auparavant pour la saisie d'une phrase qui se termine par # mais ça ne marche pas,la saisie continue meme en tapant #
voici le code

.model small
.stack 100h
.data
msg db 10,13,"saisir une phrase :  $"
msg1 db 10,13,"la chaine saisie est : $"
msg2 db 10,13,"la taille de la chaine saisie est : $"
chaine db ?
taille dw ?
.code
start:
mov ax,@data
mov ds, ax
mov ah,9
mov dx,offset msg \affichage du 1er msg
int 21h
mov taille,0 \initialisation de la taille

lea si,chaine
comparaison:
mov ah,1     \saisie de la chaine
int 21h 
mov ah,[si]
cmp ah,'$'
jz fin
cmp ah,"#"  \COMPARAISON AVEC # QUI MARQUE LA FIN DE LA CHAINE
je yess
inc taille
inc si
jmp comparaison
yess:
mov ah,09h
mov dx,offset msg1
int 21h
mov ah,09h
mov dx, offset chaine
int 21h
mov ah,09h
mov dx,offset msg2
int 21h
mov bx,taille
mov ah,01h
int 21h
fin:
mov ah,4ch
int 21h
end start

ToutEnMasm · Answer

Je peux me tromper mais il me semble que la fonction 1 de l'interruption 21h retourne le caractere lu par ax (ah,al..).
Après il faut mettre al dans le tampon si on veut le lire dans le tampon.
Sinon,on evite de modifier al par:
"mov al,[si]"  et à la place:
cmp al,"#"
jz fin
mov [si],al
inc si
jmp comparaison
fin:
......

marooh · Answer

re,
merci pour cette remarque,le probleme reside maintenant dans l'affichage de la taille de la chaine saisie,il m'affiche une valeur fausse
voici le code aprés modification
.model small
.stack 200h
.data
msg db 10,13,"saisir une phrase :  $"
msg2 db 10,13,"la taille de la chaine saisie est : $"
chaine db ?
taille dw ?
.code
start:
mov ax,@data
mov ds, ax
mov ah,9
mov dx,offset msg
int 21h
mov taille,0

lea si,chaine
comparaison:
mov ah,1
int 21h 
cmp al,"0"
jz yess
mov [si],al
inc si
inc taille
jmp comparaison
yess:
mov ah,9
mov dx,offset msg2
int 21h
mov ah,2
mov bx,taille
int 21h
mov ah,4ch
int 21h
end start

cs_patatalo · Answer

chaine db ? n'est toujours pas bon. Tu dois déclarer un tableau. Je ne sais pas comment en tasm, regarde la doc.

masm:
#define TAILLE_CHAINE 128
chaine db TAILLE_CHAINE du(?)

nasm:
%define TAILLE_CHAINE 128
chaine resb TAILLE_CHAINE
ou
times TAILLE_CHAINE db 0

pour que tu arrête la saisie avec '#', tu dois tester caractère par caractère. fonction dos avec taille = 1 ou fonction bios (16h).

cs_patatalo · Answer

pardon,

masm: 
#define TAILLE_CHAINE 128 
chaine db TAILLE_CHAINE dup(?); ou dup(0)


@++

cs_patatalo · Answer

ça fait longtemps:

masm:
TAILLE_CHAINE EQU 128
...

marooh · Answer

re,
la saisie jusqu'à # tapé fonctionne mais comme j'ai dit le probleme reside maintenant dans l'affichage de la taille de cette chaine,il ya un  truc qui cloche

cs_patatalo · Answer

Tu parles avec la machine alors, il faut apprendre à la connaître un peu. C'est passionnant mais pas très utilisé dans l'industrie. D'ou peut-être le peu de temps passé dessus.

Soit tu regarde des docs mais ça va prendre du temps soit je comprends pourquoi on nous demande toujours les codes tout faits. La doc intel386 est toujours d'actualité pour connaître le processeur.

cs_patatalo · Answer

TU ECRASES TOUT CE QUI EST APRES chaine DANS TA SAISIE.

marooh · Answer

re,
comme j'ai dit auparavant je fais mon maximum pour apprendre ce langage,mais la pratique a un grand role c'est pour cela que je pose des questions à des specialistes en la matiere que vous etes pour apprendre de mes erreurs et me corriger

cs_patatalo · Answer

ça fait 3 ou 4 fois que je te le dis:

déjà, ça serait mieux:
chaine db 0
msg db 10,13,"saisir une phrase :  $"
msg2 db 10,13,"la taille de la chaine saisie est : $"
taille dw ?

ça c'est comme d'hab:
chaine db 0
msg db 10,13,"saisir une phrase :  $"
msg2 db 10,13,"la taille de la chaine saisie est : $"
chaine db TAILLE_CHAINE dup('$')
taille dw ?

et tu n'as pas a ajouter le caractère final '$' puisque ça en est rempli.

marooh · Answer

re,j'ai declaré la chaine comme étant un tableau comme suit:
chaine db 1024 dup("$")

cs_patatalo · Answer

voilà la tronche de la sequence que tu veux sortir:

chaine = "m",?,0,"s que ce passe-t-il ?",13,10,"$"

cs_patatalo · Answer

voilà comment j'envisage le travail de macro a fournir après:

tu crée une structure string qui comprend:
1 byte taille_courant = 0;
1 byte taille_max = TAILLE_CHAINE_MAX
1 tableau de byte data[TAILLE_CHAINE_MAX] = que des '$'

ensuite, tu crée les macros:

str_stdin_char pstring
str_add_char pstring, char
str_add_crlf pstring
str_affiche pstring
str_size pstring
str_is_full pstring
...

marooh · Answer

re,
j'ai ajouté au code quelques procedures pour convertir du binaire vers une chaine,mais ça n'affiche pas,pourtant cette procedure je l'ai deja utilisé et elle fonctionne correctement
voici le nouveau code
.model small
.stack 200h
.data
msg db 10,13,"saisir une phrase :  $"
msg1 db 10,13,"la chaine saisie est : $"
msg2 db 10,13,"la taille de la chaine saisie est :             $"
chaine db 1024 dup("$") 
compteur dw 0
fincompteur  LABEL BYTE
.code
start:
mov ax,@data
mov ds, ax
mov ah,9
mov dx,offset msg
int 21h

lea si,chaine
comparaison:
mov ah,1
int 21h 

cmp al,"0"
jz yess 
inc [compteur]
mov [si],al
inc si
jmp comparaison
yess:
mov ah,9
mov dx,offset msg1
int 21h
mov ah,9
lea dx,chaine
int 21h 
xor ax,ax
mov     ax,[Compteur] 
mov     bx,OFFSET fincompteur-1 
mov     cx,5 
call ConvertBinEnChaine
mov bx,offset msg2
call affichage 
mov ah,4ch
int 21h

affichage     PROC
push ax
push dx
mov ah,9
mov dx,bx
int 21h
pop dx
pop ax
ret
affichage     ENDP
ConvertBinEnChaine   PROC
mov     si,10                     
ConvertLoop:
sub     dx,dx                
 div     si                                                  
 add     dl,'0'           
 mov     [bx],dl                   
 dec     bx                       
                                          
loop    ConvertLoop                
ret
ConvertBinEnChaine    ENDP

 end start

cs_patatalo · Answer

le petit soucis, c'est qu'il faudrait que je me tape les fonctions D.O.S. afin de savoir comment elles fonctionnent et leur retour vu que les commentaires ne sont pas mis et je n'ai vraiment pas envie tout de suite et le D.O.S. je ne m'en sert plus depuis longtemps...

marooh · Answer

Re,
je comprend.
Bon la 1ere partie du code on l'a deja traite.
La nouvelle partie:
mov bx,offset fincompteur-1 \pointe sur la fin 
mov cx,5 \ le nombre de digit à traiter
convertBinEnChaine proc
mov si,10 \le diviseur
convert loop:
sub dx,dx \ on converti ax à un doubleword
div si\ on effectue la div
add al,'0' \ on ajoute un 0 à la chaine converti
mov [ bx],dl\on stocke la valeur converti 
dec bx
loop

cs_patatalo · Answer

type word en mémoire:
00: low byte
01: high byte
fincompteur.

*(fincompteur - 1) = high byte.

@++

cs_patatalo · Answer

que se passe-t-il si tu entre 1024 caractères ?

cs_patatalo · Answer

que fais-tu du caractère final '$' ?

où est la fontion de sortie du nombre ?

tel que c'est, ce qui serait illogique, c'est que cela fonctionne.

@++

cs_patatalo · Answer

ce qui définit l'arret de la boucle, ce n'est pas cx, c'est ax=0.

cs_ghuysmans99 · Answer

@patatalo : Une boucle loop s'arrête lorsque (E)CX=0 

VB.NET is good ... VB6 is better
Utilise Réponse acceptée quand un post répond à ta question

marooh · Answer

re,
fincompteur est déclarée comme un  LABEL BYTE
chaine est déclarée comme un db 1024 dup("$")
pour afficher le resultat j'ai écrit une procedure printstring
push ax
push dx
mov ah,9
mov dx,offset msg
int 21h
pop dx
pop ax

marooh · Answer

re,
j'ai enfin résolu le probléme!!!!!!!!!

cs_patatalo · Answer

@ ghuysmans99: le problème ce n'est pas le loop et comment il se termine, le problème est qu'il n'a rien a faire là étant donné que c'est ax qui doit nous donner la fin du traitement.

le nombre sera converti entièrement quand ax sera à zero et non quand cx sera à 0. Sinon, on aura des 0 devant le nombre et je ne vois pas trop l'interêt dans le cas présent. De plus, en utilisant ax, tu économises un registre.

@++

Erreur en assemblage

59 réponses

Votre réponse

Discussions similaires