4

完全な開示：これは課題のためのものなので、実際のコードソリューションを投稿しないでください！

ユーザーから文字列を取得してスタックとキューに渡し、それら2つを使用して文字を比較し、文字列が回文であるかどうかを判断する必要がある割り当てがあります。プログラムを作成しましたが、どこかに論理エラーがあるようです。関連するコードは次のとおりです。

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

キューとスタックが正しいと仮定すると、一般的なアルゴリズムは正しく機能します（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 に答える
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 に答える