SUPPRIMER LES CARACTERES D'UNE CHAINERésolu

Question

bonjour
comment procéder pour supprimer quelques derniers caractères d'une chaine
merci
char ChaineCaractères[MAX_PATH];

ben92600 · Answer

ba soit tu refait une chaine ou tu les retire soi tu renplace ces
carateres par des \0 pouisque ce son les dernier de la cahine sa posera
pas de probleme ;)

SAKingdom · Answer

Tien essais cette fonction à cette addresse:
http://www.codyx.org/snippet_effacer-caracteres-partir-fin_168.aspx

___________________________________________
Les plus grands esprits trouvent toujours une solution

vecchio56 · Answer

C'est même pas utile de mettre tous la caractères à zéro, seul le premier est important:

void EraseLastChar (char *value, int count)
{
  *(value + strlen(value) - count) = 0;
}

SAKingdom · Answer

Oui on peu aussi faire quelque chose comme ça:


void EraseLastChar (char *value, int count)

{

    char *c = value;


    while(*(++c));


    *(c-count) = 0;

}


Je modifirais bien ma fonction sur Codyx mais je suis pas encore capable (problème avec les routines qu'a dit Nix je crois)

___________________________________________
Les plus grands esprits trouvent toujours une solution

vecchio56 · Answer

Ta variable c ne sert a rien, autant utiliser value directement
Et tu as quelque chose contre strlen?

SAKingdom · Answer

Ben c qu'on peut faire directement l'opération de strlen sans passer par lui. C'est plus rapide non? Pas besoin de l'appeler.

___________________________________________
Les plus grands esprits trouvent toujours une solution

vecchio56 · Answer

strlen doit être optimisé donc peut être plus rapide

SAKingdom · Answer

Tien j'ai fais un listing de ta fonction et de la mienne. Voici la tienne:

    mov    edx, DWORD PTR _value$[esp-4]
    push    edi
    mov    edi, edx
    or    ecx, -1
    xor    eax, eax
    repne scasb
    mov    eax, DWORD PTR _count$[esp]
    pop    edi
    not    ecx
    dec    ecx
    sub    edx, eax
    mov    BYTE PTR [ecx+edx], 0

    12 opérations et ce sans compter les opérations de strlen

Voici la mienne:

    mov    eax, DWORD PTR _value$[esp-4]
    inc    eax
    cmp    BYTE PTR [eax], 0
    je    SHORT $L57476
$L57475:
    mov    cl, BYTE PTR [eax+1]
    inc    eax
    test    cl, cl
    jne    SHORT $L57475
$L57476:
    sub    eax, DWORD PTR _count$[esp-4]
    mov    BYTE PTR [eax], 0

    10 opérations

C'est pas moi qui a inventé ce codage. C'est généré avec le compilateur. En plus, j'ai activé les options du compilateur pour une optimisation de la vitesse.

___________________________________________
Les plus grands esprits trouvent toujours une solution

vecchio56 · Answer

Et?? Tu vois comme ca laquelle est la plus rapide? Une code plus court n'est pas forcément plus rapide
Peut être que ta fonction est plus rapide, mais faudrait tester pour voir.

SAKingdom · Answer

Ahhh si BruNews passait par ici, il nous dirait ça tout de suite lui. Bon je vais essayer de tester la rapidité. Je reviend avec les resultats plus tard.

___________________________________________
Les plus grands esprits trouvent toujours une solution

vecchio56 · Answer

Pour avoir des résultats assez precis:

http://www.cppfrance.com/codes/TESTS-RAPIDITE-WIN32_17060.aspx

J'avais la flemme de le faire ;)

SAKingdom · Answer

Hihihi ses codes ne sont pas de tout repos. Bon, j'ai patenté un peu le code pour qu'il teste nos 2 fonctions. J'ai tester 4 fois et j'ai fait une moyenne:
Ta fonction: 992
Ma fonction: 158

Voila. Je te jure que je n'est pas trafiqué les resultats.

___________________________________________
Les plus grands esprits trouvent toujours une solution

BruNews · Answer

pas le temps d'essayer mais, quel compilo a généré du 'repne scasb' ??? ces antiques instructions de chaine sont à proscrire, elles forcent une sauvegarde de EDI et sont bien plus lentes qu'une boucle ordinaire. Intel en préconisait l'abandon depuis le pentium.

ciao...
BruNews, MVP VC++

BruNews · Answer

oups, trop tard...

ciao...
BruNews, MVP VC++

SAKingdom · Answer

C'est le vieux compilateur de VC98. JE SAIS c'est pas le mieu. J'ai effectuer cependant les tests avec VC2005.

___________________________________________
Les plus grands esprits trouvent toujours une solution

vecchio56 · Answer

C'est ma faute aussi si ton compilo génère des vieilles instructions de chaine

SAKingdom · Answer

Bon bon je m'excuse mais j'ai bel et bien effectué les tests de performance avec VC2005 je le jure.

___________________________________________
Les plus grands esprits trouvent toujours une solution

vecchio56 · Answer

Pas la peine de jurer, je te crois

cs_Batman60 · Answer

bin j'ai essayé la première bonne  solution de sakingdom
ca fonctionne impec
pour les autres je sais pas
merci à tous

BruNews · Answer

void EraseLastChar(char *value, int count)
{
  char *c = value;
  while(*(++c));
  while(count--) *(--c) = 0;
}

while(*(++c)); Si la chaine est vide (1er octet deja 0), tu pars au dela. Pas forcément grave mais est-ce le but ???

while(count--) *(--c) = 0;
Enorme le risque de plantage, quand on isole dans une fonction c'est qu'elle est dument testée, count pourrait être largement supérieur au strlen() et en ce cas....

void EraseLastChar(char *value, int count)
{
  char *c = value;
  while(*c) c++; // CERTAIN SE STOPPER AU ZERO
  c -= count;  if(c >value) *c 0;
  else value[0] = 0; // BON A TOUT COUP
}

Ceci dit je trouve ce genre de fonction totalement inutile, avec un mois de dev on écrit cela comme on respire donc on le met direct dans le code quand besoin sans appel de fonction.
Laissez donc les snippets aux dotnetteurs, n'a aucun intérêt réel en C/C++ ou alors serait un gros morceau de code.

ciao...
BruNews, MVP VC++

SAKingdom · Answer

Je note. Merci pour les corrections et pour le conseil sur Codyx.
En passent, je sais bien que faire une fonction pour si peu est inutile mais c'est pour montrer le principe mais bon, si tu dit que ça n'a pas sa place sur Codyx, je ne vais pas te contredire. Après tout, c'est toi le pro :) .

___________________________________________
Les plus grands esprits trouvent toujours une solution

JejeLand · Answer

Moi j'ai ça :

  chaine[strlen(chaine)] = 0x00

vecchio56 · Answer

Ca c'est une affectation qui ne fait jamais rien d'intéressant
Car chaine[strlen(chaine)] vaut toujours 0 (par définition de strlen)

SAKingdom · Answer

À la limite il faudrait au moin faire ceci chaine[strlen(chaine-nomdecaractère)] = 0x00 pour que ça donne quelque chose. Et coté performance, c'est une vrai calamité. J'en vien presqu'a dire qu'il faudrait s'absetenir d'utiliser strlen quand il n'est pas absolument nécessaire.

___________________________________________
Les plus grands esprits trouvent toujours une solution

vecchio56 · Answer

chaine[strlen(chaine-nomdecaractère)] = 0x00
Non, ce n'est pas (forcément) bon

Plutot: chaine[strlen(chaine)-nomdecaractère] = 0x00

SAKingdom · Answer

Ah oups. Petit erreur dans le post. Pourtant je l'ai bien écrit quand j'ai effectuer les tests.

___________________________________________
Les plus grands esprits trouvent toujours une solution

SUPPRIMER LES CARACTERES D'UNE CHAINE

26 réponses

Votre réponse

Discussions similaires