Conversion C++ 1990 à Dev C++

**Orion2** · 19/12/2018, 07h03

Bonjour à tous,
Y a-t-il quelqu’un qui pourrait me renseigner sur un ancien programme que j’avais fait dans les années 1993 et que je voudrais reprendre sur dev c++, apparemment beaucoup de problème avec le graphisme le code est est si dessous ou en pièce jointe :

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
25
26
27
28
29
30
31
32
33
34
35
36
37
38
39
40
41
42
43
44
45
46
47
48
49
50
51
52
53
54
55
56
57
58
59
60
61
62
63
64
65
66
67
68
69
70
71
72
73
74
75
76
77
78
79
80
81
82
83
84
85
86
87
88
89
90
91
92
93
94
95
96
97
98
99
100
101
102
103
104
105
106
107
108
109
110
111
112
113
114
115
116
117
118
119
120
121
122
123
124
125
126
127
128
129
130
131
132
133
134
135
136
137
138
139
140
141
142
143
144
145
146
147
148
149
150
151
152
153
154
155
156
157
158
159
160
161
162
163
164
165
166
167
168
169
170
171
172
173
174
175
176
177
178
179
180
181
182
183
184
185
186
187
188
189
190
191
192
193
194
195
196
197
198
199
200
201
202
203
204
205
206
207
208
209
210
/***************************************************************************
Programme : digitalisation pour TARM
Auteur    : ROBER SERGE
Date      : 30/10/1993
****************************************************************************/
#include <dos.h>
#include <stdlib.h>
#include <string.h>
#include <io.h>
#include <ctype.h>
#include <graphics.h>
#include <stdio.h>
#include <conio.h>
#include <bios.h>
 
short cpt=0,np=0,xdp=0,bl,bll,apd;
int xd,yd,sx,sy,vid,da,dp,dap,v=0,col=12;
FILE *in;
void fload();
void fquit();
 
main()
{
int code,cara;
if (_setvideomode(_VRES16COLOR) !=0)
 {
 _clearscreen(_GCLEARSCREEN);
 _setcliprgn(0,0,639,479);
 _rectangle(_GBORDER,0,0,639,479);
 _rectangle(_GBORDER,479,0,479,479);
 _rectangle(_GBORDER,479,35,639,35);
 _rectangle(_GBORDER,479,230,639,230);
 _settextcolor(15);
 _settextposition(2,67);
 _outtext("DISPLAY\n");
 _settextcolor(11);
 _settextposition(4,62);
 _outtext("Name   :");
 _settextposition(6,62);
 _outtext("Nb pts :");
 _settextposition(6,71);
  printf("%d",np);
 _settextposition(8,62);
 _outtext("Blanck :");
 _settextcolor(15);
 _settextposition(11,67);
 _outtext("RS Laser");
 _settextposition(13,63);
 _outtext("Copyright 1994(c)");
 _settextcolor(11);
 _settextposition(17,62);
 _outtext("F1 : Load");
 _settextposition(18,62);
 _outtext("F2 : Quit");
 _settextposition(20,62);
 _outtext("Nb XY  :");
 _settextposition(21,62);
 _outtext("Coord X:    Y:");
 _settextposition(22,62);
 _outtext("Nb Delt:");
 _settextposition(23,62);
 _outtext("Delta X:    Y:");
 
 
 while((cara=getch())!= '\r')
   if(cara != 0) {}
 
     else
      {
       code=getch();
       if (code==59) fload();
       else if (code==60) {_setvideomode(_DEFAULTMODE);break;}
       else continue;
       }
 
 _setvideomode(_DEFAULTMODE);
 }
  else
   {
    _setvideomode(_DEFAULTMODE);
    printf ("MODE VIDEO NON SUPPORTE\n");
    }
}
/***************************************************************************
                         CHARGEMENT DES IMAGES
****************************************************************************/
void fload()
 
