Help with loading

[waker3210]

I'm having trouble doing a load that i thought to be pretty simple. I've checked the code and i don't know what is going wrong.

here's the code:

public boolean[] Load(String file)
	{
		boolean[] fill = new boolean[288];
		int count = 0;
		
		try
		{
			Scanner loader = new Scanner(new File(file));
			while(loader.hasNext())
			{
				if(loader.nextLine().equals("true"))
				{
					fill[count] = true;
				}
				else if(loader.nextLine().equals("false"))
				{
					fill[count] = false;
				}
				
				count++;
			}
			loader.close();
		}catch(IOException e){}
	
		return fill;
	}

The problem i'm having is that for some reason even when the file i'm reading from has true in that line, it still saves the boolean as false sometimes, and vice-versa. For instance, my sample files first 6 lines are true, true, true, false, false, false but the array reads true, true, true, false, false, true. The last false is read as true! Does anyone know why this might be?

[DarrylBurke]

How are you running that code?

db

[waker3210]

What do you mean? I just pasted code for that one method. Um, i call it after the user presses a JButton if that's what you mean.If you let me know what you want to know more specifically ill be glad to help. I should add that i've done some more testing and found that the Scanner seems to find something else other than true or false in the line, but looking at the file that's impossible. The only things in that file are true or false and no double spacing or anything. I'm not really sure what that means.

[awinston]

How are you running that code?

What do you mean?

I think he's asking if you are using an IDE or just a text editor and command prompt.

[waker3210]

Oh, well i'm using an IDE, JCreator to be more exact.

[DarrylBurke]

Well, I've never seen JCreator but if it has a build process anything like NetBeans, you may be inadvertently accessing an old version of the file.

Is the String parameter an absolute path or relative?

If that's not it, post a SSCCE (Short, Self Contained, Correct (Compilable), Example) that demonstrates the problem, along with a sample text file.

db

[Tolls]

You're reading the line twice if nextLine() returns "false".

if(loader.nextLine().equals("true"))  // Reads a line from the Scanner.
{
    fill[count] = true;
}
else if(loader.nextLine().equals("false")) // If the above line is "false" then this will read another line...if this line is "true" then you completely skip the if/else
{
    fill[count] = false;
}

So store the String returned by nextLine() into a variable then test the variable. Don't read and test in one go.

[DarrylBurke]

You're reading the line twice if nextLine() returns "false".

D'oh. I should have spotted that!

db

[waker3210]

Oh, wow. Thanks a lot for the help. i really appreciate it. I'll go do that right away.

[waker3210]

Yep. That fixed it, thanks again.