Pointeurs, fonctions pour calculette

Version imprimable

Bonjour, je fais appel à votre aide car je créer actuellement un programme de calculatrice en C, la contrainte étant d'utiliser des pointeurs et différentes fonctions. Mon programme SEGFAULT, en entrant dans un switch ou je fais appel à ces mêmes fonctions.

Sans forcément copier tout mon programme, je vous montre les principaux éléments essentiel à mon avis à la compréhension.

Code:

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
/** Déclaration des variables et pointeurs **/
int *nbr1 = NULL;
nbr1 = malloc(sizeof(int));
int *nbr2 = NULL;
nbr2 = malloc(sizeof(int));
int *resultat;
resultat = malloc(sizeof(int));
 
/** Boucle switch pour appeller mes fonctions **/
switch (choix2)
        {
        case 1:
            addition (resultat, nbr1, nbr2);
            printf("%d", resultat);
            break;
 
        case 2:
            soustraction (resultat, nbr1, nbr2);
            printf("\n  \4    ----     %d - %d = %d      ---- \n", nbr1, nbr2, resultat);
            break;
 
        case 3:
            multiplication (resultat, nbr1, nbr2);
            break;
 
 
/** Dans mon calcul.c voici la déclaration de ces fonctions **/
void addition (int resultat, int *nbr1, int *nbr2){
        resultat = *nbr1 + *nbr2;
        }
 
int soustraction (int resultat, int *nbr1, int *nbr2){
        resultat = (*nbr1) - (*nbr2);
        }
void multiplication (int *resultat, int *nbr1, int *nbr2){
        (*resultat) = (*nbr1) * (*nbr2);
        }
 
 
/** Prototype (header) **/
void addition(int resultat, int *nbr1, int *nbr2);
int soustraction (int *nbr1, int *nbr2, int resultat);
void multiplication (int *resultat, int *nbr1, int *nbr2);

Vous pourrez voir que j'ai essayer de plusieurs manière différentes, sans succès :( .
Je vous remercie d'avance pour votre aide.

23/10/2014, 18h16
gerald3d

Bonjour.

Dans les fonctions addition(); et soustraction(); le paramètre resultat doit être déclaré en tant que pointeur et non en tant que variable.

Petite question au passage : est-ce une contrainte de l'exercice d'allouer dans le tas tes différentes variables ?

Merci de ton aide, cependant cela ne marche toujours pas.

ps : le fait d'allouer mes variables ne fait pas partie de la contraintes, cependant je ne pense pas que ça soit l'origine du problème.
Mais encore je n'en suis pas sûr. Je précise que je libère l'espace mémoires utilisé avec free dans mon programme.

Code:

1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
 
    /** Tant que choix1 différent de 1 ou 2, on recommence **/
    while (choix1 != 1 && choix1 != 2) {
        printf("\n  !!! ERROR !!! Choisissez une fonction valide \2 \n\n\n");
        printf("\n\n Souhaitez-vous toujours utiliser la belle calculatrice ?\n Ou partir ? \n\n 1: Rester et Calculer \n 2: Exit \n\n");
        scanf ("%d", &choix1);
        }
 
            /** Quitte si choix1 égal à 2 **/
            if (choix1 == 2){ // Vérification de la demande de l'utilisateur pour, rester ou quitter //
                free(nbr1);
                free(nbr2);
                free(resultat);
                printf("\n    Sayonara !! \2 \n\n");
                return EXIT_SUCCESS;
                }

Pour revenir à l'histoire des pointeurs, j'ai essayé de ne pas déclarer la variable résultat en tant que pointeur (sans succès encore une fois).
Je suis pas sûr que les modérateurs apprécieront, mais étant donné que je penche dessus depuis toute la journée, que j'ai l'impression de ne plus rien comprendre et surtout que l'erreur peut venir de n'importe où; je me permet de joindre mes 3 fichiers, car si j'indique par bout de code on peut y passer la soirée :ptdr:.

Fichier main.c :

Code:

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
 
#include "calcul.h"
 
int main()
{
int choix1;
int choix2;
 
int *nbr1 = NULL;
nbr1 = malloc(sizeof(int));
int *nbr2 = NULL;
nbr2 = malloc(sizeof(int));
int *resultat;
resultat = malloc(sizeof(int));
 
    printf(" Bienvenue Dans La Calculatrice !! \n");
 
    do {
            /** Demande la confirmation de l'utilisateur de rester ou de quitter **/
    printf("\n\n Souhaitez-vous toujours utiliser la belle calculatrice ?\n Ou partir ? \n\n 1: Rester et Calculer \n 2: Exit \n\n");
    scanf ("%d", &choix1);
 
    /** Tant que choix1 différent de 1 ou 2, on recommence **/
    while (choix1 != 1 && choix1 != 2) {
        printf("\n  !!! ERROR !!! Choisissez une fonction valide \2 \n\n\n");
        printf("\n\n Souhaitez-vous toujours utiliser la belle calculatrice ?\n Ou partir ? \n\n 1: Rester et Calculer \n 2: Exit \n\n");
        scanf ("%d", &choix1);
        }
 
            /** Quitte si choix1 égal à 2 **/
            if (choix1 == 2){ // Vérification de la demande de l'utilisateur pour, rester ou quitter //
                free(nbr1);
                free(nbr2);
                free(resultat);
                printf("\n    Sayonara !! \2 \n\n");
                return EXIT_SUCCESS;
                }
 
 
    printf("\n Choisissez Le Calcul A Effectuer : \n");
    // Menu du programme de calcul //
 
 
    /** Menu choisir opération **/
    printf("\n   1: \4 \4 Addition \1\n   2: \4 \4 Soustraction \1\n   3: \4 \4 Multiplication \1\n   4: \4 \4 Division \1\n   5: \4 \4 Modulo \1\n   6: \4 \4 HELP \1 \n\n");
    scanf ("%d", &choix2);
 
        /** Tant que choix2 invalide, on recommence **/
        while (choix2 == 0 || choix2 >= 7){
            printf("\n  !!! ERROR !!! Choisissez une fonction valide \2 \n\n\n");
            printf("\n   1: \4 \4 Addition \1\n   2: \4 \4 Soustraction \1\n   3: \4 \4 Multiplication \1\n   4: \4 \4 Division \1\n   5: \4 \4 Modulo \1\n   6: \4 \4 HELP \1 \n\n");
            scanf ("%d", &choix2);
            } /** Fin du While **/
 
 
        /** Début de la saisie des différentes valeurs si choix différents de HELP **/
 
            if(choix2 != 6){
                printf("\n Entrez un nombre\n\n");
                scanf("%d" , &nbr1);
                printf("\n Entrez un deuxieme nombre\n\n");
                scanf("%d", &nbr2);}
 
 
 
        switch (choix2)
        {
        case 1:
            addition (resultat, nbr1, nbr2);
            printf("%d", resultat);
            break;
 
        case 2:
            soustraction (resultat, nbr1, nbr2);
            printf("\n  \4    ----     %d - %d = %d      ---- \n", nbr1, nbr2, resultat);
            break;
 
        case 3:
            multiplication (resultat, nbr1, nbr2);
            break;
 
        case 4:
            division (nbr1, nbr2, resultat);
            printf("\n  \4    ----     %d / %d = %d      ---- \n", nbr1, nbr2, resultat);
            break;
 
        case 5:
            modulo (nbr1, nbr2, resultat);
            printf("\n  \4    ----     %d % %d = %d      ---- \n", nbr1, nbr2, resultat);
            break;
 
        case 6: /** HELP **/
                printf("\n\n Dans ce programme de calculatrice vous ne pouvez calculer que 2 nombres a\n la fois. \n Pour cela, vous disposez de deux menu : \n\n 1- Le premier vous permet de quitter aisement le programme en entrant la\n touche approprie, dans ce cas il s'agit de la touche (2),\n Et inversement si vous souhaitez continuez votre utilisation (1). \n\n 2- Le deuxieme menu vous demande de choisir entre differents moyen de calcul,\n validez votre choix de la meme maniere que le menu precedent. \n\n Quand vous aurez valider votre mode de calcul souhaite, le programme vous\n demandera de saisir de valider deux valeurs pour effecter le calcul désire. \n Une fois ceci fait, vous serez en mesure de connaitre le resultat de votre\n operation. \n A partir de la, recommencez cette methode aussi longtemps que necessaire. \n\n\n Sachez que si vous rentrer une valeur incorrecte dans les menus, le programme\n vous permettra toujours de recommencer tant que le choix reste invalide. \n\n La societe G&B Technologies vous souhaite une agreable utilisation\n de ce programme, et vous invite a decouvrir de nouvelles applications\n sur son site web :\n <a href="https://g&btechnologies/nouveautes.php" target="_blank">https://g&btechnologies/nouveautes.php</a> \n\n Createur : \n Guillaume BONHOMMEAU - Etudiant a l'Exia CESI de St-Nazaire - France\n \4 \4 \4 La Belle Calculatrice -- Copyright@2014 -- Tous Droits Reserves \4 \4 \4\n\n\n");
                break;
 
        default : /** Si jamais l'utilisateur parviens à entrer autre chose de non valide **/
            printf("\n  !!! ERROR !!! Choisissez une fonction valide \2 \n\n\n");
 
            }
 
        printf("Resultat : %d\n", *resultat);
 
 
    } while (choix1 != 2);
 
    return EXIT_SUCCESS;
}

Fichier calcul.c :

Code:

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>
#include "calcul.h"
 
void addition (int *resultat, int *nbr1, int *nbr2){
        resultat = *nbr1 + *nbr2;
        }
int soustraction ( int *resultat, int *nbr1, int *nbr2){
        resultat = (*nbr1) - (*nbr2);
        }
void multiplication (int *resultat, int *nbr1, int *nbr2){
        (*resultat) = (*nbr1) * (*nbr2);
        }
int division (int *nbr1, int *nbr2, int resultat){
        resultat = (*nbr1) / (*nbr2);
        }
int modulo (int *nbr1, int *nbr2, int resultat){
        resultat = (*nbr1) % (*nbr2);
        }

Fichier calcul.h :

Code:

1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
 
#ifndef __include_fichier_h__
#define __include_fichier_h__
 
#include <stdio.h>
#include <stdlib.h>
#include <math.h>
 
void addition(int *resultat, int *nbr1, int *nbr2);
int soustraction (int *resultat, int *nbr1, int *nbr2);
void multiplication (int *resultat, int *nbr1, int *nbr2);
int division (int *nbr1, int *nbr2, int resultat);
int modulo (int *nbr1, int *nbr2, int resultat);
 
 
#endif /* #ifndef __include_fichier_h__ */

Si dans le switch mes conditions ne sont pas toutes les mêmes c'est justement parce que je testé beaucoup de possibilités.
Encore désolé pour la longueur du message :calim2:, mais aussi merci de m'aider :) .

23/10/2014, 19h15
gerald3d

Avancons par petits pas ;).

La fonction scanf(); attend un pointeur sur une variable. Tu lui transmets **nbr1 et non *nbr1 (idem pour nbr2).

Pourquoi la fonction soustraction(); renvoie un int ?

En effet, je viens de changer la fonction scanf() :
Code:

1 2 3 4 5 if(choix2 != 6){ printf("\n Entrez un nombre\n\n"); scanf("%d" , *nbr1); printf("\n Entrez un deuxieme nombre\n\n"); scanf("%d", *nbr2);}
Cependant ça ne marche toujours pas. La saisie des nombres fonctionne comme toujours, mais au moment d'afficher le résultat le programme plante.

Pour ce qui est du type int de la fonction soustraction(); je dois bien avouer que je ne sais plus trop qu'elle type de fonction utilisé dans ce cas précis, pendant un moment je pensais que utilisé un void résoudrais mes problèmes ... mais bon, honnêtement je besoin d'un nouveau regard la dessus, si tu étais à ma place qu'elle type de fonction utiliserais tu ? et pourquoi ?

23/10/2014, 19h46
gerald3d

Pour ce qui est du type de fonction voila l'idée générale.

La fonction doit-elle retournée une ou plusieurs informations ?

Si une seule information est à retourner alors elle renvoie cette information par l'intermédiaire d'un return;. Dans le cas contraire on utilise les pointeurs transmis en paramètre.

Une autre question à se poser. Les paramètres transmis peuvent-ils être modifiés à l'interieur de celle-ci. Si ce n'est pas le cas on utilise le mot réservé const devant leur déclaration. Ainsi, si on fait "une fausse manip" le compilateur va hurler :).

