Discussion :

[Maestro_pikal]

Bonjour à tous,

je deviens dingue et je rage comme un fou tellement ça m'énerve. Je suis complètement désolé par avance parce que la question a du être gérée un milliard de fois.

J'ai regardé la FAQ avant de venir ici mais elle ne répond pas à ma question car elle ne traite que chaînes de longueur fixées (char[50], char[100], etc. etc.)

Je souhaite faire quelque chose d'incommensurablement complexe et original : concaténer dans une chaîne de caractère de taille indéfinie (un char * quoi) une variable chaîne dont je peux fixer la longueur là par contre, mais dont le contenu va varier

Exemple :

Code :

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

1
2
char * chaine_finale ;
char[50] mon_buffer ;

au lancement : je veux que chaine_finale = "" et mon_buffer = "tour 1;" par exemple
et mettre le contenu de mon_buffer à la suite du contenu de chaine_finale.

tour 2 : chaine_finale vaut "tour 1;" et mon_buffer vaut "tour 2;".
tour 3 : chaine_finale vaut "tour 1;tour 2;" et mon_buffer vaut "tour 3;".
...
tour n : chaine_finale vaut "tour 1;tour 2;...;tour n-1;" et mon_buffer vaut "tour_n".

me dites pas qu'il faut que je passe par du malloc ou du realloc quand même ... ? si ? y a pas de librairie ni de fonctions qui encapsule la gestion de concaténation dynamique de chaîne en C ?

