Pb de pointeurRésolu

Question

Bonjour, J'ai un problème avec un pointeur... Contexte : j'envoie une trame à un periphérique connecté sur port série, le périphérique répond. Mon problème se situe à la réception de la réponse. Dans mon main j'appel une fonction "recevoir" qui va lire le buffer via une fonction lireBuffer après un WaitCommEvent (le port série ne me pose pas de problèmes). Lorsque je lis le buffer récupéré via ReadFile dans la fonction 'lireBuffer" ca marche, mais dans "recevoir" j'ai une erreur deboguage / envoyer le rapport d'erreur / ne pas envoyer). // Permet de lire les octets dans le buffer du port série int LiaisonSerie::lireBuffer(void * telBuffer) { if(handleCom == NULL) { // si la connexion n'est pas établie messageErreur(Err_Connexion); return Err_Connexion; } //Pour connaitre le nombre d'octets dans le buffer d'entrée COMSTAT stat; DWORD errors; ClearCommError(handleCom,&errors,&stat); // nb de caracteres dans le buffer int nbCarALire = stat.cbInQue; if(nbCarALire==0) { // si le buffer est vide messageErreur(Err_Buffer_Vide); return Err_Buffer_Vide; } telBuffer = new char(nbCarALire); //On effectue la lecture si il y a des caractères présents unsigned long nbCarLus = 0; if(ReadFile(handleCom,telBuffer,nbCarALire,&nbCarLus,NULL)==0) { // Echec messageErreur(Err_Reception); return Err_Reception; } cout<<"nb caracteres a lire : "< Message d'erreur (deboguage / envoyer le rapport d'erreur / ne pas envoyer) cout<<"(recevoir) telBuffer : "<<(char*)telBuffer<recevoir(TrameRecueMoteur); ... } Note : mon main est OK le problème c'est mon buffer Auriez-vous une idée ?? Merci pour votre aide ++

dmk04 · Accepted Answer

celui-là marche, mais je me demande si il n'existerai pas une écriture plus propre ?

#include 
#include <string.h>
using namespace std;


int test(void * &buf)
{
    buf = new char[5];
    memset(buf,'\0',5);
    strcpy((char*)buf,"test");
    cout<<"test : "<<(char*)buf<<endl;
    return 0;
}


int main()
{
    char * leBuf;
    test((void*&)leBuf);
    cout<<"main : "<<leBuf<<endl;
    delete(leBuf);
    system("PAUSE");
    return 0;
}

Merci

cs_Joky · Answer

C'est bizarre tes allocations mémoire
new char(nbCarALire);

moi j'aurais fait comme ça new char[nbCarALire+1]; pour le \0 qui ne faut pas oublié.
Ensuite un memset(telBuffer, 0, nbCarALire+1);

Voilà
if(!Meilleur("Joky")) return ERREUR;<

ymca2003 · Answer

new char(nbCarALire);
alloue en effet un seul caractère et l'initialise à la valeur nbCarALire
new char[nbCarALire];
alloue le tableau

cs_Joky · Answer

Ah j'savais pas qu'il y avait ce genre de constructeur pour les char enfin constructeur c'est vite dit mais bon
Ca se rapproche de ce concept quand même

if(!Meilleur("Joky")) return ERREUR;<

dmk04 · Answer

ok,

j'ai essayer new char[nbCarALire], char[nbCarALire+1], char(nbCarALire+1), avec un memset(telBuffer,0,nbCarALire) ou memset(telBuffer,0,nbCarALire+1), mais toujours le même résultat.

Voilà ce qui s'affiche :

nb caracteres a lire : 1
contenu buffer lu : r  taille : 1
nb caracteres lus : 1
(recevoir) telBuffer :

Message d'erreur (deboguage / envoyer le rapport d'erreur / ne pas envoyer).

C'est un problème de pointeur... pas d'allocation

merci quand même

++

dmk04 · Answer

voilà un petit programme qui isole juste mon problème :

#include 
#include <string.h>
using namespace std;


int test(void * buf)
{
    buf = new char(5);
    memset(buf,'\0',5);
    strcpy((char*)buf,"test");
    cout<<"test : "<<(char*)buf<<endl;
    return 0;
}


int main()
{
    char * leBuf;
    test(leBuf);
    cout<<"main : "<<leBuf<<endl;
    delete(leBuf);
    return 0;
}

Merci pour votre aide

a+

cs_Joky · Answer

C'est l'allocation dans la fonction qui pose problème
Fait le hors de la fonction.

if(!Meilleur("Joky")) return ERREUR;<

cs_Joky · Answer

et on ne fait pas new char(5)
mais new char[5];

comme expliqué ci-dessus

if(!Meilleur("Joky")) return ERREUR;<

dmk04 · Answer

je vais réessayer avec tes [], tu y tiens tant :)

en attendant vla un programme qui donne le résultat souhaité, mais je préfererai que l'ancien marche :

#include 
#include <string.h>
using namespace std;


void * test(void * buf)
{
    buf = new char(5);
    memset(buf,'\0',5);
    strcpy((char*)buf,"test");
    cout<<"test : "<<(char*)buf<<endl;
    return buf;
}


int main()
{
    char * leBuf;
    leBuf = (char*)test(leBuf);
    cout<<"main : "<<leBuf<<endl;
    delete(leBuf);
    system("PAUSE");
    return 0;
}

merci

a+

dmk04 · Answer

new char[5] ça marche pas non plus...

cs_Joky · Answer

Bé c'est pour toi qu'on dit ça sur les [] parce que new char( ... ) ça te construit un seul caractère et pas le nombre que tu veux donc forcément
après si tu veux écrire 5 caractères dedans pas possible...

if(!Meilleur("Joky")) return ERREUR;<

dmk04 · Answer

ok. En attendant je peux écrire 5 caractères dedans, essaye tu verras, et c'est avec mes pointeur le problème pas avec l'allocation.

Merci

cs_laurent1024 · Answer

salut.
tu peux peut etre faire comme ca :
char * test()
{
    char *buf = new char[5];
    memset(buf,'\0',5);
    strcpy((char*)buf,"test");
    cout<<"test : "<<(char*)buf<<endl;
    return buf;
}

int main()
{
    char * leBuf;
    leBuf = test();
    cout<<"main : "<<leBuf<<endl;
    delete(leBuf);
    system("PAUSE");
    return 0;
}

ymca2003 · Answer

char* test()
{
    char* buf = new char[5];
    memset(buf,'\0',5);
    strcpy(buf,"test");
    cout<<"test : "<<buf<<endl;
    return buf;
}


int main()
{
    char * leBuf = test();
    cout<<"main : "<<leBuf<<endl;
    delete[] leBuf;
    system("PAUSE");
    return 0;
}

dmk04 · Answer

euhhh, je voulais une écriture plus propre pour :

#include 
#include <string.h>
using namespace std;


int test(void * &buf)
{
    buf = new char[5];
    memset(buf,'\0',5);
    strcpy((char*)buf,"test");
    cout<<"test : "<<(char*)buf<<endl;
    return 0;
}


int main()
{
    char * leBuf;
    test((void*&)leBuf);
    cout<<"main : "<<leBuf<<endl;
    delete(leBuf);
    system("PAUSE");
    return 0;
}

je voudrais que test retourne toujours un entier, c'est pour les erreurs... je dois laisser les void*& ? pas d'autre écriture ?

Merci

ymca2003 · Answer

si tu veux que test renvoi un entier et alloue la mémoire, il faut qu'un des 2 soit passé en référence...
Le mieux c'est quand même d'allouer le buffer avant d'appeler la fct, de le passer en paramètre ainsi que la taille
ton code d'erreur pourra alors renvoyer si des octets sonyt encore à lire parceque le buffer était trop petit

dmk04 · Answer

ok, merci.

Pb de pointeur

17 réponses

Votre réponse

Discussions similaires