Discussion :

[Tolhsadum]

Bonjour à tous,
Je suis vraiment très peu doué en C mais j'aurais besoin de faire un programme qui formate un fichier contenant des numéros de téléphone.

Par exemple : un fichier nommé "numero.txt" contient au départ :
06,00,00,00,00
06,00,00,00,00
06,00,00,00,00
06,00,00,00,00
06,00,00,00,00
06,00,00,00,00
06,00,00,00,00
06,00,00,00,00

à la fin il devrait contenir :

0600000000,0600000000,0600000000,0600000000,0600000000,0600000000,0600000000,0600000000

Donc plus d'espace ni de passage à la ligne et suppression des virgules de séparation des chiffre 2 à 2 mais rajout de séparation 10 à 10.

J'ai commencé à codé ça : (je sais que je code comme un sagouin j'en suis désolé mais je n'arrives vraiment pas à m'y mettre au C)

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
 
#include <stdio.h>
#include <stdlib.h>
#include <string.h>
 
int main(int argc, char *argv[]) {
   FILE* fichier = NULL;
   fichier = fopen("numero.txt", "r+");
   char chaine[2000];
   char tmp = 'a';
   char temp[1];
   int c1 = 0;
 
   if (fichier != NULL)
   {
   	// Boucle de lecture des caractères un à un
   	do
   	{
       	tmp = fgetc(fichier); // On lit le caractère
       	if ((tmp < '0') || (tmp > '9')) {
   		} else {
   			temp[0] = tmp;
   			strcat(chaine, temp);
   			c1 += 1;
   			if (c1 % 10 == 0) {
   				strcat(chaine, ",");
   			}
   		}
 
   	} while (tmp != EOF);
   	printf("%s", chaine);
 
   	fclose(fichier);
   }
   else
   {
   	// On affiche un message d'erreur si on veut
   	printf("Impossible d'ouvrir le fichier numero.txt");
   }
   return 0;
}

Je me retrouve avec une erreur de segmentation et je n'arrive pas à m'en dépatouiller.

Merci d'avance pour votre aide.

[ram-0000]

Tu as au moins une erreur ici :

Code :

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

strcat(chaine, temp);

temp est un tableau de char de 1 caractère

quand tu fais temp[0] = tmp;
ta chaine temp n'est pas "zero terminated" et donc il est probable que le strcat() plante ensuite.

Au fait, le debugger, c'est pas fait pour les chiens, cela aide justement à localiser (et résoudre) ce genre de problèmes.

[ram-0000]

Au fait, question subsidiaire, comment tu feras lorsque tu aura 200 numéros de téléphone ou plus dans ton fichier "numero.txt" ?

Je te parie un autre crash lorsque la variable chaine[2000] va déborder

[Tolhsadum]

Merci pour ta réponse, mais qu'est ce que "zero terminated" et que dois-je faire pour qu'elle le soit ?

Sinon pour la longueur de la chaîne, j'ai fixé 2000 un peu au hasard, mais à priori on ne devrait jamais dépasser se nombre quand on utilisera le programme, on sera de très loin en dessous.

[Bktero]

mais qu'est ce que "zero terminated" et que dois-je faire pour qu'elle le soit ?

Quelqu'un n'a pas été attentif au cours sur les chaines de caractères

En C, une chaine de caractères est une suite d'objets de type char, placés de manière continue en mémoire. Comment savoir où se termine cette suite sachant qu'une variable char* ou char[] ne donne que l'adresse du premier élément de cette suite ? La réponse est que cette suite doit être terminée par un caractère spécial indiquant sa fin. Ce caractère en C est '\0' qui vaut 0 (zéro), aussi appelé NUL. D'où le terme de "zero terminated" ou "null terminated".

Pour ce qui est du code lui-même, pour faire un IF vide et tout faire dans le ELSE et non inverser la condition ? De plus, il existe une fonction isdgit() qui pourrait rendre ton code plus clair (c'est un détail bien sûr).

strcat() n'est peut-être pas la fonction la plus adaptée à ton besoin. Elle est fait concaténer deux chaines (null terminated donc ^^) et toi tu veux ajouter un caractère dans un tableau. Pourquoi ne pas écrire dans chainedirectement si tmpest OK et une virgule si besoin ? D'ailleurs, tu as vu le mic mac que tu es obligé de faire à cause de strcat(), tmpet temp?

Quant au problème évoqué par Raymond, tu peux simplement ouvrir ton fichier, déterminer le nombre de numéros de téléphone (soit exactement en comptant le nombre de lignes, soit de manière peut-être moins exact en regardant la taille du fichier avec ftell()), faire une allocation dynamique avec malloc() de la bonne taille et le tour est joué. Ou allouer une taille qui parait OK pour la plupart des fichiers et réallouer si le fichier est trop grand.

[ram-0000]

J'oserais, et sans vouloir froisser Tolhsadum car on a tous débuté, je dirais que ce que tu dis est parfaitement valide (isdigit(), ftell(), malloc(), ...) mais d'un niveau encore trop élévé par rapport à ce qui est présenté.

Donc d'abors, une correction immédiate du problème :

Code :

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

1
2
3
4
char temp[2];
...
temp[0] = tmp;
temp[1] = 0; /* c'est cette ligne qui fait le "zero terminated" de la chaine */