请在2024 Clojure 状态调查!中分享您的想法。

欢迎!请查看关于页面了解更多关于其工作方式的信息。

0 投票
Spec
every-kv 文档说明 "需要独立的关键字和值前缀,并在关联集合上工作"。向量对关联?返回 true,但当前不起作用


user=> (s/conform (s/every-kv any? any?) [])
[]
user=> (s/conform (s/every-kv any? any?) [1 2 3])
:clojure.spec/invalid
user=> (s/conform (s/every-kv integer? string?) [])
[]
user=> (s/conform (s/every-kv integer? string?) ["x"])
:clojure.spec/invalid


另一个类似问题


(s/explain-data (s/every-kv int? int?) [{:a :b}])
UnsupportedOperationException nth 不支持此类型:PersistentArrayMap  clojure.lang.RT.nthFrom (RT.java:903)


**原因**:向量不应与 every-kv 一起使用。every-kv 和 every-impl 的组合假设传递给 every-kv 的集合可以提供映射项的序列。在解释的情况下,由 every-kv 创建的 ::kfn 用于使用键而不是元素索引来创建更好的路径段。kfn 假设集合的元素可以调用 `(nth entry 0)` 来调用元素。在上面的解释失败中,映射 {:a :b} 将在调用 nth 时引发。

**建议**:做以下操作以更清楚地说明 coll 元素是映射项的要求

**修改文档字符串以将 "seqs to map entries"(序列到映射项)而不是 "is a map"(是映射)
**修改 kfn 以添加检查元素是否是项。如果是,则使用它的键。如果不是,则使用元素的索引(即集合中的索引)。在这种情况下,当传入项时,它将报告实际键,但当传入不是项的东西时,它将报告非项的集合基于索引。

修补后,explain-data 调用将提供有用的错误而不是上面的异常


user=> (s/explain-data (s/every-kv int? int?) [{:a :b}])
#:clojure.spec.alpha{:problems ({:path [], :pred vector?, :val {:a :b}, :via [], :in [0]})}



**补丁**:clj-2080-8.patch

8 答案

0 投票

评论者:alexmiller

目前,我倾向于将every-kv中的文档从“关联集合”改为“映射”,但将与Rich确认。

0 投票

评论者:alexmiller

更新以适用于master

0 投票

评论者:alexmiller

更新补丁以适用于spec.alpha。

0 投票
_评论者:stu_

请更新这张票据,包括在描述中所示的示例的预期行为,以及显示预期行为的测试。我仍在看到向量对于符合失败。 


(s/explain-data (s/every-kv integer? string?) ["x"])
=> #:clojure.spec.alpha{:problems ({:path [], :pred vector?, :val "x", :via [], :in [0]})}
0 投票

评论者:alexmiller

向量应该失败 - every-kv需要一个映射条目的序列。该补丁在字符串中澄清了这一点,并修复了在生成explain-data时可能抛出(错误地)抛出的异常。我更新了标题,将行为后的explain-data包含在描述中,并添加了补丁中的测试。

0 投票

评论者:alexmiller

之前丢失了文档更改,现在在-7补丁中重新添加。

0 投票

评论者:alexmiller

向 master 应用 -8 个更新,无语义变化

0 投票
参考:https://clojure.atlassian.net/browse/CLJ-2080(由 bbloom 报告)
...