distinguish container-sub-commands and container-dom-commands
[mudsync.git] / mudsync / gameobj.scm
index 65ddc296262c7d2a767e53db858330170d4aed3d..ebd4db7276c44aecd5b307c10a43edc5f7b48486 100644 (file)
@@ -21,6 +21,7 @@
 
 (define-module (mudsync gameobj)
   #:use-module (mudsync command)
+  #:use-module (mudsync utils)
   #:use-module (8sync actors)
   #:use-module (8sync agenda)
   #:use-module (8sync rmeta-slot)
@@ -46,7 +47,7 @@
 
             ;; Some of the more common commands
             cmd-take cmd-drop
-            cmd-take-from cmd-put-in))
+            cmd-take-from-no-op cmd-put-in-no-op))
 
 ;;; Gameobj
 ;;; =======
                                                       '("in" "inside" "on"))))))
 
   ;; Commands we can handle by being something's container
-  (container-commands #:allocation #:each-subclass
-                      #:init-thunk (build-commands))
+  ;;   dominant version (goes before everything)
+  (container-dom-commands #:allocation #:each-subclass
+                          #:init-thunk (build-commands))
+  ;;   subordinate version (goes after everything)
+  (container-sub-commands #:allocation #:each-subclass
+                          #:init-thunk (build-commands))
 
   ;; Commands we can handle by being contained by something else
   (contained-commands #:allocation #:each-subclass
   ;; (gameobj whos-acting where)
   (drop-me? #:init-value #t
             #:init-keyword #:drop-me?)
-  ;; Can be a boolean or a procedure accepting
-  ;; (gameobj whos-acting take-what)
-  (take-from-me? #:init-value #f
-                 #:init-keyword #:take-from-me?)
-  ;; Can be a boolean or a procedure accepting
-  ;; (gameobj whos-acting put-what)
-  (put-in-me? #:init-value #f
-              #:init-keyword #:put-in-me?)
 
   ;; TODO: Remove this and use actor-alive? instead.
   ;; Set this on self-destruct
             ;; Commands for co-occupants
             (get-commands gameobj-get-commands)
             ;; Commands for participants in a room
-            (get-container-commands gameobj-get-container-commands)
+            (get-container-dom-commands gameobj-get-container-dom-commands)
+            (get-container-sub-commands gameobj-get-container-sub-commands)
             ;; Commands for inventory items, etc (occupants of the gameobj commanding)
             (get-contained-commands gameobj-get-contained-commands)
 
 
             ;; Common commands
             (cmd-take cmd-take)
-            (cmd-take-from cmd-take-from)
-            (cmd-put-in cmd-put-in)
-            (cmd-drop cmd-drop))))
+            (cmd-drop cmd-drop)
+            (cmd-take-from cmd-take-from-no-op)
+            (cmd-put-in cmd-put-in-no-op))))
 
 
 ;;; gameobj message handlers
@@ -211,7 +209,7 @@ Assists in its replacement of occupants if necessary and nothing else."
 
 (define (gameobj-act-goes-by actor message)
   "Reply to a message requesting what we go by."
-  (<-reply message #:goes-by (gameobj-goes-by actor)))
+  (<-reply message (gameobj-goes-by actor)))
 
 (define (val-or-run val-or-proc)
   "Evaluate if a procedure, or just return otherwise"
@@ -231,10 +229,16 @@ Assists in its replacement of occupants if necessary and nothing else."
            #:commands candidate-commands
            #:goes-by (gameobj-goes-by actor)))
 
-(define* (gameobj-get-container-commands actor message #:key verb)
-  "Get commands as the container / room of message's sender"
+(define* (gameobj-get-container-dom-commands actor message #:key verb)
+  "Get (dominant) commands as the container / room of message's sender"
+  (define candidate-commands
+    (get-candidate-commands actor 'container-dom-commands verb))
+  (<-reply message #:commands candidate-commands))
+
+(define* (gameobj-get-container-sub-commands actor message #:key verb)
+  "Get (subordinate) commands as the container / room of message's sender"
   (define candidate-commands
-    (get-candidate-commands actor 'container-commands verb))
+    (get-candidate-commands actor 'container-sub-commands verb))
   (<-reply message #:commands candidate-commands))
 
 (define* (gameobj-get-contained-commands actor message #:key verb)
@@ -379,12 +383,20 @@ By default, this is whether or not the generally-visible flag is set."
          (gameobj-replace-data* gameobj)))
 
 (define (gameobj-ok-to-be-taken-from gameobj message whos-acting)
-  (<-reply message (slot-ref-maybe-runcheck gameobj 'take-me?
-                                            whos-acting #:from #t)))
+  (call-with-values (lambda ()
+                      (slot-ref-maybe-runcheck gameobj 'take-me?
+                                               whos-acting #:from #t))
+    ;; This allows this to reply with #:why-not if appropriate
+    (lambda args
+      (apply <-reply message args))))
 
 (define (gameobj-ok-to-be-put-in gameobj message whos-acting where)
-  (<-reply message (slot-ref-maybe-runcheck gameobj 'drop-me?
-                                            whos-acting where)))
+  (call-with-values (lambda ()
+                      (slot-ref-maybe-runcheck gameobj 'drop-me?
+                                               whos-acting where))
+    ;; This allows this to reply with #:why-not if appropriate
+    (lambda args
+      (apply <-reply message args))))
 
 \f
 ;;; Utilities every gameobj has
@@ -407,8 +419,9 @@ By default, this is whether or not the generally-visible flag is set."
 ;;; Basic actions
 ;;; -------------
 
-(define* (cmd-take gameobj message #:key direct-obj)
-  (define player (message-from message))
+(define* (cmd-take gameobj message
+                   #:key direct-obj
+                   (player (message-from message)))
   (define player-name
     (mbody-val (<-wait player 'get-name)))
   (define player-loc
@@ -418,25 +431,28 @@ By default, this is whether or not the generally-visible flag is set."
     (slot-ref-maybe-runcheck gameobj 'take-me? player))
   ;; @@: Is there any reason to allow the room to object in the way
   ;;   that there is for dropping?  It doesn't seem like it.
-  ;; TODO: Allow gameobj to customize
-  (if self-should-take
-      ;; Set the location to whoever's picking us up
-      (begin
-        (gameobj-set-loc! gameobj player)
-        (<- player 'tell
-            #:text (format #f "You pick up ~a.\n"
-                           our-name))
-        (<- player-loc 'tell-room
-            #:text (format #f "~a picks up ~a.\n"
-                           player-name
-                           our-name)
-            #:exclude player))
-      (<- player 'tell
-          #:text (format #f "It doesn't seem like you can take ~a.\n"
-                         our-name))))
-
-(define* (cmd-drop gameobj message #:key direct-obj)
-  (define player (message-from message))
+  (call-with-values (lambda ()
+                      (slot-ref-maybe-runcheck gameobj 'take-me? player))
+    (lambda* (self-should-take #:key (why-not
+                                      `("It doesn't seem like you can take "
+                                        ,our-name ".")))
+      (if self-should-take
+          ;; Set the location to whoever's picking us up
+          (begin
+            (gameobj-set-loc! gameobj player)
+            (<- player 'tell
+                #:text (format #f "You pick up ~a.\n"
+                               our-name))
+            (<- player-loc 'tell-room
+                #:text (format #f "~a picks up ~a.\n"
+                               player-name
+                               our-name)
+                #:exclude player))
+          (<- player 'tell #:text why-not)))))
+
+(define* (cmd-drop gameobj message
+                   #:key direct-obj
+                   (player (message-from message)))
   (define player-name
     (mbody-val (<-wait player 'get-name)))
   (define player-loc
@@ -478,141 +494,18 @@ By default, this is whether or not the generally-visible flag is set."
                        our-name)
         #:exclude player))))
 
-;; @@: Moving this to a container subclass/mixin could allow a lot more
-;;   customization of take out / put in phrases
-(define* (cmd-take-from gameobj message
-                        #:key direct-obj indir-obj preposition)
-  (define player (message-from message))
-  (define player-name
-    (mbody-val (<-wait player 'get-name)))
-  (define player-loc
-    (mbody-val (<-wait player 'get-loc)))
-  (define our-name (slot-ref gameobj 'name))
-  ;; We need to check if we even have such a thing
-  (define this-thing
-    (call/ec
-     (lambda (return)
-       (for-each (lambda (occupant)
-                   (mbody-receive (_ #:key goes-by)
-                       (<-wait occupant 'goes-by)
-                     (when (member direct-obj goes-by)
-                       (return occupant))))
-                 (gameobj-occupants gameobj))
-       ;; nothing found
-       #f)))
-  (define (this-thing-name)
-    (mbody-val (<-wait this-thing 'get-name)))
-  (define (should-take-from-me)
-    (and this-thing
-         (slot-ref-maybe-runcheck gameobj 'take-from-me? player this-thing)))
-  (define (default-objection)
-    `("Unfortunately, it doesn't seem like you can take "
-      (this-thing-name) " " preposition " " our-name "."))
-
-  (define (this-thing-objection)
-    (mbody-receive (_ taken-ok? #:key why-not) ; does the object object to being removed?
-        (<-wait this-thing 'ok-to-be-taken-from? player) ; @@ no need to supply from where
-      (and (not taken-ok?)
-           ;; Either give the specified reason, or give a boilerplate one
-           (or why-not
-               (default-objection)))))
-  (cond
-   ;; Wait, aren't we going to check (should-take-from-me) later?
-   ;; Well yes, but this checks if there's a #f as the value, which
-   ;; is a much clearer indication that this doesn't take *anything*.
-   ((not (slot-ref gameobj 'take-from-me?))
-    (<- player 'tell
-        #:text `("It's not really clear how to take something " ,preposition
-                 " " ,our-name ".")))
-
-   ;; Unfortunately this does leak information about what is contained
-   ;; by us.  Maybe not what's wanted in all circumstances.
-   ((not this-thing)
-    (<- player 'tell
-        #:text `("You don't see any such " ,direct-obj " to take from "
-                 ,our-name ".")))
-   ;; A particular objection to taking this thing.
-   ;; We should allow customizing the reason here, which could be
-   ;; provided by the 'ok-to-be-taken-from? slot.
-   ((not (should-take-from-me))
-    (<- player 'tell
-        #:text (default-objection)))
-   ;; the thing we wsant to take itself has objected...
-   ((this-thing-objection) =>
-    (lambda (objection)
-      (<- player 'tell
-          #:text objection)))
-   ;; looks like we can take it
-   (else
-    ;; Wait to announce to the player just in case settting the location
-    ;; errors out or something.  Maybe it's overthinking things, I dunno.
-    (<-wait this-thing 'set-loc! #:loc player)
-    (<- player 'tell
-        #:text `("You take " ,(this-thing-name) " from "
-                 ,our-name "."))
-    (<- player-loc 'tell-room
-        #:text `(,player-name " takes " ,(this-thing-name) " from "
-                              ,our-name ".")
-        #:exclude player))))
-
-(define* (cmd-put-in gameobj message
-                     #:key direct-obj indir-obj preposition)
-  (define player (message-from message))
-  (define player-name
-    (mbody-val (<-wait player 'get-name)))
-  (define player-loc
-    (mbody-val (<-wait player 'get-loc)))
-  (define our-name (slot-ref gameobj 'name))
-  ;; We need to check if we even have such a thing
-  (define this-thing
-    (call/ec
-     (lambda (return)
-       (for-each (lambda (occupant)
-                   (mbody-receive (_ #:key goes-by)
-                       (<-wait occupant 'goes-by)
-                     (when (member direct-obj goes-by)
-                       (return occupant))))
-                 (mbody-val (<-wait player 'get-occupants)))
-       ;; nothing found
-       #f)))
-  (define (this-thing-name)
-    (mbody-val (<-wait this-thing 'get-name)))
-  (define (should-put-in-me)
-    (and this-thing
-         (slot-ref-maybe-runcheck gameobj 'put-in-me? player this-thing)))
-  (define (default-objection)
-    `("As much as you'd like to, it doesn't seem like you can put "
-      ,(this-thing-name) " " ,preposition " " ,our-name "."))
-  (define (this-thing-objection)
-    (mbody-receive (_ put-in-ok? #:key why-not) ; does the object object to being moved?
-        (<-wait this-thing 'ok-to-be-put-in? player (actor-id gameobj))
-      (and (not put-in-ok?)
-           ;; Either give the specified reason, or give a boilerplate one
-           (or why-not (default-objection)))))
-  (cond
-   ;; Is it not there, or maybe we won't allow it to be taken?
-   ((not this-thing)
-    (<- (message-from message) 'tell
-        #:text `("You don't seem to have any such " ,direct-obj " to put "
-                 ,preposition " " ,our-name ".")))
-
-   ((or (not (should-put-in-me)))
-    (<- (message-from message) 'tell
-        #:text (default-objection)))
-   ;; the thing we wsant to take itself has objected...
-   ((this-thing-objection) =>
-    (lambda (objection)
-      (<- (message-from message) 'tell
-          #:text objection)))
-   ;; looks like we can take it
-   (else
-    ;; Wait to announce to the player just in case settting the location
-    ;; errors out or something.  Maybe it's overthinking things, I dunno.
-    (<-wait this-thing 'set-loc! #:loc (actor-id gameobj))
-    (<- player 'tell
-        #:text `("You put " ,(this-thing-name) " " ,preposition " "
-                 ,our-name "."))
-    (<- player-loc 'tell-room
-        #:text `(,player-name " puts " ,(this-thing-name) " " ,preposition " "
-                              ,our-name ".")
-        #:exclude player))))
+(define* (cmd-take-from-no-op gameobj message
+                              #:key direct-obj indir-obj preposition
+                              (player (message-from message)))
+  (<- player 'tell
+      #:text `("It doesn't seem like you can take anything "
+               ,preposition " "
+               ,(slot-ref gameobj 'name) ".")))
+
+(define* (cmd-put-in-no-op gameobj message
+                           #:key direct-obj indir-obj preposition
+                           (player (message-from message)))
+  (<- player 'tell
+      #:text `("It doesn't seem like you can put anything "
+               ,preposition " "
+               ,(slot-ref gameobj 'name) ".")))