Systéme de contrôle des appels téléphoniques

**mad0210** · 29/03/2016, 13h49

Bonjour,

j'ai besoin d'aide

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
 
#include <stdio.h>
#include <stdlib.h>
#include <string.h>
#define N 100
 
 
int bonNumero(char *numero)
{
    int okLong, okNum;
 
    okLong = strlen(numero) == 9;
    okNum = numero[0] == '7' && (numero[1] == '7' ||
                                 numero[1] == '6' ||
                                 numero[1] == '0' ||
                                 numero[1] == '8');
 
    if (!okLong && okNum)
        printf("Le numero doit comporter 9 chiffres\n");
    if (!okLong && !okNum)
        printf("Le numero doit comporter 9 chiffres et commence par 77 ou 76 ou 70 ou 78\n");
    if (okLong && !okNum)
        printf("Le nummero doit commencer par : 77 ou 76 ou 70 ou 78\n");
 
    return okNum && okLong; 
}
 
int bonNumerocorres(char *numerocorres)
{
    int i, oknumcorres = 0;
    int preO = 5, preT = 5, preE = 5;
    int ind1, ind2, ind3;
    int mat[3][N] ={{772314440, 782314440, 774517070, 784517070, 778868181},
                    {762314440, 764517070, 768868181, 763570521, 765181304}, 
                    {702314440, 704517070, 708868181, 703570521, 705181304}};
    char numero[30];
 
 
    if (oknumcorres == numero)
        printf("Le numero du correspondant doit etre different de votre numero\n");
    if (ind1 == 77 || ind1 == 78)
    {
        for (i = 0; i <= preO; i++)
        {
            if (mat[0][i] == numero)
            {
            oknumcorres = 1;
            i = preO;
            }
        }
        if (oknumcorres == 0)
        printf("Ce numero n'est pas enregistre\n");
    }
    if (ind1 == 76)
    {
        for (i = 0; i <= preT; i++)
        {
            if (mat[1][i] == numero)
            {
            oknumcorres = 1;
            i = preT;
            }
        }
        if (oknumcorres == 0)
        printf("Ce numero n'est pas enregistre\n");
    }
    if (ind1 == 70)
    {
        for (i = 0; i <= preE; i++)
        {
            if (mat[2][i] == numero)
            {
            oknumcorres = 1;
            i = preE;
            }
        }
        if (oknumcorres == 0)
        printf("Ce numero n'est pas enregistre\n");
    }
 
    return oknumcorres;
}
 
 
int main()
{
    int credit, dureecomsouhaite;
    char numero[30], numerocorres[30];
 
    do 
    {
        printf("Entrer votre numero\n");
        scanf("%s", &numero);
    }
    while (!bonNumero(numero));
 
    do
    {
        printf("Entrer le montant de votre credit\n");
        scanf("%d",&credit);
 
        if (credit < 60)
            printf("Le credit doit etre superieur ou egale a 60\n");
    }
    while (credit < 60);
 
    do
    {
        printf("Entrer le numero du correspondant\n");
        scanf("%s", &numerocorres);
    }
    while (!bonNumero(numerocorres) || !bonNumerocorres(numerocorres));
 
    do
    {
        printf("Entrer la duree de communication souhaitee\n");
        scanf("%d", &dureecomsouhaite);    
    }
    while (dureecomsouhaite < 0);
 
 
 
 
 
    return 0;
}

Merci d'avance.

**Obsidian** · 29/03/2016, 13h57

Bonjour,

Quel est le problème ?

**mad0210** · 29/03/2016, 18h12

j'arrive pas à faire le controle sur le numéro du correspondant

**dinobogan** · 30/03/2016, 14h35

Sur les 126 lignes de code postées, dans quelle fonction se situe le problème ?

**picodev** · 30/03/2016, 15h07

Bonjour,
le premier qui t'apporte une aide est ton compilateur …

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
$ gcc -Wall -Wextra -Wmissing-prototypes -c t.c
t.c:8:5: warning: no previous prototype for ‘bonNumero’ [-Wmissing-prototypes]
 int bonNumero(char *numero)
     ^
t.c:28:5: warning: no previous prototype for ‘bonNumerocorres’ [-Wmissing-prototypes]
 int bonNumerocorres(char *numerocorres)
     ^
t.c: In function ‘bonNumerocorres’:
t.c:39:21: warning: comparison between pointer and integer
     if (oknumcorres == numero)
                     ^
t.c:45:27: warning: comparison between pointer and integer
             if (mat[0][i] == numero)
                           ^
t.c:58:27: warning: comparison between pointer and integer
             if (mat[1][i] == numero)
                           ^
