tools.reader的parse-symbol在读取新n维类数组时太宽松了

Question

Tools.reader接受这个符号：MyClazz:/3，但不接受符号MyClass:。这似乎不一致。这是这个"提问"的主要主题。

顺便说一下，我发现这个提交https://github.com/clojure/tools.reader/commit/31adc06cd56753ed5a3cd443cea32158ba68c45c#diff-11ba7fe0994caede16eeb1c6ed52fc3aba112849ce1fd23dff0b9fb72441a09fR111使用正则表达式+re-matches来检查字符串sym是否为1个大小的数字字符串。我发现这比版本1.4.2略有性能下降。我认为可以使用稍微复杂但性能更好的检查来实现该检查

• 检查字符串大小为1
• 只有在此之后才检查该字符串是否代表1到9之间的数字

例如，这个版本似乎比re-matches方法快大约6倍

user=> (defn array-dim? [^String s] (and (= 1 (.length s)) (some-> (try (Integer/parseInt s) (catch Exception e nil)) pos?)))
#'user/array-dim?
user=> (time (dotimes [i 10000000] (array-dim? "1")))
"Elapsed time: 45.287416 msecs"

user=> (defn array-dim2? [^String s] (re-matches #"[1-9]" s))
#'user/array-dim2?
user=> (time (dotimes [i 10000000] (array-dim2? "1")))
"Elapsed time: 304.30275 msecs"
nil

注意clojure自己的读取器是如何实现的

https://github.com/clojure/clojure/commit/1fa5b038a434da34f787e3aec56f9cb48ed4dd99#diff-9216bc4fd055e0567fa034ec14a8ba470d36ba5be16418c8f2e2ebe1939bbb21R259

Comments

这是我的错误，因为我实现了正则表达式版本。一旦有JIRA票据，我将编写一个补丁。

Answer 1

创建工单
https://clojure.atlassian.net/browse/TRDR-74

Comments

谢谢。性能问题是意外收获。主要问题是

Tools.reader接受这个符号：MyClazz:/3，但不接受符号MyClass:.这似乎不一致。这是这个“问题”的主要内容。