Discussion :

[VinceBassman]

Bonjour,

Je suis a la recherche d'une fonction en C++ pour remplacer un fragment par un autres dans une chaine de caracteres...
Les fragments n'ont pas forcement la meme longueur.
Par exemple:
char *s = "je cherche une fonction";
char *old = "cherche une fonction";
char *new = "'ai trouve!";
// printf de s donne "je cherche une fonction"
string_replace(s,old,new);
// printf de s donne "j'ai trouve une fonction"

Si qqun peut me donner un coup de main, ca serait sympa!
Merci d'avance,
VB.

[Médinoc]

Déjà, ta question parait incohérente.
Ensuite, on ne pond pas de code sur demande, comme indiqué dans les règles du forum. Tu présentes ton code, et peut-être qu'on t'aidera à corriger les erreurs...

[VinceBassman]

Bon ben je ne vois pas en quoi c'est incoherent, mais bon, je vais simplifier: je souhaite remplacer 'abc' par 'defghij' dans une chaine de caracteres.
Voila ma fonction:

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
 
// Replaces a fragment of a string by another one
char *strstr_rep(char *source, const char *oldfrag, const char *newfrag) {
        char *original = source;
 
printf("INFO --> old fragment: -%s-\n", oldfrag);
printf("INFO --> new fragment: -%s-\n", newfrag);
        char temp[1024];
        int old_length = strlen(oldfrag);
        int i,j,k,l,place=-1;
 
        for (i=0; source[i] && (place == -1); ++i) {
                for (j=i,k=0; source[j] == oldfrag[k]; j++,k++) {
                        if (!oldfrag[k+1]) {
                                place = i;
                        }
                }
        }
        if (place != -1) { 
                // First part of string source is not modified, until place-1
                for (j=0; j < place; j++) {
                        temp[j] = source[j];
                }
                // Insert newfrag from index place
                for (i=0; newfrag[i]; i++,j++) {
                        temp[j] = newfrag[i];
                }
                // Last part of string
                for (k=place+old_length; source[k]; k++,j++) {
                        temp[j] = source[k];
                }
                temp[j] = '\0';
                for (i=0; source[i]=temp[i]; i++) { }
        }
printf("INFO --> modified string: -%s-\n", original);
        return(original);
}

et voila comment je l'appelle:

Code :

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

1
2
3
4
5
6
7
8
9
10
11
12
13
14
const char deffilename = "${RAZ}/@@essai";
char *newdeffilename = new char[strlen(deffilename) + 1];
strcpy(newdeffilename, deffilename);
printf("\nINFO --> this is a copy:-%s-\n", newdeffilename);
if (strchr(newdeffilename, '@')) {
        printf("\nINFO --> this is a metafile:-%s-\n", newdeffilename);
        const char *RAZ_VAR;
        if ((RAZ_VAR = getenv("RAZ")) != 0 && *RAZ_VAR) {
                if (!strncmp(newdeffilename,"${RAZ}",6)) {
                        strstr_rep(newdeffilename,"${RAZ}",RAZ_VAR);
                        printf("\nINFO --> after subsitution, new:%s-!!!\n",newdeffilename);
                }
        }
}

Et en sortie, je me tape un bus error... Donc je voulais savoir s'il n'existe pas une fonction toute faite faisant partie des librairies C/C++ qui fait le meme boulot --> je n'ai jamais demande explicitement qu'on me ponde du code!!!!

Merci d'avance pour vos idees,
VB.

[Médinoc]

Ah oui, parce qu'en plus tu bosses en char*...
Tu devrais déjà voir s'il n'y a rien qui marcherait mieux dans la classe std::string...

Et relis ta question: string_replace("je cherche une fonction", "cherche une fonction", "ai trouvé!") doit-il retourner "je ai trouvé! une fonction" ou plutôt "je ai trouvé" ?

Déjà, ton code appelant est trop compliqué pour un test. Tu devrais juste tenter d'appeler ta fonction avec rien d'autre autour...

Ensuite, je ne vois aucune allocation, alors que ta fonction:

1. Modifie potentiellement la taille de la chaîne, potentiellement en augmentation;
2. Ecrit sur la chaîne originale qui peut être sur une zone en lecture seule (heureusement, ce n'est pas le cas).

Le code char *str="abc" est extrêmement dangereux car les chaînes littérales sont généralement placées dans une zone de mémoire en lecture seule: Toute tentative d'écriture provoque donc au mieux un plantage, au pire un mauvais résultat. Une chaîne dont le contenu (et non la taille) est modifiable se déclare char[] str="abc". Une chaîne littérale devrait toujours être déclarée char const *str="abc" (si tu as GCC, ajouter l'option -Wwrite-strings te permettra d'éviter ce genre d'erreurs).
Bien sûr, en C++, l'usage des std::string peut avantageusement remplacer ces problèmes, le contenu d'une std::string étant toujours alloué dynamiquement...

Edit: Petite correction...

[VinceBassman]

Merci pour tes conseils...
En fait string_replace("je cherche une fonction", "cherche une fonction", "'ai trouvé!") devrait retourne "j'ai trouve!", tu as raison, je me suis un peu emballe dans le premier post...
Par contre, la chaine de caracteres que je veux modifier a ete definie par:
char *newdeffilename = new char[strlen(deffilename) + 1];
Ca n'est pas une allocation?

VB.

[Ti-R]

Salut,

Le code que tu présentes n'est pas du C++, mais du C.

Sinon, en C++ il y a les std::string qui simplifie la vie pour ce qui est des manipulations de chaîne de caractères, Il existe aussi la fonction str_replace qui remplace une chaine de caractere par une autre .

Son code est plus du C++ que du C car il utilise new !
C'est "l'ancienne" manière de coder du C++. Et oui les std::string simplifie la vie

char *newdeffilename = new char[strlen(deffilename) + 1];
Ca n'est pas une allocation?

?
Si...
?

[frenchi]

Son code est plus du C++ que du C car il utilise new !
C'est "l'ancienne" manière de coder du C++. Et oui les std::string simplifie la vie

J'avais pas vu le new
Les printf m'ont perturbés, désolé!

[frenchi]

Salut,

Le code que tu présentes n'est pas du C++, mais du C.

Sinon, en C++ il y a les std::string qui simplifie la vie pour ce qui est des manipulations de chaîne de caractères, Il existe aussi la fonction str_replace qui remplace une chaine de caractere par une autre .