Discussion :

[lahlaly]

Salut Tout le monde !

Bon mon Problème est le suivant , j'essaie de lire a partir d'un fichier de mots ces mots et les mettre dans d'autre fichier en fonction du nombre de caractères de chaque mot ,sans oublier d'enlever les mots contenant des caractères avec des accents ('é','è'...)

voila mon code entier:

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
#include <stdio.h>
#include <stdlib.h>
#include <string.h>
 
 
 
int motValable (char *mot)
{
    int i,j=strlen(mot);
    for (i=0;i<j;i++)
    {
        if (mot[i]<'a' && mot[i]>'z')
        {
            return 0;
        }
    }
    return 1;
}
int main()
{
    FILE *f,*six,*sept,*huit,*neuf,*dix;
    char t[20];
    f=fopen("dic.txt","r");
    six=fopen("6.txt","w");
    sept=fopen("7.txt","w");
    huit=fopen("8.txt","w");
    neuf=fopen("9.txt","w");
    dix=fopen("10.txt","w");
    fseek(f,0,SEEK_SET);
    while (!feof(f))
    {
 
        fscanf(f,"%s \n",t);
        if (motValable(t)){
        if (strlen(t)==6)
        {
            fprintf(six,"%s ",t);
        }
        else if (strlen(t)==7)
        {
            fprintf(sept,"%s \n",t);
        }
        else if (strlen(t)==8)
        {
            fprintf(huit,"%s \n",t);
        }
        else if (strlen(t)==9)
        {
            fprintf(neuf,"%s \n",t);
        }
        else if (strlen(t)==10)
        {
            fprintf(dix,"%s \n",t);
        }         }
    }
 
 
    fclose (f);
    fclose (six);
    fclose (sept);
    fclose (huit);
    fclose (neuf);
    fclose (dix);
    return 0;
}

Les Mots sont mis dans les fichier malgré la seg fault , mais ils ne sont pas filtrées , Merci Bonne journée .

[moins1]

Dans ta fonction motValable() ça serais mieux un || il me semble. Une lettre peut difficilement être plus grande que 'z' et en même temps plus petite que 'a'.

Dans le main() j'utiliserais un switch()

Code :

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

1
2
3
4
5
6
7
8
 
len = strlen(t)
switch(len)
{
    case 6: fprintf(six,"%s\n",t); break;
    case 7: fprintf(sept,"%s\n",t); break;
    // etc
}

EDIT: En testant je n'ai pas eu de seg fault. Peut être que la taille de t est trop petite pour certain mot.

[ternel]

attention, on ne teste pas avec feof pour la boucle.

il faut tester le retour de la fonction de lecture, puis, en cas d'erreur, vérifier si feof.

[darkwall_37]

En effet, soit dès qu'une lettre n'est pas dans [a,z], ton mot n'est plus valable et tu retournes 0.

Code :

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

1
2
3
4
5
6
7
8
9
int motValable (char *mot)
{
    int i,j=strlen(mot);
    for (i=0;i<j;i++)
        if ( !('a'<=mot[i] && mot[i]<='z') )
            return 0;
 
    return 1;
}

Soit, tu développes

Code :

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

!('a'<=mot[i] && mot[i]<='z')

ce qui revient à

Code :

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

mot[i] < 'a' || mot[i] > 'z'

et qui est parfaitement la même chose.

Code :

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

1
2
3
4
5
6
7
8
9
int motValable (char *mot)
{
    int i,j=strlen(mot);
    for (i=0;i<j;i++)
        if ( mot[i] < 'a' || mot[i] > 'z' )
            return 0;
 
    return 1;
}

Aussi, au lieu de recalculer la taille de t à chaque fois : Imagines que tu aies 1000000 de mots et qu'il faut 1/10000de sec pour effectuer l'instruction strlen. Dans le pire des cas tu fais les 5 tests. 5*1/10000*1000000/60 =+- 8min20 de perdu en strlen.

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
 
#include <stdio.h>
#include <stdlib.h>
#include <string.h>
 
 
 
int motValable (char *mot)
{
    int i,j=strlen(mot);
    for (i=0;i<j;i++)
        if (mot[i]>='a' && mot[i]<='z')
            return 0;
 
    return 1;
}
 
int main()
{
    FILE *f,*six,*sept,*huit,*neuf,*dix;
    char t[20];
    f=fopen("dic.txt","r");
    six=fopen("6.txt","w");
    sept=fopen("7.txt","w");
    huit=fopen("8.txt","w");
    neuf=fopen("9.txt","w");
    dix=fopen("10.txt","w");
    fseek(f,0,SEEK_SET);
    while (!feof(f))
    {
        fscanf(f,"%s \n",t);
        if (motValable(t))
        {
            unsigned int len = strlen(t)
            if (len==6)
                fprintf(six,"%s ",t);
 
            else if (len==7)
                fprintf(sept,"%s \n",t);
 
            else if (len==8)
                fprintf(huit,"%s \n",t);
 
            else if (len==9)
                fprintf(neuf,"%s \n",t);
 
            else if (len==10)
                fprintf(dix,"%s \n",t);
        }
    }
 
 
    fclose (f);
    fclose (six);
    fclose (sept);
    fclose (huit);
    fclose (neuf);
    fclose (dix);
    return 0;
}

Cordialement

[Médinoc]

On peut aussi se passer complètement de l'appel à strlen():

Code C :

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

1
2
3
4
5
6
7
8
9
10
int motValable (char const *mot)
{
	size_t i;
	for (i=0 ; mot[i] != '\0' ; i++)
	{
		if( ! (mot[i]>='a' && mot[i]<='z') )
			return 0;
	}
	return 1;
}

On peut aussi remplacer le test par une fonction de <ctype.h> (isalpha(), islower(), etc.)

[lahlaly]

Merci a Vous Problème résolu Merci Énormément