Discussion :

[Invité]

Bonjour,

J'essaie de faire un démineur en GTK, et c'est la première fois que j'utilise cette librairie donc j'y connais vraiment rien.
Donc voilà, mon démineur marchait bien dans la console uniquement avec les printf, mais un affichage graphique c'est toujours mieux. Donc j'ai fait les fonctions principales du démineur le problème étant que j'arrive pas à rafraîchir les cases que j'ai découvert. En gros, je veux qu'à chaque clicks de l'utilisateur, la programme réexécute ma fonction aff2 (qui permet d'afficher les images dans un grid). J'ai testé en faisant un affichage console et en cliquant sur la fenêtre GTK, ça marche, le problème vient donc de ma fonction aff je pense.
Voilà le code simplifié du programme:
https://pastebin.com/qfcUPt4S

Cordialement.

[gerald3d]

Bonjour.

Le principe d'un affichage fenêtré repose sur celui des poupées russes. On crée une fenêtre principale dans la quelle on insère un widget quelconque et ainsi de suite...

Au premier affichage tu insères dans ta table des GtkImage*. Pour pouvoir "rafraîchir" ta table il te faut supprimer ces GtkImage avant de pouvoir en insérer de nouveaux.

Tu ne respectes pas le principe des poupées russes si tu essayes de mettre deux GtkWidget dans le même conteneur.

[gerald3d]

Pendant que j'y suis quelques remarques .

Tu as créé une structure s_data. C'est une bonne chose . Elle contient le pointeur sur la fenêtre principale et le pointeur sur la table.

Code :

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

1
2
3
4
struct s_data {
  GtkWidget *window;
  GtkWidget *table;
};

Pourquoi alors la fonction aff2(); à ce prototype : void aff2(GtkWidget * window, GtkWidget*table); ? Il suffit de transmettre le pointeur de data. Le prototype serait plutôt void aff2(struct s_data *data);. Tout ceci serait vrai bien entendu si tu initialises data avant l'appel à aff2();

Histoire de pousser un peu plus loin, évite tant que faire ce peut les déclarations de variable en global, c'est à dire en dehors de toute fonction. Pour un code de 200 lignes ce n'est pas très génant, on arrive à retrouver nos petits, mais si tu codes un projet beaucoup plus gros et que tu gardes cette habitude, tu vas rapidement t'y perdre. Il serait peut être intéressant d'insérer dans la structure les deux tableaux :

Code :

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

1
2
3
4
5
6
7
struct s_data {
  GtkWidget *window;
  GtkWidget *table;
 
  int tdj[12][12];
  int tdv[12][12];
};

Ainsi, il te suffit de propager data à toutes tes fonctions pour accéder aux données nécessaires. Et il n'y a plus de variables globales !

Pour finir, tu charges les images depuis le disque dur à chaque rafraîchissement. Ce n'est pas très performant. Essaye de faire en sorte de ne les charger qu'une seule fois lors de l'initialisation. Tu les places dans un tableau auquel tu peux faire référence dans aff2 ();.

J'arrête là mes petites remarques, constructives je l'espère .

[liberforce]

Bonjour, voilà quelques conseils:

• - sépare le fond (l'état de test cases) de la forme (la manière dont tu les affiches)
• - précharge et mets de côté toutes les images donc tu auras besoin
• - utilise gtk_image_set_from_pixbuf pour mettre à jour le contenu des cases. Comme tu as déjà l'état de la case et chargé les images, tu as très peu de choses à faire dans cette fonction
• - utilise snprintf pour générer le nom du fichier d'image ou g_strdup_printf, ça évite les d'écrire 15 fois la même chose avec just un chiffre qui change:

Code :

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

image = gtk_image_new_from_file("0.png");

Autre remarque, à quoi servent tdj et tdv ? On dirait que tu gères le contenu de la table et sa visibilité dans des tables différentes. Utilise plutôt une table unique et un masque de bits pour savoir si une case a été découverte. Pas besoin d'un int pour l'état d'une case, tu as 10 états possibles seulement (4 bits), et pour chaque état savoir si la case est découverte ou pas (1 bit). Donc un guint8 (équivalent d'un unsigned char) est suffisant.

Coté GTK+, le delete-event ne te sert à rien, tu n'as que le signal destroy à gérer. Pour celà, on appelle en général gtk_main_quit() directement en tant que callback:

Code :

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

g_signal_connect(window, "destroy", G_CALLBACK (gtk_main_quit), NULL);

[Invité]

Bonjour,

J'ai réussi à faire fonctionner mon code aujourd'hui, mais je vais l'améliorer avec tout ce que vous avez cité.
Merci beaucoup.

Cordialement.