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

欢迎!请查看关于页面了解有关此功能的一些更多信息。

+2
IO
重新标记

以下对 cl-format 的调用引起了一个 Java 级别空指针异常。

(cl-format false "
~@")

当然,这个代码示例没有太多意义。导致这个最小测试用例的真实例子如下。

(cl-format false "~&~
                            item=~A~@
                            transitions=~A~@
                            lhs=~A~@
                            rhs=~A~@"
                      1
                      2
                      3
                      4
                      )

据我所知,问题似乎是处理最后的 ~@,clojure 代码似乎(我猜)在 cl_format.clj 中的函数 compile-directive 里解构了 extract-params 返回的 nil
值。

(defn- compile-directive [s offset]
  (let [[raw-params [rest offset]] (extract-params s offset)
        [_ [rest offset flags]] (extract-flags rest offset)
        directive (first rest)
        def (get directive-table (Character/toUpperCase ^Character directive))
        params (if def (map-params def (map translate-param raw-params) flags offset))]
    (if (not directive)
      (format-error "Format string ended in the middle of a directive" offset))
    (if (not def)
      (format-error (str "Directive \"" directive "\" is undefined") offset))
    [(struct compiled-directive ((:generator-fn def) params offset) def params offset)
     (let [remainder (subs rest 1) 
           offset (inc offset)
           trim? (and (= \newline (:directive def))
                      (not (:colon params)))
           trim-count (if trim? (prefix-count remainder [\space \tab]) 0)
           remainder (subs remainder trim-count)
           offset (+ offset trim-count)]
       [remainder offset])]))

这里是看到的堆栈跟踪。

1. Unhandled java.lang.NullPointerException
   (No message)

             cl_format.clj: 1717  clojure.pprint/compile-directive
             cl_format.clj: 1861  clojure.pprint/compile-format/fn
                  AFn.java:  154  clojure.lang.AFn/applyToHelper
                  AFn.java:  144  clojure.lang.AFn/applyTo
                  core.clj:  665  clojure.core/apply
             utilities.clj:   37  clojure.pprint/consume
             cl_format.clj: 1845  clojure.pprint/compile-format
             cl_format.clj: 1845  clojure.pprint/compile-format
             cl_format.clj:   62  clojure.pprint/cl-format
             cl_format.clj:   27  clojure.pprint/cl-format
               RestFn.java:  521  clojure.lang.RestFn/invoke
                      REPL: 1391  clojure-rte.rte-core/eval24978
                      REPL: 1391  clojure-rte.rte-core/eval24978
             Compiler.java: 7176  clojure.lang.Compiler/eval
             Compiler.java: 7131  clojure.lang.Compiler/eval
                  core.clj: 3214  clojure.core/eval
                  core.clj: 3210  clojure.core/eval
    interruptible_eval.clj:   87  nrepl.middleware.interruptible-eval/evaluate/fn/fn
                  AFn.java:  152  clojure.lang.AFn/applyToHelper
                  AFn.java:  144  clojure.lang.AFn/applyTo
                  core.clj:  665  clojure.core/apply
                  core.clj: 1973  clojure.core/with-bindings*
                  core.clj: 1973  clojure.core/with-bindings*
               RestFn.java:  425  clojure.lang.RestFn/invoke
    interruptible_eval.clj:   87  nrepl.middleware.interruptible-eval/evaluate/fn
                  main.clj:  414  clojure.main/repl/read-eval-print/fn
                  main.clj:  414  clojure.main/repl/read-eval-print
                  main.clj:  435  clojure.main/repl/fn
                  main.clj:  435  clojure.main/repl
                  main.clj:  345  clojure.main/repl
               RestFn.java: 1523  clojure.lang.RestFn/invoke
    interruptible_eval.clj:   84  nrepl.middleware.interruptible-eval/evaluate
    interruptible_eval.clj:   56  nrepl.middleware.interruptible-eval/evaluate
    interruptible_eval.clj:  152  nrepl.middleware.interruptible-eval/interruptible-eval/fn/fn
                  AFn.java:   22  clojure.lang.AFn/run
               session.clj:  202  nrepl.middleware.session/session-exec/main-loop/fn
               session.clj:  201  nrepl.middleware.session/session-exec/main-loop
                  AFn.java:   22  clojure.lang.AFn/run
               Thread.java:  834  java.lang.Thread/run

1 答案

+1

经过一番研究,我现在对错误有了更好的了解。cl-format 给出的格式字符串可以包含零个或多个 指令。每个指令从一个开始 ~ 并以一个调度字符结束,比如 %as 等。在 ~ 和调度字符之间可以发现一些中间项,比如 :@,并且可能有参数,如 ~2,4f~3,-4s~3,-4:@s。其中 :@ 是布尔标志,将其传递给调度函数。

在我的情况下,我给了它 ~@ 然后是字符串末尾,因此 cl-format 理想情况下应该检测到在调度字符之前已达到字符串末尾。

这里是在 common-lisp 中遇到的错误

CL-USER> (format nil "~@")
error in FORMAT: String ended before directive was found
  ~@
  ^
   [Condition of type SB-FORMAT:FORMAT-ERROR]

Restarts:
 0: [RETRY] Retry SLIME REPL evaluation request.
 1: [*ABORT] Return to SLIME's top level.
 2: [ABORT] abort thread (#<THREAD "repl-thread" RUNNING {1004289BC3}>)

Backtrace:
  0: (SB-FORMAT::FORMAT-ERROR-AT* "~@" 0 "String ended before directive was found" NIL)
  1: (SB-FORMAT::FORMAT-ERROR-AT "~@" 0 "String ended before directive was found")
  2: ((FLET SB-FORMAT::GET-CHAR :IN SB-FORMAT::PARSE-DIRECTIVE))
  3: (SB-FORMAT::PARSE-DIRECTIVE "~@" 0 NIL)
  4: (SB-FORMAT::%TOKENIZE-CONTROL-STRING "~@" 0 2 NIL)
  5: (SB-FORMAT::TOKENIZE-CONTROL-STRING "~@")
  6: (SB-FORMAT::%FORMAT #<SB-IMPL::CHARACTER-STRING-OSTREAM {1004883B33}> "~@" NIL NIL)
  7: (FORMAT NIL "~@")
  8: (FORMAT NIL "~@") [more]
  9: (SB-INT:SIMPLE-EVAL-IN-LEXENV (FORMAT NIL "~@") #<NULL-LEXENV>)
 10: (EVAL (FORMAT NIL "~@"))
顺便说一下,看起来 `cl_format.clj` 中的 `compile-directive` 函数有代码来检测 `"格式字符串在指令中间结束"` 并调用 `format-error` 来显示有用的错误信息。不幸的是,Java 错误的出现似乎阻止了这段代码的执行。
我认为可以通过这个补丁修复 `compile-directive` 函数的问题。注意,我使用的是没有 `else` 的 `if` 传统,这是前一位作者所使用的。修改只是简单地检测 `let` 中声明的 `directive` 是否是 `nil` 而不是字符。

```



















```

通过这个修改,我得到了一个有用的错误信息,我相信这是原作者的意图。

```


在 nrepl.middleware.interruptible-eval/evaluate$fn$fn (interruptible_eval.clj:87) 处发生执行错误。
格式字符串在指令中间结束
~:@
   ^

```
我尝试用这个简单的补丁创建一个pull request。然而,我发现这个项目不接受pull request。我试图遵循说明,但我承认我迷路了。
我已收到并记录了您的贡献者协议,并发送了邀请邮件到您的邮箱用于jira。我在 https://clojure.atlassian.net/browse/CLJ-2653 上提交了一个工单。如果您喜欢,可以参照 https://clojure.org/dev/developing_patches 上的说明来制作补丁。谢谢!
这个补丁被拒绝了吗?今天我又遇到了相同的问题,Clojure 1.11.1 中的代码似乎还存在同样的错误。

也许我在补丁中做错了什么? 

有人能帮忙吗?
我们还没有看过它,抱歉。
by
这是一个非常低风险的修复。在我看来
...