AbstractGridCrud constructor calling protected method

[mmkathurima]

I suggest finding an alternative way of:

public AbstractGridCrud(Class<T> domainType, CrudLayout crudLayout, CrudFormFactory<T> crudFormFactory, CrudListener<T> crudListener) {
    super(domainType, crudLayout, crudFormFactory, crudListener);
    initLayout();
  }

I want to initialize a final variable before I override initLayout() and of course it is null by the time I use it in the overriden method,

[alejandro-du]

The only solutions I can quickly think of is making initLayout final or requiring clients to call some init method after construction. Do you have suggestions?

[mmkathurima]

initLayout should remain protected and non final. I suggest converting the constructor into a static factory method.

[mmkathurima]

Take inspiration from the below sample:

private abstract static class Parent {
    private final String name;

    protected Parent(String name) {
        this.name = name;
        //NOTE Remove this.init();
    }

    protected void init() {
        System.out.printf("Called from Parent %s: init\n", this.name);
    }

    public static <T extends Parent> T create(T instance) {
        instance.init();
        return instance;
    }
}

private static class Child extends Parent {
    private final String name;

    private Child(String name) {
        super(name);
        this.name = name;
    }

    @Override
    protected void init() {
        super.init();
        System.out.printf("Called from Child %s: init\n", this.name);
    }
}

public static void main(String[] args) {
    Parent.create(new Child(Child.class.getSimpleName()));
}

Output:

Called from Parent Child: init
Called from Child Child: init