Discussion :

[Tittom]

Bonjour,

Je débute en C, et je me demandais s'il est possible de passer une concaténation à une fonction.

Exemple :

Code :

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

1
2
3
4
5
6
7
8
9
10
11
12
13
14
 
void exploiter_chaine(const char *texte)
{
    /* j'utilise ma chaine pour l'ecrire dans un fichier, l'afficher, etc... */
}
 
int main(int argc, char * const argv[])
{
    char *chaine="John";
    exploiter_chaine(strcat("Bonjour ", chaine));
    chaine = "Marie";
    exploiter_chaine(strcat("Bonjour ", chaine));
    return 0;
}

Quand j'utilise ce genre de chose, car je crois qu'on peut parler de chose , j'ai des choses bizarres dans la chaine que je récupère..., le deuxième appel de exploiter_chaine ne contient pas "Bonjour Marie", mais quelque chose comme "jour JohnBonjour Marie"...

Je pense que je fais quelque chose pas bien du tout !

[Franck.H]

Le mieux serais de créer un buffer temporaire (un espace de type char* alloué dynamiquement ou un tableau static si tu connais par avance la longueure de ta chaîne) qui contiendrais la chaîne puis le passer à la fonction, là c'est pas bon ce que tu fait !

[crocodilex]

Bonjour,

Je débute en C, et je me demandais s'il est possible de passer une concaténation à une fonction.

Exemple :

Code :

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

1
2
3
4
5
6
7
8
9
10
11
12
13
14
 
void exploiter_chaine(const char *texte)
{
    /* j'utilise ma chaine pour l'ecrire dans un fichier, l'afficher, etc... */
}
 
int main(int argc, char * const argv[])
{
    char *chaine="John";
    exploiter_chaine(strcat("Bonjour ", chaine));
    chaine = "Marie";
    exploiter_chaine(strcat("Bonjour ", chaine));
    return 0;
}

Quand j'utilise ce genre de chose, car je crois qu'on peut parler de chose , j'ai des choses bizarres dans la chaine que je récupère..., le deuxième appel de exploiter_chaine ne contient pas "Bonjour Marie", mais quelque chose comme "jour JohnBonjour Marie"...

Je pense que je fais quelque chose pas bien du tout !

Toi, tu n'as pas lu le man de strcat. Pourtant c'est très clair :

#include <string.h>
char *strcat (char *dest, const char *src);

La fonction strcat() ajoute la chaîne src à la fin de la chaîne dest en écrasant le caractère `\0' a la fin de dest, puis en ajoutant un nouveau caractère `\0' final. Les chaînes ne doivent pas se chevaucher, et la chaîne dest doit être assez grande pour accueillir le resultat.

[Tittom]

Le mieux serais de créer un buffer temporaire (un espace de type char* alloué dynamiquement ou un tableau static si tu connais par avance la longueure de ta chaîne) qui contiendrais la chaîne puis le passer à la fonction, là c'est pas bon ce que tu fait !

Justement, ma question de fond consistait à tater le terrain pour voir s'il est possible de passer des chaines à une fonction, sans utiliser de buffer, par simple flemmardise. Mais je crois comprendre que le C n'aime pas les flemmards, et requiert rigueur et prévoyance dans la déclaration des structures destinées à manipuler les données ! Je me trompe ? (ce n'est pas une critique, c'est un constat )

Toi, tu n'as pas lu le man de strcat. Pourtant c'est très clair

J'imagine que ça fait partie des habitudes évidentes qu'un débutant - que je suis - doit prendre, en effet. Merci pour la remarque .

[fearyourself]

Le mieux serais de créer un buffer temporaire (un espace de type char* alloué dynamiquement ou un tableau static si tu connais par avance la longueure de ta chaîne) qui contiendrais la chaîne puis le passer à la fonction, là c'est pas bon ce que tu fait !

Je ne dirais pas "le mieux", le code présenté est faux.

Une chaîne littérale n'est pas forcément dans une zone de texte modifiable. Aucune allocation suppléméntaire n'est faite pour ajouter derrière ces chaînes des informations.

Donc n'importe quel ajout ou modification d'une chaîne littérale rend le code inutilisable.

Jc

[Franck.H]

Je ne dirais pas "le mieux", le code présenté est faux.

C'est ce que je sous-entendais ... la prochaine fois je serais un peu plus direct alors

[Tittom]

Je ne dirais pas "le mieux", le code présenté est faux.

Ha, alors je suis recalé ?


Bon bon, je vais essayer de préciser ce que je veux faire et comment je comptais le faire, car je suis sûr que vos lumières seront un chemin de sagesse qu'il sera bon pour moi de suivre

Je veux faire une fonction d'affichage de log, ayant les caractéristiques suivantes :
- reçoit une chaine en paramètre
- affiche sur stdout (printf ou autre ?) la chaine, préfixée par la date et l'heure formatées en AAAAMMJJ-HHMMSS
- pas de valeur de retour

Je veux pouvoir utiliser cette fonction en lui passant des chaines concaténées. (par exemple concatenation de "Itération numero " et "27" si vous voyez ce que je veux dire).

Voici ce que j'ai pondu :

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
 
/*
Juste un petit test de fonction d'affichage de chaine
*/
#include <stdio.h> 
#include <time.h> 
 
/**
 * wlog
 * This function sends a timestamped message to stdout
 * In : 
 * string : message text
 * Out : 
 * Return : 
 */
void wlog(const char *text)
{
 char date_time[16];
 time_t ltime; 
 
 time(&ltime);
 strftime(date_time, sizeof date_time, "%Y%m%d-%H%M%S", localtime(&ltime));
 printf("[%s] %s\n", date_time, text);
} /* wlog */
 
int main(int argc, char * const argv[])
{
 int i = 0;
 char *message = "";
 
 for (i=0; i<10; ++i)
 {
  sprintf(message, "Iteration numero %d ", i);
  wlog(message);
 }
 return 0;
} /* main */

Je me demande :
- si l'affichage de la date ne peut pas être plus simple ?
- si la déclaration de "message" est propre ?

Merci de vos commentaires

[emardjean]

Salut

char *message = "";
n'est pas bon. Tu utilises un pointeur sur une zone d'un seult caratere
utilise plutot

char message[MAXSIZE];

[Emmanuel Delahaye]

Bonjour,

Je débute en C, et je me demandais s'il est possible de passer une concaténation à une fonction.

Exemple :

Code :

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

1
2
3
4
5
6
7
8
9
10
11
12
13
14
 
void exploiter_chaine(const char *texte)
{
    /* j'utilise ma chaine pour l'ecrire dans un fichier, l'afficher, etc... */
}
 
int main(int argc, char * const argv[])
{
    char *chaine="John";
    exploiter_chaine(strcat("Bonjour ", chaine));
    chaine = "Marie";
    exploiter_chaine(strcat("Bonjour ", chaine));
    return 0;
}

Quand j'utilise ce genre de chose, car je crois qu'on peut parler de chose , j'ai des choses bizarres dans la chaine que je récupère..., le deuxième appel de exploiter_chaine ne contient pas "Bonjour Marie", mais quelque chose comme "jour JohnBonjour Marie"...

Je pense que je fais quelque chose pas bien du tout !

En C, il n'y a pas de magie. Si tu veux créer une chaine de caractères à partir d'autres (concaténation), il faut définir un tableau de char suffisament grand et recopier les chaines dedans.

Code :

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

1
2
3
4
5
6
 
 char out[BIG_ENOUGH];
 
   strcpy(out, "Hello");   
   strcat(out, " ");   
   strcat(out, "world");

ou

Code :

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

1
2
 
 sprintf (out, "%s %s", "Hello", "World");

etc.

Pour définir le tableau de destination, on peut calculer sa taille à partir des chaines à concaténer, sans oublier ce qu'il peut y avoir en plus (ici, un espace) et le 0 final... puis allouer le bloc avec malloc().

Enfin, on peu se construire un objet chaine 'avancé' qui s'agrandit automatiquement selon les besoins (à coup de realloc() invisibles)...

http://emmanuel-delahaye.developpez.com/clib.htm
Module FSTR

[zul]

emmanuel il est l'heure d'aller te reposer Il faut lire bien sur

Code :

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

1
2
3
4
5
6
 
 char out[BIG_ENOUGH];
 
   strcpy(out, "Hello");   
   strcat(out, " ");   
   strcat(out, "world");

ou

Code :

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

1
2
 
 sprintf (out, "%s %s", "Hello", "World");

[Emmanuel Delahaye]

emmanuel il est l'heure d'aller te reposer Il faut lire bien sur

Code :

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

1
2
3
4
5
6
 
 char out[BIG_ENOUGH];
 
   strcpy(out, "Hello");   
   strcat(out, " ");   
   strcat(out, "world");

ou

Code :

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

1
2
 
 sprintf (out, "%s %s", "Hello", "World");

Oui, merci. Corrigé. (Maudzit copié/collé...)