0

私は Lisp World の "Nil" または () です。
エッジ リスト内のすべてのノードのリストを取得したかったので、この問題を解決するコードを書きました。しかし、私はいくつかの予期しない問題に遭遇しました。

(「Land of 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)"

スタックオーバーフロー？私のコードの再帰関数が原因だと思います。どうすればこれを適切に修正できますか?

4

1 に答える 1

2

あなたの再帰的（あなたがリスパーならaddNodeもっとよく呼ばれます）は停止条件を必要とします。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))
于 2015-01-26T04:12:40.007 に答える