标签：system strong nop 数字 htm scanner length import 编写
1、测试帖链接：http://www.cnblogs.com/hsy123/p/6602191.html
2、测试人员提出的问题、发现的缺陷：
测试结果都通过，并没有什么大问题。
对代码优化提出的建议：
（1）建议规范代码，代码全都挤到一块 ，看起来既不美观，又影响代码的可读性。
（2）可以添加循环代码，在程序结束的时候，询问是否继续。
（3）应对scan进行关闭，避免占用内存。
3、修正后的代码：
import java.util.Scanner;
public class homeWork01 {
public static String triangle(String s1,String s2,String s3){
String str="abcdefghijklmnopqrstuvwxyz!@#$%^&*()_+-=";
if(str.indexOf(s1)!=-1||str.indexOf(s2)!=-1||str.indexOf(s3)!=-1){
return "输入的是非法字符";
}
//判断输入的值是否是整数
if(isNumeric(s1)&&isNumeric(s2)&&isNumeric(s3)){
}else{
return"输入的数字不是整数";
}
int a = 0,b=0,c=0;
a=Integer.parseInt(s1);
b=Integer.parseInt(s2);
c=Integer.parseInt(s3);
if(a>=1&&a<=100&&b>=1&&b<=100&&c>=1&&c<=100){
if((a<b+c)&&(b<a+c)&&(c<a+b)){
if(a==b&&b==c){
return "等边三角形";
}
if(a==b||a==c||b==c){
return "等腰三角形";
}
if(a*a==b*b+c*c||b*b==a*a+c*c||c*c==a*a+b*b){
return"直角三角形";
}
return "一般三角形";
}else{
return"不构成三角形";
}
}else{
return "边的值不在范围内";
}
}
public static boolean isNumeric(String str1){
for (int i = str1.length();--i>=0;){
if (!Character.isDigit(str1.charAt(i))){
return false;
}
}
return true;
}
public static void main(String []args){
System.out.println("请输入三角形的三条边：");
String s1=null;
String s2=null;
String s3=null;
Scanner scan=new Scanner(System.in);
s1=scan.next();
//如果输入的边长为0，则退出程序
if(s1==0){
break;
}
s2=scan.next();
s3=scan.next();
System.out.println(triangle(s1,s2,s3));
Scan.close();
}
}
4、心得体会：
代码写的太过生硬，增加了许多注释，让人读起来更容易理解。编写的该程序没有找到比较大的毛病。个人增添了一些输入字符的判定，如输出非法字符或者小数都会提示出相应的错误。在修改的过程中，比较难以修改的就是判定数字是否为整数，上网找资料话了挺多时间，比对判定整数的方法的实用性。
标签：system strong nop 数字 htm scanner length import 编写
原文地址：http://www.cnblogs.com/molaoshi/p/6628617.html
