我在Lisp世界里是个“无”或()。
我想得到边缘列表中所有节点的列表,我编写了一个代码来解决这个问题但我遇到了一些意想不到的问题。
(摘自《Lisp之地》第8章)
;; Creating edge list
(defun random-node ()
(1+ (random *node-num*)))
(defun edge-pair (a b)
(unless (eql a b)
(list (cons a b) (cons b a))))
(defun make-edge-list ()
(apply #'append (loop repeat *edge-num*
collect (edge-pair (random-node) (random-node)))))
(defparameter el (make-edge-list))
我编写了一个代码,将所有节点提取为如下列表。
;; el : list of dotted list
;; I want to extract all the first element from every dotted lists in el.
;; el : ((25 . 6) (6 . 25) (2 . 13) (13 . 2) (25 . 16) (16 . 25) ....)
;; What I want to get: (25 6 2 13 25 16 ... )
(defun extract-nodes (el)
(let ((nodes nil))
(labels ((addNode (edgeList)
(push (caar edgeList) nodes)
(addNode (cdr edgeList))))
(addNode el))
nodes))
我以为我的代码没那么糟糕,但结果显示了一条令人尴尬的错误消息。
"Stack overflow (deep)"
堆栈溢出?
我认为这是由代码中的递归函数引起的。
我怎样才能把它修好?
谢谢。
最佳答案
递归addNode
(如果您是lisper,最好称为add-node
)需要一个停止条件。
例如。,
(add-node (edge-list)
(push (car (pop edge-list)) nodes)
(when edge-list
(add-node (cdr edge-list))))
注意,这里没有理由使用递归,一个简单的
mapcar
就可以了:(defun extract-nodes (el)
(mapcar #'car el))