Discussion :

[fomblardo]

Bonjour,

j'ai un problème que je comprends absolument pas lié à mon GTK_Entry.

Voici le code :

Code :

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

1
2
3
4
 text = gtk_entry_get_text(GTK_ENTRY(entry1));
 aux = gtk_entry_get_text(GTK_ENTRY(entry2));
 strcat (text,aux);
 strcat (text,".txt");

Jusque là tout marche, cela me renvoie un nom de fichier texte.

Mais le problème est que lorsque je refais un get_text pour mon entry1, et bien ce qu'il y avait dedans a changé, ce n'est plus le texte que j'avais tapé dans le entry, mais le texte présent dans la variable text.

Exemple :
entry1 : Salut
entry2 : Bonjour

après le code, dans la variable text il y a : "SalutBonjour.txt".

Lorsque j'essaie de voir ce qu'il y a dans le entry1, je trouve "Salut Bonjour.txt"....

Je ne vois pas du tout d'où ça vient.

Quelqu'un pour m'aider please.
Merci d 'avance.

EDIT : Déclaration des variabales

Code :

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

1
2
3
 
 const gchar * text;
 const gchar * aux;

[Franck.H]

Ainsi qu'il est écrit dans la ref de GTK+ sur le GtkEntry:

Returns : a pointer to the contents of the widget as a string. This string points to internally allocated storage in the widget and must not be freed, modified or stored.

En d'autres termes, il te faut créer une nouvelle chaîne faisant au minimum la taille totale de ta future chaîne complète + 1 pour le zéro final:

Code :

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

1
2
3
4
5
6
7
 
gchar * str = g_malloc (strlen (text) + strlen (aux) + 1);
 
if (str != NULL)
{
   sprintf ("%s%s.txt", text, aux);
}

[fomblardo]

Merci pour ta réponse rapide.

Donc ça donnerait ça avec la nouvelle variable ?

Code :

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

1
2
3
4
5
6
7
8
9
10
11
12
const gchar * text;
const gchar * aux;
gchar * contact; 
 
text = gtk_entry_get_text(GTK_ENTRY(entry1));
aux = gtk_entry_get_text(GTK_ENTRY(entry2));
str = g_malloc (strlen (text) + strlen (aux) + 1);
 
 if (str != NULL)
    {
    str = sprintf ("%s %s.txt", text, aux);
    }

On aurait alors dans str = "%s %s.txt" et le entry1 serait inchangé ?

[Franck.H]

Merci pour ta réponse rapide.

Donc ça donnerait ça avec la nouvelle variable ?

Code :

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

1
2
3
4
5
6
7
8
9
10
11
12
const gchar * text;
const gchar * aux;
gchar * contact; 
 
text = gtk_entry_get_text(GTK_ENTRY(entry1));
aux = gtk_entry_get_text(GTK_ENTRY(entry2));
str = g_malloc (strlen (text) + strlen (aux) + 1);
 
 if (str != NULL)
    {
    str = sprintf ("%s %s.txt", text, aux);
    }

On aurait alors dans str = "%s %s.txt" et le entry1 serait inchangé ?

Oui sauf que je me suis juste trompé dans l'utilisation de sprintf, j'ai oublié le premier argument ce qui nous donne alors:

Code :

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

1
2
3
4
5
6
7
8
9
10
11
12
const gchar * text;
const gchar * aux;
gchar * contact; 
 
text = gtk_entry_get_text(GTK_ENTRY(entry1));
aux = gtk_entry_get_text(GTK_ENTRY(entry2));
str = g_malloc (strlen (text) + strlen (aux) + 1);
 
if (str != NULL)
{
   sprintf (str, "%s %s.txt", text, aux);
}

On aurait alors dans str = "%s %s.txt" et le entry1 serait inchangé

En réalité, les deux GtkEntry restent inchangés !

[fomblardo]

Ok merci beacoup à vous deux, j'ai fait un mix des 2 fonctions (sprintf , gstrdup) et ça marche impec

Un grand merci

[gege2061]

Ainsi qu'il est écrit dans la ref de GTK+ sur le GtkEntry:

En d'autres termes, il te faut créer une nouvelle chaîne faisant au minimum la taille totale de ta future chaîne complète + 1 pour le zéro final:

Code :

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

1
2
3
4
5
6
7
 
gchar * str = g_malloc (strlen (text) + strlen (aux) + 1);
 
if (str != NULL)
{
   sprintf ("%s%s.txt", text, aux);
}

g_strdup est fait pour ça

[Franck.H]

g_strdup est fait pour ça

Certes mais ca ne créé qu'une copie de la chaîne alors que là il s'agit de faire une concaténation de chaînes, je pense que c'est inutile de faire une copie de chaque chaînes pour ensuite faire une concaténation dans un nouvelle, etc...

[gege2061]

Certes mais ca ne créé qu'une copie de la chaîne alors que là il s'agit de faire une concaténation de chaînes, je pense que c'est inutile de faire une copie de chaque chaînes pour ensuite faire une concaténation dans un nouvelle, etc...

Dans ce cas g_strdup_printf

[Franck.H]

Dans ce cas g_strdup_printf

Ouais ca c'est une solution, j'y avais pas pensé