reading an inventory.txt file and placing the price and item into an array

[yolkman]

The text file is below:

21.95 Water Filter Pack
.51 Pork and Beans
1.79 Chicken Salad Sandwich
5.99 Coke 24-pack
3.79 Barbara's Oat Bran Cereal
2.19 Sandwich Bags
1.07 Potato Chips

Here is part of my code to read the text file.

    try{
           FileReader fr=new FileReader(filename);
           BufferedReader br=new BufferedReader(fr);
 
           String line;
           int index=0;
 
                   while((line=br.readLine())!=null){
                        StringTokenizer tokenizer=new StringTokenizer(line,"  ");
 
                          while(tokenizer.hasMoreTokens()){
 
                              String item=tokenizer.nextToken();
                              Double price=Double.parseDouble(item);
 
 
 
                           System.out.println(price);
 
               } 
 
 
           }
 
         br.close();
            }    

I used a printline statement in order to determine whether or not all of the prices will be read properly. However, after running this code, it only prints the first number which is 21.95 and it doesn't loop through the rest.

[evilchia]

The text file is below:

21.95 Water Filter Pack
.51 Pork and Beans
1.79 Chicken Salad Sandwich
5.99 Coke 24-pack
3.79 Barbara's Oat Bran Cereal
2.19 Sandwich Bags
1.07 Potato Chips

Here is part of my code to read the text file.

First of all your code is really compuationally expensive. The BufferedReader is fine, but StringTokenizer is a pig. Try something of the following using the RegExp packages from jakarta

RECompiler rec = new RECompiler();
REProgram rep = rec.compile("^([\\d\\.]+)\\s+(.*)");
RE myre = new RE(rep);
 
BufferedReader br = new BufferedReader(new FileReader(new File(<path>)));
String wkline = null;
while((wkline = br.readLine()) != null) {
  if(myre.match(wkline) {
    try {
      double wkprice = Double.parseDouble(myre.getParen(1));
      String item = myre.getParen(2);
      <work to insert in array here>
    } catch (NumberFormatException nfe) {
      nfe.printStackTrace(System.err);
    }
  }
}
br.close();
 

This should address both the performance and the issue you were dealing with. You have to remember that the StringTokenizer you're using is looking for spaces and will tokenize the entire line wherever there is a space.

[r035198x]

your inner while loop should be throwing a number format exception when it tries to access the second token. It should be something like this

boolean done = false;
while(tokenizer.hasMoreTokens() && done == false) {
     int price = Integer.parseInt(tokenizer.nextToken());//get only the first word
     done == true;
}

java also has
java.util.regex package which is probably what you would want to use here for the reason already given.