MORSEPLAYER, JOUER UN MESSAGE MORSE

Commentaire de Neo_Fr le 02/12/2007 22:14:53

Ta fonction ConvertirEnMorse ne libere pa la memoire qu'elle alloue..
Tu devrait plutot passer un pointeur en argument pour le resultat  et pourquoi utilise tu malloc?
L'equivalent win32 c'est HeapAlloc ou VirtualAlloc..

Sinon l'idée est pa mal :]

Neo_Fr

Commentaire de BruNews le 02/12/2007 22:36:04 administrateur CS

char enMorse[TailleMorse+1];
Quelle horreur, plus aucun controle de la mémoire, autant aller faire de l'interprété.

renvoi = malloc(strlen(enMorse) + 1);
strcpy(renvoi,enMorse); // TOUJOURS VERIFIER UNE ALLOC !!!
return renvoi;

char * partition = ConvertirEnMorse( partitionTemp );
IL EST OU LE free(partition) ???

if( IsDlgButtonChecked( hDlg, IDR_FERMER ) ) killHWND = TRUE;
else killHWND = FALSE;
Inutile d'enregistrer ceci en début de procédure puisque c'est en sortie qu'on fermera ou non.

Le C est le langage PRO, ce qui implique qu'on agite qlq neurone avant les doigts sur le clavier. Le nbr de caractères obtenus par GetWindowText() n'est même pas vérifié pour savoir si on appelle ConvertirEnMorse(), etc, etc...
Laisse la production de masse aux VBistes et autres Csharpiens, sur cppfrance c'est la qualité qu'il faut montrer en sources. Tu en es très certainement capable alors montre le nous en arrangeant tout ceci et en prenant le temps de nous proposer nettement mieux pour la suite.

Bonne continuation.

Commentaire de ndubien le 04/12/2007 19:44:57

Salut GAMEMONDE,

j'ai essayé de remplacer (comme vous me l'avez suggéré) :
if( temp<0 ) temp += 256;
int temp2 = strlen( CodeMorse[temp] );
par:
strlen( CodeMorse[(unsigned char)temp] );

mais celà ne fonctionne pas (enfin avec moi et dev-c++):
exemple si le texte saisie est: ççççççç, rien ne se passe (aucun beep pour le 'ç')...
tandis qu'avec mon ancien code celà fonctionnait...

Sinon, je ne comprend de quoi voulait parler Brunews par TOUJOURS VERIFIER UNE ALLOC ?
Que fallait-il ajouter pour vérifier cette alloc?

Sinon, pour les problèmes de libération de mémoire abordés par Neo_Fr dans son premier commentaire, je suis en train de regarder! ;)

A+ et merci pour toutes ces indications

Commentaire de ndubien le 04/12/2007 20:00:46

D'accord merci pour l'information!

Commentaire de ndubien le 14/12/2007 19:57:54

Salut DESK_BSD,
moi aussi j'ai déjà eut un problème comparable avec mon vista: il ne faisait pas les beeps mais n'a jamais planté... Je ne sais pas du tout d'où pourrait venir le problème... Si quelqu'un le sait merci de me le transmettre...
A+ et encore merci pour l'information...