Discussion :

[~Brouette~]

Bonjour, je tient a signaler que je suis un bon vrai débutant avec tout ce que celà implique

Voici mon prog : (J'utilise Dev-C++ V4.9.9.2)

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
#include <stdio.h>
#include <stdlib.h>
 
int Somme;
 
int main(void)
{
              int X;
              int Y;
              char Buff[10];
              printf("Entrez un premier nombre entier : ");
              gets(Buff);
              X = atoi(Buff);
              printf("Entrez un second nombre entier : ");
              gets(Buff);
              Y = atoi(Buff);
              Somme = X + Y;
              printf("La somme de %d et de %d est %d\n",X,Y,Somme);
              return 0;
}

Alors quand je lance le programme par F9, une fenetre MS-DOS s'ouvre et me demande un premier nombre. J'ecris un chiffre au hasard, disons 5, je fais entrée pour valider; Ensuite on me demande un deuxieme chiffre, j'ecris au hasard 4, et à ce moment quand je fais entrée, la fenetre se ferme. Je n'ai pas la somme qui s'affiche...

Quelqu'un à une idée du problème ???

Merci par avance

[Emmanuel Delahaye]

Voici mon prog : (J'utilise Dev-C++ V4.9.9.2)
<...>
Alors quand je lance le programme par F9, une fenetre MS-DOS s'ouvre et me demande un premier nombre. J'ecris un chiffre au hasard, disons 5, je fais entrée pour valider; Ensuite on me demande un deuxieme chiffre, j'ecris au hasard 4, et à ce moment quand je fais entrée, la fenetre se ferme. Je n'ai pas la somme qui s'affiche...

Ce problème est bien connu et se résout ainsi : Désinstalle Dev-C++ et installe Code::Blocks à la place.

http://www.codeblocks.org/

A part ça, le code est correct, mais dangereux. Et attention à ton vocabulaire...

• Il n'y a pas de 'fenêtre MS-DOS'. Il y a une 'console Windows'.
• Les cas où les variables globales sont utiles sont très rares. Déplace la dans le main().
• Il ne faut pas utiliser gets() qui est un bug bien connu. Utilise fgets() à la place.

[Médinoc]

Code::Blocks a de nombreux avantages sur Dev-C++, mais aussi quelques inconvénients.

Le workaround pour Dev-C++ se trouve dans la FAQ :
http://c.developpez.com/faq/c/?page=...onsole_windows

[Emmanuel Delahaye]

Code::Blocks a de nombreux avantages sur Dev-C++, mais aussi quelques inconvénients.

Comme ?

[Médinoc]

Comme ?

Je n'ai plus utilisé Code::Blocks depuis un moment, mais j'avais vu:

• Le refus de compiler des fichiers solitaires (appelle ça de la paresse si tu veux, mais seul Dev le permet)
• L'arborescence du projet est grosse et hideuse
• L'absence de séparation des flags supplémentaires entre C et C++ (et moi, j'ai pas mal de projets mixtes (mais un fichier source passe par un seul compilo, hein))
• Des problèmes bizarres aussi dans la gestion des options (mais c'est assez flou dans ma mémoire)

[Emmanuel Delahaye]

* Le refus de compiler des fichiers solitaires (appelle ça de la paresse si tu veux, mais seul Dev le permet)

Normal, Mauvaise Pratique

* L'arborescence du projet est grosse et hideuse

Bah, elle est pas pire que celle de Visual C++... Et j'aime bien l'organisation automatique des fichiers par sources/headers/others. Et puis on peut avoir plusieurs projets sous la main, ce qui est très pratique.

* L'absence de séparation des flags supplémentaires entre C et C++ (et moi, j'ai pas mal de projets mixtes (mais un fichier source passe par un seul compilo, hein))

C'est possible, en effet... Est-ce vraiment génant ?

* Des problèmes bizarres aussi dans la gestion des options (mais c'est assez flou dans ma mémoire)

Attendons d'en savoir plus...

[~Brouette~]

Merci pour ta réponse

Malheureusement je ne peu desinstaller dev-c++, je prend des cours par le cnam, et tous les "éleves" doivent utiliser ce logiciel pour faciliter les resolutions de problemes...

J'ai remplacé gets par fgets dans mon code et du coup j'ai des erreurs qu is'affichent, le logiciel m'ouvre la bibliotheque stdio.h et me souligne cette ligne :

Code :

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

_CRTIMP char* __cdecl	fgets (char*, int, FILE*);

Sinon, pourquoi ce code est-il dangereux ?

Merci pour vos reponses

[Emmanuel Delahaye]

Sinon, pourquoi ce code est-il dangereux ?

A cause de gets(). Il n'y a aucun moyen portable d'utiliser gets() sans créer un trou de sécurité dans le programme, étant donné qu'il n'y a pas de contrôle de taille. On peut toujours déborder du tableau de char et provoquer un comportement indéfini. Il suffit d'entrer une chaine assez longue...

Cette faille bien connue est à l'origine de l'exploit dit du Great Internet Worm.

[Thierry Chappuis]

Salut,

Tout d'abord, quelques remarques générales sur ton programme:
1. Utiliser gets(), c'est très mal d'un point de vue sécurisation du code. On ne doit pas utiliser gets(), mais fgets(char *s, int n, FILE *file).
2. Pourquoi utilises-tu une variable globale pour Somme. Il n'y a aucune raison de le faire ici. D'ailleurs, on essaie également de ne jamais utiliser de variables globales. Les rares cas où tu ne peux pas faire autrement doivent être justifiés.
3. Utilise plutôt strtol() que atoi().

Ton programme sinon semble correct. Essaie de mettre de mettre à la fin de ton programme avant l'instruction return:

Code :

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

1
2
 
system("PAUSE");

C'est un problème assez classique sous windows. La console se ferme automatiquement dès la fin du programme. Si tu veux résoudre ce problème, utilise Code::blocks à la place de Dev-C++ (dont le développement a maintenant été arrete).

Thierry

[Thierry Chappuis]

D'autres ont été plus rapide que moi, cela m'apprendra à retourner 50 fois la langue dans ma bouche pour ne dire trop de bétises, et de relire 3x mon message avant de poster.

En ce qui concerne Dev-C++ comparé à Code::Blocks, mes souvenirs sous Windows sont lointains, mais je trouve Code::Blocks très bien.

Thierry

[Thierry Chappuis]

fgets te permet de lui indiquer la taille de Buff en deuxième paramètre, ce qui évite les débordement de tampon. Ainsi, fgets lira, si l'entrée de l'utilisateur est trop grande, les 9 premiers caractères sur le flux que tu lui indique en 3ème paramètre et placera un caractère nul à la fin de la chaîne.

Un utilisation correcte de fgets dans ton cas est:

Code :

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

fgets(Buff, 10, stdin);

Thierry

[~Brouette~]

fgets te permet de lui indiquer la taille de Buff en deuxième paramètre, ce qui évite les débordement de tampon. Ainsi, fgets lira, si l'entrée de l'utilisateur est trop grande, les 9 premiers caractères sur le flux que tu lui indique en 3ème paramètre et placera un caractère nul à la fin de la chaîne.

Un utilisation correcte de fgets dans ton cas est:

Code :

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

fgets(Buff, 10, stdin);

Thierry

Cela fonctionne très bien effectivement...

Y'a du boulot....

[Emmanuel Delahaye]

Un utilisation correcte de fgets dans ton cas est:

Code :

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

fgets(Buff, 10, stdin);

Sous réserve que Buff soit un tableau (et non un pointeur), on préfèrera écrire

Code :

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

fgets(Buff, sizeof Buff, stdin);

ce qui rend le code 'auto-dém*rdant'.

[~Brouette~]

Ton programme sinon semble correct. Essaie de mettre de mettre à la fin de ton programme avant l'instruction return:

Code :

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

1
2
 
system("PAUSE");

Thierry

Merci Thierry, tout fonctionne correctement maintenant
Merci aussi à tous ceux qui m'ont repondu, je pense que je vais devoir apprendre avec 2 logiciels en parrallele...