4

完全披露：这是一个作业，所以请不要发布实际的代码解决方案！

我有一个任务要求我从用户那里获取一个字符串并将其传递到一个堆栈和一个队列中，然后使用这两个来比较字符以确定该字符串是否是回文。我已经编写了程序，但某处似乎存在一些逻辑错误。以下是相关代码：

public static void main(String[] args) {

    UserInterface ui = new UserInterface();
    Stack stack = new Stack();
    Queue queue = new Queue();
    String cleaned = new String();
    boolean palindrome = true;

    ui.setString("Please give me a palindrome.");
    cleaned = ui.cleanString(ui.getString());

    for (int i = 0; i < cleaned.length(); ++i) {
        stack.push(cleaned.charAt(i));
        queue.enqueue(cleaned.charAt(i));
    }

    while (!stack.isEmpty() && !queue.isEmpty()) {
        if (stack.pop() != queue.dequeue()) {
            palindrome = false;
        }
    }

    if (palindrome) {
        System.out.printf("%s is a palindrome!", ui.getString());
    } else
        System.out.printf("%s is not a palindrome :(", ui.getString());

    stack.dump();
    queue.clear();

}

 public class Stack {

   public void push(char c) {
    c = Character.toUpperCase(c);
    Node oldNode = header;
    header = new Node();
    header.setData(c);
    header.setNext(oldNode);
  }

  public char pop() {
    Node temp = new Node();
    char data;
    if (isEmpty()) {
        System.out.printf("Stack Underflow (pop)\n");
        System.exit(0);
    }
    temp = header;
    data = temp.getData();
    header = header.getNext();
    return data;
  }

}

public class Queue {

  public void enqueue(char c) {
    c = Character.toUpperCase(c);
    Node n = last;
    last = new Node();
    last.setData(c);
    last.setNext(null);
    if (isEmpty()) {
        first = last;
    } else n.setNext(last);     
  }

  public char dequeue() {
    char data;
    data = first.getData();
    first = first.getNext();
    return data;
  }

}

public String cleanString(String s) {
    return s.replaceAll("[^A-Za-z0-9]", "");
}

基本上，当通过 Eclipse 中的调试器运行我的代码时，我的 pop 和 dequeue 方法似乎只选择某些字母数字。我replaceAll("[^A-Za-z0-9]", "")用来“清理”用户的任何非字母数字字符（！、？、& 等）的字符串。当我说它只选择某些字符时，似乎没有任何我可以辨别的模式。有任何想法吗？

4

2 回答 2

0

我建议使用调试器来查看正在比较的确切内容，或者至少吐出一些打印行：

while (!stack.isEmpty() && !queue.isEmpty()) {
    Character sc = stack.pop();
    Character qc = queue.dequeue();
    System.out.println(sc + ":" + qc);
    if (sc != qc) {
        palindrome = false;
    }
}
于 2012-06-08T18:20:27.703 回答
0

假设您的队列和堆栈是正确的，您的一般算法可以正常工作（我使用 jdk 中的 Deque 实现进行了尝试）。由于您的任务涉及数据结构，我几乎只是采用了您的主要逻辑并用 ArrayDequeue 替换了数据结构，所以我不觉得我在为您回答这个问题。

    String word = "ooffoo";

    word = word.replaceAll("[^A-Za-z0-9]", "");

    Deque<Character> stack = new ArrayDeque<Character>(word.length());
    Deque<Character> queue = new ArrayDeque<Character>(word.length());

    for (char c : word.toCharArray()) {
        stack.push(c);
        queue.add(c);
    }

    boolean pal = true;

    while (! stack.isEmpty() && pal == true) {
        if (! stack.pop().equals(queue.remove())) {
            pal = false;
        }
    }

    System.out.println(pal);
于 2012-06-08T18:01:31.697 回答