compilateur

Marsh Posté le 19-05-2017 à 21:34:39

Code :

//<blocvar> -> VAR<sdecl>FVAR
//<sdecl> -> <decl> | <sdecle>,<decl>
//<decl> -> i<chvar> = <nbrevar>
//<nbrevar> -> <chvar> | <nbrevar><chvar>
//<chvar> -> 0|1|2|3|4|5|6|7|8|9
//<affectation> ? INSTR<sinstruction>SINSTR
//<sinstruction> -> <intruction> | <sinstruction>,<instruction>
//<instruction> -> i<chiffre> = <oulogique>
//<oulogique> -> <etlogique>| <oulogique> | <etlogique>
//<etlogique> -> <PlusMoins> | <etlogique> & <PlusMoins>
// <plusmoins> ? <foisdiv> | <plusmoins> + <foisdiv> | <plusmoins> - <foisdiv>
// <foisdiv> ? <opunaire> | <foisdiv> * <opunaire> | <foisdiv> / <opunaire>
//<opunaire> -> <facteur> | - <facteur> | ~ <facteur> | + <facteur>
// <facteur> ? <nombre> | ( <plusmoins> ) | i<chiffre>
//<nombre>-> <nombredeci> | 0b<nombrebin> | 0x<nombrehexa>
//<nombrehexa> -> <chiffrehexa> | <nombrehexa><chiffrehexa>
//<chiffrehexa> -> 0|1|2|3/4/5/6/7/8/9/A/B/C/D/E/F
// <nombredeci> -> <chiffredeci> | <nombredeci><chiffredeci>
// <Chiffre> -> 0 | 1 | 2 | 3 | 4 | 5 | 6 | 7 | 8 | 9
// <nombrebin> -> <chiffrebin> | <nombrebin><chiffrebin>
//<chiffrebin> -> 0 | 1
#include <stdio.h>
#include <ctype.h>
#include <string.h>
#include <stdlib.h>
#include <stdio.h>
#include <stdlib.h>
//#include <unistd.h>
//#include "SortieErreur.h"
void PlusMoins();
void FoisDiv();
void Facteur();
void Nombre();
int NombreDeci();
int NombreBin();
int ChiffreBin();
int nombrehexa();
int chiffrehexa();
void CaractereSuivant();
char CaractereCourant();
void etlogique();
void oulogique();
void sinstruction();
void instruction();
void opunaire();
void blocvar();
void decl();
void prog();
int nbrevar();
void sdecl();
void affectation();
char acExpr[500] ;
// "VAR i1=3,i3=8 SVAR INSTR i1=i1*2,i3=i1+i1 SINSTR";//code source dans fichier source
// expression à compiler
char bCourant = 0; // indice du caractère courant dans l'expression
void main()
{
FILE *hfF=NULL;
if ((hfF = fopen("F:\\virton\\Bureau\\micro quadri 2\\compilateur\\compilateur\\expression.txt","r+" )) == NULL)
{ printf("(Fichier1) Err. d'ouverture du fichier" );
exit(1) ;
}
//fseek(hfF, SEEK_SET, 0);
fread(acExpr,500,1,hfF);/*) != 1)
{ printf("(Fichier14) Err. d'ouverture du fichier" );
exit(1) ;
}*/
printf("%s\n", &acExpr[0]);
//oulogique(); // démarrer la compilation
//
blocvar();
affectation();
if (CaractereCourant() != 0)
printf("Le caractere %c a la position %d dans %s est invalide!",
acExpr[bCourant], bCourant + 1, acExpr);
else
printf("La compilation s'est tres bien deroulee!" );
fclose(hfF);
}
//<blocvar> VAR<sdecl>FVAR
void blocvar()
{
if (strncmp(&acExpr[bCourant], "VAR",3) == 0)
{
// printf("\nint i%d =\n",CaractereCourant());
CaractereSuivant();
CaractereSuivant();
CaractereSuivant();
sdecl();
if (strncmp(&acExpr[bCourant], "SVAR",4) == 0)
{
CaractereSuivant();
CaractereSuivant();
CaractereSuivant();
CaractereSuivant();
}
}
}
//<affectation> ? INSTR<sinstruction>SINSTR
void affectation()
{
if (strncmp(&acExpr[bCourant], "INSTR", 5) == 0)
{
printf("\nint main()\n{\n\t_asm\n\t{\n" );
CaractereSuivant();
CaractereSuivant();
CaractereSuivant();
CaractereSuivant();
CaractereSuivant();
sinstruction();
if (strncmp(&acExpr[bCourant], "SINSTR", 6) == 0)
{
CaractereSuivant();
CaractereSuivant();
CaractereSuivant();
CaractereSuivant();
CaractereSuivant();
CaractereSuivant();
printf("\n}\n}" );
}
}
}
//<sinstruction> -> <intruction> | <sinstruction>,<instruction>
void sinstruction()
{
instruction();
while ( CaractereCourant()== ',' )
{
CaractereSuivant();
instruction();
}
}
//<sdecl> -> <decl> | <sdecl>,<decl>
void sdecl()
{
decl();
while ( CaractereCourant()== ',' )
{
CaractereSuivant();
decl();
}
}
//<instruction> -> i<chiffre> = <oulogique>
void instruction()
{
char nbre;
if (CaractereCourant()== 'i')
{
CaractereSuivant();
if(isdigit(CaractereCourant()))
{
//printf("\t\tpush i%d,eax\n", CaractereCourant());
nbre=CaractereCourant();
CaractereSuivant();
if(CaractereCourant()=='=')
{
CaractereSuivant();
oulogique();
printf("\t\tpop i%c\n",nbre);
// printf("\t\tmov i%c,eax\n",nbre);
//printf("push i1\npush 2\npop ebx\npop eax\nimul eax ebx \npush eax\n pop i1\n" );
}
}
}
}
//<decl> -> i<chvar> = <nbrevar>
void decl()
{
char nbre;
if (CaractereCourant()== 'i')
{
printf("int i" );
CaractereSuivant();
if(isdigit(CaractereCourant()))
{
//printf("\t\tpush i%d,eax\n", CaractereCourant());
nbre=CaractereCourant();
printf("%c ",nbre);
CaractereSuivant();
if(CaractereCourant()=='=')
{
//printf("=%c\n",CaractereCourant());
CaractereSuivant();
printf("= %c;\n",CaractereCourant());
nbrevar();
}
}
}
}
// Cette fonction prend en compte les règles de grammaire:
//<opunaire> -> <facteur> | - <facteur> | ~ <facteur> | + <facteur>
void opunaire()
{
switch(CaractereCourant())
{
case '~':
CaractereSuivant();
Facteur();
printf("pop eax\n" );
printf("not eax\n" );
printf("push eax\n" );
break;
case '-':
CaractereSuivant();
Facteur();
printf("pop eax\n" );
printf("imul eax,-1\n" );
printf("push eax\n" );
break;
case '+':
CaractereSuivant();
Facteur();
break;
default:
Facteur();
break;
}
}
// Cette fonction prend en compte les règles de grammaire:
//<oulogique> -> <oulogique> | <etlogique>
void oulogique()
{
etlogique();
while ( CaractereCourant()== '|' )
{
CaractereSuivant();
etlogique();
printf("\tpop ebx\n" );
printf("\tpop eax\n" );
printf("\tor eax, ebx\n" );
printf("\tpush eax\n" );
}
}
// Cette fonction prend en compte les règles de grammaire:
//<etlogique> -> <etlogique> | <PlusMoins> & <PlusMoins>
void etlogique()
{
PlusMoins();
while( CaractereCourant()== '&' )
{
CaractereSuivant();
PlusMoins();
printf("\tpop ebx\n" );
printf("\tpop eax\n" );
printf("\tand eax, ebx\n" );
printf("\tpush eax\n" );
}
}
// Cette fonction prend en compte les règles de grammaire:
// <plusmoins> ? <foisdiv> | <plusmoins> + <foisdiv> | <plusmoins> - <foisdiv>
void PlusMoins()
{
FoisDiv();
while (CaractereCourant() == '+' || CaractereCourant() == '-')
{
switch(CaractereCourant())
{
case '+':
CaractereSuivant();
FoisDiv();
printf("pop ebx\n" );
printf("pop eax\n" );
printf("add eax, ebx\n" );
printf("push eax\n" );
break;
case '-':
CaractereSuivant();
FoisDiv();
printf("pop ebx\n" );
printf("pop eax\n" );
printf("sub eax, ebx\n" );
printf("push eax\n" );
}
}
}
// Cette fonction prend en compte les règles de grammaire:
// <foisdiv> ? <opunaire> | <foisdiv> * <opunaire> | <foisdiv> / <opunaire>
void FoisDiv()
{
opunaire();
while (CaractereCourant() == '*' || CaractereCourant() == '/')
{
switch (CaractereCourant())
{
case '*':
CaractereSuivant();
opunaire();
printf("pop ebx\n" );
printf("pop eax\n" );
printf("imul eax, ebx\n" );
printf("push eax\n" );
break;
case '/':
CaractereSuivant();
opunaire();
printf("pop ebx\n" );
printf("pop eax\n" );
printf("cdq\n" );
printf("idiv ebx\n" );
printf("push eax\n" );
}
}
}
// Cette fonction prend en compte les règles de grammaire:
// <facteur> ? ( <plusmoins> ) | <nombre> | <nbrevar>
void Facteur()
{
if (CaractereCourant() == '(')
{
CaractereSuivant();
oulogique();
if (CaractereCourant() == ')')
CaractereSuivant();
}
else
{
if (CaractereCourant()== 'i')
{
CaractereSuivant();
if(isdigit(CaractereCourant()))
{
printf("\t\tpush i%c\n", CaractereCourant());
CaractereSuivant();
}
}
else
Nombre();
}
}
// Cette fonction prend en compte les règles de grammaire:
// <nombre> -> <nombredeci> | 0b<nombrebin>
void Nombre()
{
if(strncmp(&acExpr[bCourant], "0b", 2) == 0)
{
CaractereSuivant();
CaractereSuivant();
NombreBin();
}
if(strncmp(&acExpr[bCourant], "0x", 2) == 0)
{
CaractereSuivant();
CaractereSuivant();
nombrehexa();
} else
{
NombreDeci();
}
}
// Cette fonction prend en compte les règles de grammaire:
// <nombredeci> -> <chiffredeci> | <nombredeci><chiffredeci>
// <Chiffre> -> 0 | 1 | 2 | 3 | 4 | 5 | 6 | 7 | 8 | 9
int NombreDeci()
{
int iVal = 0;
if (isdigit(CaractereCourant()) != 0)
{
while (isdigit(CaractereCourant()) != 0)
{
iVal = (iVal * 10) + (CaractereCourant() - 0x30);
CaractereSuivant();
}
printf("push %d\n", iVal);
}
return iVal;
}
//<nbrevar> -> <chvar> | <nbrevar><chvar>
int nbrevar()
{
int iVal = 0;
if (isdigit(CaractereCourant()) != 0)
{
while (isdigit(CaractereCourant()) != 0)
{
iVal = (iVal * 10) + (CaractereCourant() - 0x30);
CaractereSuivant();
}
//printf("push %d\n", iVal);
}
return iVal;
}
// Cette fonction prend en compte les règles de grammaire:
// <nombrebin> -> <chiffrebin> | <nombrebin><chiffrebin>
int NombreBin()
{
int iVal = 0;
if (ChiffreBin() != 0)
{
while (ChiffreBin() != 0)
{
iVal = (iVal * 2) + (CaractereCourant() - 0x30);
CaractereSuivant();
}
printf("push %d\n", iVal);
}
return iVal;
}
// Cette fonction prend en compte les règles de grammaire:
// <chiffrebin> -> 0 | 1
int ChiffreBin()
{
if (CaractereCourant() == '0' || CaractereCourant() == '1')
return 1;
else return 0;
}
// Cette fonction prend en compte les règles de grammaire:
//<chiffrehexa> -> 0|1|2|3/4/5/6/7/8/9/A/B/C/D/E/F
int chiffrehexa()
{
if((CaractereCourant() >= '0' && CaractereCourant() <= '9')|| (CaractereCourant() >= 'A' && CaractereCourant() <= 'F'))
return 1;
else return 0;
}
// cette fonction prend en compte les règles de grammaire:
//<nombre>-> <nombredeci> | 0b<nombrebin> | 0x<nombrehexa>
//<nombrehexa> -> <chiffrehexa> | <nombrehexa><chiffrehexa>
//<chiffrehexa> -> 0|1|2|3/4/5/6/7/8/9/A/B/C/D/E/F
int nombrehexa()
{
int iVal=0;
if(chiffrehexa() != 0)
{
while(chiffrehexa() != 0)
{
if(isdigit(CaractereCourant()))
{
iVal=(iVal*16)+(CaractereCourant()-0x30);
CaractereSuivant();
}
else
{
iVal=(iVal*16)+(CaractereCourant()-55);
CaractereSuivant();
}
}
printf("push %d\n", iVal);
}
return iVal;
}
// Cette fonction passe au caractère suivant dans l'expression
void CaractereSuivant()
{
do
bCourant++;
while (acExpr[bCourant] == ' '); // éliminer les espaces
}
// Cette fonction retourne le caractère courant de l'expression
char CaractereCourant()
{
return acExpr[bCourant];
}
//mettre à jour hexa et chiffre héxa >>>0x

Message cité 1 fois
Message édité par bjs le 20-05-2017 à 00:20:30

Reply

Marsh Posté le 19-05-2017 à 21:34:39

Reply

Marsh Posté le 19-05-2017 à 22:20:48

bjs a écrit :

avec les fichier j'ai un souci

C'est à dire? "un soucis" c'est une très mauvaise description, il faut être précis.

Il vient faire quoi ce fflush() ligne 66? :??:

Ton appel de fread() + le if() c'est douteux, tu demandes 500*1 octets alors que d'après le commentaire ligne 59 le fichier en contient beaucoup moins, du coup ton code indique une erreur (avec le mauvais texte d'ailleurs).
Citation du K&R:

Citation :

size_t fread(void *ptr, size_t size, size_t nobj, FILE *stream)
fread reads from stream into the array ptr at most nobj objects of size size.
fread returns the number of objects read; this may be less than the number requested. feof and ferror must be used to determine status.

Aussi tu devrais activer les warnings de ton compilateur (ou les prendre en compte ):
(Attention les n° de lignes ne correspondent pas forcément à ton code!)

Citation :

> "gcc" "t2.c" -Wall -O3 -std=c99 -o "t2.exe"
t2.c:56:6: warning: return type of 'main' is not 'int' [-Wmain]
t2.c: In function 'main':
t2.c:83:16: warning: array subscript has type 'char' [-Wchar-subscripts]
t2.c: In function 'blocvar':
t2.c:95:5: warning: array subscript has type 'char' [-Wchar-subscripts]
t2.c:102:9: warning: array subscript has type 'char' [-Wchar-subscripts]
t2.c: In function 'affectation':
t2.c:114:5: warning: array subscript has type 'char' [-Wchar-subscripts]
t2.c:123:9: warning: array subscript has type 'char' [-Wchar-subscripts]
t2.c: In function 'Nombre':
t2.c:355:5: warning: array subscript has type 'char' [-Wchar-subscripts]
t2.c:361:5: warning: array subscript has type 'char' [-Wchar-subscripts]
t2.c: In function 'CaractereSuivant':
t2.c:467:5: warning: array subscript has type 'char' [-Wchar-subscripts]
t2.c: In function 'CaractereCourant':
t2.c:472:5: warning: array subscript has type 'char' [-Wchar-subscripts]

Reply

Marsh Posté le 20-05-2017 à 00:13:15

j'ai réedité le code ça marche maintenant

Message cité 1 fois

Reply

Marsh Posté le 20-05-2017 à 00:18:43

j'ai un seul warning dans mon visual c++ 2010 et on nous a dit d'ignorer ça et je pensais que fread retourne le nombre d'octets lus ??
1>f:\virton\bureau\micro quadri 2\ex1\ex1\exercice1.c(66): warning C4996: 'fopen': This function or variable may be unsafe. Consider using fopen_s instead. To disable deprecation, use _CRT_SECURE_NO_WARNINGS. See online help for details.
1> c:\program files (x86)\microsoft visual studio 10.0\vc\include\stdio.h(234) : voir la déclaration de 'fopen'

Message édité par bjs le 20-05-2017 à 00:20:01

Reply

Marsh Posté le 20-05-2017 à 16:59:26

bjs a écrit :

j'ai réedité le code ça marche maintenant

C'est pas forcément une bonne idée de modifier ton message initial après-coup, ça peut prêter à confusion...

Aussi ta solution

Citation :

Code :

//fseek(hfF, SEEK_SET, 0);
fread(acExpr,500,1,hfF);/*) != 1)
{ printf("(Fichier14) Err. d'ouverture du fichier" );
exit(1) ;
}*/

c'est moche, tu as complètement viré le test plutôt que de le corriger et de faire quelque chose de propre (voir citation du K&R).

Citation :

j'ai un seul warning dans mon visual c++ 2010

alors ton compilateur est mal configuré! Les warnings ne sont pas là pour rien, il faut les activer et les prendre en compte!

Par ailleurs attention aux "magic numbers": Ta taille de buffer (500) apparaît à deux endroits différents, si tu veux changer la valeur et tu oublies de modifier partout bonjour les ennuis. Utiliser des #define!

Je viens aussi de penser à un truc: Tu utilises fread() pour lire une chaîne de caractère, sauf que fread() c'est une fonction générique qui ne connaît pas cette notion et va donc pas terminer la chaîne lue par un '\0', ça peut créer des gros problèmes ça aussi. Si tu as un fichier texte qui remplit le buffer jusqu'à la derniere place je dirais que les appels strncmp() risquent de lire en dehors de la mémoire allouée sous certaines conditions, mais j'ai pas la tête suffisamment reposée à vrai dire pour y voir clair... À regarder de près.

Citation :

et on nous a dit d'ignorer ça

On vous a expliqué pourquoi aussi? (Non je ne sais pas, d'où ma question). Il ne faut jamais ignorer un warning à moins de savoir exactement ce qu'on fait.

Citation :

et je pensais que fread retourne le nombre d'octets lus ??

Voir citation du K&R. Ca retourne le nombre d'objets (pas forcément octets!!) lus. Regarde bien ton if() et la taille de ton fichier expression.txt, tu ne remarques rien?

Reply

Marsh Posté le 20-05-2017 à 18:15:17

je vais essayer de corriger une erreur à la fois
je voudrais savoir si ce test de fread est juste ou pas ? ça marche chez moi et ça met pas le message d'erreur mais bon je suis pas sur du test ==-1
je pense que fread retourne le nombre d'enregistrements lus et pas le nombre d'octets lus et comme mon enregistrement fait moins de 500 alors il retourne 0 ??????? si mon fichier faisait 500 octets alors le test se deroulerait bien ?

if(fread(acExpr,500,1,hfF) != 1)
{ printf("(Fichier14t) Err. d'ouverture du fichier" );
exit(1) ;
}
// j'ai essayé comme ça et je vois que fread retourne 0 je comprends pas pourquoi pourtant j'ai modifié mon fichier texte et je vois que il est bien lu et il est mis dans le vecteur acExpr donc je comprends pas pourquoi ça marche sans test du fread et quand je mets un test ça marche plus
nbrad= fread(acExpr,500,1,hfF);
printf("%d\n",nbrad);
if (nbrad != 1)
{ printf("(Fichier14t) Err. d'ouverture du fichier" );
exit(1) ;
}

Message édité par bjs le 20-05-2017 à 19:09:00

Reply

Marsh Posté le 20-05-2017 à 19:12:05

J'ai été voir la taille réelle du fichier et ça fait 40 octets maintenant. Le test fonctionne mais si j'avais fait une allocation dynamique ça aurait marché je pense??

nbrad= fread(acExpr,40,1,hfF);
printf("%d\n",nbrad);
if (nbrad != 1)
{ printf("(Fichier14t) Err. d'ouverture du fichier" );
exit(1) ;
}

Message cité 1 fois
Message édité par bjs le 20-05-2017 à 20:36:50

Reply

Marsh Posté le 20-05-2017 à 19:12:26

Merci de faire un effort au niveau lisibilité: Mettre le code entre balises et faire des phrases qui commencent avec une majuscule et se terminent par un point (ou un point d'interrogation).

Oui fread() retourne le nombre d'enregistrements (si on veut appeller ça comme ça) lus, je l'ai dit et c'est très clair dans la citation du K&R. Après ton code montre que tu n'as pas encore bien compris. Juste pour lever un doute, tu comprends l'anglais?

Refléchis tranquillement: Quelle est la taille de ton fichier? Que retourne donc fread()? Quel sera donc le résultat du test fread(...)!=1 ? Avant de modifier ton test, pense au fait que ton fichier peut avoir une autre taille. C'est vraiment pas bien compliqué cette histoire. Si on veut vraiment bien faire il faut regarder feof() et ferror(), sinon un simple if(fread(...) ...) avec la bonne condition (que je ne te donnes pas car c'est toi qui doit apprendre) peut suffir.

Par ailleurs ton message d'erreur est toujours faux, fread() n'ouvre pas le fichier mais lit le contenu.

>vecteur acExpr
Ce n'est pas un vecteur mais un tableau. Il est très important d'utiliser les bon termes (dés le début quand on apprend encore histoire de ne pas créer de mauvaises habitudes) pour éviter les malentendus!

EDIT:

bjs a écrit :

j'ai été voir la taille du fichier et ça fait 40 octets maintenant le test fonctionne

Modifie donc ton ficher, p.ex. en changeant le "code" à l'intérieur ou en rajoutant un simple espace et relance ton programme...

Message édité par rat de combat le 20-05-2017 à 19:13:57

Reply

Marsh Posté le 20-05-2017 à 19:24:14

Je devrais faire une allocation dynamique ? pour que ça marche quelque soit la taille ? et je mets sizeof?

Message édité par bjs le 20-05-2017 à 20:26:05

Reply

Marsh Posté le 20-05-2017 à 20:13:47

Je voudrais faire une allocation dynamique, comme ça quelque soit la taille du fichier, ça sera ecrit dans acExpr. J'ai modifié le code mais ça ne marche pas et je dois surement avoir une petite erreur. Le ftell me retourne une bonne valeur pour la taille du fichier.

Code :

char *acExpr ;
int nbenreg=0;
// "VAR i1=3,i3=8 SVAR INSTR i1=i1*2,i3=i1+i1 SINSTR";//code source dans fichier source
// expression à compiler
char bCourant = 0;
int nbrad=0;
// indice du caractère courant dans l'expression
void main()
{
FILE*hfF=NULL;
if ((hfF = fopen("F:\\virton\\Bureau\\micro quadri 2\\compilateur\\compilateur\\expression.txt","r+" )) == NULL)
{
printf("(Fichier1) Err. d'ouverture du fichier" );
exit(1) ;
}
/*je test avec ftell la taille du fichier.
Je declare un maloc de la taille, ensuit je passe
l'adr du maloc a fread et je lis tout le fichier en un fois
et je passe l adr a acexpr */
fseek(hfF,0,SEEK_END);
nbenreg= ftell(hfF)/sizeof(char);
printf("nbenre=%d\n",nbenreg);
acExpr=(char*) malloc(nbenreg);
nbrad= fread(acExpr,nbenreg,1,hfF);
printf("nbrad = %d\n",nbrad);
if (nbrad != 1)
{ printf("(Fichier14t) Err. de lecture du fichier" );
exit(1) ;
}
printf("%s\n", &acExpr[0]);

Message édité par bjs le 20-05-2017 à 20:43:55

Reply

Marsh Posté le 20-05-2017 à 20:13:47

Reply

Marsh Posté le 20-05-2017 à 20:15:38

Citation :

Merci de faire un effort au niveau lisibilité: Mettre le code entre balises et faire des phrases qui commencent avec une majuscule et se terminent par un point (ou un point d'interrogation).

(éditer ta réponse précédente)

Reply

Marsh Posté le 20-05-2017 à 20:49:04

>fseek(hfF,0,SEEK_END)
Il faut revenir au début du fichier ensuite si tu veux lire quelque chose...

Il y a plusieurs d'autres trucs dans ton code qu'on peut critiquer, mais je ne veux pas faire ton boulot. Juste une chose, ton commentaire "je test avec ftell ..." c'est n'importe quoi. ftell() ne teste rien du tout, la fonction malloc qui prend deux 'L' par ailleurs est déclarée dans je ne sais quel header standard, toi tu l'utilises, tu ne passes pas l'adresse de malloc à fread ni à acExpr qui en plus est une variable alors qu'on passe des arguments (p.ex. des adresses) à des fonctions.
Non, ce n'est pas de la méchanceté de ma part, il est très important de s'exprimer correctement pour éviter les malentendus et surtout pour un débutant histoire de ne pas prendre de mauvaises habitudes et bien comprendre les bases.

Reply

Marsh Posté le 20-05-2017 à 20:59:11

Merci pour ton aide, tu es un pro. Le programme se compile bien et ça fait les bonnes instructions en assembleur mais j'ai des caractères qui se sont "introduits" à la fin de mon fichier... Je ne comprends pas d'où viennent ces caractères ; ç'est un exposant deux, quatre fois de suite, à la fin du fichier que j'ai crée et pourtant ils sont invisibles. Est ce que je dois rajouter un caractère 0 ? et comment ou bien à quel endroit du code je dois rajouter ce caractère 0 ?

Message cité 1 fois

Reply

Marsh Posté le 20-05-2017 à 21:06:51

bjs a écrit :

Merci pour ton aide, tu es un pro.

Oh non, loin de là!

Citation :

Le programme se compile bien

Tu as activé les warnings et corrigé ces derniers aussi?

Citation :

j'ai des caractères qui se sont "introduits" à la fin de mon fichier... Je ne comprends pas d'où viennent ces caractères ; ç'est un exposant deux, quatre fois de suite, à la fin du fichier que j'ai crée et pourtant ils sont invisibles. Est ce que je dois rajouter un caractère 0 ? et comment ou bien à quel endroit du code je dois rajouter ce caractère 0 ?

Où est-ce que tu vois ces caractères exactement?
Laisse moi deviner: C'est sur l'écran au moment du printf("%s\n", &acExpr[0]); c'est ça? Qu'est ce que je disais plus haut concernant fread() et les chaînes de caractères? Le '\0' c'est la bonne piste.

Reply

Marsh Posté le 20-05-2017 à 21:30:10

J'ai essayé plusieurs fois de rajouter un caractère '\0' de fin mais je ne sais pas en fait où je dois faire ça. Mon programme bug quand je rajoute un caractère zéro et je sais pas comment le faire ni où le faire.

Message édité par bjs le 20-05-2017 à 21:38:15

Reply

Marsh Posté le 20-05-2017 à 21:40:25

Essayer au pif c'est pas la bonne solution.
Qu'est-ce que c'est qu'une chaîne de caractère en C? Exact, un tableau avec des codes ASCII et '\0' en dernière position.
Donc si j'ai char str[]="hardware"; en mémoire je vais avoir (quelque part, peu importe l'adresse) une suite d'octets: 104 97 114 100 119 97 114 101 0 (en supposant '\0' == 0). Dans ton code c'est fread qui s'occupe de remplir ton tableau quelque part en mémoire.

Donc il faut mettre le '\0' où dans le code et où en mémoire?

Reply

Marsh Posté le 20-05-2017 à 21:50:52

Je ne sais pas; là je vois pas.

Reply

Marsh Posté le 20-05-2017 à 22:25:45

Est ce que ça marcherait si je modifie cette fonction ? je rajoute le caractère zéro après INSTR.
//<affectation> ? INSTR<sinstruction>SINSTR

Code :

void affectation()
{
if (strncmp(&acExpr[bCourant], "INSTR", 5) == 0)
{
printf("\nint main()\n{\n\t_asm\n\t{\n" );
CaractereSuivant();
CaractereSuivant();
CaractereSuivant();
CaractereSuivant();
CaractereSuivant();
sinstruction();
if (strncmp(&acExpr[bCourant], "SINSTR", 6) == 0)
{
CaractereSuivant();
CaractereSuivant();
CaractereSuivant();
CaractereSuivant();
CaractereSuivant();
CaractereSuivant();
acExpr[bCourant]=0;
printf("\n}\n}" );
}
}
}

Message édité par bjs le 20-05-2017 à 22:28:25

Reply

Marsh Posté le 20-05-2017 à 22:48:26

Mais non, c'est pourtant evident, il faut rajouter le '\0' à la fin de acExpr (attention, il faut la place pour!) après le fread()!

Reply

Marsh Posté le 20-05-2017 à 23:06:52

Comme ça ?

Code :

fseek(hfF,0,SEEK_END);
nbenreg= ftell(hfF)/sizeof(char);
printf("nbenre=%d\n",nbenreg);
acExpr= (char*) malloc (nbenreg+1);
fseek(hfF,0,SEEK_SET);
nbrad= fread(acExpr,nbenreg,1,hfF);
acExpr[nbenreg]=0;
printf("nbrad = %d\n",nbrad);

Message édité par bjs le 20-05-2017 à 23:11:14

Reply

Marsh Posté le 20-05-2017 à 23:26:52

à première vue c'est bon.

Reply

Marsh Posté le 20-05-2017 à 23:35:02

Mettre à jour le compilateur pour qu'il puisse lire le programme source depuis un fichier et écrire le programme en assembleur dans un autre fichier dont l'extension est .c

ah ok j'ai compris je dois mettre des fprintf comme ça ça copie mes instructions assembleur dans un fichier que je declare en global ???
J'ai aussi rajouté des lignes pour fermer le fichier et libérer la mémoire à la fin du main.
fclose(hfF);
free(acExpr);
acExpr=NULL;

Message édité par bjs le 21-05-2017 à 00:07:54

Reply

Marsh Posté le 21-05-2017 à 01:12:33

J'ai modifié juste les printf en fprintf pour copier mon code assembleur généré dans un fichier que j'ai crée en .c et j'ai déclaré un pointeur de fichier en global.
Si tu as le temps tu peux jeter un coup d'oeil, mais je pense que ça marche car j'ai compilé, puis j'ai vu que ça chargeait bien le code assembleur dans le fichier.c que j'ai crée et puis j'ai recompilé et regardé en debogeur pour voir si ça mettait les bon calculs et ç'était ok.
Quelques messages avant celui ci tu m'a écrit : "Il y a plusieurs d'autres trucs dans ton code qu'on peut critiquer, mais je ne veux pas faire ton boulot."
Je voudrais savoir quelles choses sont pas bien dans mon code. Merci de me dire quoi. Je voudrais savoir aussi comment activer les warnings sur visual c++ 2010. J'ai déjà des warnings mais peut être qu'il y a un level supérieur en warnings ??
J'ai mis enable all warnings et ça me rien de spécial en warnings à par ce même warning pour toutes mes fonctions=>
1>f:\virton\bureau\micro quadri 2\ex1\ex1\exercice1.c(453): warning C4255: 'Nombre' : aucun prototype de fonction fourni : conversion de '()' en '(void)'

Code :

//<blocvar> -> VAR<sdecl>FVAR
//<sdecl> -> <decl> | <sdecle>,<decl>
//<decl> -> i<chvar> = <nbrevar>
//<nbrevar> -> <chvar> | <nbrevar><chvar>
//<chvar> -> 0|1|2|3|4|5|6|7|8|9
//<affectation> ? INSTR<sinstruction>SINSTR
//<sinstruction> -> <intruction> | <sinstruction>,<instruction>
//<instruction> -> i<chiffre> = <oulogique>
//<oulogique> -> <etlogique>| <oulogique> | <etlogique>
//<etlogique> -> <PlusMoins> | <etlogique> & <PlusMoins>
// <plusmoins> ? <foisdiv> | <plusmoins> + <foisdiv> | <plusmoins> - <foisdiv>
// <foisdiv> ? <opunaire> | <foisdiv> * <opunaire> | <foisdiv> / <opunaire>
//<opunaire> -> <facteur> | - <facteur> | ~ <facteur> | + <facteur>
// <facteur> ? <nombre> | ( <plusmoins> ) | i<chiffre>
//<nombre>-> <nombredeci> | 0b<nombrebin> | 0x<nombrehexa>
//<nombrehexa> -> <chiffrehexa> | <nombrehexa><chiffrehexa>
//<chiffrehexa> -> 0|1|2|3/4/5/6/7/8/9/A/B/C/D/E/F
// <nombredeci> -> <chiffredeci> | <nombredeci><chiffredeci>
// <Chiffre> -> 0 | 1 | 2 | 3 | 4 | 5 | 6 | 7 | 8 | 9
// <nombrebin> -> <chiffrebin> | <nombrebin><chiffrebin>
//<chiffrebin> -> 0 | 1
#include <stdio.h>
#include <ctype.h>
#include <string.h>
#include <stdlib.h>
#include <stdio.h>
#include <stdlib.h>
//#include <unistd.h>
//#include "SortieErreur.h"
void PlusMoins();
void FoisDiv();
void Facteur();
void Nombre();
int NombreDeci();
int NombreBin();
int ChiffreBin();
int nombrehexa();
int chiffrehexa();
void CaractereSuivant();
char CaractereCourant();
void etlogique();
void oulogique();
void sinstruction();
void instruction();
void opunaire();
void blocvar();
void decl();
void prog();
int nbrevar();
void sdecl();
void affectation();
char *acExpr ;
int taillefichier=0;
FILE* hfich=NULL;
// "VAR i1=3,i3=8 SVAR INSTR i1=i1*2,i3=i1+i1 SINSTR";//code source dans fichier source
// expression à compiler
char bCourant = 0;
int nbretourread=0;
// indice du caractère courant dans l'expression
void main()
{
FILE*hfF=NULL;
if ((hfF = fopen("F:\\virton\\Bureau\\micro quadri 2\\ex1\\expression.txt","r+" )) == NULL) //"F:\\virton\\Bureau\\micro quadri 2\\compilateur\\compilateur\\expression.txt","r+" )) == NULL)
{
printf("(Fichier1) Err. d'ouverture du fichier" );
exit(1) ;
}
fseek(hfF,0,SEEK_END);
taillefichier= ftell(hfF)/sizeof(char);
printf("//taillefichier=%d\n",taillefichier);
acExpr= (char*) malloc (taillefichier+1);
if(acExpr==NULL)
{
printf("Erreur d'allocation dynamique" );
}
fseek(hfF,0,SEEK_SET);
nbretourread= fread(acExpr,taillefichier,1,hfF);
acExpr[taillefichier]='\0';
printf("//nbrretourread = %d\n",nbretourread);
if (nbretourread != 1)
{ printf("(Fichier14t) Err. de lecture du fichier" );
exit(1) ;
}
printf("//%s\n", &acExpr[0]);
// démarrer la compilation
//
if ((hfich = fopen("F:\\virton\\Bureau\\micro quadri 2\\ex1\\fichier.c","w" )) == NULL) //"F:\\virton\\Bureau\\micro quadri 2\\compilateur\\compilateur\\expression.txt","r+" )) == NULL)
{
printf("(Fichier1) Err. d'ouverture du fichier" );
exit(1) ;
}
blocvar();
affectation();
if (CaractereCourant() != 0)
printf("Le caractere %c a la position %d dans %s est invalide!",
acExpr[bCourant], bCourant + 1, acExpr);
else
printf("La compilation s'est bien deroulee!" );
fclose(hfF);
free(acExpr);
acExpr=NULL;
}
//<blocvar> VAR<sdecl>FVAR
void blocvar()
{
if (strncmp(&acExpr[bCourant], "VAR",3) == 0)
{
CaractereSuivant();
CaractereSuivant();
CaractereSuivant();
sdecl();
if (strncmp(&acExpr[bCourant], "SVAR",4) == 0)
{
CaractereSuivant();
CaractereSuivant();
CaractereSuivant();
CaractereSuivant();
}
}
}
//<affectation> ? INSTR<sinstruction>SINSTR
void affectation()
{
if (strncmp(&acExpr[bCourant], "INSTR", 5) == 0)
{
//printf("\nint main()\n{\n\t_asm\n\t{\n" );
fprintf(hfich,"\nint main()\n{\n\t_asm\n\t{\n" );
CaractereSuivant();
CaractereSuivant();
CaractereSuivant();
CaractereSuivant();
CaractereSuivant();
sinstruction();
if (strncmp(&acExpr[bCourant], "SINSTR", 6) == 0)
{
CaractereSuivant();
CaractereSuivant();
CaractereSuivant();
CaractereSuivant();
CaractereSuivant();
CaractereSuivant();
//printf("\n}\n}" );
fprintf(hfich,"\n}\n}" );
}
}
}
//<sinstruction> -> <intruction> | <sinstruction>,<instruction>
void sinstruction()
{
instruction();
while ( CaractereCourant()== ',' )
{
CaractereSuivant();
instruction();
}
}
//<sdecl> -> <decl> | <sdecl>,<decl>
void sdecl()
{
decl();
while ( CaractereCourant()== ',' )
{
CaractereSuivant();
decl();
}
}
//<instruction> -> i<chiffre> = <oulogique>
void instruction()
{
char nbre;
if (CaractereCourant()== 'i')
{
CaractereSuivant();
if(isdigit(CaractereCourant()))
{
//printf("\t\tpush i%d,eax\n", CaractereCourant());
nbre=CaractereCourant();
CaractereSuivant();
if(CaractereCourant()=='=')
{
CaractereSuivant();
oulogique();
fprintf(hfich,"\t\tpop i%c\n",nbre);
//printf("\t\tpop i%c\n",nbre);
// printf("\t\tmov i%c,eax\n",nbre);
//printf("push i1\npush 2\npop ebx\npop eax\nimul eax ebx \npush eax\n pop i1\n" );
}
}
}
}
//<decl> -> i<chvar> = <nbrevar>
void decl()
{
char nbre;
if (CaractereCourant()== 'i')
{
//printf("int i" );
fprintf(hfich,"int i" );
CaractereSuivant();
if(isdigit(CaractereCourant()))
{
nbre=CaractereCourant();
//printf("%c ",nbre);
fprintf(hfich,"%c ",nbre);
CaractereSuivant();
if(CaractereCourant()=='=')
{
CaractereSuivant();
//printf("= %c;\n",CaractereCourant());
fprintf(hfich,"= %c;\n",CaractereCourant());
nbrevar();
}
}
}
}
// Cette fonction prend en compte les règles de grammaire:
//<opunaire> -> <facteur> | - <facteur> | ~ <facteur> | + <facteur>
void opunaire()
{
switch(CaractereCourant())
{
case '~':
CaractereSuivant();
Facteur();
// printf("pop eax\n" );
// printf("not eax\n" );
// printf("push eax\n" );
fprintf(hfich,"pop eax\nnot eax\npush eax\n" );
break;
case '-':
CaractereSuivant();
Facteur();
//printf("pop eax\n" );
// printf("imul eax,-1\n" );
// printf("push eax\n" );
fprintf(hfich,"pop eax\nimul eax,-1\npush eax\n" );
break;
case '+':
CaractereSuivant();
Facteur();
break;
default:
Facteur();
break;
}
}
// Cette fonction prend en compte les règles de grammaire:
//<oulogique> -> <oulogique> | <etlogique>
void oulogique()
{
etlogique();
while ( CaractereCourant()== '|' )
{
CaractereSuivant();
etlogique();
// printf("pop ebx\n" );
// printf("pop eax\n" );
// printf("or eax, ebx\n" );
// printf("push eax\n" );
fprintf(hfich,"pop ebx\npop eax\nor eax, ebx\npush eax\n" );
}
}
// Cette fonction prend en compte les règles de grammaire:
//<etlogique> -> <etlogique> | <PlusMoins> & <PlusMoins>
void etlogique()
{
PlusMoins();
while( CaractereCourant()== '&' )
{
CaractereSuivant();
PlusMoins();
// printf("\tpop ebx\n" );
// printf("\tpop eax\n" );
// printf("\tand eax, ebx\n" );
// printf("\tpush eax\n" );
fprintf(hfich,"pop ebx\npop eax\and eax, ebx\npush eax\n" );
}
}
// Cette fonction prend en compte les règles de grammaire:
// <plusmoins> ? <foisdiv> | <plusmoins> + <foisdiv> | <plusmoins> - <foisdiv>
void PlusMoins()
{
FoisDiv();
while (CaractereCourant() == '+' || CaractereCourant() == '-')
{
switch(CaractereCourant())
{
case '+':
CaractereSuivant();
FoisDiv();
// printf("pop ebx\n" );
// printf("pop eax\n" );
// printf("add eax, ebx\n" );
// printf("push eax\n" );
fprintf(hfich,"pop ebx\npop eax\nadd eax, ebx\npush eax\n" );
break;
case '-':
CaractereSuivant();
FoisDiv();
// printf("pop ebx\n" );
// printf("pop eax\n" );
// printf("sub eax, ebx\n" );
// printf("push eax\n" );
fprintf(hfich,"pop ebx\npop eax\nsub eax, ebx\npush eax\n" );
}
}
}
// Cette fonction prend en compte les règles de grammaire:
// <foisdiv> ? <opunaire> | <foisdiv> * <opunaire> | <foisdiv> / <opunaire>
void FoisDiv()
{
opunaire();
while (CaractereCourant() == '*' || CaractereCourant() == '/')
{
switch (CaractereCourant())
{
case '*':
CaractereSuivant();
opunaire();
// printf("pop ebx\n" );
// printf("pop eax\n" );
// printf("imul eax, ebx\n" );
// printf("push eax\n" );
fprintf(hfich,"pop ebx\npop eax\nimul eax, ebx\npush eax\n" );
break;
case '/':
CaractereSuivant();
opunaire();
// printf("pop ebx\n" );
// printf("pop eax\n" );
// printf("cdq\n" );
// printf("idiv ebx\n" );
// printf("push eax\n" );
fprintf(hfich,"pop ebx\npop eax\ncdq\nidiv ebx\npush eax\n" );
}
}
}
// Cette fonction prend en compte les règles de grammaire:
// <facteur> ? ( <plusmoins> ) | <nombre> | <nbrevar>
void Facteur()
{
if (CaractereCourant() == '(')
{
CaractereSuivant();
oulogique();
if (CaractereCourant() == ')')
CaractereSuivant();
}
else
{
if (CaractereCourant()== 'i')
{
CaractereSuivant();
if(isdigit(CaractereCourant()))
{
//printf("push i%c\n", CaractereCourant());
fprintf(hfich,"push i%c\n", CaractereCourant());
CaractereSuivant();
}
}
else
Nombre();
}
}
// Cette fonction prend en compte les règles de grammaire:
// <nombre> -> <nombredeci> | 0b<nombrebin>
void Nombre()
{
if(strncmp(&acExpr[bCourant], "0b", 2) == 0)
{
CaractereSuivant();
CaractereSuivant();
NombreBin();
}
if(strncmp(&acExpr[bCourant], "0x", 2) == 0)
{
CaractereSuivant();
CaractereSuivant();
nombrehexa();
} else
{
NombreDeci();
}
}
// Cette fonction prend en compte les règles de grammaire:
// <nombredeci> -> <chiffredeci> | <nombredeci><chiffredeci>
// <Chiffre> -> 0 | 1 | 2 | 3 | 4 | 5 | 6 | 7 | 8 | 9
int NombreDeci()
{
int iVal = 0;
if (isdigit(CaractereCourant()) != 0)
{
while (isdigit(CaractereCourant()) != 0)
{
iVal = (iVal * 10) + (CaractereCourant() - 0x30);
CaractereSuivant();
}
//printf("push %d\n", iVal);
// fprintf(hfich,"push %d\n", iVal);
}
return iVal;
}
//<nbrevar> -> <chvar> | <nbrevar><chvar>
int nbrevar()
{
int iVal = 0;
if (isdigit(CaractereCourant()) != 0)
{
while (isdigit(CaractereCourant()) != 0)
{
iVal = (iVal * 10) + (CaractereCourant() - 0x30);
CaractereSuivant();
}
//printf("push %d\n", iVal);
//fprintf(hfich,"push %d\n", iVal);
}
return iVal;
}
// Cette fonction prend en compte les règles de grammaire:
// <nombrebin> -> <chiffrebin> | <nombrebin><chiffrebin>
int NombreBin()
{
int iVal = 0;
if (ChiffreBin() != 0)
{
while (ChiffreBin() != 0)
{
iVal = (iVal * 2) + (CaractereCourant() - 0x30);
CaractereSuivant();
}
//printf("push %d\n", iVal);
fprintf(hfich,"push %d\n", iVal);
}
return iVal;
}
// Cette fonction prend en compte les règles de grammaire:
// <chiffrebin> -> 0 | 1
int ChiffreBin()
{
if (CaractereCourant() == '0' || CaractereCourant() == '1')
return 1;
else return 0;
}
// Cette fonction prend en compte les règles de grammaire:
//<chiffrehexa> -> 0|1|2|3/4/5/6/7/8/9/A/B/C/D/E/F
int chiffrehexa()
{
if((CaractereCourant() >= '0' && CaractereCourant() <= '9')|| (CaractereCourant() >= 'A' && CaractereCourant() <= 'F'))
return 1;
else return 0;
}
// cette fonction prend en compte les règles de grammaire:
//<nombre>-> <nombredeci> | 0b<nombrebin> | 0x<nombrehexa>
//<nombrehexa> -> <chiffrehexa> | <nombrehexa><chiffrehexa>
//<chiffrehexa> -> 0|1|2|3/4/5/6/7/8/9/A/B/C/D/E/F
int nombrehexa()
{
int iVal=0;
if(chiffrehexa() != 0)
{
while(chiffrehexa() != 0)
{
if(isdigit(CaractereCourant()))
{
iVal=(iVal*16)+(CaractereCourant()-0x30);
CaractereSuivant();
}
else
{
iVal=(iVal*16)+(CaractereCourant()-55);
CaractereSuivant();
}
}
//printf("push %d\n", iVal);
fprintf(hfich,"push %d\n", iVal);
}
return iVal;
}
// Cette fonction passe au caractère suivant dans l'expression
void CaractereSuivant()
{
do
bCourant++;
while (acExpr[bCourant] == ' '); // éliminer les espaces
}
// Cette fonction retourne le caractère courant de l'expression
char CaractereCourant()
{
return acExpr[bCourant];
}

Message cité 1 fois
Message édité par bjs le 21-05-2017 à 11:51:32

Reply

Marsh Posté le 21-05-2017 à 15:31:48

bjs a écrit :

Si tu as le temps tu peux jeter un coup d'oeil, mais je pense que ça marche car j'ai compilé, puis j'ai vu que ça chargeait bien le code assembleur dans le fichier.c que j'ai crée et puis j'ai recompilé et regardé en debogeur pour voir si ça mettait les bon calculs et ç'était ok.

J'ai pas les connaissances pour vérifier le code asm généré mais ton programme fonctionne oui.

Citation :

Quelques messages avant celui ci tu m'a écrit : "Il y a plusieurs d'autres trucs dans ton code qu'on peut critiquer, mais je ne veux pas faire ton boulot." Je voudrais savoir quelles choses sont pas bien dans mon code. Merci de me dire quoi.

Désolé. La note que tu vas avoir pour ce travail est censé représenter/évaluer tes compétences/..., pas les miennes. Si je commence à corriger ton code (en t'indiquant ce qui ne va pas) ça va falsifier le truc et puis j'ai pas trop le temps ni l'envie non plus. Il faut aussi dire que bien des choses au niveau code sont subjectives, ton prof ne pense peut-être pas du tout comme moi et je ne connais pas le cahier des charges. Je t'ai aidé à faire fonctionner ton code globalement, je vais m'arrêter là.

Citation :

Je voudrais savoir aussi comment activer les warnings sur visual c++ 2010.

Je ne sais pas et je ne vais pas lire la doc (ou utiliser un moteur de recherche, en anglais c'est mieux) à ta place. Vu le "C++" dans le nom de ce logiciel (que je ne connais pas), ton code est bien dans un fichier .c (et non .cpp) et compilé avec un compilateur C et non C++?

Citation :

J'ai déjà des warnings mais peut être qu'il y a un level supérieur en warnings ??
J'ai mis enable all warnings et ça me rien de spécial en warnings à par ce même warning pour toutes mes fonctions=>
1>f:\virton\bureau\micro quadri 2\ex1\ex1\exercice1.c(453): warning C4255: 'Nombre' : aucun prototype de fonction fourni : conversion de '()' en '(void)'

Il doit y avoir un autre niveau, le compilateur doit au moins te dire que main() doit retourner int. GCC (que j'utilises) avec -Wall parle aussi d'autres trucs, voir en haut, j'ai pas regardé. Concernant le warning C4255, tu as bien des prototypes dans ton code, je pense que le compilateur essaye de te dire que si une fonction ne prend pas d'arguments il faut indiquer (void) et non ().

Reply

Marsh Posté le

Reply

compilateur

Sujets relatifs:

Leave a Replay