プロジェクトオイラー＃4：Java

Question

なぜコードが機能しないのかわかりません。基本的に私はプロジェクトオイラー＃4をコーディングしようとしています。これは2桁の3桁の数字を掛け合わせることで得られる最大のパリンドローム数です。このコードは動作していないようです（コンパイルされます）。コードを実行するたびに "0"が出力されます。私はそれがアルゴリズム的に効率的ではないかもしれないことは知っていますが、何が間違っていますか？

public class Palindrome { 
public static void main (String [] args) { 

    int largestnum = 0; 
    Palindrome pp = new Palindrome(); 

    for (int i = 100; i <= 999; i++) { 
     for (int k = 100; k <= 999; k++) { 
      if (pp.checkPal(i * k) == true) { 
       largestnum = i * k; 
      } 
     } 
    }   

    System.out.println(largestnum); 
} 


public boolean checkPal(int y) { 
    String snum = Integer.toString(y); 
    int snum_length = snum.length() - 1; 
    String palindrome = new String(""); 

    for (int i = snum_length; i >= 0; i--) { 
     palindrome.concat(Character.toString(snum.charAt(i))); 
    } 

    if (snum.equals(palindrome)) { 
     return true; 
    } 
    else { 
     return false; 
    }  
} 
} 

Answer 1

Javaの文字列は不変です。

あなたがやっている：

palindrome.concat(Character.toString(snum.charAt(i))); 

これはpalindromeとCharacter.toString(snum.charAt(i))の連結である新しい文字列を返す式です。 ではなく、を変更します。palindromeです。

あなたはこのようにそれを修正することができます：文字列を構築するためにStringBufferまたはStringBuilderを使用する方が良いでしょうが、

palindrome = palindrome.concat(Character.toString(snum.charAt(i))); 

。

Answer 2

checkPal関数内にprintステートメントがあるので、空白が表示されている可能性があります。あなたがpalindrome.concatを行う際

はまた、あなたはそうのような結果に回文を割り当てる必要があります： palindrome = palindrome.concat(Character.toString(snum.charAt(i)));