java 演習の質問

Question

import java.io.IOException; // 例外処理用
import java.io.BufferedReader; // バッファ処理用
import java.io.InputStreamReader; // 文字列入力用

/**
*

• @author *

public class Horoscope {

 public static void main( String[] args ) throws   IOException {

     final char[] vowel = { ' ', 'A', 'I', 'U', 'E', 'O' }; // 母音チェック用配列
     int point = 0; // 相性度

      // 自分の氏名を入力する
     System.out.print( "自分の氏名（半角ローマ 字）：");
     BufferedReader br = new BufferedReader( new InputStreamReader( System.in ) );
     String myName = br.readLine();

     // 相手の氏名を入力する
     System.out.print( "相手の氏名（半角ローマ字）：");
     String yourName = br.readLine();
     System.out.println();


     String pairName = myName + yourName; // 二人の氏名を１つの配列にまとめる

//Start--------------------
System.out.println(pairName.toUpperCase
()); //二人のローマ字氏名を連結し、大文字に変換して表示する。
System.out.println(pairName);
int nums[]=new int[pairName.length()];
int index;
index=0;
int i;
int j;
int k;
for(i=0;i<pairName.length();i++){
for(j=1;j<vowel.length;j++){
if(pairName.charAt(i)==vowel[j]){
if(j==vowel.length -1){
System.out.println("");
}
System.out.println(pairName.charAt (j));
nums[index]=j;
index=index+1;
}
}
}
System.out.println();
System.out.println();
for(i=0;i<index;i++){
System.out.println(nums[i]);
}
for(i=index-1;i>1;i--){
for(j=0;j<index-i;j++){
System.out.println();
}
for(k=1;k<i;k++){
nums[k]=(nums[k]+nums[k+1])%10;
System.out.println (nums[k]);
}
System.out.println();
}
System.out.println();
point=nums[0]*10+nums[1];

//End----------------------

     // 相性度を表示する
     System.out.println( "相性度 "+ point + "％");

 } // end of main()

} // end of Horoscope

Comments

[swordone]

質問編集画面で、コードの部分を反転選択し、<code>を押して、
「ここに言語を入力」の部分を「java」に書き換えて
編集を完了してください。

[swordone]

「コードの部分を反転選択し」が伝わらなかったでしょうか？

Answer 1

if(pairName.charAt(i)==vowel[j]){

ここの文字比較に問題があります

あなたはローマ字の大文字変換を行った値を
どこにも保持せずにこの比較を行っているため
indexの値が加算されていません

なので下のループ文が即終わります。

Comments

[退会済みユーザー]

どのように書き換えたらいいか教えて頂きたいです。

[ransuS_T]

System.out.println(pairName.toUpperCase());
あなたはここで大文字にする処理をしているので
この値を使って比較すればよいのではないでしょうか？

pairName = pairName.toUpperCase()
こういうふうに元の変数につっこむかして

[ransuS_T]

それからソースをのせるときはインデントをそろえて
余計な処理はなるべく省き要点となりそうな箇所にしぼると
早くたくさんの回答がもらえると思います。

[m.ts10806]

>インデントをそろえて

マークダウン使わないと行頭詰められますので仕方ないかと(マークダウン使ってももとのソースグチャグチャなコードも少なくないですが)