Discussion :

[Sassy]

Bonjour !

J'ai un ennui avec une fonction qui ne marche pas ! Elle permet de créer un numéro de téléphone tout en s'assurant que le numéro est valide. Les numéros valides ont comme caractéristiques :

— Les caractères ne doivent être que des chiffres ;
— Ils doivent avoir 9 chiffres ;
— Il y a deux catégories de numéros : mobile et fixes ! les mobiles commencent soit par 77,76,70,78. Et les fixes par 33 ou 30. Voici mon code qui ne marche pas !

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
void ajout_tel(FILE*pt_fichier,Contact* p)
{
int ok = 1 ;
do{
printf("entrer le numero de telephone: ");
scanf("%s",p->tel);
fprintf(pt_fichier,"%s",p->tel);
system("cls");
if(strlen(p->tel)> 9) ok = 0 ; 
if(p->tel[i]!='0' && p->tel[i]!='1' && p->tel[i]!='2' && p->tel[i]!='3' && p->tel[i]!='4' && p->tel[i]!='5' && p->tel[i]!='6' && p->tel[i]!='7' && p->tel[i]!='8' && p->tel[i]!='9')
 ok=0;
 if(p->tel[0]!= 7 && p->tel[0]!= 3) //ici on s'assure que le premier caractere du numero indique soit le type mobile ou fixe//
   ok = 0 ;
if(p->tel[0]== 7 && (p->tel[1]!= 7  && p->tel[1]!= 6 && p->tel[1]!= 8 && p->tel[1]!= 0 ) ) ok = 0 ; //un num ki debute par 7 ne peut aller qu'qvec un '7','6','8','0'.
if(p->tel[0]== 3 && (p->tel[1]!= 0 && p->tel[1]!= 3) ) ok = 0 ;
}while(!ok);
}

[Bktero]

Salut,

Et qu'est ce qui ne marche pas et que souhaites-tu savoir pour le faire marcher ?

[moins1]

Pourquoi à certains endroits tu vérifies des caractères alors qu'à d'autres tu vérifies des entiers?

Sinon la variable i sort de où?



EDIT: Et qu'est-ce que vient faire FILE*pt_fichier dans les arguments de la fonctions?

[Neckara]

Bonjour,

Je trouve ton code pas très lisible et pas très bien indenté, fait aussi attention aux "nombres magiques".
Image que tu aies 200 débuts de numéro de téléphones valides, ton code serait totalement illisible.

Voici le pseudo-code que je te proposerais :

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
 
int isValide( char * numero )
{
          static char ** debutTelValide  = {"77", "76"}; // ici tu liste les débuts de numéros de téléphones valides.
 
         // si numero n'a pas exactement 9 caractères (+ '\0' final)
                  return 0; // false
 
         for( int i = 0; i != 9 ; i++) // on parcours les 9 caractères
                 // si le caractère n'est pas un nombre (cf la fonction is_digit)
                          return 0; // false
 
 
         // On parcours chaque chaînes de caractères de debutTelValide
                 // On le compare au début du numéro de téléphone ( cf strncmp )
                 // Si la comparaison est correcte
                          return 1; // true
 
        return 0; // false
)
}

Je pense qu'il sera un peu plus lisible et un peu plus simple à comprendre et te permettra de voir plus facilement où il y a une erreur si tu en a une.

EDIT : quelques pages de man utiles
http://man.developpez.com/man3/isalpha.3.php
http://man.developpez.com/man3/strcmp.3.php

[Sassy]

moins1 je devai tester des caracteres pour tout! le FILE*pt_fichier c parce que les numeros seront ecri dan un fichier! et le i je voulai tester tous les caracteres du numero

[Sassy]

neckara le truc c'est que je dois repeter a l'utilisateur de saisir un numero de telephone tant que les numeros qu'il donne ne sont pas valides

[Bktero]

Voici le pseudo-code que je te proposerais :

Code :

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

1
2
3
4
5
 
int isValide( char * numero )
{
          static char ** debutTelValide  = {"77", "76"}; // ici tu liste les débuts de numéros de téléphones valides.
}

Je doute que cette écriture soit correcte, à la vue de la tripotée de warnings qu'elle génère. Ne serait-ce pas plutôt ?

Code :

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

static const char* debutTelValide[2]  = {"77", "76"};

[Neckara]

Je doute que cette écriture soit correcte, à la vue de la tripotée de warnings qu'elle génère. Ne serait-ce pas plutôt ?

Code :

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

static const char* debutTelValide[2]  = {"77", "76"};

Je reconnais ne pas avoir compilé

En fait, ce serait plutôt :

Code :

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

static const char* debutTelValide[]  = {"77", "76"};

Pour permettre des débuts de tailles différentes (mais bon, après tous ces warnings, je ne suis pas très bien placé pour chipoter ).