Help par pitié je vais finir par tuer le chien (qui n'a rien fait) tellement ça m'énerve !

[gerald3d]

Bonjour.

Le prototype de la fonction est-il libre ou imposé ?

[Bousk]

Pourtant dans la FAQ il y a les solutions avec sprintf et strcat.
https://c.developpez.com/faq/?page=L...-de-caracteres
Que les chaînes soient déclarées comme [length] n'a absolument aucune incidence, il suffit de regarder les prototypes de ces fonctions pour s'en assurer.
Et oui si ta chaîne qui reçoit les données n'est pas assez grande, il va bien évidemment falloir passer par du malloc/realloc.

[Maestro_pikal]

Bonjour et merci de votre réponse

Je ne suis pas sûr de bien comprendre la question.

En fait en reprenant mon exemple précédent, ma "chaine_finale" est une ressource partagée.

J'ai un thread qui vient la remplir (avec mon_buffer du coup dans mon exemple)

Et derrière j'ai un autre thread qui vient "vider" cette chaîne dans un fichier selon d'autres critères

Je peux bien sûr partir sur une taille fixe surdimensionnée, mais je trouve ça moche d'aller allouer 500000 char de mémoire si je n'en utilise au max que 5000 :s

Vous parlez du prototype de ma fonction du premier thread du coup ?

[Maestro_pikal]

Pourtant dans la FAQ il y a les solutions avec sprintf et strcat.
https://c.developpez.com/faq/?page=L...-de-caracteres
Que les chaînes soient déclarées comme [length] n'a absolument aucune incidence, il suffit de regarder les prototypes de ces fonctions pour s'en assurer.
Et oui si ta chaîne qui reçoit les données n'est pas assez grande, il va bien évidemment falloir passer par du malloc/realloc.

Comme je l'ai dit dans mon premier message, j'avais bien sûr été voir la FAQ avant d'aller poser une question que je sais être posée déjà mille fois.

j'ai regardé les prototypes et du coup oui je ne comprends pas parce que pour moi ça devrait marcher :

le proto de strcat est le suivant, sauf erreur :

Code :

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

char *strcat(char *dest, const char *src)

or dans mon test, je lui passe bien un char * en premier argument :

Code :

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

1
2
3
4
 
char * chaine_finale = "GRRRRRR!" ;
...
strcat(chaine_finale ,"POUUUUUUUUUUUUUUUUUET");

Et derrière j'ai un Seg Fault à chaque fois ... d'où mon incompréhension. :'(

Quant au malloc ou realloc, j'avais le secret espoir qu'il y avait des fonctions qui encapsulaient déjà ce type de fonctionnalités tellement ce type de besoin me semble fréquent ...

[gerald3d]

Dans ce bout de code chaine_finale est de taille finie. En y ajoutant des données tu dépasses sa taille. D’où Le segdefault.

[Bktero]

C'est surtout que ce code essaye d'écrire dans la mémoire constante... Aucune RAM n'est ici allouée pour stocker les caractères.

[Maestro_pikal]

Ok c'est ce que je craignais, merci pour vos réponses ...

Je vais prendre ma croix et aller recoder une usine à gaz de malloc pour gérer ça ... snif

quand tu pense que même un pauvre korn shell sait faire ça les doigts dans le nez ... ça fait de la peine ... enfin, tant pis.

Merci encore à tous.

[Bktero]

Tu crois Korn Shell fait ça comment ?

C'est pas super compliqué non plus, le code doit ressembler à ceci :

Code :

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

1
2
3
length = strlen(a) + strlen(b)
p = malloc(length)
snprintf(p, length, "%s%s", a, b)

[gerald3d]

Je répondrais donc comme tu le soupçonnes. Passer par un malloc / free me semble être le plus simple.

Tu pourrais par exemple faire ceci en considérant que les chaînes transmises sont correctement formatées, c’est à dire se terminant par \0. De plus si on part du principe que la chaîne destination str1 est allouée dans le tas alors on peut la liberer dans la même fonction.

Code :

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

1
2
3
4
5
6
7
8
9
10
char *perso_strcat (char *str1, char *str2)
{
  char *result = malloc (strlen (str1) + strlen (str2) + 1);
  result = strcpy (result, str1);
 
  result = strcat (result, str2);
 
  free (str1); // à voir en fct de ton code
  return result;
}

Je n’ai pas testé ce code mais l’idée générale est là.

[Maestro_pikal]

Yep merci à tous les deux, vous dégainez plus vite que moi ^^

Non je trouve juste dommage que la fonction que tu as décrite Gerald3d ne fasse pas partie d'une bibliothèque intégrée du style <string.h> par exemple.

merci encore à tous

[chrtophe]

Tu as std:string dans la bibliothèque standard, mais du C++.

Si tu débutes, je t'invites à jeter un œil sur C++.

Mais attention C++ c'est pas du C. La syntaxe reste similaire, donc ce que tu as déjà appris en C te servira en cas de passage à C++.

La difficulté est qu'en C, une chaine de caractère ça n'existe pas vraiment. C'est un tableau de char qui doit se terminer par \0. Ce tableau est adressable sous forme de pointeur, celui-ci pointant alors sur le premier caractère de la chaine/
Donc soit tu crée implicitement une chaine de 5 caractères avec un code comme char chaine[5], soit tu dois allouer la mémoire avec malloc (et éventuellement realloc pour modifier la taille). Toutes les fonctions de manipulation de chaine <string.h> considèrent que les chaines traitées ont la place nécessaire et ne font aucun contrôle.

En c++, on parle d'objet. Tu n'as pas à te préoccuper de la taille d'un objet string C++, le système fait le boulot pour toi.

[gl]

C'est pas super compliqué non plus, le code doit ressembler à ceci :

Code :

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

1
2
3
length = strlen(a) + strlen(b)
p = malloc(length)
snprintf(p, length, "%s%s", a, b)

Attention, il manque l'emplacement du 0 terminal, c'est

Code :

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

length = strlen(a) + strlen(b) + 1

[Sve@r]

Bonjour

Je vais prendre ma croix et aller recoder une usine à gaz de malloc pour gérer ça ... snif

Ben oui, c'est justement fait pour ça malloc !!! Il ne faut pas en avoir peur. Et surtout ce n'est pas parce que tu l'utilises que ce que tu fais devient forcément une usine à gaz...

quand tu pense que même un pauvre korn shell sait faire ça les doigts dans le nez ... ça fait de la peine ... enfin, tant pis.

Python aussi et bien d'autres choses (essaye par exemple de coder en C le append() de Python et là tu comprendras vraiment ta douleur). Mais faut savoir ce que tu veux. Si tu veux utiliser un langage aussi fin qu'un cheveu qui te permet de toucher le coeur de la machine avec une précision chirurgicale faut aussi assumer ensuite de devoir tout lui faire faire.

Non je trouve juste dommage que la fonction que tu as décrite Gerald3d ne fasse pas partie d'une bibliothèque intégrée du style <string.h> par exemple.

Moui pourquoi pas. Effectivement puisqu'il existe strdup() et readline() je t'accorde qu'il pourrait exister aussi cette fonction là. On mettra ça sur le compte de la faute à "pas de chance"...

Je répondrais donc comme tu le soupçonnes. Passer par un malloc / free me semble être le plus simple.

Non seulement le plus simple mais aussi et surtout la seule et unique solution possible.

en considérant que les chaînes transmises sont correctement formatées, c’est à dire se terminant par \0.

Pareil, tu es en réalité obligé de faire cette hypothèse. Sinon déjà tu ne peux plus nommer ce que tu reçois "chaine" et surtout tu ne peux plus le traiter

De plus si on part du principe que la chaîne destination str1 est allouée dans le tas alors on peut la liberer dans la même fonction.

Moui. Mais ce que tu présentes comme un avantage est en réalité une faiblesse car tu limites désormais les possibilités d'utilisation de ta fonction (interdit par exemple d'appeler désormais perso_strcat("xxx", "yyy") !!!).

