在参数之间验证关系 - spec fdef

Question

我有一个迷你餐厅应用程序的以下数据结构作为状态

    {:orders {:e4d55743-c964-48e8-9cd1-cb1cf9075617 [#:helloworld.restaurant{:name "chilly parotta", :quantity 3}  #:helloworld.restaurant{:name "masal dosa", :quantity 2}]}, :menu {:kothu-parotta #:helloworld.restaurant{:name "Kothu Parotta", :price 9.5, :quantity 50},  :butter-naan #:helloworld.restaurant{:name "Butter Naan", :price 3, :quantity 50}, :paneer-butter-masala #:helloworld.restaurant{:name "Paneer Butter Masala", :price 9.5, :quantity 50}}}

我有一个函数用来更新订单中的每项订单对应菜单项，以从库存中减去订单数量并返回新的菜单。我需要验证

1. 订单项中的项应该存在于菜单中
2. 库存中的项数量应该大于订单数量

我有以下规范（注意：我现在还没有验证 2）。当我运行这个时，update-menu-item-for-order-item 函数调用没有验证我的谓词 1。我在 fdef 规范中做错了什么？

(s/def ::name string?)
(s/def ::price number?)
(s/def ::quantity number?)
(s/def ::order-item (s/keys :req [::name ::quantity]))
(s/def ::order-items (s/coll-of ::order-item))
(s/def ::menu-item (s/keys :req [::name ::price ::quantity]))
(s/def ::menu-item-id keyword?)
(s/def ::menu (s/map-of ::menu-item-id ::menu-item))

(s/fdef restaurant-save-menu :args (s/cat :menu-items (s/coll-of ::menu-item)))

(s/fdef update-menu-item-for-order-item
        :args (s/and (s/cat :menu ::menu :order-item ::order-item)
                     #(contains? (:menu %) (keyword (slugify (::name (:order-item %))))))
        :ret ::menu)

(stest/instrument `update-menu-item-for-order-item)
(stest/instrument `restaurant-save-menu)

Answer 1

你调用的是什么？你看到了什么结果？

Comments

另一种调试技术是独立于函数测试函数的参数规范（您可以将它分割并命名为更容易，但这也可以从函数规范中检索到）

    (s/valid? (:args (s/get-spec `update-menu-item-for-order-item)) [a-menu a-item])

---

以下是代码。当我调用restaurant-create-order时，它会调用create-order，而create-order又会调用update-menu-item-for-order-item。现在，断言1（订货的项目应该在菜单中存在）工作正常。之前，它不起作用，我不知道我做了什么，但现在它工作了！不管怎样，谢谢。

<pre>
(def restaurant (atom {}))

(defn update-menu-item-for-order-item [menu order-item]
  (let [name (::name order-item)
        item-id (keyword (slugify (::name order-item)))
        order-quantity (::quantity order-item)]
    (update-in menu
               [item-id]
               (fn [item]
                 (if (nil? item)
                   (throw (IllegalArgumentException. (str "抱歉，我们没有 " name " 在我们的菜单中")))
                   (if (< (::quantity item) order-quantity)
                     (throw (IllegalArgumentException.
                              (str "不够 " (::name item) "，我们缺了 "
                                   (- order-quantity (::quantity item))
                                   "。请更正订单并重新提交")))
                     (assoc item ::quantity (- (::quantity item) order-quantity))))))))

(defn create-order [orders menu order-items]
  (let [order-id (keyword (str (UUID/randomUUID)))]
    {::orders (assoc orders order-id order-items)
     ::menu   (reduce #(update-menu-item-for-order-item %1 %2) menu order-items)}))

(defn restaurant-create-order [order-items]
  (swap! restaurant #(create-order (::orders %) (::menu %) order-items)))

(restaurant-create-order [{::name "chilly parotta" ::quantity 3}
                          {::name "masal dosa" ::quantity 2}])
<pre>

---

我没有时间阅读所有这些代码 - 你能简化成单次函数调用，并用字面数据吗？

---

代码目前对我跟Prabu都有效。我认为是旧REPL状态的问题。

---

是的，谢谢 @alexmiller。代码现在可以工作了