2024 Clojure状态调查中分享你的想法!

欢迎!请查看关于这一页以获取更多关于如何工作的信息。

+1 表扬
ClojureScript
{{clojure.string/split}}带着限制不能正确按以下方式分割字符串


(clojure.string/split "quaqb" #"q(?!u)") ; <- 匹配后面非'u'的'q'
;;=> ["qua" "b"]

(clojure.string/split "quaqb" #"q(?!u)" 2)
;;=> ["" "uaqb"]


第一种情况的结果是我们想要的,但第二种情况的结果是错误的。
因为,如果有限制,它将使用自己的算法而不是JavaScript的{{String.prototype.split}}函数来分割字符串。
并且当正则表达式中使用[前瞻或后顾|https://regexper.cn/lookaround.html]时,该算法有问题。


;; clojure.string/split
(let [re #"q(?!u)"]
  (loop [s "quaqb"
         limit 2
         parts []]
    (if (== 1 limit)
      (conj parts s)
      (let [m (re-find re s)] ; <- 1!
        (if-not (nil? m)
          (let [index (.indexOf s m)] ; <- 2!
            (recur (.substring s (+ index (count m)))
                   (dec limit)
                   (conj parts (.substring s 0 index))))
          (conj parts s))))))
;;=> ["" "uaqb"]

(re-find #"q(?!u)" "quaqb") ; <- 1!
;; => "q"

(.indexOf "quaqb" "q") ; <- 2!
;;=> 0


我们应该从JavaScript的{{RegExp.prototype.exec}}函数中得到索引,而不是进行计算。


;; clojure.string/split
(let [re #"q(?!u)"]
  (loop [s "quaqb"
         limit 2
         parts []]
    (if (== 1 limit)
      (conj parts s)
      (let [m (.exec re s)]
        (if-not (nil? m)
          (let [index (.-index m)]
            (recur (.substring s (+ index (count (aget m 0))))
                   (dec limit)
                   (conj parts (.substring s 0 index))))
          (conj parts s))))))
;;=> ["qua" "b"]


i 在 V8, Spidermonkey, Nashorn 上进行了测试。

6 答案

0 投票

评论由:vmfhrmfoaj发表

添加补丁

0 投票

评论由:vmfhrmfoaj发表

我更新了补丁以更改测试

0 投票

评论由:vmfhrmfoaj发表

我更新了补丁以修复边缘情况

0 投票

评论由:vmfhrmfoaj发表

我更新了补丁以修复错误并添加测试
(我为多次更新补丁道歉,我想这很简单)

0 投票

评论由:vmfhrmfoaj发表

更新补丁

0 投票
参考:https://clojure.atlassian.net/browse/CLJS-2528 (由vmfhrmfoaj报告)
...