2016-10-08 4 views
0

これでforループ内とその上/下にprintlnチェックを埋め込みましたが、ループ自体に問題は見つかりませんでした。なぜforループが実行されないのか分からない

機能は単純な回文チェッカーの一部です。ループのために、物事を解決しようとする際に、私は実行していないループを確認したループのために内側に移動した上で、「それはある」と私はもともとZが設定されていた

public static String checkPalindrome(String x) { 
    x=x.toLowerCase(); 
    String trimmedX = x.replaceAll("\\W",""); 
    String reverseX = new StringBuilder(trimmedX).reverse().toString(); 
    int length= trimmedX.length(); 
    for (int i=0;i>length;i++){ 
     if (trimmedX.charAt(i)!=reverseX.charAt(i)) { 
      String z = "It is not a palindrome."; 
      return z; 
     } 
     String z = "It is a palindrome."; 
    } 
    return z; 
} 

+0

長さ= 0を確認して>交換することがあります。 – zgc7009

+0

このコードは実際にはコンパイルされません。だからあなたが走っているものは、少し違うものでなければなりません。 –

答えて

2

おそらくあなたは、より密接にループを検査する必要があります

for (int i=0;i>length;i++){ 

私はiのチェックで問題があると思います。

+0

これは唯一の問題ですが、ループチェックは間違っています。長さが負でない限りループの本体を実行することはありません。この場合、長時間実行されます。 :) – sfjac

1

は、あなたは彼らがあることを期待値であることを確認する<

for (int i=0;i<length;i++){ 
関連する問題