Discussion :

[mitherkiller]

Bonjour alors voila j'ai deux questions en 1
Alors dabord je n'ai vraiment pas compris a quoi servé un debugger ??(
si c bien ça sa sert a remplaçer et corriger automatiquement des faute par exemple printfff au lieu de printf)?? et si oui su dev comment faire pour que ça corrige un code source.


2 question
alors voila mon programme je débute un:

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 main()
{
	long resultat, nombre1, nombre2;
	printf("Bonjour, voiçi un programme dos pour faire des multiplications ");
 
	printf("Vueillez entrez un nombre");
	scanf("%ld", &nombre1"); // içi la groose ligne rouge
 
	printf("vueillez entrez un deuxième nombre ");
 
	scanf("%ld", &nombre2);
 
	resultat = nombre1 * nombre2;
	printf(" %ld + %d = %ld \n", nombre1, nombre2, resultat);
 
	return 0;
}

et la sur code blocks il me mets une grosse ligne rouge sur scanf("%ld", &nombre1");
Bonvoila merci

[gauguin]

Tu as tout simplement oublié une guillemet après &nombre1 dans ton scanf.

Code :

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

1
2
 
scanf("%ld", &nombre1"); // içi la groose ligne rouge

Par ailleurs je crois savoir que certains compilateurs sont sensibles aux fautes d'orthographes

Sinon bravo, ça marche.

[spawntux]

Oublié ou rajouté une guillemet en trop il me semble non ?

[Trap D]

Oublié ou rajouté une guillemet en trop il me semble non ?

Ça dépend comment on l'entend

[David.Schris]

Oublié ou rajouté une guillemet en trop il me semble non ?

Tu l'as oublié là, donc il est en trop (car sans son jumeau).

[gauguin]

je voulais dire qu'il y a une guillemet de trop, celle qui est après &nombre1.

[David.Schris]

[...] Par ailleurs je crois savoir que certains compilateurs sont sensibles aux fautes d'orthographes [...]

Et d'autres aux fautes de grammaire (<- sans "s").

[David.Schris]

[...] 1
Alors dabord je n'ai vraiment pas compris a quoi servé un debugger ??(
si c bien ça sa sert a remplaçer et corriger automatiquement des faute par exemple printfff au lieu de printf)??[...]

Non, cela serait trop facile.
Lis ça.

[Thierry Chappuis]

Salut,

Voici quelques petits commentaires sur ton programme...

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
#include <stdio.h> 
#include <stdlib.h>   
 
/* -TC- La fonction main() ne possède que deux formes valides en C: 
    int main(void) ou int main(int argc, char *argv[]) */
int main(void) 
{  
    /* -TC- Pour des questions de lisibilité, je préfère une déclaration par ligne*/
    long nombre1;
    long nombre2; 
    long resultat;
 
    printf("Bonjour, voici un programme console pour faire des multiplications\n");     
    printf("Veuillez entrez un nombre: "); 
    /* -TC- Il est nécessaire ici de vider le tampon pour s'assurer
        que l'invite sera affichée */
    fflush(stdout);
    scanf("%ld", &nombre1); 
 
    printf("veuillez entrez un deuxième nombre: ");   
    /* Ici aussi, il faut vider le tampon */
    fflush(stdout);
    scanf("%ld", &nombre2);   
 
    resultat = nombre1 * nombre2; 
    printf("%ld + %ld = %ld\n", nombre1, nombre2, resultat);   
 
    return 0; 
}

Maintenant, ce code fonctionne, mais dépend de la bonne volonté de l'utilisateur qui peut très bien entrer n'importe quoi. Il y a deux solutions pour sécuriser les entrées utilisateur.

• Se passer de scanf() (j'utilise uniquement fgets + strtol/strtod/sscanf)
• Apprendre à utiliser scanf() et vérifier sa valeur de retour.

Pour plus d'infos sur scanf(), cette lecture est très instructive: http://xrenault.developpez.com/tutoriels/c/scanf/

En ce qui concerne la sécurisation de ton code, voici un exemple simple. J'ai encapsulé le code qui se répétait dans les fonctions m_saisir_long() et m_stdin_vider_tampon():

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
#include <stdio.h>
#include <stdlib.h>
 
/* Vide les caractères résiduels contenus dans le tampon de stdin */
void m_stdin_vider_tampon(void)
{
    scanf("%*[^\n]");
    getchar();
}
 
/* Le code de saisie d'un long est utiliser deux fois, on peut le mettre
    dans une fonction */
long m_saisir_long(char const *message)
{
    long nombre;
 
    printf("%s", message);
    fflush(stdout);
    while (scanf("%ld", &nombre) != 1)
    {
        /* On vide le tampon */
        m_stdin_vider_tampon();
 
        printf("Erreur: %s", message);
        fflush(stdout);
    }
 
    /* On vide le tampon et on retourne nombre */
    m_stdin_vider_tampon();
 
    return nombre;
}
 
int main(void)
{
    long nombre1;
    long nombre2;
    long resultat;
 
    printf("Bonjour, voici un programme console pour faire des multiplications\n");
    nombre1 = m_saisir_long("Veuillez saisir un nombre: ");
    nombre2 = m_saisir_long("Veuillez saisir un 2ème nombre: ");
 
    resultat = nombre1 * nombre2;
    printf("%ld * %ld = %ld\n", nombre1, nombre2, resultat);
 
    return 0;
}

Thierry

[GG_L-aspi]

parfait
a part la fin, je mettrais un system("PAUSE"); avant le return sinon la fenêtre DOS se ferme après que l'on insert la deuxième valeur et qu'on valide.

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
#include <stdio.h>
#include <stdlib.h>
 
/* Vide les caractères résiduels contenus dans le tampon de stdin */
 
void m_stdin_vider_tampon(void)
{
     scanf("%*[^\n]");
     getchar();
}
 
/* Le code de saisie d'un long est utiliser deux fois, on peut le mettre
dans une fonction */
 
long m_saisir_long(char const *message)
{
     long nombre;
     printf("%s", message);
     fflush(stdout);
     while (scanf("%ld", &nombre) != 1)
     {
           m_stdin_vider_tampon();           /* On vide le tampon */
           printf("Erreur: %s", message);
           fflush(stdout);
     }
     m_stdin_vider_tampon();     /* On vide le tampon et on retourne nombre */
     return nombre;
}
 
int main(void)
{
    long nombre1;
    long nombre2;
    long resultat;
 
    printf("Bonjour, voici un programme console pour faire des multiplications\n");
 
    nombre1 = m_saisir_long("Veuillez saisir un nombre: ");
    nombre2 = m_saisir_long("Veuillez saisir un 2ème nombre: ");
 
    resultat = nombre1 * nombre2;
 
    printf("%ld * %ld = %ld\n", nombre1, nombre2, resultat);
 
    system("PAUSE");
 
    return 0;
}

[Thierry Chappuis]

J'ai jamais utilisé de system("PAUSE")... L'auteur de ce fil semble utiliser Code::Blocks comme EDI. Il n'est pas utile de recourir à l'instruction system("PAUSE") avec cet environnement de développement. D'ailleurs, ce n'est absolument pas portable, et cette pratique est déconseillée. Si, pour des raisons purement pratiques, j'ai besoin d'éviter que la console Windows (N.B. et pas DOS) ne disparaisse dès la fin de mon programme, j'utilise de préférence la fonction getchar(), qui fait partie de la bibliothèque standard du C.

Thierry

[Emmanuel Delahaye]

parfait
a part la fin, je mettrais un system("PAUSE"); avant le return sinon la fenêtre DOS se ferme après que l'on insert la deuxième valeur et qu'on valide.

Huh ? Quelle 'fenêtre DOS' ? Sous Windows XP, la console n'est certainement pas une 'fenêtre DOS...'

Et le problème évoqué n'apparait qu'avec l'ancien IDE Dev-C++. Avec Code::Blocks, une pause est ajoutée automatiquement par l'IDE.

En ce qui concerne l'application elle-même, il est normal que quand elle se termine, la console se ferme, si on n'a pas lancée celle-ci au préalable.

Bref, avant de mettre n'importe quoi dans le code, il faut commencer par utiliser les bons outils et utiliser correctement son système.