代码无错便是优？

   首先给一段简单的计算器的代码，基本上所有的菜鸟都做过的

package com.kiritor;

import java.io.BufferedReader;
import java.io.Console;
import java.io.IOException;
import java.io.InputStream;
import java.io.InputStreamReader;
import java.util.Scanner;

/**
* 一个简单的+，-，*，/运算器
* 通过控制台进行读入和计算
* 基于正整数的
* */
public class Calculator {
public static void main(String[] args) throws NumberFormatException, IOException {
System.out.print("请输入第一个数字：");
BufferedReader reader = new BufferedReader(new InputStreamReader(System.in));
double a = Double.parseDouble(reader.readLine());
System.out.print("\n请输入第二个数字：");
double b = Double.parseDouble(reader.readLine());
System.out.print("\n请输入运算符:");
String opr = reader.readLine();
double result=0 ;
if(opr.equals("+"))
{
result = a+b;
}
if(opr.equals("-"))
{
result =a-b;
}
if(opr.equals("*"))
{
result = a*b;
}
if(opr.equals("/"))
{
result = a/b;
}
System.out.println("结果为："+result);
}
}

    上述代码若按照正确的输入是可以得出正确的结果的，

        不过真的没有问题了吗？

   代码毛病：

          1、变量命名规范

          2、分支判断的地方极不合理，计算机进行了3次无用功的判断！

          3、除法运算被除数为零的情况并未考虑！

          4、用户输入的不确定性，非法输入的处理没有，造成程序的稳定性差！

  代码进一步规范：

  

package com.kiritor;

import java.io.BufferedReader;
import java.io.Console;
import java.io.IOException;
import java.io.InputStream;
import java.io.InputStreamReader;
import java.util.Scanner;

/**
* 一个简单的+，-，*，/运算器 通过控制台进行读入和计算 基于正整数的
* */
public class Calculator {
public static void main(String[] args) {
try {
System.out.print("请输入第一个数字：");
BufferedReader reader = new BufferedReader(new InputStreamReader(
System.in));
double num1 = Double.parseDouble(reader.readLine());
System.out.print("\n请输入第二个数字：");
double num2 = Double.parseDouble(reader.readLine());
System.out.print("\n请输入运算符:");
String opr = reader.readLine();
double result = 0;
switch (opr) {
case "+":
result = num1 + num2;
System.out.println("结果为：" + result);
break;
case "-":
result = num1 - num2;
System.out.println("结果为：" + result);
break;
case "*":
result = num1 * num2;
System.out.println("结果为：" + result);
break;
case "/":
if (num2 == 0) {
System.out.println("除数不能为零！");
} else {
result = num1 / num2;
System.out.println("结果为：" + result);
}
break;
default:
break;
}

} catch (Exception e) {
// TODO: handle exception
}

}
}