javadesign-patternsclass-constructors

Argument errorr in Java Nested Builder Pattern


I am learning the Java Builder Pattern from Section 6.1 (Builder Pattern Example) at: https://www.baeldung.com/creational-design-patterns#factory-method

But after I run my own code, it shows the error, wonder if could anyone help to point out where is wrong? Thanks a lot in advance.

error

class Test {
    public static void main(String[] args) {
        BankAccount newAccount = new BankAccount.BankAccountBuilder("Jon", "22738022275").withEmail("jon@example.com").wantNewsletter(true).build();
        System.out.print(newAccount);
    }
}

public class BankAccount {

    private String name;
    private String accountNumber;
    private String email;
    private boolean newsletter;

    // constructors/getters
    
    public static class BankAccountBuilder {
    
    private String name;
    private String accountNumber;
    private String email;
    private boolean newsletter;
    
    public BankAccountBuilder(String name, String accountNumber) {
        this.name = name;
        this.accountNumber = accountNumber;
    }

    public BankAccountBuilder withEmail(String email) {
        this.email = email;
        return this;
    }

    public BankAccountBuilder wantNewsletter(boolean newsletter) {
        this.newsletter = newsletter;
        return this;
        
    }
    
    public BankAccount build() {
        return new BankAccount(this);
        
    }
        
    }
}



Solution

  • You are missing the private constructor, something like:

    //The constructor that takes a builder from which it will create the object
    //the access to this is only provided to builder
    private BankAccount(BankAccountBuilder builder) {
        this.name = builder.name;
        this.accountNumber = builder.accountNumber;
        this.email = builder.email;
        this.newsletter = builder.newsletter;
    }
    

    I assume the article leaves it to the reader in the section that says // constructors/getters on the code.

    In addition, not completely related but the System.out.print you have will not print the object but its reference, you don't have a toString() method.