NullPointerException

[Brian LaRue]

after creating my class, at run-time, I'm getting a NullPointerException. I'm wondering why this is occuring. JVM doesn't like my array for some reason.
Here's my code so far:

import java.io.*;
 
public class Test {
    public static void main(String[] args) throws IOException {
            File inputFile = new File("customer.txt");
            FileReader in = new FileReader(inputFile);
            BufferedReader bfr = new BufferedReader(in);
 
            String line;
 
            while ((line = bfr.readLine()) != null); {
 
                String[] parts = new String[4];
                parts = line.split(",");
 
                String custID = parts[0];
                int age = Integer.parseInt(parts[1]);
                boolean a_status = Boolean.parseBoolean(parts[2]);
                boolean c_status = Boolean.parseBoolean(parts[3]);
                double gpa = Double.parseDouble(parts[4]);
 
                Customer c1 = new Customer(custID, age, a_status, c_status, gpa);
                c1.calculateSurcharge();
                c1.display();
 
            }  
 
            in.close();
    }
}
 

What am I doing wrong? Any help is greatly appreciated.

Brian

[Ernest Friedman-Hill]

Hi Brian,

The stack trace should tell you exactly what line of what file the NullPOinterException occurs on; can you tell show us exactly that line of code?

[Brian LaRue]

Exception in thread "main" java.lang.NullPointerException
        at Test.main(Test.java:14)

Thanks for offering your help...

[Christophe Verré]

remove the semicolon :
while ((line = bfr.readLine()) != null); {

Be careful 1) You should check the size of the array returned by the split method.
Be careful 2) You are declaring a array of 4 elements, while trying access 5

In fact, you don't even need to declare a new instance.
Just do :

String[] parts;
parts = line.split(",");

and check the size of the array with parts.length
[ February 16, 2006: Message edited by: Satou kurinosuke ]

[Brian LaRue]

Awe man, I'm an idiot. I should have guessed that it was something obvious! Doh! That did it, thanks for the insight!