私は文字列パラメータを受け取り、次のように文字の大文字と小文字を変更した結果の文字列を返すコード書かれている

Question

1行に文字を配置します：

1. すべての母音は母音がある（大文字でなければなりませんが、E、I、O、U）
2. すべての子音は
3. 小文字で文字でない任意の文字は、ここで

を変更してはならないしなければならない私のコードです：

public class Simple { 

    public char ChangeCase() { 
     Scanner scanner = new Scanner(System.in); 
     System.out.println("Enter an input String: "); 
     String inputString = scanner.nextLine(); 
     //String isVowel = "aeiou"; 
     char c='\0'; 
     for (int i = 0; i < inputString.length(); i++) { 
      c = inputString.charAt(i); 
      if (c == 'a' || c == 'e' || c == 'i' || c == 'o' || c == 'u'||c=='A'||c=='E'||c=='I'||c=='O'||c=='U') { 
       c = Character.toLowerCase(c); 

       System.out.println(c); 
      } 
      else if (c=='b'||c=='c'||c=='d'||c=='f'||c=='g'||c=='h'||c=='j'||c=='k'||c=='l'||c=='m'||c=='n'||c=='p'||c=='q'||c=='r'||c=='s'||c=='t'||c=='v'||c=='w'||c=='x'||c=='y'||c=='z'|| 
       c=='B'||c=='C'||c=='D'||c=='F'||c=='G'||c=='H'||c=='J'||c=='K'||c=='L'||c=='M'||c=='N'||c=='P'||c=='Q'||c=='R'||c=='S'||c=='T'||c=='V'||c=='W'||c=='X'||c=='Y'||c=='Z'){ 
       c = inputString.charAt(i); 
       c =Character.toUpperCase(c); 
       System.out.println(c); 
      } 
      else if(c=='@'||c=='!'||c=='"'||c==' '||c=='!'||c=='"'||c=='#'||c=='$'||c=='%'||c=='&'||c=='('|| c==')'||c=='*'||c=='+'||c==','||c=='-'||c=='.'||c=='/'||c==':'||c==';'||c=='<'||c=='='||c=='>'||c=='?'||c=='['||c==']'||c=='^'||c=='_'||c=='`'||c=='{'||c=='|'||c=='}'||c=='~'||c=='"'){ 
       c=inputString.charAt(i); 
       c=c; 
       System.out.println(c); 
      } 
      else 
       c=c; 

     } 

     return c; 

    } 
} 

ランナークラス：私はハロー入力したときに私が取得

public class Runner { 
    public static void main(String[] args) { 
     Simple smpl=new Simple(); 
     smpl.ChangeCase(); 

    } 

} 

結果：

Enter an input String: hello 
H 
e 
L 
L 
o 

期待される結果：

私は期待どおりの結果を得るために変更する必要がありますどのような

HeLLo 

？

Answer 1

printlnからprintに変更すると、printlnは常に余分な行を追加するためです。

Answer 2

変更println()〜出力から改行文字を省略する場合。 toUpperCaseとtoLowerCaseを交換して、仕様に必要なコードを作成します。

現在、改行などの他の文字も印刷していません（「文字以外の文字は変更できません」という要件）。 二else ifを取り除くとelse支店と

else { 
    System.out.print(c); 
} 

また、余分なc = inputString.charAt(i);文を取り除くと交換しなさい。最初のものだけが必要です。

あなたが解決している問題に関して、私があなただったら私は正規表現を使用します。

とにかく、ここで同様に適用される他のいくつかの改良を加えた正規表現無しSimple.javaためのソリューションである：

import java.util.Scanner; 

public class Simple { 

    public void ChangeCase() { 
     Scanner scanner = new Scanner(System.in); 
     System.out.println("Enter an input String: "); 
     String inputString = scanner.nextLine(); 
     String vowels = "aeiouAEIOU"; 
     for (char c: inputString.toCharArray()) { 
      if (vowels.indexOf(c) != -1) { // found vowel 
       c = Character.toUpperCase(c); 
      } else if (Character.isLetter(c)) { // must be a consonant 
       c = Character.toLowerCase(c); 
      } 
      System.out.print(c); 
     } 
    } 
} 

Answer 3

最初のスターターとしては、 - 単にそれぞれ

c = Character.toLowerCase(c); 

他の呼交換しますあなたのifブロック内にtoUpperCase()があります。

コードは基本的には正しいものです。間違った場所でケースを変更するだけです。だから、単にこれらの呼び出しを "交換"してください！もちろん、println()は使用しないでください。これは "行"を印刷して、最後に "新しい行"を追加します。

Answer 4

println（）は常に新しい行を追加するため、println（）の代わりにprint（）メソッドを使用してください。

Answer 5

より良いコードのために最初にいくつかのヒント：

1. Structorize酸味コードので、各ステップ（IF）のための方法を使用してください。
2. 1文字の変数のみが反復変数である必要がありますが、変数が1文字のみの変数でない方が良いです。 （C）ここで

あなたのためにいくつかの簡単な例（最適化することができます）：

public class Simple { 
    public void ChangeCase() { 
     String resultString; 
     Scanner scanner = new Scanner(System.in); 
     System.out.println("Enter an input String: "); 
     String inputString = scanner.nextLine(); 

     // make alle chars lower-case 
     resultString = inputString.toLowerCase(); 

     // make vowels upper-case 
     resultString = uppercaseVowels(resultString); 

     // print the result or change to return String for main() 
     System.out.println("Input: " + inputString); 
     System.out.println("Result: " + resultString); 
    } 

    private String uppercaseVowels(String inputString) { 
     inputString = inputString.replace('a', 'A'); 
     inputString = inputString.replace('e', 'E'); 
     inputString = inputString.replace('i', 'I'); 
     inputString = inputString.replace('o', 'O'); 
     inputString = inputString.replace('u', 'U'); 
     return inputString; 
    } 
}