{
int a=0x11,b=0x10,c=15,cpt,sw=0,dx,dy,tes=0,test=0,nd=0,nde=0;
fpos_t  posi;
short x,xx,rx,y,yy,ty,xy,ry,np,ex,ey;
FILE *fichier;
char ligne[8];
np=0;
_settextposition(4,71);
_outtext("        ");
_settextcolor(15);
_settextposition(27,04);
_outtext("8 LETTRES MAX.");
_settextposition(28,04);
_outtext("Entrez le nom du fichier :");
_settextcolor(11);
 gets(ligne);
_setcliprgn(1,1,478,478);
_clearscreen(_GVIEWPORT);
_settextposition(4,71);
 printf("%s",ligne);
  if((fichier=fopen(ligne,"r"))==NULL)
   {
      _settextposition(28,10);
      _outtext("Impossible d'ouvrir le fichier");
      return;
   }
rewind(fichier);
while(!feof(fichier))
  {  xy=fgetc(fichier);
     for ( ; sw!=1 ; xy=fgetc(fichier) )
/****************************************************************************
                       RECHERCHE & CALCUL DE LA MATRIX
****************************************************************************/
	 if(xy==a)sw=1;
     if(xy==a){}
	else
	  {
	     for (cpt=1 ; !feof(fichier) && xy!=a ; cpt++)
		 {
		    if (cpt==1)
		       {
			 x=xy;
			 if(x<127)xx=x+128;
			   else xx=x-128;
		       }
		    else if (cpt==2)
		      {
			y=xy;
			if(y<127)yy=y+128;
			  else yy=y-128;
			    ty=255-yy;
			    rx=xx*1.8710937;
			    ry=ty*1.8710937;
			    np=np+1;
			    nd=nd+1;
			    _settextposition(21,71);
			     printf("%X",x);
			    _settextposition(21,77);
			     printf("%X",y);
			    _settextposition(20,71);
			    printf("%d",nd);
			    _setcolor(c);
			    _setpixel(rx,ry);
		      }
		    else if (cpt>=3)
		      {
			if(xy==0x10)
			  {
			     xy=fgetc(fichier);
			     if(xy==0x80)
				  {
				     _settextposition(8,71);
				     printf("OPEN ");
				     c=15;
				     xy=fgetc(fichier);
				 }
			     if(xy==0x00)
			     {
			       _settextposition(8,71);
			       printf("CLOSE");
			       c=10;
			       xy=fgetc(fichier);
			   }
			 }
			if(xy==00) {}
			   else
			     {
				dx=xy/b;
				dy=xy%b;
				ex=dx-8;
				ey=dy-8;
				xx=xx+ex;
				yy=yy+ey;
				ty=255-yy;
				rx=xx*1.8710937;
				ry=ty*1.8710937;
				np=np+1;
				nde=nde+1;
			       _settextposition(23,71);
				printf("   ");
			       _settextposition(23,77);
				printf("   ");
			       _settextposition(23,71);
				printf("%X",dx);
			       _settextposition(23,77);
				printf("%X",dy);
 
				_settextposition(22,71);
				printf("%d",nde);
				_setcolor(c);
				_setpixel(rx,ry);
			    }
		      }
		    xy=fgetc(fichier);
		   _settextposition(6,71);
		    printf("%d",np);
		 }
	    if(feof(fichier));
	  }
  }
fclose(fichier);
}

Merci d'avance. Serge

**Médinoc** · 19/12/2018, 10h17

Bonjour,

Quitte à passer à du C++, tu devrais prendre un EDI plus récent que Dev-C++ (Code::Blocks par exemple).
Ah, les graphismes sur terminal... Il me semble que le plus portable pour ça, c'est la bibliothèque pdcurses (l'autre solution étant d'abandonner le TUI pour passer à un terminal "bête" ou à une vraie application fenêtrée).
Ton code possède un nombre dérangeant de variables globales, variables déclarées mais non-référencées, variables au nom non-informatif, usages de fonctions non-sécurisées (comme gets()), nombres magiques (pourquoi 1.8710937?), usages incorrects de feof()...Le nombre de commentaires dérange aussi, mais dans l'autre sens.

**Orion2** · 20/12/2018, 06h53

Bonjour,
Tous d’abord un grand merci pour la réponse, je sais qu’il y a un paquet de variable inutile car a la base le programme est plus conséquent et j’ai récupéré la partie « lecture de fichier » qui fonctionne très bien sous dos.
Du fait hier j’ai essayer le Code::Blocks avec le graphics.h a rajouter, mais bon, même une simple fenêtre c’est tout un art de programmation surtout d’un Windows a l’autre.
Donc actuellement je travaille avec Win 7 pro intégrale, dans l’espoir de savoir tiré quelque chose de la ?, ce n’est pas gagné !, je me posais la question serai ce pas plus facile sous VB, qui je pense est plus adéquat pour se genre de programme.
Bien à toi.

**Médinoc** · 20/12/2018, 11h06

Aurais-tu un échantillon de fichier à traiter pour qu'on voie un peu mieux à quoi la sortie est censée ressembler?

