第二次作业+105032014037

1、测试帖链接：http://www.cnblogs.com/ypzeng/p/6600986.html

2、测试人员提出的问题、发现的缺陷

测试人员提出的问题为：对不符合数据的要求的数据应进行处理，不能任由程序出错而放任不过，应以更为优化的界面给予视觉上的舒适感，还有对与一下代码

Scanner scana=new Scanner(System.in);

int a = scana.nextInt();

Scanner scanb=new Scanner(System.in);

int b = scanb.nextInt();

Scanner scanc=new Scanner(System.in);

int c = scanc.nextInt();

应对 scana scanb scanc 进行关闭，对所占用的资源进行关闭。

解答：

（1）已对不符合数据进行处理，将输入定义为String类型，再进行转换为int型是否判断正确，正确方进行下一步。应该把这个小程序当成一个完整的应用来看待，运行程序时不在非人为的情况下发生崩溃。

（2）界面舒适感在改过代码后应该会有所提高，谢谢提醒。

（3）在新代码中scana、scanb、scanc不做关闭处理，因为这三个输入放在do···while（）里，如果关闭了，因为没有键盘输入，程序将不断的进行case：“1”。当程序进行人为的退出后，show（）将停止执行，系统会退出，所以不进行处理。出于eclipse提醒，添加 @SuppressWarnings("resource")在show（）前，让它阻止这个提醒。鉴于学艺不精，可能这个回答不是很令人满意，请见谅。

 

3、修正后的代码清单

package TSort;
public class Client {
    public static void main(String args[]) throws Exception{
        new Menu();
    }
}

package TSort;
import java.util.Scanner;
public class Menu {
    private static Scanner in = new Scanner(System.in);
    public Menu() throws Exception
    {
        while(true){
            this.show();
        }
    }
    @SuppressWarnings("resource")
    public void show() throws Exception{
        
        System.out.println("======三角形种类=====");
        System.out.println("[1] 进行三角形类型判断");
        System.out.println("[e] 退出。");
        while (in.hasNext()) {
        String i = in.nextLine();
        switch(i){
            case "1":{
                int i_a,i_b,i_c;
                System.out.println("请输入三角形的三条边：");        
                do {
                    Scanner scana=new Scanner(System.in);             
                    String a=scana.next();
                    Scanner scanb=new Scanner(System.in); 
                    String b=scanb.next();                
                    Scanner scanc=new Scanner(System.in);
                    String c=scanc.next();
                    if(IsInteger.IsNum(a)&&IsInteger.IsNum(b)&&IsInteger.IsNum(c))
                    {
                         i_a = Integer.parseInt(a);
                         i_b = Integer.parseInt(b);
                         i_c = Integer.parseInt(c);
                         System.out.println(Sort.triangle(i_a, i_b, i_c));
                         break;
                    }
                    else{
                        System.out.println("输入不是整数！请重请输入三角形的三条边：");
                    }
                } while (true);    
                System.out.println("==========请继续选择===========");
                System.out.println("[1] 进行三角形类型判断");
                System.out.println("[e] 退出。");    
                break ;
        }    
            case "e":{
                System.exit(1) ;        // 系统退出
                break ;
            }
            default:{
                System.out.println("请选择正确的操作！") ;
            }
        }
    }
    }
}

package TSort;
public class IsInteger {
    public static boolean IsNum(String str)
     {
      if(str==null)
       {
         return false;
        }
      else{
           try{
            @SuppressWarnings("unused")
            int num=Integer.valueOf(str);//把字符串强制转换为数字 
            return true;//是数字,返回true
            }catch (Exception e) {
             return false;
            }
        }
    }
}

package TSort;
public class Sort{
    static String t1="边的值不在范围内";
    static String t2="不构成三角形";
    static String t3="等边三角形";
    static String t4="等腰三角形";
    static String t5="直角三角形";
    static String t6="一般三角形";
    public static String triangle(int a,int b,int c){    
        if(a<1 || a>100 ||b<1 || b>100|| c<1|| c>100)
        {
            return t1;
        }
        else if(a>=b+c ||b>=a+c||c>=a+b)
        {
            return t2;
        }
        else if(a==b &&b==c &&c==a){
            return t3;
        }
        else if(a==b ||b==c ||c==a){
            return t4;
        }
        else if(a*a+b*b==c*c ||b*b+c*c==a*a ||c*c+a*a==b*b){
            return t5;
        }
        else {
            return t6;
        }
    }
}

4、修正后心得体会

（1）做一个功能简单的程序，也要当成一个完整的应用，要考虑到程序在非人为情况下不出现闪退崩溃等情况；

（2）对于参数的声明定义要明确这个类型的范围（比如int最大最小是多少，如果超过又怎么处理）；

（3）从Scanner中获取数据，要习惯进行处理，得到期望输入的类型再进行下一步功能实现；

（4）if（）条件处理可以进行封装成一个方法，这样子复杂的条件判断能够单独进行，方便检查处理。

（5）对于编程开发人员可能只是为了完成这个功能，但是对于测试人员来说，功能只是程序正确的一个体现，所以如果不想让测试人员检查出更过错误，就要养成严谨的代码习惯，正确的处理数据，代码整洁，功能完善。