Cleanup code to remove file-scope the_rack

[wordgame] / rack-fancy.c

diff --git a/rack-fancy.c b/rack-fancy.c
index f2caa13adae6daa53bd43cf896903fda61598dde..8aae0f8b2e58233c620b00f0136e2dc5d495d122 100644 (file)
--- a/rack-fancy.c
+++ b/rack-fancy.c
@@ -46,10 +46,9 @@ typedef struct _rack
     char guess[MAX_TILES+1];
     int guess_length;
     dict_t *solution;
+    GooCanvasItem *solution_item;
 } rack_t;
 
-static rack_t the_rack;
-
 #define LETTER_SIZE 60
 #define LETTER_PAD 5
 
@@ -250,17 +249,32 @@ shuffle (int *array, int length)
 }
 
 static void
-rack_init (rack_t *rack, GooCanvasItem *parent, char *word, dict_t *solution)
+rack_init (rack_t      *rack,
+          dict_t       *solution)
 {
     int i;
 
-    for (i = 0; i < MIN (MAX_TILES, strlen (word)); i++)
-       rack->tiles[i] = tile_create (parent, word[i], i);
-    rack->num_tiles = i;
-    while (i < MAX_TILES)
+    for (i = 0; i < MAX_TILES; i++)
        rack->tiles[i] = NULL;
+    rack->num_tiles = 0;
+
     rack->guess_length = 0;
     rack->solution = solution;
+    rack->solution_item = NULL;
+}
+
+static void
+rack_create_tiles (rack_t      *rack,
+                  GooCanvas    *canvas,
+                  char         *word)
+{
+    int i;
+
+    GooCanvasItem *root = goo_canvas_get_root_item (canvas);
+
+    for (i = 0; i < MIN (MAX_TILES, strlen (word)); i++)
+       rack->tiles[i] = tile_create (root, word[i], i);
+    rack->num_tiles = i;
 }
 
 static gboolean
@@ -322,9 +336,7 @@ on_key_press_event (GtkWidget       *widget,
        entry = dict_lookup (rack->solution, rack->guess);
        if (DICT_ENTRY_IS_WORD (entry)) {
            *entry = *entry | RACK_DICT_ENTRY_FOUND;
-           printf ("%s is good\n", rack->guess);
-       } else {
-           printf ("%s is not a word\n", rack->guess);
+           goo_canvas_item_simple_changed (GOO_CANVAS_ITEM_SIMPLE (rack->solution_item), FALSE);
        }
        rack_return_all (rack);
        return TRUE;
@@ -378,7 +390,7 @@ on_key_press_event (GtkWidget       *widget,
 }
 
 static GtkWidget *
-create_window (void)
+create_window (rack_t *rack)
 {
     GtkWidget *window, *scrolled_window;
 
@@ -390,7 +402,7 @@ create_window (void)
 
     gtk_widget_add_events (window, GDK_KEY_PRESS_MASK);
     g_signal_connect (window, "key_press_event",
-                     (GtkSignalFunc) on_key_press_event, &the_rack);
+                     (GtkSignalFunc) on_key_press_event, rack);
 
     scrolled_window = gtk_scrolled_window_new (NULL, NULL);
     gtk_scrolled_window_set_shadow_type (GTK_SCROLLED_WINDOW (scrolled_window),
@@ -404,8 +416,8 @@ create_window (void)
     return scrolled_window;
 }
 
-static void
-create_canvas (GtkWidget *parent, char *word, dict_t *solution)
+static GooCanvas *
+create_canvas (GtkWidget *parent, rack_t *rack)
 {
     GtkWidget *canvas;
     GooCanvasItem *root;
@@ -418,14 +430,14 @@ create_canvas (GtkWidget *parent, char *word, dict_t *solution)
 
     root = goo_canvas_get_root_item (GOO_CANVAS (canvas));
 
-    rack_init (&the_rack, root, word, solution);
+    rack->solution_item = goo_demo_item_new (root,
+                                            LETTER_PAD,
+                                            LETTER_PAD + 2 * (LETTER_SIZE + 2 * LETTER_PAD),
+                                            400, 400 - (2 * (LETTER_SIZE + 2 * LETTER_PAD)),
+                                            dict_paint, rack->solution,
+                                            NULL);
 
-    goo_demo_item_new (root,
-                      LETTER_PAD,
-                      LETTER_PAD + 2 * (LETTER_SIZE + 2 * LETTER_PAD),
-                      400, 400 - (2 * (LETTER_SIZE + 2 * LETTER_PAD)),
-                      dict_paint,
-                      solution, NULL);
+    return GOO_CANVAS (canvas);
 }
 
 int
@@ -434,9 +446,11 @@ main (int argc, char *argv[])
     dict_t dict, solution;
     struct timeval tv;
     bag_t bag;
-    char rack[8];
+    rack_t rack;
+    char word[8];
     int i;
     GtkWidget *window;
+    GooCanvas *canvas;
 
     gettimeofday (&tv, NULL);
     srand (tv.tv_sec ^ tv.tv_usec);
@@ -444,22 +458,27 @@ main (int argc, char *argv[])
     bag_init (&bag);
     bag_shuffle (&bag);
 
-    memcpy (rack, bag.tiles, 7);
-    rack[7] = '\0';
+    memcpy (word, bag.tiles, 7);
+    word[7] = '\0';
 
     for (i = 0; i < 7; i++)
-       rack[i] = toupper (rack[i]);
+       word[i] = toupper (word[i]);
 
     dict_init (&dict);
     dict_add_words_from_file (&dict, "words.txt");
 
     dict_init (&solution);
-    subanagram_expand (rack, &dict, &solution);
+    subanagram_expand (word, &dict, &solution);
 
     gtk_init (&argc, &argv);
-    window = create_window ();
 
-    create_canvas (window, rack, &solution);
+    rack_init (&rack, &solution);
+
+    window = create_window (&rack);
+
+    canvas = create_canvas (window, &rack);
+
+    rack_create_tiles (&rack, canvas, word);
 
     gtk_main ();