Discussion :

[swiixz]

Bonjour dans le cadre d'un projet j'ai un programme de traduction de morse à faire en C.

Je recupere à partir d'un fichier texte, le code morse correspondant à chaque lettre.

Ce code est dans un char*. Je dois ensuite remplir une liste chainée (obligatoirement) contenant ce meme code et la lettre correspondant.


j'ai donc une liste:

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
 
#ifndef liste_h
#define liste_h
#include<stdio.h>
#include<stdlib.h>
 
 
struct elem {     char alpha;
                  char *morse;
                  struct elem* suiv;
              };
 
typedef struct elem element;
typedef struct elem* liste;
 
 
#endif

Les fonctions pour connaitre les valeurs de la liste

Code :

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

1
2
3
4
5
6
7
8
9
 
char alphaValeur(liste l)
{
return l->alpha;
}
char morseValeur(liste l)
{
return l->morse;
}

Celle pour construire ma liste

Code :

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

1
2
3
4
5
6
7
8
9
10
11
12
 
liste cons(char a,char *m,liste l)
{
 
liste p;
p=(liste)malloc(sizeof(element));
p->alpha=a;
//strcpy(p->morse,m);
p->morse=*m;
p->suiv=l;
return p;
}

Et celle pour l'afficher:

Code :

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

1
2
3
4
5
6
7
8
9
10
11
 
void afficheliste(liste l) 
{
liste p=l;
while(p)
{
printf("(%c,%c) ",alphaValeur(p),morseValeur(p));
p=reste(p);
}
printf("\n");
}

Voila le souci c'est que je n'ai pas le code morse complet, seulement la premiere lettre.
J'ai deja essayé de mettre un "%s" dans la fonction d'affichage mais le programme plante, une "*" dans la fonction qui me retourne le code morse mais le programme plante.

Je ne trouve pas de solutions si vous pouviez me donner un coup de pouce ça serait plutot cool.

Voici au passage ma procedure qui lit mon fichier texte:

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
 
 
void essai2()
{
     FILE* fichier = NULL;
    fichier = fopen("txtMorse.txt", "r");
    char alpha;
    char *morse=NULL;
    char chaine[TAILLE_MAX] = "";
    liste l=NULL;
    int index;
 
 
    if (fichier != NULL)
    {
        while (fgets(chaine, TAILLE_MAX, fichier)!= NULL) // On lit le fichier tant qu'on ne reçoit pas d'erreur (NULL)
        {
            alpha=chaine[0];
            printf("%c",alpha); 
            morse=strpbrk(chaine,".-");
            printf("%s\n",morse);
            l=cons(alpha,morse,l);
 
        }
    }
     fclose(fichier); // On ferme le fichier qui a été ouvert 
 
     afficheliste(l);       
}

Merci d'avance.

[Climoo]

Code :

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

1
2
3
4
5
 
struct elem {     char alpha;
                  char *morse;
                  struct elem* suiv;
              };

Tu as un simple char *morse.

Code :

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

1
2
 
p->morse=*m;

Tu ne fais que copier un pointeur ici. Tu ne recopies pas chaque caractère de la chaîne.

Code :

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

//strcpy(p->morse,m);

Tu as enlevé le strcpy, tu aurais du le laisser ! MAIS, il faut auparavant que dans ta chaîne il y ait la place nécessaire. Deux solutions : tu alloues l'espace dynamiquement. Ou, plus simples, tu changes un peu ta structure en

Code :

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

char morse[MAX_CHR];

[swiixz]

Merci de ta réponse qui a été rapide je test ça demain, en fait juste avant mon strcpy il faut que je fasse un malloc?

Et ma fonction qui me retourne le char* de ma liste reste en:

Code :

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

1
2
3
4
char morseValeur(liste l)
{
return l->morse;
}

ou passe en

Code :

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

1
2
3
4
char *morseValeur(liste l)
{
return l->morse;
}

[Climoo]

Code :

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

char morse[MAX_CHR];

Si tu choisis cette solution, tu n'auras pas besoin de faire de malloc. La mémoire est DEJA présente quand tu crées la structure. Du moment que tu n'essaies pas de copier plus de caractères que MAX_CHR , tu n'auras pas de problème.

En effet, je navais pas fait attention à ça, mais cest bien:

Code :

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

1
2
3
4
5
 
char *morseValeur(liste l)
{
return l->morse;
}

car tu retournes un pointeur vers un caractère.
Mais après faire une fonction pour ça, cest pas forcément très intéressant. Une macro à la limite, mais ca reste sans grand intérêt.

Code :

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

1
2
 
#define morseValeur(l) ( (*(l)).morse)

[swiixz]

Merci pour l'astuce du #define, c'est simplement notre prof qui nous a montrer par le biais d'une fonction.

Le probleme c'est que le nombre de caracteres morse est variable donc je ne vais pas allouer un tab de taille fixe.

Je test tout ça j'espere que ça va fonctionner.

Merci encore de ton aide.

[swiixz]

Ca marche nikel j'ai juste eu un ptit soucis car dans mon fichier texte j'ai un saut de ligne à la fin de mes caracteres en morse.

Je l'ai réglé avec un:

if(morse[strlen(morse)-1]=='\n')
morse[strlen(morse)-1]='\0';

Car avec un strncpy(morse2,morse,strlen(morse)-1) j'avais des caracteres farfelus qui apparaissaient pour certaines lettres.

[swiixz]

Je pensais en avoir fini et bien non pourtant je suis au bout du bout mais j'ai une erreur que je ne trouve pas.

je lis un fichier de morse pour le traduire.

Apres avoir construit ma liste contenant mon code morse et les lettres, j'ai fais une fonction qui me retourne la lettre correspondant au code morse passé en paramètre.

Code :

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

1
2
3
4
5
6
7
8
9
10
 
char retourneAlpha(liste l,char *m)
{
liste p=l;
while(morseValeur(p)!=m)
{
p=reste(p);
}
return alphaValeur(p);
}

Elle fonctionne car ceci

Code :

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

1
2
 
printf("%c\n",retourneAlpha(l,morseValeur(l)));

me retourne la premiere lettre de ma liste. morseValeur(l) est un char*


En revanche quand je lis un fichier morse et que je fais ça:

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
 
void lireFichierMorse(liste l)
{
    FILE* fichier = NULL;
    fichier = fopen("morse.txt", "r");
    char chaine[TAILLE_MAX] = "";
    char chaine2[5] = "";
    int index=0;
    int index2=0;
    char *morse=NULL;
 
 
    if (fichier != NULL)
    {
        while (fgets(chaine, TAILLE_MAX, fichier)!= NULL) // On lit le fichier tant qu'on ne reçoit pas d'erreur (NULL)
        {
              while(chaine[index]!='\n')
              {
                      if(chaine[index]!=' ')
                      {
                              chaine2[index2]=chaine[index];
                              index2++;
                      } 
                      else 
                      {
                          morse=strpbrk(chaine2,".-");
                          printf("%s",morse);
                          printf("%c",retourneAlpha(l,morse)); // ça plante ici pourtant morse est un char* ne contient rien d'autre que du morse
                          index2=0;
                          while(index2<5){chaine2[index2]='\0';index2++;}
                          index2=0;
 
                      }
                      index++;
 
              }   
 
        }
        printf("\n"); 
    }
     fclose(fichier);     
}

A noter que l'inverse fonctionne parfaitement (lecture d'un fichier texte et traduction en morse.) Donc je ne comprend vraiment pas le probleme qui se pose ici.

Merci à ceux qui m'aideront.

[diogene]

Code :

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

1
2
3
4
char retourneAlpha(liste l,char *m)
{
liste p=l;
while(morseValeur(p)!=m)

L'expression dans le while ne compare pas les chaînes de caractères d'adresse morseValeur(p) et m, mais les adresses elles-même. Le résultat est donc en cas général toujours Vrai.
Pour comparer des chaînes de caractères, utiliser la fonction strcmp.

Elle fonctionne car ceci

Code :

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

printf("%c\n",retourneAlpha(l,morseValeur(l)));

...

Voila un test très mal choisi :
m est morseValeur(l) et la condition devient

Code :

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

while(morseValeur(l)!=morseValeur(l))

cas très particulier qui donne Faux !

[swiixz]

Ouh pinaise le boulet... Bien vue... Merci beaucoup le projet est finie... Enfin j'espere...