2013-04-29 63 views
0

我很难确保双打在我的程序中正确验证。用户可以输入一个金额存入账户,这应该是一个双倍(我知道,这不是我应该使用的,但它是分配指南的一部分)。理论上,用户应该可以存入任何金额 - 不只是30英镑,而是15.23英镑。 这是我目前的验证,它允许数字,但阻止进入一个句号,这会产生一些问题。(应该是)简单的双重验证

这里是我的代码至今:

public static String getBalanceValidation() 
{ 
    //Allow user input capabilities 
    Scanner input = new Scanner (System.in); 
    //Declare variables needed for validation 
    double dblInput = 0; //dblInput is set as 0 
    String strNumber = ""; //strNumber is blank 
    boolean bolSuccessful, bolNumeric; 
    int intCount; 
    char charLetter; 


    do 
    { 
     //set bolSuccessful and bolNumeric as true 
     bolSuccessful = true; 
     bolNumeric = true; 

     try //try user input 
      { 
       System.out.println("Enter the balance to be deposited: "); //User prompt 
       strNumber = input.next(); //User input as string 
       dblInput = Double.parseDouble(strNumber) ; //String input converted to double 


      }// end of try 

     catch (NumberFormatException e) //NumberFormatException disallows letters or symbols in value 
      { 
       System.out.println("Deposit value cannot contain letters!"); //Error message 
       bolSuccessful = false; //set bolSuccessful as false 

       continue; //Return to try 
      }//end of number format catch 


      //create for loop which checks each character throughout the string 
      for (intCount = 0; intCount < strNumber.length(); intCount++) 
       { 
        charLetter = strNumber.charAt(intCount); //charLetter is the alphanumeric value of a character in the string at the point dictated by intCount 


        if (!(charLetter >= '0') && (charLetter <= '9') //if charLetter is not between 0 and 9 
          || (charLetter == '.')) //or charLetter is not a full stop 
         { 
          bolNumeric = false; //Set bolNumeric as false 
         }//end of if construct 
       }//end of for loop 

      if (!bolNumeric) //if bolNumeric is false 
       { 
        System.out.println("Incorrect input format! The balance must be numbers only!"); //Error message 
        bolSuccessful = false; //Set bolSuccessful as false 
       }//end of if construct 

    }while (!bolSuccessful); //While bolSuccessful is false, return to top 


    return strNumber; //return strNumber to be used in main method 
    //end of do method 
}//end of getBalanceValidation method 

我不知道是否是因为我使用NumberFormatException的(有双别的东西吗?)

非常感谢

+1

定义'一些问题'。不确定是否因为...而发生了什么?不是一个真正的问题。 – EJP 2013-04-29 10:01:50

+0

我真的很讨厌那些告诉我一个变量被设置为'true'的意见:'// set bolSuccessful and bolNumeric as true' – Henrik 2013-04-29 10:04:22

+0

我也讨厌这些评论 - 但就像我说的,这是一个任务。我刚刚被告知要疯狂发表评论。 – 2013-04-29 10:05:20

回答

0

您的布尔表达式2个错误:

if (!(charLetter >= '0') && (charLetter <= '9') || (charLetter == '.')) 

这个条件等效于:

if ((charLetter < '0') && (charLetter <= '9') || (charLetter == '.')) 

哪个可被简化为:

if ((charLetter < '0') || (charLetter == '.')) 

所以!应适用于表达的第一两个部分:

if (!((charLetter >= '0') && (charLetter <= '9')) || (charLetter == '.')) 

此外,由于.不是一个数字,所以这个表达式是等价的吨至:

if (!((charLetter >= '0') && (charLetter <= '9'))) 

你可能是指&&||

if (!((charLetter >= '0') && (charLetter <= '9')) && (charLetter != '.')) 

这意味着if(not_a_number AND not_a_full-stop)

+0

工程处理 - 非常感谢。应该知道它不是或者在这种情况下,我主要是在验证名称。谢谢! – 2013-04-29 10:19:16

0

strNumber = input.next();你可以double number = input.nextDouble();代替。这将允许您直接输入number作为double而不是String

你将不得不在你的catch块中处理InputMismatchException,你很好。您将不需要验证以检查是否包含.

0

它会使用一个正则表达式容易得多:

bolNumeric = strNumber.matches("[1-9][0-9]*(\\.[0-9]{1,2})?"); 

说明:第一个数字必须是内1-9。然后尽可能多的(包括没有)其他数字可能会跟随。这可以选择跟一个点,然后至少一个,最多2个数字。