我正在编写一个将八进制数转换为小数的程序。它编译正确和一切,但我的转换代码有一些重大错误。这对我来说似乎是完全合乎逻辑的,但不管怎样,当我运行程序时,转换是错误的(即1转换为36),有人可以指出出错的地方吗?
public static int convert(int octal)
{
int d1=0,d2=0,d3=0,d4=0,d5=0,d6=0,d7=0,d8=0;
if(octal >=9999999){
d8 = (octal-(octal%10000000));}
if(octal >=999999){
d7 = (octal-(octal%1000000));}
if(octal >=99999){
d6 = (octal-(octal%100000));}
if(octal >=9999){
d5 = (octal-(octal%10000));}
if(octal >= 999){
d4 = (octal-(octal%1000));}
if(octal >= 99){
d3 = (octal-(octal%100));}
if(octal >= 9){
d2 = (octal-(octal%10));}
if(octal >= 0){
d1 = (octal-(octal%1));}
octal = (d8 * 8^7) + (d7 * 8^6) + (d6 * 8^5) + (d5 * 8^4) + (d4 * 8^3) + (d3 * 8^2) + (d2 * 8^1) + (d1 * 8^0);
return octal;
}
这只是我的转换方法,我的主要方法是收集的 int octal;
答案 0 :(得分:7)
这是问题所在:
8^7
^
operator没有按照您的想法行事。它做二进制XOR ......
然而,我会说整个设计显然是可疑的。 int
值不是“in”八进制或任何其他基数 - 它只是一个整数。数字10是数字10,无论是十进制的“10”,十六进制的“A”还是八进制的“12”。如果您有一个要解析为八进制数字的字符序列,则该方法的输入应为String
,而不是int
。
答案 1 :(得分:4)
由于您的方法接受一个整数(通常以字符串表示形式表示为十进制),并且您输出的内容也是一个整数,您实际上并没有变成“八进制整数”,您已将其更改为完全是其他整数。您正在尝试将整数正确转换为八进制,然后错误地将该八进制解释为十进制。
如果希望将整数转换为八进制字符串表示形式,则只需使用以下方法:
public static String convert(int number) {
Integer.toOctalString(number);
}
并且,如果你真的想要返回一个表示该八进制字符串的int,就像它是十进制一样,你可以这样做:
public static int convert(int number) {
return Integer.parseInt(Integer.toOctalString(number));
}
答案 2 :(得分:1)
如果您有重复的代码,可以考虑循环。
public static void main(String... args) {
for (long i = 7, j = 7; i > 0; i = i * 10 + 1, j = j * 8 + 1) {
long c = convert(i);
if (c != j)
throw new AssertionError(i + ": " + c + " != " + j);
System.out.println(i + " = > " + j);
}
}
public static long convert(long octal) {
long ret = 0;
for (long i = 1000000000000000000L, j = 1L << (3 * 18); i > 0; i /= 10, j >>= 3)
ret += j * (octal / i % 10);
return ret;
}
打印
7 = > 7
71 = > 57
711 = > 457
7111 = > 3657
71111 = > 29257
711111 = > 234057
7111111 = > 1872457
71111111 = > 14979657
711111111 = > 119837257
7111111111 = > 958698057
71111111111 = > 7669584457
711111111111 = > 61356675657
7111111111111 = > 490853405257
71111111111111 = > 3926827242057
711111111111111 = > 31414617936457
7111111111111111 = > 251316943491657
71111111111111111 = > 2010535547933257
711111111111111111 = > 16084284383466057
7111111111111111111 = > 128674275067728457
答案 3 :(得分:1)
你有一个潜在的误解,就是一直在弄乱你。
本身没有“八进制数”这样的东西。 int
不是十进制数,罗马数字或八进制数 - 它只是一个数字。
“八进制数”实际上是指“包含数字的八进制表示的字符串”。