t.c:71:27: warning: comparison between pointer and integer
             if (mat[2][i] == numero)
                           ^
t.c:32:21: warning: unused variable ‘ind3’ [-Wunused-variable]
     int ind1, ind2, ind3;
                     ^
t.c:32:15: warning: unused variable ‘ind2’ [-Wunused-variable]
     int ind1, ind2, ind3;
               ^
t.c:28:27: warning: unused parameter ‘numerocorres’ [-Wunused-parameter]
 int bonNumerocorres(char *numerocorres)
                           ^
t.c: In function ‘main’:
t.c:93:15: warning: format ‘%s’ expects argument of type ‘char *’, but argument 2 has type ‘char (*)[30]’ [-Wformat=]
         scanf("%s", &numero);
               ^
t.c:110:15: warning: format ‘%s’ expects argument of type ‘char *’, but argument 2 has type ‘char (*)[30]’ [-Wformat=]
         scanf("%s", &numerocorres);
               ^
t.c: In function ‘bonNumerocorres’:
t.c:41:8: warning: ‘ind1’ is used uninitialized in this function [-Wuninitialized]
     if (ind1 == 77 || ind1 == 78)
        ^

Quelles erreurs ne comprends-tu pas ?

**mad0210** · 30/03/2016, 15h09

Au niveau de la fonction bonNumerocorres

**picodev** · 30/03/2016, 15h15

Tu compares des int et des char* … pas bon. Ton compilateur te le dis. Compare des variables du même type, transforme l'une en l'autre (int→chaîne sprintf, chaîne→int strtol) et compare les correctement (== pour des nombres, strcmp pour des chaînes).
Comprendre les messages du compilateur pour résoudre les warnings et erreurs est bien plus rapide de laisser des messages sur les forums. Sur ce problème tu auras perdu 24H …

**mad0210** · 30/03/2016, 15h25

La fonction bonNumerocorres je l'ai appelé pour qu'il m'aide à faire le contrôle de saisie quand l'utilisateur entre le numerocorres pour donner les resultats suivants:
s'il entre un numérocorres qui est identique à son numéro, le programme lui informe que le numero de votre correspondant doit etre different de votre numero.
s'il entre un numerocorres qui n'est pas enregistré, le programme lui informe que ce numéro n'est pas enregistré.

**mad0210** · 30/03/2016, 15h31

Envoyé par picodev

Tu compares des int et des char* … pas bon. Ton compilateur te le dis. Compare des variables du même type, transforme l'une en l'autre (int→chaîne sprintf, chaîne→int strtol) et compare les correctement (== pour des nombres, strcmp pour des chaînes).
Comprendre les messages du compilateur pour résoudre les warnings et erreurs est bien plus rapide de laisser des messages sur les forums. Sur ce problème tu auras perdu 24H …

moi je suis débutant je veux de l'aide mais pas de jugement dis moi remplace cette ligne par ça oubien tu répond pas à mes post c 'est simple

**picodev** · 30/03/2016, 15h38

Envoyé par mad0210

moi je suis débutant je veux de l'aide mais pas de jugement dis moi remplace cette ligne par ça oubien tu répond pas à mes post c 'est simple

La susceptibilité c'est jamais bon quand on apprend.
Je t'apprends à lire les messages de ton compilateur, à les comprendre et à débuguer ton programme.
Je t'apprends à exposer ton problème sur un forum car un «voilà mon code, ça fonctionne pas, aidez moi» c'est pas la meilleure manière.
Je t'apprends qu'il ne faut pas mélanger les pommes et les poires, je te signale quelle est une de tes erreurs, comment y remédier et quelles fonctions utiliser. Si tu n'es pas capable de chercher un peu par toi-même et si tu attends une solution toute faite sans rien nous expliquer ni montrer de bonne volonté c'est clair que tu n'auras pas énormément de réponse.
Je t'apprends surtout qu'un forum ce n'est ni un générateur automatique de code, ni un débugueur automatique, ni une boule de cristal magique.

Bon courage pour la suite.

**dinobogan** · 30/03/2016, 15h50

Envoyé par mad0210

moi je suis débutant je veux de l'aide mais pas de jugement dis moi remplace cette ligne par ça oubien tu répond pas à mes post c 'est simple

C'est même encore plus simple : personne ici ne te fera le travail à ta place.
Si tu postes ici c'est pour avoir de l'aide et pour t'améliorer toi-même (cf. la charte du forum que tu as acceptée en t'inscrivant). Si tu montres de la mauvaise volonté, les bénévoles ne prendront plus la peine de te lire.
Les bénévoles utilisent un peu de leur temps libre pour aider ceux qui ont des problèmes. Ils donnent des réponses ou des pistes d'amélioration, à toi d'avoir l'humilité d'accepter les conseils.

