[GDubz]

Getting error...
at java.util.Scanner.throwFor(Unknown Source)
at java.util.Scanner.next(Unknown Source)
at java.util.Scanner.nextDouble(Unknown Source)
at Lab4.loadFile(Lab4.java:53)
at Lab4.main(Lab4.java:25)

public static void loadFile(Scanner in, HashMap<String, Product> productMap) throws IOException
	{
		System.out.println("Enter database file name: ");
		Scanner inFile = new Scanner(new File(in.nextLine()));
		while (inFile.hasNextLine())
		{
			Product p = new Product();
			System.out.println("here");
			p.setName(inFile.nextLine().trim());
			p.setType(inFile.nextLine().trim());
			p.setPrice(inFile.nextDouble());
			p.setStock(inFile.nextInt());
			productMap.put(p.getName(), p);
			
		}
	}

[GregBrannon]

The error suggests the contents of your file do not match your data type being read from it. Compare the nextLine(), nextDouble(), etc. calls to the file's contents and make sure they match. If you still need help, it would help to post the contents of your file.

[GDubz]

I belive it matches, this is the file

The Shawshank Redemption
DVD
19.95
1
Dracula
Book
4.95
1
The Dark Knight
DVD
19.95
1
Lego Ultimate Building Set
Toy
29.95
3
The Girl With The Dragon Tattoo
Book
14.95
1
Iron Man
DVD
19.95
1
Under The Dome
Book
19.95
1

This post has been edited by GDubz: 18 February 2012 - 06:02 AM

[GregBrannon]

I think you need to parse the numeric input from a String object to the desired format. So your first number read (the Double where you're getting the error) would be:

p.setPrice( Double.parseDouble( inFile.readLine() ) );

[smohd]

The problem is that after reading one loop, the last reading method was nextInt() which do not read the line feed, so the next iteration nextLine() reads the line feed from previous iteration and so it mixed the flow.
To solve the problem, just put nextLine() at the end of the loop like:

p.setStock(inFile.nextInt());  inFile.nextLine(); //add nextLine()

This is the normal problem when you mix nextLine() with other next() methods, better use nextLine() for this case like what Greb showed above instead of nextDouble() and nextInt()

This post has been edited by smohd: 18 February 2012 - 06:47 AM

[GDubz]

that was it smohd, thank you