実行結果があいません。どこを直したらよいか教えてください。

Question

9進数のものを3進数にするプログラムです。これまでいただいたアドバイスをもとに自分のレベルでできるくらいのものをつくってみましたが実行結果があいません。
10進数のところで表示したところまずそこが違うので9進数から10進数にするところが間違っているのだと思います。どこを直したらよいか教えてください。

public class kadai3_1{
	public static void main(String[] args)throws IOException{
	
		BufferedReader br=new BufferedReader(new InputStreamReader(System.in));
		System.out.print("9進数"+" ");
		String sinsu=br.readLine();
		char a;  int b=0; int c=0; String d=" "; int k=0;
		int sinsu1[]=new int[sinsu.length()]; int s=0;
		int sinsu2[]=new int [1000];
		
		for(int i=0;i<sinsu.length();i++){
			sinsu1[i]=sinsu.charAt(i);
			if(sinsu1[i]==9){
				System.out.println("9進数ではない");
				break;
			}
		}
	
		for(int j=sinsu.length()-1;j>=0;j--){
			double kaijyou=Math.pow(9,k);

			b+=sinsu1[j]*kaijyou;
			k++;
		}
			System.out.println(b);
		
		
		while(b>=3){
			c=b%3;
			b=b/3;
			sinsu2[s]=c;
			s++;
		}
		sinsu2[s]=b;
		for(int y=s;y>=0;y--){
			System.out.print(sinsu2[y]);
		}
	
	}
}
			
コード

Answer 1

この質問に対しては禁じ手かもしれないが

int i = Integer.parseInt(sinsu, 9);
String result = Integer.toString(i, 3);

Comments

[maisumakun]

「自分で作らずに既存のライブラリ（特に標準ライブラリで存在すればなお良い）を使い倒せる」というのも、立派な能力の1つだとは思います。

# 自分自身、必死で書いたコードが使用中のライブラリに存在したことにあとから気づいて微妙な気分になったこともあります

[abs123]

文字列から文字列の変換であればこっちのほうがよさそう
BigInteger(String val, int radix) -> BigInteger#toString(int radix)

[退会済みユーザー]

そんな方法もあったんですね！
ありがとうございます！

Answer 2

一般的に使える方法ではありませんが、9は3の2乗なので、いちいち10進法に変換しなくても、9進法の1桁を3進法の2桁に変換すれば、だいぶ楽に処理が進むのではないかと思います。

Comments

[退会済みユーザー]

やってみます！
ありがとうございます！

Answer 3

以下の部分、これだと入力された「文字列」を1桁ずつ分解してsinsu1[]に入れることになりますが、ここで入るのは「文字コード」であり数値ではありません。「1」と入力したなら、「1」を表す文字コードである49（0x31）が入ることでしょう。これをその先の計算で用いているから合わないのでしょう。

        for(int i=0;i<sinsu.length();i++){
            sinsu1[i]=sinsu.charAt(i);
            if(sinsu1[i]==9){
                System.out.println("9進数ではない");
                break;
            }
        }

次のように改めてみては？

            sinsu1[i]=Character.digit(sinsu.charAt(i), 10);

Comments

[退会済みユーザー]

わかりました！
ありがとうございます！

Answer 4

おはようございます。いろいろ訂正させてください。
1.

sinsu1[i]=sinsu.charAt(i);

左辺はint型に対して右辺はchar型になっているため正しい値が代入されてないと考えられます。
char型をint型に変換するときは

sinsu1[i] = Character.getNumericValue(sinsu.charAt(i));

という方法があります。

2.入力した値の位に少なくとも１つ9がある場合は9進数ではないと表示していますが
9進数でない場合も3進数に変換する処理を行っているのはおかしいですよね?
if文を用いて９進数ではないときは3進数に変換する処理をしないようにすべきだと思います。
(以前の質問のところで私が回答したものを参考にしてください。)

<追記>
もっと簡単な方法がありました。

import java.util.Scanner;

public class Main {
    public static void main(String[] args) {
        Scanner sc = new Scanner(System.in);
        String line = sc.nextLine();
        int i = Integer.parseInt(line, 9); //9進数に変換
        String result = Integer.toString(i, 3); //3進数に変換
        
        System.out.println(result);
    }
}

こっちのほうがすっきりしてますね。
参考にしたサイト->Javaで進数変換を行う方法

Comments

[退会済みユーザー]

わかりました！
ありがとうございます！

Answer 5

次のように、デバッグのための print 文を入れて実行してみました。(22 行目, 23 行目)

charAt(i) と sinnsu1[i] が異なっています。
他の回答にあるように、
charAt(0) は 文字の '1'
sonsu1[0] 文字 '1' のアスキーコード値 49 になっています。
sousu1[0] に 1 をいれたいのですから
sinsu1[i] = sinsu.charAt(i) - '0'
にして、実行してみます。

eclipse などのプログラミング環境を使えば。print 文を埋め込まなくても、
好きな場所で実行を一時停止し、そのときの変数の値を調べたりできます。

Comments

[退会済みユーザー]

わかりました！
ありがとうございます。