Des erreurs de programmation java ou mauvaises pratiques

Bonjour, quels sont les erreurs ou bien mauvaises pratiques dans le suivant code:
public class CustomerRepository {
	private static  CustomerRepository INSTANCE = null;
	private CustomerRepository(){}
	
	private static void createInstance() {
		if (INSTANCE == null) {
			INSTANCE = new CustomerRepository();
		}
	}
	public static CustomerRepository getInstance() {
		if (INSTANCE == null) createInstance();
		return INSTANCE;
	}
		
}

2 réponses

  1. if (INSTANCE == null) createInstance();


    Tu ne peux pas comparer une instance avec null.
    C'est comme si tu comparais une variable int par rapport à 0.
    1. Mais INSTANCE c'est un objet de type CustomerRepository
      1. Tu as regardé avec le débug?