我想编写一个接收字符串值并打印十进制数的程序。
另外,如果字符串值不是1或0,我需要打印一条消息。
我编写了这段代码,但它总是进入if命令。
感谢您的支持!
谢谢
import java.util.Random;
public class Decimal {
public static void main(String[] args) {
String input = (args[0]);
int sum = 0;
for (int i = 0; i <= input.length(); i++) {
if (!(input.charAt(i) == '0') || (input.charAt(i) == '1')) {
System.out.println("wrong string");
break;
}
char a = input.charAt(i);
if (a == '1') {
sum |= 0x01;
}
sum <<= 1;
sum >>= 1;
System.out.println(sum);
}
}
}
答案 0 :(得分:1)
!
语句的if
(not)运算符仅适用于第一部分:
if ( ! (input.charAt(i) == '0')
||
(input.charAt(i) == '1')
) {
所以这与:
相同if ((input.charAt(i) != '0') || (input.charAt(i) == '1')) {
当你真正打算做的时候:
if (input.charAt(i) != '0' && input.charAt(i) != '1') {
但这是一件好事,因为一旦有效,你就会在IndexOutOfBoundsException
时获得i == input.length()
。将循环更改为:
for (int i = 0; i < input.length(); i++) {
为了提高性能,请将变量a
向上移动并在第一个if
语句中使用它。重命名为c
或ch
更具描述性/普遍性。
同时执行sum <<= 1
和sum >>= 1
会使您离开。那是你想要的吗?你也应该在设置最右边位之前进行左移。
应用所有这些,我相信你的意思是这样做:
String input = args[0];
int sum = 0;
for (int i = 0; i < input.length(); i++) {
char c = input.charAt(i);
if (c != '0' && c != '1') {
System.out.println("wrong string");
break;
}
sum <<= 1;
if (c == '1')
sum |= 1;
}
System.out.println(sum);