Des erreurs de programmation java ou mauvaises pratiques

Mouha -  
Pierre1310 Messages postés 8854 Statut Membre -
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. Pierre1310 Messages postés 8854 Statut Membre 652
     
    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.
    0
  2. Mouha
     
    Mais INSTANCE c'est un objet de type CustomerRepository
    0
    1. Pierre1310 Messages postés 8854 Statut Membre 652
       
      Tu as regardé avec le débug?
      0