Discussion :

[minibus]

Bonjour ! J'ai encore une question à propos du tuto GTK+ par l'exemple : dans le chapitre 8 on convertit le contents du codage locale vers utf8, tandis que dans la même fonction dans le chapitre 9 on ne le fait plus. La fonction a été un peu modifiée entre les deux chapitres, mais je ne comprends pas pourquoi il n'y a plus la conversion locale vers utf8...

dans le chapitre 8 :

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
static void open_file (const gchar *file_name, GtkTextView *p_text_view)
{
  g_return_if_fail (file_name && p_text_view);
  {
    gchar *contents = NULL;
 
    if (g_file_get_contents (file_name, &contents, NULL, NULL))
    {
      /* Copie de contents dans le GtkTextView */
      gchar *utf8 = NULL;
      GtkTextIter iter;
      GtkTextBuffer *p_text_buffer = NULL;
 
      /* Pour l'instant il faut allouer la memoire, par la suite on modifiera
         simplement le pointeur */
      docs.actif = g_malloc (sizeof (*docs.actif));
      docs.actif->chemin = g_strdup (file_name);
      /* Pour l'instant, on se contente de stocker le seul GtkTextView qui existe,
        par la suite, il faudra en creer un nouveau ! */
      docs.actif->p_text_view = p_text_view;
      /* Le document vient d'etre ouvert, il n'est donc pas modifie */
      docs.actif->sauve = TRUE;      p_text_buffer = gtk_text_view_get_buffer (p_text_view);
      gtk_text_buffer_get_iter_at_line (p_text_buffer, &iter, 0);
      utf8 = g_locale_to_utf8 (contents, -1, NULL, NULL, NULL);
      g_free (contents), contents = NULL;
      gtk_text_buffer_insert (p_text_buffer, &iter, utf8, -1);
      g_free (utf8), utf8 = NULL;
    }
    else
    {
    	print_warning ("Impossible d'ouvrir le fichier %s\n", file_name);
    }
  }
}

dans le chapitre 9 :

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
static void open_file (const gchar *file_name, GtkTextView *p_text_view)
{
  g_return_if_fail (file_name && p_text_view);
  {
    gchar *contents = NULL;
 
    if (g_file_get_contents (file_name, &contents, NULL, NULL))
    {
      /* Copie de contents dans le GtkTextView */
      GtkTextIter iter;
      GtkTextBuffer *p_text_buffer = NULL;
 
      cb_new (NULL, p_text_view);
      gtk_widget_set_sensitive (GTK_WIDGET (docs.actif->p_text_view), TRUE);
      p_text_buffer = gtk_text_view_get_buffer (p_text_view);
      gtk_text_buffer_get_iter_at_line (p_text_buffer, &iter, 0);
      gtk_text_buffer_insert (p_text_buffer, &iter, contents, -1);
      /* Nous sommes obliges de remetre sauve a TRUE car l'insertion du contenu
         du fichier dans le GtkTextView a appele cb_modfie */
      docs.actif->sauve = TRUE;
    }
    else
    {
    	print_warning ("Impossible d'ouvrir le fichier %s\n", file_name);
    }
  }
}

le dernier code fait appel à cb_new():

Code :

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

1
2
3
4
5
6
7
8
9
10
11
12
13
void cb_new (GtkWidget *p_widget, gpointer user_data)
{
  /* Pour l'instant il faut allouer la memoire, par la suite on modifiera
     simplement le pointeur */
  docs.actif = g_malloc (sizeof (*docs.actif));
  docs.actif->chemin = NULL;
  /* Pour l'instant, on se contente de stocker le seul GtkTextView qui existe,
    par la suite, il faudra en creer un nouveau ! */
  docs.actif->p_text_view = GTK_TEXT_VIEW (user_data);
  /* Le document vient d'etre creer, il n'est donc pas modifie */
  docs.actif->sauve = TRUE;
  gtk_widget_set_sensitive (GTK_WIDGET (docs.actif->p_text_view), TRUE);
}

D'autre part dans cb_new on voit qu'on utilise :

Code :

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

 gtk_widget_set_sensitive (GTK_WIDGET (docs.actif->p_text_view), TRUE);

donc pourquoi le remettre dans cb_open_file (dans le chapitre 9) alors qu'on fait appel à cb_new juste avant ?
Merci pour votre aide !

[teuf13]

Ma position sur le sujet, c'est qu'il ne faut pas utiliser g_locale_*, mais les gens qui font les tutoriaux adorent en mettre. Pas de commentaire à ajouter en dehors de ça

[minibus]

Bah, ça améliore la portabilité du programme, non ?

[teuf13]

De quelle façon? Au contraire, ça rend ton programme dépendant de la locale que tu as choisie d'utiliser, alors que si tu mets toutes tes chaînes en utf8, tu n'as pas ce pb, ça sera interprété pareil partout.

PS: j'ai relu le code que tu as donné, c'est utilisé pour convertir des données entréees par l'utilisateur, donc effectivemnet pourquoi pas, c'est même mieux

[minibus]

j'ai relu le code que tu as donné, c'est utilisé pour convertir des données entréees par l'utilisateur, donc effectivemnet pourquoi pas, c'est même mieux

justement, l'auteur le retire à partir du chapitre 9 et je n'y vois pas de raison...

[gege2061]

PS: j'ai relu le code que tu as donné, c'est utilisé pour convertir des données entréees par l'utilisateur, donc effectivemnet pourquoi pas, c'est même mieux

Sauf que rien ne garantie que le fichier est au même encodage que le système, mais c'est mieux que rien, ce n'est pas évident de faire un bon convertisseur.

justement, l'auteur le retire à partir du chapitre 9 et je n'y vois pas de raison...

Sûrement un problème de copier/coller

[minibus]

justement, l'auteur le retire à partir du chapitre 9 et je n'y vois pas de raison...

Sûrement un problème de copier/coller

OK, merci !
et pour l'utilisation deux fois de suite de :

Code :

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

gtk_widget_set_sensitive (GTK_WIDGET (docs.actif->p_text_view), TRUE);

C'est juste une erreur d'inattention, ou j'ai mal compris quelque chose ?

[gege2061]

OK, merci !
et pour l'utilisation deux fois de suite de :

Code :

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

gtk_widget_set_sensitive (GTK_WIDGET (docs.actif->p_text_view), TRUE);

C'est juste une erreur d'inattention, ou j'ai mal compris quelque chose ?

C'est aussi une bonne remarque

Peux-tu les reporter dans le sujet de l'article : http://www.developpez.net/forums/d65...gtkp-lexemple/ ?

ça évitera que d'autres se posent la question, et ça m'évitera d'oublier de corriger

[minibus]

J'ai laissé un petit commentaire, merci de m'avoir éclairé !