Question

我正在研究自己在Common Lisp中自己编写一次，实现Shunting-yard算法。我认为它没关系，即使它出来相当难看，如果我怀疑它的Lispy-ness，但是在测试REPL中的函数时，我在标题中得到错误。

代码如下，测试用例为(shunting-yard '(3 + 5))。

(defparameter *output-queue*   nil)
(defparameter *operator-stack* nil)

(defun determine-precedence (operator)
  (case operator
    (('+ '-) 2)
    (('* '/) 3)
    ('^      4)))

(defun shunting-yard (stmt)
  (loop until (null stmt) do
       (let ((token (car stmt)))
         (cond ((or (numberp token)
            (eq token '\())
            (setf *output-queue* (cons token *output-queue*)))
               ((mapcar #'(lambda (x) (eq token x)) '(+ - * / ^))
            (let* ((token-precedence (determine-precedence token))
                   (stack-topmost (car *operator-stack*))
                   (stack-precedence (determine-precedence stack-topmost)))
              (when (< token-precedence stack-precedence)
                (setf *output-queue* (cons stack-topmost *output-queue*))
                (setf *operator-stack* (cdr *operator-stack*)))
              (setf *operator-stack* (cons token *operator-stack*))))
               ((eq token '\))
            (loop for stack-topmost in *operator-stack*
               until (eq stack-topmost '\()
               do (progn
                (setf *output-queue* (cons stack-topmost *output-queue*))
                (setf *operator-stack* (cdr *operator-stack*)))
               finally (setf *operator-stack* (cdr *operator-stack*)))))
         (setf stmt (cdr stmt))))
  (loop while (not (null *operator-stack*))
     do (progn
      (setf *output-queue* (cons (car *operator-stack*) *output-queue*))
      (setf *operator-stack* (cdr *operator-stack*))))
  (nreverse *output-queue*))

代码本身的错误（我的猜测）还是在我的测试用例中？

非常感谢，这真的很有趣，而且我迫不及待地想要开展其他工作，但只有在我开始工作之后。

Answer 1

有几个错误：

首先：

(defun determine-precedence (operator)
  (case operator
    (('+ '-) 2)
    (('* '/) 3)
    ('^      4)))

引号需要去。所有

第二

(mapcar #'(lambda (x) (eq token x)) '(+ - * / ^))

上面没有做你的想法。将其替换为对MEMBER的调用。

第三

 (when (< token-precedence stack-precedence)

您需要确保变量确实绑定到数字。使用像

这样的东西

 (check-type token-precedence number)
 (check-type stack-precedence number)

之前作为支票。

快乐的进一步调试......

Common Lisp Error：Expected-type：REAL datum：NIL

1 个答案: