I know there's probably no ideal answer to this, but I was wondering what's the best way to organise your command handlers in the on_commands{} section. In the past, I organised them by room, then object in room, then commands for each object. The advantage of this is that all commands for a particular object are grouped together. For example:

: match "examine door" {
   : if (is_present "door") {
      // Do something
   }
}
: match "open door" {
   : if (is_present "door") {
      // Do something
   }
}
: match "close door" {
   : if (is_present "door") {
      // Do something
   }
}

Alternatively, you could do something like this:

: if (is_present "door) {
   : match "examine door" {
      // Do something
   }
   : match "open door" {
      // Do something
   }
   : match "close door" {
      // Do something
   }
}

I'm now wondering if it's better to organise them by verb, then statements for each object within the verb. For example:

: match "examine _" {
   : if (noun1_is "door" && is_present "door") {
      // Do something
   }
   : if (noun1_is "cupboard" && is present "cupboard") {
      // Do something
   }
}

What do others think?

The third approach is neatest I think (personally speaking).

For 8-bit compatibility sake, this works, and was specifically coded to map to DAAD. You are permitted to have matches inside an outer match so long as every element of an outer match is also a match:

: match "examine _" {
   : match "_ door" {
      : if (is_present "door") {
         // Do something
      }
   }
   : match "_ cupboard" {
      : if (is_present "cupboard") {
         // Do something
      }
   }
}

POINT OF NOTE : There is an open issue (in the mysterious issue tracker) for being able to combine PRESENT, CARRIED, WORN, BESIDE, NOTWORN in match statements as a short form for the if statement within the match. This should eliminate the need for an inner if per item (in many  cases).

I think, I'll start by managing the GET/DROP objects globally, and the others in locations.  Then, I'll see how I organize my code inside the Global and the Local.

on_command {
   : match "_ _"  {
      : if (is_at "room1") {
         : gosub "room1Sub" ;
         : done ;
      }
      : if (is_at "room2") {
         : gosub "room2Sub" ;
         : done ;
      }
      : if (is_at "room3") {
         : gosub "room3Sub" ;
         : done ;
      }
   }
}