Don't display slots for target words less than 5 letters long

[wordgame] / rack-fancy.c

diff --git a/rack-fancy.c b/rack-fancy.c
index 8659d9ac9f3e09dcfb70cc13f0adeba95991c22d..2a60f88e336f94d4a902df362f94e462112674b1 100644 (file)
--- a/rack-fancy.c
+++ b/rack-fancy.c
@@ -1,7 +1,7 @@
 /*
  * Copyright © 2006 Carl Worth
  *
- * This program is free software; you can redistribute it and\/or modify
+ * This program is free software; you can redistribute it and/or modify
  * it under the terms of the GNU General Public License as published by
  * the Free Software Foundation; either version 2, or (at your option)
  * any later version.
@@ -29,7 +29,7 @@
 
 #define RACK_DICT_ENTRY_OBSCURE        (1<<1)
 #define RACK_DICT_ENTRY_FOUND  (1<<2)
-#define MAX_TILES 6
+#define MAX_TILES 7
 
 typedef struct _tile
 {
@@ -98,7 +98,7 @@ dict_paint_action (void *closure, char *word, dict_entry_t *entry)
     double new_x, new_y;
     int found, show_blanks = FALSE;
 
-    if (strlen (word) < 3)
+    if (strlen (word) < 5)
        return;
 
     found = *entry & RACK_DICT_ENTRY_FOUND;
@@ -354,13 +354,22 @@ _flag_obscure_word (void *closure, char *word, dict_entry_t *entry)
        *entry |= RACK_DICT_ENTRY_OBSCURE;
 }
 
+static void
+_at_least_one_is_not_obscure (void *closure, char *word, dict_entry_t *entry)
+{
+    int *result = closure;
+
+    if ((*entry & RACK_DICT_ENTRY_OBSCURE) == 0)
+       *result = 1;
+}
+
 static void
 rack_new_game (rack_t *rack)
 {
     int i, bottom;
     char word[MAX_TILES + 1];
     int length = MAX_TILES;
-    int count;
+    int has_full_length_non_obscure_word;
 
     /* We'll shuffle as many times as necessary until we can find a
      * sequence of <length> letters with at least one full-length
@@ -382,24 +391,29 @@ rack_new_game (rack_t *rack)
            }
        }
 
+       /* Look at each successive run of tiles in the bag until
+        * finding one that has at least one non-obscure word using
+        * all of its leters. */
        for (i = 0; i + length <= bottom + 1; i++) {
            memcpy (word, &rack->bag.tiles[i], length);
            word[length] = '\0';
            dict_fini (&rack->solution);
            dict_init (&rack->solution);
            subanagram_expand (word, &rack->dict, &rack->solution);
-           count = dict_for_each_of_length (&rack->solution,
-                                            NULL, NULL,
-                                            length, length);
-           if (count)
+           dict_for_each (&rack->solution,
+                          _flag_obscure_word, &rack->obscure);
+           has_full_length_non_obscure_word = 0;
+           dict_for_each_of_length (&rack->solution,
+                                    _at_least_one_is_not_obscure,
+                                    &has_full_length_non_obscure_word,
+                                    length, length);
+           if (has_full_length_non_obscure_word)
                goto DONE;
            i++;
        }
     }
 
   DONE:
-    dict_for_each (&rack->solution,
-                  _flag_obscure_word, &rack->obscure);
     rack->solution_total = dict_count (&rack->solution);
     goo_canvas_item_simple_changed (GOO_CANVAS_ITEM_SIMPLE (rack->solution_item), FALSE);