Question

我遇到了问题。我收到了为回文检查程序编写伪代码的任务。我的问题是，虽然我在伪代码分配上得到了很好的分数，但当我尝试在java中编写代码用于我自己的启发时，我无法使其能够检查int和string。

import java.util.Scanner;
public class palindromeCheck {

private static Scanner in;

public static void main(String[] args) {

    in = new Scanner(System.in);
    String forward;
    String reverse = "";
    int reverseCountdown;
    System.out.println("enter a string and I will tell you if its a palindrome");
    forward= in.next();
    int stringLength= forward.length();


    for(reverseCountdown = stringLength-1; stringLength>-1; stringLength--);
    reverse=reverse+forward.charAt(reverseCountdown);

    if(forward.equals(reverse))
        System.out.println("Bro you got a palindrome!");
    else
        System.out.println("Thats not a palindrome...");

    }
 }

现在我的问题就是我可以用我可怜的技能找到它，就是在我的for循环中，我将字符值逐个转换为字符串，但是，我无法想出一个代码解决方案，将采取所有角色;它似乎除了一个以外都是它们。（或者我的错误可能是其他的东西。）但是这就像我的代码将会运行一样，但我从来没有得到回文的回应（即使对于像222这样明显的东西），除了像单个字符项一样0或1。

任何帮助解决这个问题，甚至理解更优雅的检查方式都将受到赞赏。

Answer 1

这不是伪代码吗？为什么你的for循环后有一个分号？

我会删除它。

这不是唯一的问题。你应该使用reverseCountdown作为循环检查而不是stringLength，你应该减少reverseCountdown。

for(reverseCountdown = stringLength-1; reverseCountdown >= 0; reverseCountdown--)

Answer 2

您的for循环由分号终止。

for(reverseCountdown = stringLength-1; stringLength>-1; stringLength--);
reverse=reverse+forward.charAt(reverseCountdown);

我相信你想（并且你需要测试和修改reverseCountdown）像

这样的东西

for(reverseCountdown = stringLength-1; reverseCountdown>-1; reverseCountdown--) {
   reverse=reverse+forward.charAt(reverseCountdown);
}

我个人更喜欢StringBuilder（因为它有reverse方法），比如

System.out.println("enter a string and I will tell you if "
        + "it's a palindrome");
String forward = in.next();
StringBuilder sb = new StringBuilder(forward);
sb.reverse();
if (sb.toString().equals(forward)) {
    System.out.println("Bro you got a palindrome!");
} else {
    System.out.println("Thats not a palindrome...");
}

Answer 3

for循环应该是这样的

    for (reverseCountdown = stringLength-1; reverseCountdown >=0; reverseCountdown--){ //have changed the loop variables here
                reverse  += forward.charAt(reverseCountdown);
    }

for循环结束时的;导致for循环在空语句上运行（仅;被视为空语句）。删除它并改变你的for循环，

你的for循环的问题是，你应该在任何地方都使用变量'reverseCountdown'，但是你正在做StringLength--并不是很好。

使用StringBuilder也是一种方式，它具有内置函数来反转字符串，但我不确定给你作业的人是否会很高兴看到你使用内置函数。

希望这有帮助！祝你好运！

Answer 4

我认为更好的方法是在java中使用StringBuilder或使用array进行自己的实现，如下所示。

import java.util.Scanner;


public class PalindromeCheck {

private static Scanner in; 

public static void main(String[] args) {

    in = new Scanner(System.in);
    System.out.println("enter a string and I will tell you if its a palindrome");
    char[] arr = in.next().toCharArray();
    boolean bool = true;
    for(int i=0;i<Math.floor(arr.length/2);i++){
        if(arr[i] != arr[arr.length-1-i]){
            System.out.println("Not a palindrome");
            bool = false;
            break;
        }
    }
    if(bool){
        System.out.println("You got a palindrome");
    }
}
}

这样效率更高，因为它只循环了一半时间。

Answer 5

检查回文代码

public class PalindromeCheck {

    public static boolean check(String str) {
        boolean flag = true;
        char[] chs = str.toCharArray();
        int len = chs.length;
        for(int i = 0; i < len/2; i++) {
            if(chs[i] != chs[len - i -1]) {
                flag = false;
                break;
            }
        }
        return flag;
    }

    public static void main(String[] args) {
        String str0 = "123321";
        String str1 = "1234321";
        String str2 = "123421";
        System.out.println(check(str0));
        System.out.println(check(str1));
        System.out.println(check(str2));
    }
}

JAVA：错误地反转字符串以逐个检查palaindrome我认为

5 个答案: