probleme de language cRésolu

Question

Bonjour quelqu'un pourrais jetter un  oeil et me dire la les boulettes
du code suivant
PS je debute en c ne soyer pas tueur

char *CentreTexte(char *Texte,int nbCaractereMax)
{
char *tampon=NULL;
int calcul=(nbCaractereMax-strlen(Texte))/2;
tampon=malloc(sizeof(char)*nbCaractereMax+1);
ZeroMemory (tampon,nbCaractereMax) ;
if (calcul>=0)
{
    while(calcul!=0)
    {
        strcat(tampon," ");
        calcul--;
    }
    strcat(tampon,Texte);
}
else
{
    strcpy(tampon,Texte);
}
return tampon;
free (tampon);
}

merci beaucoup

SAKingdom · Accepted Answer

Ce n'est pas le système qui détermine la taille d'un char mais le compilateur. Une fois compilé, ton char fera un octet définitivement. (et de toute façon, les caractères ascii font tous 1 octets alors...).

Le ZeroMemory est, ici, complètement inutile. En toute logique, il est complètement inutile de remplir un tableau de 0 quand on insère un texte immédiatement après.

while(calcul!=0)

    {

        strcat(tampon," ");

        calcul--;

    }

strcat coute déjà assez cher comme ça, du fait qu'il parcours toute la chaine avant de copier le texte. Si c'est pour copier un seul caractère...
Un pointeur positionné une fois en fin de chaine fait l'affaire.

C++ (@++)<!--

matonzo · Answer

Salut

Pour commencer, mettre des instructions après un return ça ne sert à rien car elle ne seront pas exécutées.

Ensuite, en admettant que tu veuilles faire ton free(tampon) avant le return,  il ne faut pas faire de free sur le pointeur retourné par ta fonction sinon quand tu essaieras d'accéder à la valeur retournée tu auras une belle erreur de segmentation.

Sinon, tu peut être un peu plus spécifique sur ton problème? ça ne fait pas ce que tu veux, ça ne compile pas, ça plante?...

cs_darunia · Answer

Salut,

Boulette au niveau du free ! En effet tu fais ton return juste avant, et comme les instructions qui suivent le return ne sont pas executées (vu que le return rend la main a l'appelant), ton free ne sera jamais executé !

Autre truc, toujours verifier le retour d'un malloc (ça peut arriver qu'il renvoi null).

Et le ZeroMemory n'est pas bon, tu dois passer sizeof(char) * nbCaractereMax + 1 pour la longueur (imagine un systeme ou un char ferait plus d'1 octets)

Voila, a part pour le free qui est quand meme assez embetant, le code est correct pour un debutant !!!

D@runia

cs_darunia · Answer

Oui je sais bien, mais on peut toujours imaginer !

D@runia

SAKingdom · Answer

Tien, j'ai refais rapidement ton code:

char *CentreTexte(char *Texte, char *tampon, int nbCaractereMax)
{
    char *c = tampon;
    int len = strlen(Texte);

    if (nbCaractereMax > len) {
        int calcul = ((nbCaractereMax-len)/2)+(int)c;
        while(c < (char*)calcul) *c++ = ' ';
    } 

    strcpy(c,Texte);

    return tampon;
} 

C++ (@++)<!--

BruNews · Answer

Comme dit plus haut, acceptable venant d'un débutant mais totalement à refaire pour passer en prod.
Réfléchir à ce que fait strcat, tu verras que ce n'est pas productif dans une boucle.
ZeroMemory() est inutile si codé comme il se doit.
TOUJOURS vérifier les paramètres.


Autre version:


char *CentreTexte(char *Texte,int nmax)
{
  int len;
  char *d, *tampon = 0;
  len = strlen(Texte);
  if(!len) goto centrEXIT;
  len++;
  if(nmax < len) namx = len;
  tampon = (char*) malloc(nmax);
  if(!tampon) goto centrEXIT;
  d = tampon;
  nmax -= len;
  nmax /= 2;
  while(nmax) {
    *d++ = ' ';
    nmax--;
  }
  while(*d = *Texte) {d++; Texte++;}
 centrEXIT:
  return tampon;
}

ciao...
BruNews, MVP VC++

SAKingdom · Answer

Bizarre. 
Quand je regarde le listing de nos code optimisé, le compilo transforme mon
while(c < calcul) *c++ = ' ';
et ton
while(nmax) {
    *d++ = ' ';
    nmax--;
  }
en un appel à memset...

C++ (@++)<!--

BruNews · Answer

Rien de bizarre, que de l'habituel. Quand il reconnait la signature ou plutot l'implem de memset, il fait cela à tout coup.

On s'en aperçoit de suite quand on bosse sans CRT, on a un message d'alerte dans la fenetre de compilation. En ce cas, je vire le memset en codant inversé: 'if' avant et 'do while' ainsi il me lache la grappe (le code voulais-je dire).

ciao...
BruNews, MVP VC++

SAKingdom · Answer

Bien vue.
Je trouve ça quand même stupide de remplacer une opération sur des pointeurs par un couteux appel à une fonction que l'on ne veut pas simplement parce que "ça y ressemble".
Est-il possible de désactiver complètement cette "fonctionnalité" ?

C++ (@++)<!--

BruNews · Answer

Aucune idée, mettre if devant et pousser le while en bas coute si peu que je n'ai jamais perdu de temps à chercher.

ciao...
BruNews, MVP VC++

PADYVEN · Answer

Rebonjour et d'abord merci


BruNew dans ton exemple reecrit ne doit tu pas libere le malloc a un moment donné
sinon j'ai repri le code pour le transformer comme suivant

void CentreTexte(char *Texte, char *tampon, int nbCaractereMax)
{
    char *c = tampon;
    int len = strlen(Texte);


    if (nbCaractereMax > len)
    {
        int calcul = ((nbCaractereMax-len)/2)+(int)c;
        while(c < (char*)calcul) *c++ = ' ';
    }
    strcpy(c,Texte);
}

ce qui ce passe c'est que j'affiche un menu vertical avec texteout et les items de mon menu font au max 24characteres


comme vous l'avez compris je veux juste ajouter des espaces devant pour centrer les items de mon menu
la declaration de mon menu:
char *TableNomMenu2[9]={"Créer les liens","Configurer les cellules","Interconnecter","Effacer les liens","Effacer les configs","Effacer tous les liens","Effacer ttes les Configs","Tout effacer","Configurer"};

l'utilisation
char tampon[24]={0};
 CentreTexte(TableNomMenu2[compteur],(char *)&tampon,24);
 SetTextColor(hdc,0x0);
 TextOut(hdc,pos.x,pos.y,tampon,strlen(tampon));

est ce que tout est correct cette fois????????
encore merci

SAKingdom · Answer

&tampon
Pas nécessaire, simplement à passer tampon directement.
CentreTexte(TableNomMenu2[compteur],tampon,24);

C++ (@++)<!--

PADYVEN · Answer

Merci a tous


a +

Probleme de language c

13 réponses

Votre réponse

Discussions similaires