使用pr-str代替str打印对象来改善错误信息

Question

ClojureScript中的许多错误信息都包含类似这样的无效参数

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

如果{{x}}为nil，则生成的错误信息为{{"不支持名称: "}}，这在调试时有点令人困惑。如果用{{pr-str}}包装{{x}}，则错误信息将更为易懂：{{"不支持名称: 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] 奉告)