Sokoban game for HP Prime

[patrice]

Hi Ariel,
After the last review of your code, I have an advice for you.
When programming big pieces of code it is usually easier to put only one functionality in a subroutine, and only the top level routines put together every thing.
Example in loadsubnivel(), it is called only once by the top level routine. Making it only loading the level and nothing else make it easier to remember and to change. drawscreen() have some side effects that complicate changes.

Code:

    IF GAMEOVER == 0 THEN
      loadsubnivel();
      drawscreen();
      RECT_P(G0,20,223,300,238,#000000,#FFFFFF);
      TEXTOUT_P("LEVEL: "+NIVEL+"-"+SUBNIVEL+"  MOVES:           PUSHES:        ",25,224);
      FINAL := 0; MOVI  := 0;PUSH  := 0;

a code like that with a loadsubnivel() only loading the level is considered easier to understand and to change.