Donc actuellement je travaille avec Win 7 pro intégrale, dans l’espoir de savoir tiré quelque chose de la ?, ce n’est pas gagné !, je me posais la question serai ce pas plus facile sous VB, qui je pense est plus adéquat pour se genre de programme.

De préférence VB.Net ou C#, je conseillerais: ça permet un affichage graphique assez facilement.

**Médinoc** · 20/12/2018, 17h18

Je pense que le plus gros problème de ce code, c'est qu'il est pratiquement impossible à porter sur une plate-forme qui n'a pas les mêmes capacités d'affichage facile (positionnement du texte au pixel près, etc.) parce que la lecture du fichier est complètement entrelacée avec l'affichage. Il n'y a pas de lecture vers une représentation mémoire indépendante du reste.

Puis il y a les autres problèmes, comme l'absence totale de commentaires, les noms de variable à une ou deux lettres, qui font qu'on ne peut même pas comprendre ce que le code est censé faire, surtout sans échantillon de fichier.

L'usage incorrect de feof() complique encore les choses, parce qu'on a facilement peur que le code marche "par accident" et que le corriger casse en fait son fonctionnement...

Enfin, voici le code ré-indenté, avec des "stubs" des fonctions graphiques pour que ça compile sous Windows (j'ai utilisé Visual Studio, mais ça devrait aussi pouvoir compiler sous Code::Blocks). J'ai ajouté des commentaires sur les accolades fermantes pour qu'on voie plus facilement à quoi ça fait référence.
J'ai aussi changé les short en int parceque sinon j'avais des warnings de conversion presque partout, à cause des promotions d'entiers sur les opérations mathématiques. Mais si ça a une influence sur le déroulement du code, ça veut dire qu'il était encore pire que je pensais.

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
25
26
27
28
29
30
31
32
33
34
35
36
37
38
39
40
41
42
43
44
45
46
47
48
49
50
51
52
53
54
55
56
57
58
59
60
61
62
63
64
65
66
67
68
69
70
71
72
73
74
75
76
77
78
79
80
81
82
83
84
85
86
87
88
89
90
91
92
93
94
95
96
97
98
99
100
101
102
103
104
105
106
107
108
109
110
111
112
113
114
115
116
117
118
119
120
121
122
123
124
125
126
127
128
129
130
131
132
133
134
135
136
137
138
139
140
#include <stdio.h>
#include <windows.h>
 
void _settextposition(int x, int y) { printf("_settextposition(%d, %d)\n", x, y); }
void _outtext(char const *s) { printf("_outtext(%s)\n", s); }
void _setcliprgn(int a, int b, int c, int d) { printf("_setcliprgn(%d, %d, %d, %d)\n", a, b, c, d); }
void _clearscreen(char const *s) { printf("_clearscreen(%s)\n", s); }
void _settextcolor(int c) { printf("_settextcolor(%s)\n", c); SetConsoleTextAttribute(GetStdHandle(STD_OUTPUT_HANDLE), (WORD)c); }
void _setpixel(int x, int y) { printf("_setpixel(%d, %d)\n", x, y); }
void _setcolor(int c) {  printf("_setcolor(%s)\n", c);  }
 
#define _GVIEWPORT "_GVIEWPORT"
 
void fload()
{
	int a=0x11,b=0x10,c=15,cpt,sw=0,dx,dy,tes=0,test=0,nd=0,nde=0;
	fpos_t  posi;
	int x,xx,rx,y,yy,ty,xy,ry,np,ex,ey;
	FILE *fichier;
	char ligne[FILENAME_MAX];
	np=0;
	_settextposition(4,71);
	_outtext("        ");
	_settextcolor(15);
	_settextposition(27,04);
 
	_outtext("8 LETTRES MAX.");
	_settextposition(28,04);
	_outtext("Entrez le nom du fichier :");
	_settextcolor(11);
	fgets(ligne, FILENAME_MAX, stdin);
	_setcliprgn(1,1,478,478);
	_clearscreen(_GVIEWPORT);
	_settextposition(4,71);
	printf("%s",ligne);
 
	if((fichier=fopen(ligne,"r"))==NULL)
	{
		_settextposition(28,10);
		_outtext("Impossible d'ouvrir le fichier");
		return;
	}
	rewind(fichier);
 
	while(!feof(fichier))
	{
		xy=fgetc(fichier);
		for ( ; sw!=1 ; xy=fgetc(fichier) )
			/****************************************************************************
                        RECHERCHE & CALCUL DE LA MATRIX
                        ****************************************************************************/
			if(xy==a)sw=1;
		if(xy==a){}
		else
		{
			for (cpt=1 ; !feof(fichier) && xy!=a ; cpt++)
			{
				if (cpt==1)
				{
					x=xy;
					if(x<127)xx=x+128;
					else xx=x-128;
				}
				else if (cpt==2)
				{
					y=xy;
					if(y<127)yy=y+128;
					else yy=y-128;
					ty=255-yy;
					rx=xx*1.8710937;
					ry=ty*1.8710937;
					np=np+1;
					nd=nd+1;
					_settextposition(21,71);
					printf("%X",x);
					_settextposition(21,77);
					printf("%X",y);
					_settextposition(20,71);
					printf("%d",nd);
					_setcolor(c);
					_setpixel(rx,ry);
				}
				else if (cpt>=3)
				{
					if(xy==0x10)
					{
						xy=fgetc(fichier);
						if(xy==0x80)
						{
							_settextposition(8,71);
							printf("OPEN ");
							c=15;
							xy=fgetc(fichier);
						}
						if(xy==0x00)
						{
							_settextposition(8,71);
							printf("CLOSE");
							c=10;
							xy=fgetc(fichier);
						}
					}
					if(xy==00) {}
					else
					{
						dx=xy/b;
						dy=xy%b;
						ex=dx-8;
						ey=dy-8;
						xx=xx+ex;
						yy=yy+ey;
						ty=255-yy;
						rx=xx*1.8710937;
						ry=ty*1.8710937;
						np=np+1;
						nde=nde+1;
						_settextposition(23,71);
						printf("   ");
						_settextposition(23,77);
						printf("   ");
						_settextposition(23,71);
						printf("%X",dx);
						_settextposition(23,77);
						printf("%X",dy);
 
						_settextposition(22,71);
						printf("%d",nde);
						_setcolor(c);
						_setpixel(rx,ry);
					}
				} //successive ifs on cpt
				xy=fgetc(fichier);
				_settextposition(6,71);
				printf("%d",np);
			} //inner for
			if(feof(fichier)); //this does nothing!
		} //else of the big if
	} //outer while
	fclose(fichier);
}

**Orion2** · 20/12/2018, 17h39

Bonjour,
Déjà un grand merci de te penchez sur le cas de lecture du fichier touts se que je sais t’envoie ses le PDF en pièce jointe avec je pense un peux toutes les explications, le fichier source est déjà dans la discutions.
Dans le PDF tu trouveras, les explications, l’écran de sortie et le fichier Hexa qu’il a lu pour afficher sa a l’écran.
De mon coter j’essaye sur Code::Blocks j’arrive a recréé un peux le concept mais je n’arrive pas a afficher les variable : type les nombres de points par exemple, ou entré le nom de fichier il est vrai que depuis 93 pas mal de chose on changer.
Du genre au lieu de printf(%d,pts) ; maintenant c’est outtext(posx,posy,var) ;
Si au cas où je peux toujours t’envoie mon mail perso comme cela des que j’ai le tient je serais t’envoie un fichier Hexa que je ne sais pas mettre en pièce joindre.
Bon je continue avec acharnement pour trouver l’astuce des syntaxes dans graphics.h.

Encore Merci.
Serge

**Orion2** · 21/12/2018, 06h39

Bonjour,
Je suis un peux dégouter, hier toutes l’après midi pas de problème avec Code ::Blocks j’ai oser éteindre mon pc le soir quand je l’ai rallumé avec le petit soft que je testais voila la good news « Error » :

||=== Build: Debug in digittest (compiler: GNU GCC Compiler) ===|
C:\Program Files\CodeBlocks\MinGW\bin\..\lib\gcc\mingw32\5.1.0\..\..\..\libmingw32.a(main.o):main.c:(.text.startup+0xa7)||undefined reference to `WinMain@16|
||error: ld returned 1 exit status|
||=== Build failed: 2 error(s), 0 warning(s) (0 minute(s), 2 second(s)) ===|

Et avec n’importe quelle source que je teste ???.
je ne suis pas du genre a abandonner mais la ses Windows dépasse l’entendement. J’irais presque a dire vive le Ms Dos.
De plus j’ai du rallumer mon portable avec le bon vieux XP qui est plus stable que toute les versions de Windows qui on suivit, car sur les autres version il ne veux même pas compiler ??.
Si par le plus grand des hasard tu a une idée elle est la bien venue.

Merci.

Serge

Conversion C++ 1990 à Dev C++

C++

Vue hybride

Discussions similaires

Partager

Partager