La réponse de picodev n'a rien à voir avec un jugement. Dans son premier message, il te donne tout ce qu'il faut pour avancer dans la résolution de ton problème. N'hésite pas à poser des questions sur ce que tu n'as pas compris dans son explication et nous nous ferons un plaisir de répondre et continuerons à te donner des conseils.
Montre ta bonne volonté en prenant en compte ce que picodev te dit : corrige les erreurs et warning de compilation.

**mad0210** · 30/03/2016, 16h15

j'ai pas toujours dit de faire le projet à ma place j'ai écrit mon code je l'ai posté j'ai demandé de l'aide pour la fonction bonNumerocorres seulement

**nnovic** · 31/03/2016, 09h51

Envoyé par mad0210

j'ai pas toujours dit de faire le projet à ma place j'ai écrit mon code je l'ai posté j'ai demandé de l'aide pour la fonction bonNumerocorres seulement

... et des conseils t'ont été donnés pour résoudre ton problème. Maintenant, essaye de le résoudre par toi-même, c'est bon pour toi. Si tu n' y arrives pas, montre-nous ce que tu as essayé de corriger, explique ce qui ne marche pas, et ainsi de suite.

Tu progresseras bien mieux si tu arrives toi-même à la solution. Tu as obtenu des indices, utilise-les.

**mad0210** · 06/04/2016, 15h30

SVP!!! aidez moi je dois rendre le projet le 11 et j'ai pas encore terminé

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
#include <stdio.h>
#include <stdlib.h>
#include <string.h>
#define N 100
 
 
int bonNumero(char *numero)
{
    int okLong, okNum;
 
    okLong = strlen(numero) == 9;
    okNum = numero[0] == '7' && (numero[1] == '7' ||
                                 numero[1] == '6' ||
                                 numero[1] == '0' ||
                                 numero[1] == '8');
 
    if (!okLong && okNum)
        printf("Le numero doit comporter 9 chiffres\n");
    if (!okLong && !okNum)
        printf("Le numero doit comporter 9 chiffres et commence par 77 ou 76 ou 70 ou 78\n");
    if (okLong && !okNum)
        printf("Le nummero doit commencer par : 77 ou 76 ou 70 ou 78\n");
 
    return okNum && okLong; 
}
 