Attention, il manque l'emplacement du 0 terminal, c'est

Code :

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

length = strlen(a) + strlen(b) + 1

Très bien vu (@Bktero quelle erreur !!! )
Toutefois moi je conseillerais de laisser "length" avec sa vraie valeur correspondant à sa définition (donc la longueur exacte de la chaine concaténée) et de rajouter le "+1" au malloc...

[gerald3d]

Bonjour
...
Moui. Mais ce que tu présentes comme un avantage est en réalité une faiblesse car tu limites désormais les possibilités d'utilisation de ta fonction (interdit par exemple d'appeler désormais perso_strcat("xxx", "yyy") !!!).

En effet bonne remarque. Je pousserais le prototype comme ceci char *perso_strcat(const char *str1, const char *str2);

[Bktero]

Attention, il manque l'emplacement du 0 terminal, c'est

Code :

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

length = strlen(a) + strlen(b) + 1

C'est bien pour ça que je disais "doit ressembler"

[chrtophe]

Amélioration possible :

faire une fonction variadique permettant de concaténer un nombre de chaines variable.

[Sve@r]

En effet bonne remarque. Je pousserais le prototype comme ceci char *perso_strcat(const char *str1, const char *str2);

En fait le souci c'était pas vraiment le prototype mais plutôt le free(str1) quoi...

C'est bien pour ça que je disais "doit ressembler"

Argh cte mauvaise foi comme j'y crois trop pas quoi
Moi aussi désormais je vais dire "doit ressembler" pour couvrir mes éventuelles erreurs. Ah ben non, pas besoin vu que je ne commets jamais d'erreur

[chrtophe]

C'est bien connu, c'est l'informatique qui est en cause, pas nous.

[Bktero]

Les chaines en C, j'en fais jamais, du coup du coup je galère à chaque, du coup je fuis encore plus, du coup j'en fais encore moins. Aucun remord.

Moi si je dois concaténer deux chaines, ben je ferais comme ça :

Code :

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

1
2
3
4
5
#include <string>
 
std::string a = "hello";
std::string b = "world";
auto c = a + b;

Hop !

On remarquera d'ailleurs que je n'ai pas posté ça comme du code, c'est en balise PRE en mode pseudo-code. Si j'avais fait un vrai code j'aurais pris la peine de le tester