非常简单的练习程序无法编译

时间:2019-07-29 17:35:25

标签: clojure

我目前正在阅读《勇敢与真实的Clojure》 在本章中,他们将解释一个程序,该程序采用代表霍比特人身体部位的哈希映射向量。由于带有零件的列表仅是非对称提供的(只有左臂,左眼等是其中的一部分),因此有必要编写一个函数来添加相应的右侧零件。后来进行了一项练习,以扩展此功能以获取一个数字,并为每个左侧部分增加该身体部分的数量。第二个功能是随机选择身体部位。

这是我的代码:

(ns clojure-noob.core
  (:gen-class)
  (:require [clojure.string :as str] ))


(def asym-hobbit-body-parts [{:name "head" :size 3}
                             {:name "left-eye" :size 1}
                             {:name "left-ear" :size 1}
                             {:name "mouth" :size 1}
                             {:name "nose" :size 1}
                             {:name "neck" :size 2}
                             {:name "left-shoulder" :size 3}
                             {:name "left-upper-arm" :size 3}
                             {:name "chest" :size 10}
                             {:name "back" :size 10}
                             {:name "left-forearm" :size 3}
                             {:name "abdomen" :size 6}
                             {:name "left-kidney" :size 1}
                             {:name "left-hand" :size 2}
                             {:name "left-knee" :size 2}
                             {:name "left-thigh" :size 4}
                             {:name "left-lower-leg" :size 3}
                             {:name "left-achilles" :size 1}
                             {:name "left-foot" :size 2}])


(defn make-sym-parts [asym-set num]
  (reduce (fn [sink, {:keys [name size] :as body_part}]
           (if (str/starts-with? name "left-")
             (into sink [body_part 
                         (for [i (range num)]
                           {:name (str/replace name #"^left" (str i))
                            :size size})])
             (conj sink body_part)))
           []
           asym-set))


(defn rand-part [parts]
  (def size-sum (reduce + (map :size parts)))
  (def thresh (rand size-sum))
  (loop [[current & remaining] parts
         sum (:size current)]
    (if (> sum thresh)
      (:name current)
      (recur remaining (+ sum (:size (first remaining)))))))


(defn -main
  "I don't do a whole lot ... yet."
  [arg]
  (cond 
    (= arg "1") (println (make-sym-parts asym-hobbit-body-parts 3))
    (= arg "2") (println (rand-part asym-hobbit-body-parts))
    (= arg "3") (println (rand-part (make-sym-parts asym-hobbit-body-parts 3)))))

所以

lein run 1

工作并打印出展开的矢量

lein run 2

也可以工作并打印出随机的身体部位名称。

但是:

lein run 3

将产生以下错误:

861 me@ryzen-tr:~/clojure_practice/clojure-noob$ lein run 3
862 Exception in thread "main" Syntax error compiling at (/tmp/form-init15519101999846500993.clj:1:74).
863         at clojure.lang.Compiler.load(Compiler.java:7647)
864         at clojure.lang.Compiler.loadFile(Compiler.java:7573)
865         at clojure.main$load_script.invokeStatic(main.clj:452)
866         at clojure.main$init_opt.invokeStatic(main.clj:454)
867         at clojure.main$init_opt.invoke(main.clj:454)
868         at clojure.main$initialize.invokeStatic(main.clj:485)
869         at clojure.main$null_opt.invokeStatic(main.clj:519)
870         at clojure.main$null_opt.invoke(main.clj:516)
871         at clojure.main$main.invokeStatic(main.clj:598)
872         at clojure.main$main.doInvoke(main.clj:561)
873         at clojure.lang.RestFn.applyTo(RestFn.java:137)
874         at clojure.lang.Var.applyTo(Var.java:705)
875         at clojure.main.main(main.java:37)
876 Caused by: java.lang.NullPointerException
877         at clojure.lang.Numbers.ops(Numbers.java:1068)
878         at clojure.lang.Numbers.add(Numbers.java:153)
879         at clojure.core$_PLUS_.invokeStatic(core.clj:992)
880         at clojure.core$_PLUS_.invoke(core.clj:984)
881         at clojure.lang.ArrayChunk.reduce(ArrayChunk.java:63)
882         at clojure.core.protocols$fn__8139.invokeStatic(protocols.clj:136)
883         at clojure.core.protocols$fn__8139.invoke(protocols.clj:124)
884         at clojure.core.protocols$fn__8099$G__8094__8108.invoke(protocols.clj:19)
885         at clojure.core.protocols$seq_reduce.invokeStatic(protocols.clj:27)
886         at clojure.core.protocols$fn__8131.invokeStatic(protocols.clj:75)
887         at clojure.core.protocols$fn__8131.invoke(protocols.clj:75)
888         at clojure.core.protocols$fn__8073$G__8068__8086.invoke(protocols.clj:13)
889         at clojure.core$reduce.invokeStatic(core.clj:6824)
890         at clojure.core$reduce.invoke(core.clj:6810)
891         at clojure_noob.core$rand_part.invokeStatic(core.clj:39)
892         at clojure_noob.core$rand_part.invoke(core.clj:38)
893         at clojure_noob.core$_main.invokeStatic(core.clj:54)
894         at clojure_noob.core$_main.invoke(core.clj:48)
895         at clojure.lang.Var.invoke(Var.java:384)
896         at user$eval140.invokeStatic(form-init15519101999846500993.clj:1)
897         at user$eval140.invoke(form-init15519101999846500993.clj:1)
898         at clojure.lang.Compiler.eval(Compiler.java:7176)
899         at clojure.lang.Compiler.eval(Compiler.java:7166)
900         at clojure.lang.Compiler.load(Compiler.java:7635)
901         ... 12 more

我完全不知道为什么。还要搜索错误的第一行不会显示有用的信息。有人知道这个问题吗?

1 个答案:

答案 0 :(得分:3)

问题是您有返回混合类型的向量。一些元素是地图,一些元素是列表。请注意make-sym-parts的前几项:

(make-sym-parts asym-hobbit-body-parts 3)

=>
[{:name "head", :size 3}
 {:name "left-eye", :size 1}
 ({:name "0-eye", :size 1} {:name "1-eye", :size 1} {:name "2-eye", :size 1})
  . . .

查看我在此处列出的最后一个条目。它不是地图;这是地图列表。当您尝试将:size应用于列表时,会得到nil

(:size '({:name "0-eye", :size 1} {:name "1-eye", :size 1} {:name "2-eye", :size 1}))

=> nil

当您将:size映射到整个列表时,您会得到:

(->> (make-sym-parts asym-hobbit-body-parts 3)
     (map :size))

=> (3 1 nil 1 nil 1 1 2 3 nil 3 nil 10 10 3 nil 6 1 nil 2 nil 2 nil 4 nil 3 nil 1 nil 2 nil)

这引起了一个问题,因为您是通过+将这些值提供给reduce的,而如果您给+nil,则nil会正确地拟合(defn make-sym-parts [asym-set num] (reduce (fn [sink, {:keys [name size] :as body_part}] (if (str/starts-with? name "left-") (into sink (conj ; I threw in a call to conj here and rearranged it a bit (for [i (range num)] {:name (str/replace name #"^left" (str i)) :size size}) body_part)) (conj sink body_part))) [] asym-set)) (make-sym-parts asym-hobbit-body-parts 3) => [{:name "head", :size 3} {:name "left-eye", :size 1} {:name "0-eye", :size 1} {:name "1-eye", :size 1} {:name "2-eye", :size 1} {:name "left-ear", :size 1} {:name "0-ear", :size 1} {:name "1-ear", :size 1} {:name "2-ear", :size 1} {:name "mouth", :size 1} {:name "nose", :size 1} {:name "neck", :size 2} {:name "left-shoulder", :size 3} {:name "0-shoulder", :size 3} {:name "1-shoulder", :size 3} {:name "2-shoulder", :size 3} {:name "left-upper-arm", :size 3} {:name "0-upper-arm", :size 3} {:name "1-upper-arm", :size 3} {:name "2-upper-arm", :size 3} {:name "chest", :size 10} {:name "back", :size 10} {:name "left-forearm", :size 3} {:name "0-forearm", :size 3} {:name "1-forearm", :size 3} {:name "2-forearm", :size 3} {:name "abdomen", :size 6} {:name "left-kidney", :size 1} {:name "0-kidney", :size 1} {:name "1-kidney", :size 1} {:name "2-kidney", :size 1} {:name "left-hand", :size 2} {:name "0-hand", :size 2} {:name "1-hand", :size 2} {:name "2-hand", :size 2} {:name "left-knee", :size 2} {:name "0-knee", :size 2} {:name "1-knee", :size 2} {:name "2-knee", :size 2} {:name "left-thigh", :size 4} {:name "0-thigh", :size 4} {:name "1-thigh", :size 4} {:name "2-thigh", :size 4} {:name "left-lower-leg", :size 3} {:name "0-lower-leg", :size 3} {:name "1-lower-leg", :size 3} {:name "2-lower-leg", :size 3} {:name "left-achilles", :size 1} {:name "0-achilles", :size 1} {:name "1-achilles", :size 1} {:name "2-achilles", :size 1} {:name "left-foot", :size 2} {:name "0-foot", :size 2} {:name "1-foot", :size 2} {:name "2-foot", :size 2}] 不是数字。


那么,解决方法是什么?老实说,我现在还不到3个月才写Clojure,所以我变得生疏了,也没有阅读问题说明,但看​​来您只需要把列表弄平即可。

map

在对(map :size的调用中,您还可以检查元素是地图还是列表。如果是列表,则可以再次在子列表上调用mapcat。这取决于您是否要使用平面列表或嵌套列表作为最终结果。您也许还可以使用nil来获取平面列表,尽管那样的话您就需要处理不是地图的条目。


又如何从(非常冗长的)堆栈跟踪中找出问题所在?一旦您意识到不良的解构和不良的键查询(如上文所述)会返回; If you have an NPE, that means you have a nil being passed somewhere... Caused by: java.lang.NullPointerException 877 at clojure.lang.Numbers.ops(Numbers.java:1068) 878 at clojure.lang.Numbers.add(Numbers.java:153) ; ... so, you're passing a nil to + ("_PLUS_") 879 at clojure.core$_PLUS_.invokeStatic(core.clj:992) 880 at clojure.core$_PLUS_.invoke(core.clj:984) 881 at clojure.lang.ArrayChunk.reduce(ArrayChunk.java:63) 882 at clojure.core.protocols$fn__8139.invokeStatic(protocols.clj:136) 883 at clojure.core.protocols$fn__8139.invoke(protocols.clj:124) 884 at clojure.core.protocols$fn__8099$G__8094__8108.invoke(protocols.clj:19) 885 at clojure.core.protocols$seq_reduce.invokeStatic(protocols.clj:27) 886 at clojure.core.protocols$fn__8131.invokeStatic(protocols.clj:75) 887 at clojure.core.protocols$fn__8131.invoke(protocols.clj:75) 888 at clojure.core.protocols$fn__8073$G__8068__8086.invoke(protocols.clj:13) ; ... and it's happening inside a call to reduce 889 at clojure.core$reduce.invokeStatic(core.clj:6824) ; ... and that call to reduce is happening inside of rand-part 891 at clojure_noob.core$rand_part.invokeStatic(core.clj:39) 892 at clojure_noob.core$rand_part.invoke(core.clj:38) 893 at clojure_noob.core$_main.invokeStatic(core.clj:54) ,就可以更轻松地进行推理了。每当您获得NPE时,就可以立即假设您正在解构错误的东西,或者使用错误的密钥进行查找。这些不是进行NPE的唯一原因,但是根据我在Clojure的经验,它们是最常见的。

从上至下读取堆栈跟踪,以跟踪不良数据源自何处以及在何处读取。请注意我的评论,以获取有关如何阅读的提示:

+

reduce内部只有一次这样的rand-part实例被传递到nil的地方,因此这是一个开始寻找的好地方。从那里,您只需要使用标准调试技术来跟踪defn的来源。

这里的重点只是扫描堆栈跟踪以尝试找到您可以识别的单词。不幸的是,由于Clojure名称在翻译成Java时会变得“混乱”,因此名称往往非常冗长和嘈杂。您只需要学习“穿越噪音”即可找到相关信息。经过一些练习,它变得容易。



其他一些注意事项:

  • 请勿在{{1​​}}内使用def。 def创建不受范围限制的全局变量。请改用let

  • 尝试使用更多的缩进。单个缩进空间不是很好。

  • Clojure使用破折号。您在大多数情况下都在使用它,但是body_part似乎是Python的倒退。

如果需要,可以将此代码发布到“代码审查”中,我们可以提出建议以帮助您改进它。