int bonNumerocorres(char *numerocorres)
{
    int i, oknumcorres = 0;
    int preO = 5, preT = 5, preE = 5;
    int ind1, ind2, ind3;
    int mat[3][N] ={{772314440, 782314440, 774517070, 784517070, 778868181},
                    {762314440, 764517070, 768868181, 763570521, 765181304}, 
                    {702314440, 704517070, 708868181, 703570521, 705181304}};
    char numero[30];
 
 
    if (oknumcorres == sprintf()
        printf("Le numero du correspondant doit etre different de votre numero\n");
    if (ind1 == 77 || ind1 == 78)
    {
        for (i = 0; i <= preO; i++)
        {
            if (mat[0][i] == numero)
            {
            oknumcorres = 1;
            i = preO;
            }
        }
        if (oknumcorres == 0)
        printf("Ce numero n'est pas enregistre\n");
    }
    if (ind1 == 76)
    {
        for (i = 0; i <= preT; i++)
        {
            if (mat[1][i] == numero)
            {
            oknumcorres = 1;
            i = preT;
            }
        }
        if (oknumcorres == 0)
        printf("Ce numero n'est pas enregistre\n");
    }
    if (ind1 == 70)
    {
        for (i = 0; i <= preE; i++)
        {
            if (mat[2][i] == numero)
            {
            oknumcorres = 1;
            i = preE;
            }
        }
        if (oknumcorres == 0)
        printf("Ce numero n'est pas enregistre\n");
    }
 
    return oknumcorres;
}
 
 
int main()
{
    int credit, dureecomsouhaite;
    char numero[30], numerocorres[30];
 
    do 
    {
        printf("Entrer votre numero\n");
        scanf("%s", &numero);
    }
    while (!bonNumero(numero));
 
    do
    {
        printf("Entrer le montant de votre credit\n");
        scanf("%d",&credit);
 
        if (credit < 60)
            printf("Le credit doit etre superieur ou egale a 60\n");
    }
    while (credit < 60);
 
    do
    {
        printf("Entrer le numero du correspondant\n");
        scanf("%s", &numerocorres);
    }
    while (!bonNumero(numerocorres) || !bonNumerocorres(numerocorres));
 
    do
    {
        printf("Entrer la duree de communication souhaitee\n");
        scanf("%d", &dureecomsouhaite);    
    }
    while (dureecomsouhaite < 0);
 
 
 
 
 
    return 0;
}

**ternel** · 06/04/2016, 15h47

Ok, on veut bien t'aider.
Quelle est ta question?

**nnovic** · 06/04/2016, 22h25

Bon alors, allons-y progressivement.

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
int bonNumerocorres(char *numerocorres)
{
    int i, oknumcorres = 0;
    int preO = 5, preT = 5, preE = 5;
    int ind1, ind2, ind3;
    int mat[3][N] ={{772314440, 782314440, 774517070, 784517070, 778868181},
                    {762314440, 764517070, 768868181, 763570521, 765181304}, 
                    {702314440, 704517070, 708868181, 703570521, 705181304}};
    char numero[30];
  
    if (oknumcorres == numero)
        printf("Le numero du correspondant doit etre different de votre numero\n");

    ...
    ...

Plusieurs problèmes avec ce "if":
- "oknumcorres" vaut zéro, pourquoi tester sa valeur ?
- "numero" n'est pas initialisé, dans le sens où le contenu de "numero" est indéfini.
- tu compares un nombre entier et un tableau, ce n'est pas logique

J'ai l'impression que ce que tu voulais faire, c'est plutôt comparer "numerocorres" avec "numero". Dans ce cas, il ne faut pas utiliser "==" mais une fonction de comparaison, telle que strcmp(). Reste que "numero" n'a pas de valeur déterminée, tu ne l'as pas initialisé.

**mad0210** · 07/04/2016, 00h20

Mon problème est que j'arrive pas à contrôler le numéro du correspondant que l'utilisateur entrera...............une aide serait la bienvenue

**ternel** · 07/04/2016, 08h05

un numéro de téléphone, tel que tes messages d'erreurs le disent, vérifie deux propriétés:

il fait 10 chiffres de long
d'après tes explicatons, il commence par 077 ou 076 ou 070 ou 078

Tu es parvenu à lire le numéro dans un char[30].
Il convient donc de vérifier que:

il fait bien 10 caractères de long
chaque caractère est un chiffre
les trois premiers sont compatibles avec 077 ou 076 ou 070 ou 078

or strlen mesure la longueur, isdigit vérifie qu'un caractère est un chiffre, et a == '0' vérifie que a vaut le caractère 0 (qui ne vaut pas 0, mais 0x30 = 48).

Je te laisse voire comment l'écrire, mais ton code devrait ressembler à ceci:

soit candidat la chaine de caractères entrée par l'utilisateur.
si strlen(candidat) != 10
    raler
    retourner faux

pour chaque caractère c de candidat
    si isdigit(c) est faux
        raler
        retourner faux

si candidat[0] != '0'
ou candidat[1] != '7'
ou (candidat[2] n'est pas parmi '7', '6', '0' et '8')
    raler
    retourner faux

retourner vrai

**mad0210** · 08/04/2016, 13h32

SVP!!! je n'arrive pas à contrôler le numéro du correspondant que l'utilisateur va entrer
les conditions valides sont :
1-le numéro du correspondant doit être de celui du numéro d'appel de l'utilisateur
2-il doit figurer ici :

Code :

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

1
2
3
4
 int mat[3][AB] = {{772314440, 782314440, 774517070, 784517070, 778868181},
                      {762314440, 774517070, 768868181, 763570521, 765181304},
                      {762314440, 774517070, 768868181, 763570521, 765181304},    
                     };

**ternel** · 08/04/2016, 14h03

A la main, comment ferais-tu?
En vérifiant un à un chaque numéro de l'ensemble (ici une matrice, ce qui est un peu étrange, on s'attendrait plutot à une séquence (liste, tableau 1D) )

Pour cela, il faut que tu compares un nombre (tel que 772314440) avec une chaine de caractères ("07634567890")
Cela signifie convertir l'un des deux dans le format de l'autre, et comparer deux objets de ce format.

Tu as donc deux possibilités: convertir le nombre de la matrice en chaine et convertir la chaine de l'utilisateur en nombre.
En général, la deuxième solution est préférable pour plusieurs raisons:

il suffit de convertir une seule fois la saisie utilisateur, avant l'ensemble des vérifications
un nombre est plus léger et plus rapide à comparer qu'une chaine de caractères

Systéme de contrôle des appels téléphoniques

C

Discussions similaires

Partager

Partager