Donc pour la fonction soustraction(); je ferais ceci :

Code:

int soustraction (const int nbr1, const nbr2);

Si la transmission par pointeur est obligatoire (libellé de l'exercice) alors

Code:

void soustraction(int *resultat, const int nbr1, const int nbr2);

Maintenant pour le scanf(); nbr1 et nbr2 sont déjà des pointeurs. l'* est de trop il me semble ;)...

Mon dieu qu'est-ce que ça peut faire du bien ! Merci beaucoup beaucoup beaucoup

Code:

1
2
3
4
 for (i=0; i<=99; i++){
printf("\n   \4 MERCI BEAUCOUP Gerald3D !!!  \1\1\1");
}
return bonheur=100000000;

Bref, tu m'a compris ... Mais du coup, ceci confirme l'hypothèse comme quoi je suis le roi des erreurs à la *** (tu m'a compris là aussi ;)) .
J'ai perdu une journée à cause d'un vieux opérateur bit à bit "&" ...
Et vu que je suis sur windows (code sur CodeBlocks), je peux toujours rêver pour que le débugger m'aide ...

Tes conseils m'ont été très utile, y compris ceux sur les fonctions, portent à réflexion.
Je vais mettre mon programme correct au complet pour de futur intéressé. Bonne chance pour la suite et à plus sur le forum :D !!

Fichier main.c :

Code:

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
 
#include "calcul.h"
 
int main()
{
int choix1;
int choix2;
 
int *nbr1 = NULL;
nbr1 = malloc(sizeof(int));
int *nbr2 = NULL;
nbr2 = malloc(sizeof(int));
int *resultat;
resultat = malloc(sizeof(int));
 
    printf(" Bienvenue Dans La Calculatrice !! \n");
 
    do {
            /** Demande la confirmation de l'utilisateur de rester ou de quitter **/
    printf("\n\n Souhaitez-vous toujours utiliser la belle calculatrice ?\n Ou partir ? \n\n 1: Rester et Calculer \n 2: Exit \n\n");
    scanf ("%d", &choix1);
 
    /** Tant que choix1 différent de 1 ou 2, on recommence **/
    while (choix1 != 1 && choix1 != 2) {
        printf("\n  !!! ERROR !!! Choisissez une fonction valide \2 \n\n\n");
        printf("\n\n Souhaitez-vous toujours utiliser la belle calculatrice ?\n Ou partir ? \n\n 1: Rester et Calculer \n 2: Exit \n\n");
        scanf ("%d", &choix1);
        }
 
            /** Quitte si choix1 égal à 2 **/
            if (choix1 == 2){ // Vérification de la demande de l'utilisateur pour, rester ou quitter //
                free(nbr1);
                free(nbr2);
                free(resultat);
                printf("\n    Sayonara !! \2 \n\n");
                return EXIT_SUCCESS;
                }
 
 
    printf("\n Choisissez Le Calcul A Effectuer : \n");
    // Menu du programme de calcul //
 
 
    /** Menu choisir opération **/
    printf("\n   1: \4 \4 Addition \1\n   2: \4 \4 Soustraction \1\n   3: \4 \4 Multiplication \1\n   4: \4 \4 Division \1\n   5: \4 \4 Modulo \1\n   6: \4 \4 HELP \1 \n\n");
    scanf ("%d", &choix2);
 
        /** Tant que choix2 invalide, on recommence **/
        while (choix2 == 0 || choix2 >= 7){
            printf("\n  !!! ERROR !!! Choisissez une fonction valide \2 \n\n\n");
            printf("\n   1: \4 \4 Addition \1\n   2: \4 \4 Soustraction \1\n   3: \4 \4 Multiplication \1\n   4: \4 \4 Division \1\n   5: \4 \4 Modulo \1\n   6: \4 \4 HELP \1 \n\n");
            scanf ("%d", &choix2);
            } /** Fin du While **/
 
 
        /** Début de la saisie des différentes valeurs si choix différents de HELP **/
 
            if(choix2 != 6){
                printf("\n Entrez un nombre\n\n");
                scanf("%d" , nbr1);
                printf("\n Entrez un deuxieme nombre\n\n");
                scanf("%d", nbr2);}
 
 
 
        switch (choix2)
        {
        case 1:
            addition (resultat, nbr1, nbr2);
            printf("\n  \4   %d + %d = %d     \1\n", *nbr1, *nbr2, *resultat);
            break;
 
        case 2:
            soustraction (resultat, nbr1, nbr2);
            printf("\n  \4   %d - %d = %d     \1\n", *nbr1, *nbr2, *resultat);
            break;
 
        case 3:
            multiplication (resultat, nbr1, nbr2);
            printf("\n  \4   %d * %d = %d     \1\n", *nbr1, *nbr2, *resultat);
            break;
 
        case 4:
            division (resultat, nbr1, nbr2);
            printf("\n  \4   %d / %d = %d     \1\n", *nbr1, *nbr2, *resultat);
            break;
 
        case 5:
            modulo (resultat, nbr1, nbr2);
            printf("\n  \4   %d %% %d = %d     \1\n", *nbr1, *nbr2, *resultat);
            break;
 
        case 6: /** HELP **/
                printf("\n\n Dans ce programme de calculatrice vous ne pouvez calculer que 2 nombres a\n la fois. \n Pour cela, vous disposez de deux menu : \n\n 1- Le premier vous permet de quitter aisement le programme en entrant la\n touche approprie, dans ce cas il s'agit de la touche (2),\n Et inversement si vous souhaitez continuez votre utilisation (1). \n\n 2- Le deuxieme menu vous demande de choisir entre differents moyen de calcul,\n validez votre choix de la meme maniere que le menu precedent. \n\n Quand vous aurez valider votre mode de calcul souhaite, le programme vous\n demandera de saisir de valider deux valeurs pour effecter le calcul désire. \n Une fois ceci fait, vous serez en mesure de connaitre le resultat de votre\n operation. \n A partir de la, recommencez cette methode aussi longtemps que necessaire. \n\n\n Sachez que si vous rentrer une valeur incorrecte dans les menus, le programme\n vous permettra toujours de recommencer tant que le choix reste invalide. \n\n La societe G&B Technologies vous souhaite une agreable utilisation\n de ce programme, et vous invite a decouvrir de nouvelles applications\n sur son site web :\n https://g&btechnologies/nouveautes.php \n\n Createur : \n Guillaume BONHOMMEAU - Etudiant a l'Exia CESI de St-Nazaire - France\n \4 \4 \4 La Belle Calculatrice -- Copyright@2014 -- Tous Droits Reserves \4 \4 \4\n\n\n");
                break;
 
        default : /** Si jamais l'utilisateur parviens à entrer autre chose de non valide **/
            printf("\n  !!! ERROR !!! Choisissez une fonction valide \2 \n\n\n");
 
            }
 
 
    } while (choix1 != 2);
 
    return EXIT_SUCCESS;
}

Fichier calcul.c :

Code:

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>
#include "calcul.h"
 
void addition (int *resultat, int *nbr1, int *nbr2){
        *resultat = *nbr1 + *nbr2;
        }
int soustraction ( int *resultat, int *nbr1, int *nbr2){
        *resultat = (*nbr1) - (*nbr2);
        }
void multiplication (int *resultat, int *nbr1, int *nbr2){
        (*resultat) = (*nbr1) * (*nbr2);
        }
int division (int *resultat, int *nbr1, int *nbr2){
        *resultat = (*nbr1) / (*nbr2);
        }
int modulo (int *resultat, int *nbr1, int *nbr2){
        *resultat = (*nbr1) % (*nbr2);
        }

Fichier calcul.h :

Code:

1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
 
#ifndef __include_fichier_h__
#define __include_fichier_h__
 
#include <stdio.h>
#include <stdlib.h>
#include <math.h>
 
void addition(int *resultat, int *nbr1, int *nbr2);
int soustraction (int *resultat, int *nbr1, int *nbr2);
void multiplication (int *resultat, int *nbr1, int *nbr2);
int division (int *resultat, int *nbr1, int *nbr2);
int modulo (int *resultat, int *nbr1, int *nbr2);
 
 
#endif /* #ifndef __include_fichier_h__ */