Move containers over to their own module.
[mudsync.git] / mudsync / gameobj.scm
index 55b76bbb15dc959139a5b388fd5d6d37a83136e9..64c79f1397b67e34175dd6d6798261401786b49c 100644 (file)
@@ -47,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
 ;;; =======
   ;; (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
 
             ;; 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
@@ -491,138 +483,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
-                        (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)
-                   (define goes-by (mbody-val (<-wait occupant 'goes-by)))
-                   (when (ci-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
-                     (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)
-                   (define goes-by (mbody-val (<-wait occupant 'goes-by)))
-                   (when (ci-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)
-    (<- player 'tell
-        #:text `("You don't seem to have any such " ,direct-obj " to put "
-                 ,preposition " " ,our-name ".")))
-
-   ((or (not (should-put-in-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 (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) ".")))