Question

So basically I have a Linked List class that has all of the constructor data necessary to read a file I have and then convert it into a Linked List. However when it reads in the file data it is adding lines and spaces incorrectly. I am using system.out.println to check and it is coming out incorrectly. I don’t think it is the toString method because I have messed with it so much and nothing changes. I need help with this because I cant figure it out.
Thanks!

File Information(Basically All of the data is on separate lines):

tobi
tobi
[e-mail]
tobi
Mixed Breed
Male
3-4
Virginia
Walking
lily
lily
[e-mail]
lily
Yorkshire Terrier
Female
3-4
Hawaii
Jumping
peppy
peppy
[e-mail]
peppy
Chihuahua
Male
7-8
Alaska
Sleeping
fluffy
fluffy
[e-mail]
fluffy
MixedBreed
Female
3-4
Virginia
Walking 
flower
flower
[e-mail]
flower
Chihuahua
Female
7-8
Alaska
Sleeping

Linked List:

    import java.io.Serializable;
    import java.util.*;

    public class LinkedAccountList implements Serializable{

    private String username;
    private String password;
    private String email;
    private String name;
    private String breed;
    private String gender;
    private String age;
    private String state;
    private String hobby;

    public LinkedAccountList (String username, String password, String email, String name, String breed, String gender, String age, String state, String hobby){
    this.username = username;
    this.password = password;
    this.email = email;
    this.name = name;
    this.breed = breed;
    this.gender = gender;
    this.age = age;
    this.state = state;
    this.hobby = hobby;
    }

    public String getUsername(){
    return username;
    }

    public String getPassword(){
    return password;
    }

    public String getEmail(){
    return email;
    }

    public String getName(){
    return name;
    }

    public String getBreed(){
    return breed;
    }

    public String getGender(){
    return gender;
    }

    public String getAge(){
    return age;
    }

    public String getState(){
    return state;
    }

    public String getHobby(){
    return hobby;
    }

    @Override
    public String toString() {
    return "Username: "+username+"\nPassword: "+password+"\nEmail: "+email+"\nName: "+name+"\nBreed: "+breed+"\nGender: "+gender+"\nAge: "+age+"\nState: "+state+"\nHobby: "+hobby;
    }

    public void setUsername(String u){
    username = u;
    }

    public void setPassword(String p){
    password = p;
    }

    public void setEmail(String e){
    email = e;
    }

    public void setName(String n){
    name = n;
    }

    public void setBreed(String b){
    breed = b;
    }

    public void setGender(String g){
    gender = g;
    }

    public void setAge(String a){
    age = a;
    }

    public void setState(String s){
    state = s;
    }

    public void setHobby(String h){
    hobby = h;
    }

    }

Read from File and create Linked List Method:

    LinkedList<LinkedAccountList> account = new LinkedList<LinkedAccountList>();

    try
    {
    read(account, "file.txt");
    } catch (Exception e)
    {
    System.err.println(e.toString());
    }
    display(account);
    }

    public static void read(LinkedList<LinkedAccountList> account, String inputFileName) throws java.io.IOException
    {
    BufferedReader infile = new BufferedReader(new FileReader(inputFileName));
    while(infile.ready())
    {
    String username = infile.readLine();
    String password = infile.readLine();
    String email = infile.readLine();
    String name = infile.readLine();
    String breed = infile.readLine();
    String gender = infile.readLine();
    String age = infile.readLine();
    String state = infile.readLine();
    String hobby = infile.readLine();

    LinkedAccountList d = new LinkedAccountList(username, password, email, name, breed, gender, age, state, hobby);
    account.add(d);
    }
    infile.close();
    }

Here is what the output looks like(Look closely and you can see the problem):

Username: tobi
Password: tobi
Email: [e-mail]
Name: tobi
Breed: Mixed Breed
Gender: Male
Age: 3-4
State: Virginia
Hobby: Walking
Username: 
Password: lily
Email: lily
Name: [e-mail]
Breed: lily
Gender: Yorkshire Terrier
Age: Female
State: 3-4
Hobby: Hawaii
Username: Jumping
Password:  peppy
Email: peppy
Name: [e-mail]
Breed: peppy
Gender: Chihuahua
Age: Male
State: 7-8
Hobby: Alaska
Username: Sleeping
Password: fluffy
Email: fluffy
Name: [e-mail]
Breed: fluffy
Gender: Mixed Breed
Age: Female
State: 3-4
Hobby: Virginia
Username: Walking 
Password: flower
Email: flower
Name: [e-mail]
Breed: flower
Gender: Chihuahua
Age: Female
State: 7-8
Hobby: Alaska
Username: Sleeping
Password: null
Email: null
Name: null
Breed: null
Gender: null
Age: null
State: null
Hobby: null

Answer 1

You don’t have any validation, any logging.
You should at least create method:

public String readLine(BufferedReader br){
    String rl = br.readLine();
    if (rl.trim().length() > 2) {
        return rl;
    }else return readLine(br);
}

and use it like:

...
     while(infile.ready())
        {
          String username = readLine(infile); // instead of String username = infile.readLine();

Also using so long constructor argument is error-prone

LinkedAccountList (String username, String password, String email,
String name, String breed, String gender, String age, String state,
String hobby){

Use ony setX(String arg) method. This is common practise in DAO pattern.