X-Git-Url: https://git.distorted.org.uk/~mdw/sgt/puzzles/blobdiff_plain/c0361acdb2a1bfa484787a157cfeb5d2d7356795..33317d37fe2e8182910cc8278ad634181521cd21:/solo.c diff --git a/solo.c b/solo.c index f3afb02..cfbb869 100644 --- a/solo.c +++ b/solo.c @@ -96,8 +96,6 @@ int solver_show_working; #include "puzzles.h" -#define max(x,y) ((x)>(y)?(x):(y)) - /* * To save space, I store digits internally as unsigned char. This * imposes a hard limit of 255 on the order of the puzzle. Since @@ -109,12 +107,14 @@ int solver_show_working; typedef unsigned char digit; #define ORDER_MAX 255 -#define TILE_SIZE 32 -#define BORDER 18 +#define PREFERRED_TILE_SIZE 32 +#define TILE_SIZE (ds->tilesize) +#define BORDER (TILE_SIZE / 2) #define FLASH_TIME 0.4F -enum { SYMM_NONE, SYMM_ROT2, SYMM_ROT4, SYMM_REF4 }; +enum { SYMM_NONE, SYMM_ROT2, SYMM_ROT4, SYMM_REF2, SYMM_REF2D, SYMM_REF4, + SYMM_REF4D, SYMM_REF8 }; enum { DIFF_BLOCK, DIFF_SIMPLE, DIFF_INTERSECT, DIFF_SET, DIFF_RECURSIVE, DIFF_AMBIGUOUS, DIFF_IMPOSSIBLE }; @@ -125,6 +125,7 @@ enum { COL_CLUE, COL_USER, COL_HIGHLIGHT, + COL_ERROR, COL_PENCIL, NCOLOURS }; @@ -177,8 +178,10 @@ static int game_fetch_preset(int i, char **name, game_params **params) { "3x3 Intermediate", { 3, 3, SYMM_ROT2, DIFF_INTERSECT } }, { "3x3 Advanced", { 3, 3, SYMM_ROT2, DIFF_SET } }, { "3x3 Unreasonable", { 3, 3, SYMM_ROT2, DIFF_RECURSIVE } }, +#ifndef SLOW_SYSTEM { "3x4 Basic", { 3, 4, SYMM_ROT2, DIFF_SIMPLE } }, { "4x4 Basic", { 4, 4, SYMM_ROT2, DIFF_SIMPLE } }, +#endif }; if (i < 0 || i >= lenof(presets)) @@ -201,12 +204,22 @@ static void decode_params(game_params *ret, char const *string) } while (*string) { if (*string == 'r' || *string == 'm' || *string == 'a') { - int sn, sc; + int sn, sc, sd; sc = *string++; + if (*string == 'd') { + sd = TRUE; + string++; + } else { + sd = FALSE; + } sn = atoi(string); while (*string && isdigit((unsigned char)*string)) string++; + if (sc == 'm' && sn == 8) + ret->symm = SYMM_REF8; if (sc == 'm' && sn == 4) - ret->symm = SYMM_REF4; + ret->symm = sd ? SYMM_REF4D : SYMM_REF4; + if (sc == 'm' && sn == 2) + ret->symm = sd ? SYMM_REF2D : SYMM_REF2; if (sc == 'r' && sn == 4) ret->symm = SYMM_ROT4; if (sc == 'r' && sn == 2) @@ -237,7 +250,11 @@ static char *encode_params(game_params *params, int full) sprintf(str, "%dx%d", params->c, params->r); if (full) { switch (params->symm) { + case SYMM_REF8: strcat(str, "m8"); break; case SYMM_REF4: strcat(str, "m4"); break; + case SYMM_REF4D: strcat(str, "md4"); break; + case SYMM_REF2: strcat(str, "m2"); break; + case SYMM_REF2D: strcat(str, "md2"); break; case SYMM_ROT4: strcat(str, "r4"); break; /* case SYMM_ROT2: strcat(str, "r2"); break; [default] */ case SYMM_NONE: strcat(str, "a"); break; @@ -274,7 +291,9 @@ static config_item *game_configure(game_params *params) ret[2].name = "Symmetry"; ret[2].type = C_CHOICES; - ret[2].sval = ":None:2-way rotation:4-way rotation:4-way mirror"; + ret[2].sval = ":None:2-way rotation:4-way rotation:2-way mirror:" + "2-way diagonal mirror:4-way mirror:4-way diagonal mirror:" + "8-way mirror"; ret[2].ival = params->symm; ret[3].name = "Difficulty"; @@ -844,7 +863,12 @@ static int nsolve_intersect(struct nsolve_usage *usage, return ret; } +struct nsolve_scratch { + unsigned char *grid, *rowidx, *colidx, *set; +}; + static int nsolve_set(struct nsolve_usage *usage, + struct nsolve_scratch *scratch, int start, int step1, int step2 #ifdef STANDALONE_SOLVER , char *fmt, ... @@ -853,10 +877,10 @@ static int nsolve_set(struct nsolve_usage *usage, { int c = usage->c, r = usage->r, cr = c*r; int i, j, n, count; - unsigned char *grid = snewn(cr*cr, unsigned char); - unsigned char *rowidx = snewn(cr, unsigned char); - unsigned char *colidx = snewn(cr, unsigned char); - unsigned char *set = snewn(cr, unsigned char); + unsigned char *grid = scratch->grid; + unsigned char *rowidx = scratch->rowidx; + unsigned char *colidx = scratch->colidx; + unsigned char *set = scratch->set; /* * We are passed a cr-by-cr matrix of booleans. Our first job @@ -999,10 +1023,6 @@ static int nsolve_set(struct nsolve_usage *usage, } if (progress) { - sfree(set); - sfree(colidx); - sfree(rowidx); - sfree(grid); return TRUE; } } @@ -1021,17 +1041,33 @@ static int nsolve_set(struct nsolve_usage *usage, break; /* done */ } - sfree(set); - sfree(colidx); - sfree(rowidx); - sfree(grid); - return FALSE; } +static struct nsolve_scratch *nsolve_new_scratch(struct nsolve_usage *usage) +{ + struct nsolve_scratch *scratch = snew(struct nsolve_scratch); + int cr = usage->cr; + scratch->grid = snewn(cr*cr, unsigned char); + scratch->rowidx = snewn(cr, unsigned char); + scratch->colidx = snewn(cr, unsigned char); + scratch->set = snewn(cr, unsigned char); + return scratch; +} + +static void nsolve_free_scratch(struct nsolve_scratch *scratch) +{ + sfree(scratch->set); + sfree(scratch->colidx); + sfree(scratch->rowidx); + sfree(scratch->grid); + sfree(scratch); +} + static int nsolve(int c, int r, digit *grid) { struct nsolve_usage *usage; + struct nsolve_scratch *scratch; int cr = c*r; int x, y, n; int diff = DIFF_BLOCK; @@ -1055,6 +1091,8 @@ static int nsolve(int c, int r, digit *grid) memset(usage->col, FALSE, cr * cr); memset(usage->blk, FALSE, cr * cr); + scratch = nsolve_new_scratch(usage); + /* * Place all the clue numbers we are given. */ @@ -1204,7 +1242,7 @@ static int nsolve(int c, int r, digit *grid) */ for (x = 0; x < cr; x += r) for (y = 0; y < r; y++) - if (nsolve_set(usage, cubepos(x,y,1), r*cr, 1 + if (nsolve_set(usage, scratch, cubepos(x,y,1), r*cr, 1 #ifdef STANDALONE_SOLVER , "set elimination, block (%d,%d)", 1+x/r, 1+y #endif @@ -1217,7 +1255,7 @@ static int nsolve(int c, int r, digit *grid) * Row-wise set elimination. */ for (y = 0; y < cr; y++) - if (nsolve_set(usage, cubepos(0,y,1), cr*cr, 1 + if (nsolve_set(usage, scratch, cubepos(0,y,1), cr*cr, 1 #ifdef STANDALONE_SOLVER , "set elimination, row %d", 1+YUNTRANS(y) #endif @@ -1230,7 +1268,7 @@ static int nsolve(int c, int r, digit *grid) * Column-wise set elimination. */ for (x = 0; x < cr; x++) - if (nsolve_set(usage, cubepos(x,0,1), cr, 1 + if (nsolve_set(usage, scratch, cubepos(x,0,1), cr, 1 #ifdef STANDALONE_SOLVER , "set elimination, column %d", 1+x #endif @@ -1246,6 +1284,8 @@ static int nsolve(int c, int r, digit *grid) break; } + nsolve_free_scratch(scratch); + sfree(usage->cube); sfree(usage->row); sfree(usage->col); @@ -1327,77 +1367,103 @@ static int check_valid(int c, int r, digit *grid) return TRUE; } -static void symmetry_limit(game_params *params, int *xlim, int *ylim, int s) +static int symmetries(game_params *params, int x, int y, int *output, int s) { int c = params->c, r = params->r, cr = c*r; + int i = 0; + +#define ADD(x,y) (*output++ = (x), *output++ = (y), i++) + + ADD(x, y); switch (s) { case SYMM_NONE: - *xlim = *ylim = cr; - break; + break; /* just x,y is all we need */ case SYMM_ROT2: - *xlim = (cr+1) / 2; - *ylim = cr; - break; - case SYMM_REF4: + ADD(cr - 1 - x, cr - 1 - y); + break; case SYMM_ROT4: - *xlim = *ylim = (cr+1) / 2; - break; + ADD(cr - 1 - y, x); + ADD(y, cr - 1 - x); + ADD(cr - 1 - x, cr - 1 - y); + break; + case SYMM_REF2: + ADD(cr - 1 - x, y); + break; + case SYMM_REF2D: + ADD(y, x); + break; + case SYMM_REF4: + ADD(cr - 1 - x, y); + ADD(x, cr - 1 - y); + ADD(cr - 1 - x, cr - 1 - y); + break; + case SYMM_REF4D: + ADD(y, x); + ADD(cr - 1 - x, cr - 1 - y); + ADD(cr - 1 - y, cr - 1 - x); + break; + case SYMM_REF8: + ADD(cr - 1 - x, y); + ADD(x, cr - 1 - y); + ADD(cr - 1 - x, cr - 1 - y); + ADD(y, x); + ADD(y, cr - 1 - x); + ADD(cr - 1 - y, x); + ADD(cr - 1 - y, cr - 1 - x); + break; } + +#undef ADD + + return i; } -static int symmetries(game_params *params, int x, int y, int *output, int s) +static char *encode_solve_move(int cr, digit *grid) { - int c = params->c, r = params->r, cr = c*r; - int i = 0; + int i, len; + char *ret, *p, *sep; - *output++ = x; - *output++ = y; - i++; + /* + * It's surprisingly easy to work out _exactly_ how long this + * string needs to be. To decimal-encode all the numbers from 1 + * to n: + * + * - every number has a units digit; total is n. + * - all numbers above 9 have a tens digit; total is max(n-9,0). + * - all numbers above 99 have a hundreds digit; total is max(n-99,0). + * - and so on. + */ + len = 0; + for (i = 1; i <= cr; i *= 10) + len += max(cr - i + 1, 0); + len += cr; /* don't forget the commas */ + len *= cr; /* there are cr rows of these */ - switch (s) { - case SYMM_NONE: - break; /* just x,y is all we need */ - case SYMM_REF4: - case SYMM_ROT4: - switch (s) { - case SYMM_REF4: - *output++ = cr - 1 - x; - *output++ = y; - i++; - - *output++ = x; - *output++ = cr - 1 - y; - i++; - break; - case SYMM_ROT4: - *output++ = cr - 1 - y; - *output++ = x; - i++; - - *output++ = y; - *output++ = cr - 1 - x; - i++; - break; - } - /* fall through */ - case SYMM_ROT2: - *output++ = cr - 1 - x; - *output++ = cr - 1 - y; - i++; - break; + /* + * Now len is one bigger than the total size of the + * comma-separated numbers (because we counted an + * additional leading comma). We need to have a leading S + * and a trailing NUL, so we're off by one in total. + */ + len++; + + ret = snewn(len, char); + p = ret; + *p++ = 'S'; + sep = ""; + for (i = 0; i < cr*cr; i++) { + p += sprintf(p, "%s%d", sep, grid[i]); + sep = ","; } + *p++ = '\0'; + assert(p - ret == len); - return i; + return ret; } -struct game_aux_info { - int c, r; - digit *grid; -}; - static char *new_game_desc(game_params *params, random_state *rs, - game_aux_info **aux, int interactive) + char **aux, int interactive) { int c = params->c, r = params->r, cr = c*r; int area = cr*cr; @@ -1407,7 +1473,7 @@ static char *new_game_desc(game_params *params, random_state *rs, int ret; char *desc; int coords[16], ncoords; - int xlim, ylim; + int *symmclasses, nsymmclasses; int maxdiff, recursing; /* @@ -1425,6 +1491,31 @@ static char *new_game_desc(game_params *params, random_state *rs, grid2 = snewn(area, digit); /* + * Find the set of equivalence classes of squares permitted + * by the selected symmetry. We do this by enumerating all + * the grid squares which have no symmetric companion + * sorting lower than themselves. + */ + nsymmclasses = 0; + symmclasses = snewn(cr * cr, int); + { + int x, y; + + for (y = 0; y < cr; y++) + for (x = 0; x < cr; x++) { + int i = y*cr+x; + int j; + + ncoords = symmetries(params, x, y, coords, params->symm); + for (j = 0; j < ncoords; j++) + if (coords[2*j+1]*cr+coords[2*j] < i) + break; + if (j == ncoords) + symmclasses[nsymmclasses++] = i; + } + } + + /* * Loop until we get a grid of the required difficulty. This is * nasty, but it seems to be unpleasantly hard to generate * difficult grids otherwise. @@ -1440,22 +1531,25 @@ static char *new_game_desc(game_params *params, random_state *rs, assert(check_valid(c, r, grid)); /* - * Save the solved grid in the aux_info. + * Save the solved grid in aux. */ { - game_aux_info *ai = snew(game_aux_info); - ai->c = c; - ai->r = r; - ai->grid = snewn(cr * cr, digit); - memcpy(ai->grid, grid, cr * cr * sizeof(digit)); - *aux = ai; + /* + * We might already have written *aux the last time we + * went round this loop, in which case we should free + * the old aux before overwriting it with the new one. + */ + if (*aux) { + sfree(*aux); + } + + *aux = encode_solve_move(cr, grid); } /* * Now we have a solved grid, start removing things from it * while preserving solubility. */ - symmetry_limit(params, &xlim, &ylim, params->symm); recursing = FALSE; while (1) { int x, y, i, j; @@ -1466,13 +1560,15 @@ static char *new_game_desc(game_params *params, random_state *rs, */ nlocs = 0; - for (x = 0; x < xlim; x++) - for (y = 0; y < ylim; y++) - if (grid[y*cr+x]) { - locs[nlocs].x = x; - locs[nlocs].y = y; - nlocs++; - } + for (i = 0; i < nsymmclasses; i++) { + x = symmclasses[i] % cr; + y = symmclasses[i] / cr; + if (grid[y*cr+x]) { + locs[nlocs].x = x; + locs[nlocs].y = y; + nlocs++; + } + } /* * Now shuffle that list. @@ -1537,6 +1633,8 @@ static char *new_game_desc(game_params *params, random_state *rs, sfree(grid2); sfree(locs); + sfree(symmclasses); + /* * Now we have the grid as it will be presented to the user. * Encode it in a game desc. @@ -1586,12 +1684,6 @@ static char *new_game_desc(game_params *params, random_state *rs, return desc; } -static void game_free_aux_info(game_aux_info *aux) -{ - sfree(aux->grid); - sfree(aux); -} - static char *validate_desc(game_params *params, char *desc) { int area = params->r * params->r * params->c * params->c; @@ -1693,39 +1785,38 @@ static void free_game(game_state *state) sfree(state); } -static game_state *solve_game(game_state *state, game_aux_info *ai, - char **error) +static char *solve_game(game_state *state, game_state *currstate, + char *ai, char **error) { - game_state *ret; int c = state->c, r = state->r, cr = c*r; + char *ret; + digit *grid; int rsolve_ret; - ret = dup_game(state); - ret->completed = ret->cheated = TRUE; - /* - * If we already have the solution in the aux_info, save - * ourselves some time. + * If we already have the solution in ai, save ourselves some + * time. */ - if (ai) { + if (ai) + return dupstr(ai); - assert(c == ai->c); - assert(r == ai->r); - memcpy(ret->grid, ai->grid, cr * cr * sizeof(digit)); + grid = snewn(cr*cr, digit); + memcpy(grid, state->grid, cr*cr); + rsolve_ret = rsolve(c, r, grid, NULL, 2); - } else { - rsolve_ret = rsolve(c, r, ret->grid, NULL, 2); - - if (rsolve_ret != 1) { - free_game(ret); - if (rsolve_ret == 0) - *error = "No solution exists for this puzzle"; - else - *error = "Multiple solutions exist for this puzzle"; - return NULL; - } + if (rsolve_ret != 1) { + sfree(grid); + if (rsolve_ret == 0) + *error = "No solution exists for this puzzle"; + else + *error = "Multiple solutions exist for this puzzle"; + return NULL; } + ret = encode_solve_move(cr, grid); + + sfree(grid); + return ret; } @@ -1821,34 +1912,84 @@ static void free_ui(game_ui *ui) sfree(ui); } -static game_state *make_move(game_state *from, game_ui *ui, game_drawstate *ds, - int x, int y, int button) +static char *encode_ui(game_ui *ui) { - int c = from->c, r = from->r, cr = c*r; + return NULL; +} + +static void decode_ui(game_ui *ui, char *encoding) +{ +} + +static void game_changed_state(game_ui *ui, game_state *oldstate, + game_state *newstate) +{ + int c = newstate->c, r = newstate->r, cr = c*r; + /* + * We prevent pencil-mode highlighting of a filled square. So + * if the user has just filled in a square which we had a + * pencil-mode highlight in (by Undo, or by Redo, or by Solve), + * then we cancel the highlight. + */ + if (ui->hx >= 0 && ui->hy >= 0 && ui->hpencil && + newstate->grid[ui->hy * cr + ui->hx] != 0) { + ui->hx = ui->hy = -1; + } +} + +struct game_drawstate { + int started; + int c, r, cr; + int tilesize; + digit *grid; + unsigned char *pencil; + unsigned char *hl; + /* This is scratch space used within a single call to game_redraw. */ + int *entered_items; +}; + +static char *interpret_move(game_state *state, game_ui *ui, game_drawstate *ds, + int x, int y, int button) +{ + int c = state->c, r = state->r, cr = c*r; int tx, ty; - game_state *ret; + char buf[80]; button &= ~MOD_MASK; tx = (x + TILE_SIZE - BORDER) / TILE_SIZE - 1; ty = (y + TILE_SIZE - BORDER) / TILE_SIZE - 1; - if (tx >= 0 && tx < cr && ty >= 0 && ty < cr && - (button == LEFT_BUTTON || button == RIGHT_BUTTON)) { - /* - * Prevent pencil-mode highlighting of a filled square. - */ - if (button == RIGHT_BUTTON && from->grid[ty*cr+tx]) - return NULL; - - if (tx == ui->hx && ty == ui->hy) { - ui->hx = ui->hy = -1; - } else { - ui->hx = tx; - ui->hy = ty; - } - ui->hpencil = (button == RIGHT_BUTTON); - return from; /* UI activity occurred */ + if (tx >= 0 && tx < cr && ty >= 0 && ty < cr) { + if (button == LEFT_BUTTON) { + if (state->immutable[ty*cr+tx]) { + ui->hx = ui->hy = -1; + } else if (tx == ui->hx && ty == ui->hy && ui->hpencil == 0) { + ui->hx = ui->hy = -1; + } else { + ui->hx = tx; + ui->hy = ty; + ui->hpencil = 0; + } + return ""; /* UI activity occurred */ + } + if (button == RIGHT_BUTTON) { + /* + * Pencil-mode highlighting for non filled squares. + */ + if (state->grid[ty*cr+tx] == 0) { + if (tx == ui->hx && ty == ui->hy && ui->hpencil) { + ui->hx = ui->hy = -1; + } else { + ui->hpencil = 1; + ui->hx = tx; + ui->hy = ty; + } + } else { + ui->hx = ui->hy = -1; + } + return ""; /* UI activity occurred */ + } } if (ui->hx != -1 && ui->hy != -1 && @@ -1864,8 +2005,14 @@ static game_state *make_move(game_state *from, game_ui *ui, game_drawstate *ds, if (button == ' ') n = 0; - if (from->immutable[ui->hy*cr+ui->hx]) - return NULL; /* can't overwrite this square */ + /* + * Can't overwrite this square. In principle this shouldn't + * happen anyway because we should never have even been + * able to highlight the square, but it never hurts to be + * careful. + */ + if (state->immutable[ui->hy*cr+ui->hx]) + return NULL; /* * Can't make pencil marks in a filled square. In principle @@ -1873,16 +2020,57 @@ static game_state *make_move(game_state *from, game_ui *ui, game_drawstate *ds, * have even been able to pencil-highlight the square, but * it never hurts to be careful. */ - if (ui->hpencil && from->grid[ui->hy*cr+ui->hx]) + if (ui->hpencil && state->grid[ui->hy*cr+ui->hx]) return NULL; + sprintf(buf, "%c%d,%d,%d", + (char)(ui->hpencil && n > 0 ? 'P' : 'R'), ui->hx, ui->hy, n); + + ui->hx = ui->hy = -1; + + return dupstr(buf); + } + + return NULL; +} + +static game_state *execute_move(game_state *from, char *move) +{ + int c = from->c, r = from->r, cr = c*r; + game_state *ret; + int x, y, n; + + if (move[0] == 'S') { + char *p; + ret = dup_game(from); - if (ui->hpencil && n > 0) { - int index = (ui->hy*cr+ui->hx) * cr + (n-1); + ret->completed = ret->cheated = TRUE; + + p = move+1; + for (n = 0; n < cr*cr; n++) { + ret->grid[n] = atoi(p); + + if (!*p || ret->grid[n] < 1 || ret->grid[n] > cr) { + free_game(ret); + return NULL; + } + + while (*p && isdigit((unsigned char)*p)) p++; + if (*p == ',') p++; + } + + return ret; + } else if ((move[0] == 'P' || move[0] == 'R') && + sscanf(move+1, "%d,%d,%d", &x, &y, &n) == 3 && + x >= 0 && x < cr && y >= 0 && y < cr && n >= 0 && n <= cr) { + + ret = dup_game(from); + if (move[0] == 'P' && n > 0) { + int index = (y*cr+x) * cr + (n-1); ret->pencil[index] = !ret->pencil[index]; } else { - ret->grid[ui->hy*cr+ui->hx] = n; - memset(ret->pencil + (ui->hy*cr+ui->hx)*cr, 0, cr); + ret->grid[y*cr+x] = n; + memset(ret->pencil + (y*cr+x)*cr, 0, cr); /* * We've made a real change to the grid. Check to see @@ -1892,35 +2080,32 @@ static game_state *make_move(game_state *from, game_ui *ui, game_drawstate *ds, ret->completed = TRUE; } } - ui->hx = ui->hy = -1; - - return ret; /* made a valid move */ - } - - return NULL; + return ret; + } else + return NULL; /* couldn't parse move string */ } /* ---------------------------------------------------------------------- * Drawing routines. */ -struct game_drawstate { - int started; - int c, r, cr; - digit *grid; - unsigned char *pencil; - unsigned char *hl; -}; - -#define XSIZE(cr) ((cr) * TILE_SIZE + 2*BORDER + 1) -#define YSIZE(cr) ((cr) * TILE_SIZE + 2*BORDER + 1) +#define SIZE(cr) ((cr) * TILE_SIZE + 2*BORDER + 1) +#define GETTILESIZE(cr, w) ( (double)(w-1) / (double)(cr+1) ) -static void game_size(game_params *params, int *x, int *y) +static void game_size(game_params *params, game_drawstate *ds, + int *x, int *y, int expand) { int c = params->c, r = params->r, cr = c*r; + double ts; - *x = XSIZE(cr); - *y = YSIZE(cr); + ts = min(GETTILESIZE(cr, *x), GETTILESIZE(cr, *y)); + if (expand) + ds->tilesize = (int)(ts+0.5); + else + ds->tilesize = min((int)ts, PREFERRED_TILE_SIZE); + + *x = SIZE(cr); + *y = SIZE(cr); } static float *game_colours(frontend *fe, game_state *state, int *ncolours) @@ -1945,6 +2130,10 @@ static float *game_colours(frontend *fe, game_state *state, int *ncolours) ret[COL_HIGHLIGHT * 3 + 1] = 0.85F * ret[COL_BACKGROUND * 3 + 1]; ret[COL_HIGHLIGHT * 3 + 2] = 0.85F * ret[COL_BACKGROUND * 3 + 2]; + ret[COL_ERROR * 3 + 0] = 1.0F; + ret[COL_ERROR * 3 + 1] = 0.0F; + ret[COL_ERROR * 3 + 2] = 0.0F; + ret[COL_PENCIL * 3 + 0] = 0.5F * ret[COL_BACKGROUND * 3 + 0]; ret[COL_PENCIL * 3 + 1] = 0.5F * ret[COL_BACKGROUND * 3 + 1]; ret[COL_PENCIL * 3 + 2] = ret[COL_BACKGROUND * 3 + 2]; @@ -1968,7 +2157,8 @@ static game_drawstate *game_new_drawstate(game_state *state) memset(ds->pencil, 0, cr*cr*cr); ds->hl = snewn(cr*cr, unsigned char); memset(ds->hl, 0, cr*cr); - + ds->entered_items = snewn(cr*cr, int); + ds->tilesize = 0; /* not decided yet */ return ds; } @@ -1977,6 +2167,7 @@ static void game_free_drawstate(game_drawstate *ds) sfree(ds->hl); sfree(ds->pencil); sfree(ds->grid); + sfree(ds->entered_items); sfree(ds); } @@ -2013,10 +2204,10 @@ static void draw_number(frontend *fe, game_drawstate *ds, game_state *state, clip(fe, cx, cy, cw, ch); /* background needs erasing */ - draw_rect(fe, cx, cy, cw, ch, hl == 1 ? COL_HIGHLIGHT : COL_BACKGROUND); + draw_rect(fe, cx, cy, cw, ch, (hl & 15) == 1 ? COL_HIGHLIGHT : COL_BACKGROUND); /* pencil-mode highlight */ - if (hl == 2) { + if ((hl & 15) == 2) { int coords[6]; coords[0] = cx; coords[1] = cy; @@ -2024,7 +2215,7 @@ static void draw_number(frontend *fe, game_drawstate *ds, game_state *state, coords[3] = cy; coords[4] = cx; coords[5] = cy+ch/2; - draw_polygon(fe, coords, 3, TRUE, COL_HIGHLIGHT); + draw_polygon(fe, coords, 3, COL_HIGHLIGHT, COL_HIGHLIGHT); } /* new number needs drawing? */ @@ -2035,21 +2226,42 @@ static void draw_number(frontend *fe, game_drawstate *ds, game_state *state, str[0] += 'a' - ('9'+1); draw_text(fe, tx + TILE_SIZE/2, ty + TILE_SIZE/2, FONT_VARIABLE, TILE_SIZE/2, ALIGN_VCENTRE | ALIGN_HCENTRE, - state->immutable[y*cr+x] ? COL_CLUE : COL_USER, str); + state->immutable[y*cr+x] ? COL_CLUE : (hl & 16) ? COL_ERROR : COL_USER, str); } else { - /* pencil marks required? */ - int i, j; + int i, j, npencil; + int pw, ph, pmax, fontsize; + + /* count the pencil marks required */ + for (i = npencil = 0; i < cr; i++) + if (state->pencil[(y*cr+x)*cr+i]) + npencil++; + + /* + * It's not sensible to arrange pencil marks in the same + * layout as the squares within a block, because this leads + * to the font being too small. Instead, we arrange pencil + * marks in the nearest thing we can to a square layout, + * and we adjust the square layout depending on the number + * of pencil marks in the square. + */ + for (pw = 1; pw * pw < npencil; pw++); + if (pw < 3) pw = 3; /* otherwise it just looks _silly_ */ + ph = (npencil + pw - 1) / pw; + if (ph < 2) ph = 2; /* likewise */ + pmax = max(pw, ph); + fontsize = TILE_SIZE/(pmax*(11-pmax)/8); for (i = j = 0; i < cr; i++) if (state->pencil[(y*cr+x)*cr+i]) { - int dx = j % r, dy = j / r, crm = max(c, r); + int dx = j % pw, dy = j / pw; + str[1] = '\0'; str[0] = i + '1'; if (str[0] > '9') str[0] += 'a' - ('9'+1); - draw_text(fe, tx + (4*dx+3) * TILE_SIZE / (4*r+2), - ty + (4*dy+3) * TILE_SIZE / (4*c+2), - FONT_VARIABLE, TILE_SIZE/(crm*5/4), + draw_text(fe, tx + (4*dx+3) * TILE_SIZE / (4*pw+2), + ty + (4*dy+3) * TILE_SIZE / (4*ph+2), + FONT_VARIABLE, fontsize, ALIGN_VCENTRE | ALIGN_HCENTRE, COL_PENCIL, str); j++; } @@ -2078,7 +2290,7 @@ static void game_redraw(frontend *fe, game_drawstate *ds, game_state *oldstate, * all games should start by drawing a big * background-colour rectangle covering the whole window. */ - draw_rect(fe, 0, 0, XSIZE(cr), YSIZE(cr), COL_BACKGROUND); + draw_rect(fe, 0, 0, SIZE(cr), SIZE(cr), COL_BACKGROUND); /* * Draw the grid. @@ -2096,17 +2308,46 @@ static void game_redraw(frontend *fe, game_drawstate *ds, game_state *oldstate, } /* + * This array is used to keep track of rows, columns and boxes + * which contain a number more than once. + */ + for (x = 0; x < cr * cr; x++) + ds->entered_items[x] = 0; + for (x = 0; x < cr; x++) + for (y = 0; y < cr; y++) { + digit d = state->grid[y*cr+x]; + if (d) { + int box = (x/r)+(y/c)*c; + ds->entered_items[x*cr+d-1] |= ((ds->entered_items[x*cr+d-1] & 1) << 1) | 1; + ds->entered_items[y*cr+d-1] |= ((ds->entered_items[y*cr+d-1] & 4) << 1) | 4; + ds->entered_items[box*cr+d-1] |= ((ds->entered_items[box*cr+d-1] & 16) << 1) | 16; + } + } + + /* * Draw any numbers which need redrawing. */ for (x = 0; x < cr; x++) { for (y = 0; y < cr; y++) { int highlight = 0; + digit d = state->grid[y*cr+x]; + if (flashtime > 0 && (flashtime <= FLASH_TIME/3 || flashtime >= FLASH_TIME*2/3)) highlight = 1; + + /* Highlight active input areas. */ if (x == ui->hx && y == ui->hy) highlight = ui->hpencil ? 2 : 1; + + /* Mark obvious errors (ie, numbers which occur more than once + * in a single row, column, or box). */ + if (d && ((ds->entered_items[x*cr+d-1] & 2) || + (ds->entered_items[y*cr+d-1] & 8) || + (ds->entered_items[((x/r)+(y/c)*c)*cr+d-1] & 32))) + highlight |= 16; + draw_number(fe, ds, state, x, y, highlight); } } @@ -2115,7 +2356,7 @@ static void game_redraw(frontend *fe, game_drawstate *ds, game_state *oldstate, * Update the _entire_ grid if necessary. */ if (!ds->started) { - draw_update(fe, 0, 0, XSIZE(cr), YSIZE(cr)); + draw_update(fe, 0, 0, SIZE(cr), SIZE(cr)); ds->started = TRUE; } } @@ -2160,7 +2401,6 @@ const struct game thegame = { TRUE, game_configure, custom_params, validate_params, new_game_desc, - game_free_aux_info, validate_desc, new_game, dup_game, @@ -2169,7 +2409,11 @@ const struct game thegame = { TRUE, game_text_format, new_ui, free_ui, - make_move, + encode_ui, + decode_ui, + game_changed_state, + interpret_move, + execute_move, game_size, game_colours, game_new_drawstate, @@ -2179,6 +2423,7 @@ const struct game thegame = { game_flash_length, game_wants_statusbar, FALSE, game_timing_state, + 0, /* mouse_priorities */ }; #ifdef STANDALONE_SOLVER @@ -2193,7 +2438,7 @@ void draw_text(frontend *fe, int x, int y, int fonttype, int fontsize, void draw_rect(frontend *fe, int x, int y, int w, int h, int colour) {} void draw_line(frontend *fe, int x1, int y1, int x2, int y2, int colour) {} void draw_polygon(frontend *fe, int *coords, int npoints, - int fill, int colour) {} + int fillcolour, int outlinecolour) {} void clip(frontend *fe, int x, int y, int w, int h) {} void unclip(frontend *fe) {} void start_draw(frontend *fe) {} @@ -2266,7 +2511,7 @@ int main(int argc, char **argv) fprintf(stderr, "%s: %s\n", argv[0], err); return 1; } - s = new_game(p, desc); + s = new_game(NULL, p, desc); if (recurse) { int ret = rsolve(p->c, p->r, s->grid, NULL, 2);