Blank lines in file throwing exception

[Tony Fabeets]

Happy Holidays! The problem is the program works just fine until it hits a blank line. Then the jvm takes up 99% of my resources and the my gui locks on my screen. Sry for the long code, but figure it'll be easier to diagnose.
tried if(inputLine.equals("")continue; and if(inputLine.equals("")inputLine = reader.readLine(); no avail

while(null != (inputLine = reader.readLine()))
{
tokenizer = new StringTokenizer(inputLine," ");
if(!(inputLine.equals("")))
{
token = tokenizer.nextToken();
if(token.equalsIgnoreCase("A"))
{
addClass addObj = new addClass();
//more robust = while(.hasMoreToken()){}
addObj.itmNum = Integer.parseInt(tokenizer.nextToken());
addObj.category = tokenizer.nextToken().charAt(0);
addObj.unitPrice = Double.parseDouble(tokenizer.nextToken());
addObj.qtyOnHand = Integer.parseInt(tokenizer.nextToken());
addObj.qtyOnBackOrder = Integer.parseInt(tokenizer.nextToken());
addObj.salesPersonNum = Integer.parseInt(tokenizer.nextToken());
addObj.regionNum = Integer.parseInt(tokenizer.nextToken());
//total updater
OHValue = (addObj.qtyOnHand * addObj.unitPrice) + OHValue;
OBOValue = (addObj.qtyOnBackOrder * addObj.unitPrice) + OBOValue;
addAL.add(addObj);
}//end if
else if(token.equalsIgnoreCase("S"))
{
shipClass shipObj = new shipClass();
//more robust while(.hasMoreToken()){}
shipObj.itmNum = Integer.parseInt(tokenizer.nextToken());
shipObj.customerID = Integer.parseInt(tokenizer.nextToken());
shipObj.qtyToShip = Integer.parseInt(tokenizer.nextToken());
shipObj.priority = Integer.parseInt(tokenizer.nextToken());
//total updater
QTYShipped += shipObj.qtyToShip;
shipAL.add(shipObj);
}//end if
else main.textArea.append("check file formating on line " + currentLine + "\n");
}//end if
currentLine++;
}//end while

[ December 28, 2003: Message edited by: Tony Fabeets ]

[Joe Ess]

 

Originally posted by Tony Fabeets:

while(inputLine != null)
{
if (inputLine.equals("")) continue;
}//end while

When you hit a blank line, you get an endless loop.

[Stan James]

Good catch, Joe! This is common practice:

while( null != ( line = someReader.readLine() ) )
{
do some stuff with a line
}

This takes advantage of the fact that in Java an assignment clause (line = something) is a function. That is, it not only assigns a value to a variable, but returns that value, too. So in one swell foop we can read a line, check it for null and terminate the loop. With this in place, your "continue" clause will skip the rest of the loop and read another line.

[Tony Fabeets]

It's still blowing up on the empty line, no matter what i try... help please?

.equals("\r") fixed the prob
[ December 24, 2003: Message edited by: Tony Fabeets ]
[ December 24, 2003: Message edited by: Tony Fabeets ]
[ December 24, 2003: Message edited by: Tony Fabeets ]