使用pr-str而不是str打印对象来改进错误信息

Question

ClojureScript中的许多错误信息包括类似无效参数的这种情况

(throw (js/Error. (str "不支持名称: " x)))

如果{{x}}是nil，那么产生的错误信息是{{"不支持名称: "}}，这会使调试变得有些困难。如果{{x}}用{{pr-str}}包裹，则错误信息会更易于理解：{{"不支持名称: nil"}}。

如果您对这个问题感兴趣，我可以准备一个补丁，用pr-str包裹这类错误。

Answer 1

评论者：dnolen

去做吧

Answer 2

评论者：mfikes

我也对CLJS-2089考虑了一番。这似乎是普遍正确的事情。

Answer 3

评论者：mfikes

补丁不再适用。

Answer 4

评论者：slipset

现在应该适用了，希望如此

Answer 5

评论者：mfikes

0001-CLJS-1908-Wrap-vars-in-throw-message-with-pr-str.patch 已不再适用

Answer 6

评论者：slipset

0001-CLJS-1908-Wrap-vars-in-throw-message-with-pr-str-2.patch 已更新，应该适用

Answer 7

评论者：mfikes

0001-CLJS-1908-Wrap-vars-in-throw-message-with-pr-str-2.patch 获得LGTM

Answer 8

评论者：mfikes

0001-CLJS-1908-Wrap-vars-in-throw-message-with-pr-str-2.patch 通过CI和Canary测试(/)

Answer 9

评论者：mfikes

0001-CLJS-1908-Wrap-vars-in-throw-message-with-pr-str-2.patch 已添加到Patch Tender (i)

Answer 10

参考：[https://clojure.atlassian.net/browse/CLJS-1908](https://clojure.atlassian.net/browse/CLJS-1908) （由 [email protected] 报告）