Pour un champ booléen, quelle est la convention de dénomination de son getter / setter?

Par exemple.

boolean isCurrent = false;

Comment nommez-vous son getter et son setter?

Supposons que vous ayez

boolean active;

La méthode des accesseurs serait

public boolean isActive(){return this.active;}

public void setActive(boolean active){this.active = active;}

Voir également

• Programmation Java / Java Beans
• Conventions de code pour le langage de programmation Java

http://geosoft.no/development/javastyle.html#Specific

14. is Le préfixe doit être utilisé pour les variables et méthodes booléennes.

    isSet, isVisible, isFinished, isFound,isOpen

Il s'agit de la convention de dénomination des méthodes booléennes et des variables utilisées par Sun pour les packages Java Core. L'utilisation du préfixe is résout un problème courant de choix de mauvais noms booléens tels que status ou flag. isStatus ou isFlag ne correspondent tout simplement pas, et le programmeur est obligé de choisir des noms plus significatifs.

Les méthodes Setter pour les variables booléennes doivent avoir un préfixe défini comme dans:

void setFound(boolean isFound);

Il existe quelques alternatives au préfixe is qui s'adaptent mieux dans certaines situations. Ce sont les préfixes has, can et should:

boolean hasLicense(); 
boolean canEvaluate(); 
boolean shouldAbort = false;

Pour un champ nommé isCurrent, le nom correct du getter / setter est setCurrent()/ isCurrent()(du moins c'est ce que pense Eclipse), ce qui est très déroutant et peut être retracé au problème principal:

Votre champ ne doit pas être appelé isCurrenten premier lieu. Is est un verbe et les verbes ne sont pas appropriés pour représenter l'état d'un objet. Utilisez plutôt un adjectif, et soudainement vos noms de getter / setter auront plus de sens:

private boolean current;

public boolean isCurrent(){
    return current;
}

public void setCurrent(final boolean current){
    this.current = current;
}

Je crois que ce serait:

void setCurrent(boolean current)
boolean isCurrent()

Il est peut-être temps de commencer à réviser cette réponse? Personnellement , je voterais pour setActive()et unsetActive()(alternatives peuvent être setUnActive(), notActive(), disable(), etc. selon le contexte) depuis « setActive » implique que vous l' activer à tout moment, que vous ne faites. C'est un peu contre-intuitif de dire "setActive" mais en fait de supprimer l'état actif.

Un autre problème est que vous ne pouvez pas écouter spécifiquement un événement SetActive de manière CQRS, vous auriez besoin d'écouter un 'setActiveEvent' et de déterminer à l'intérieur de cet écouteur s'il a été activé ou non. Ou bien sûr, déterminez quel événement appeler lors de l'appel, setActive()mais cela va alors à l'encontre du principe de séparation des préoccupations.

Une bonne lecture à ce sujet est l'article FlagArgument de Martin Fowler: http://martinfowler.com/bliki/FlagArgument.html

Cependant, je viens d'un milieu PHP et je vois cette tendance de plus en plus adoptée. Je ne sais pas à quel point cela vit avec le développement Java.

private boolean current;

public void setCurrent(boolean current){
    this.current=current;
}

public boolean hasCurrent(){
    return this.current;
}

Setter: public void setCurrent(boolean val)
Getter: public boolean getCurrent()

Pour les booléens, vous pouvez également utiliser

public boolean isCurrent()

En tant que setter, que diriez-vous:

// setter
public void beCurrent(boolean X) {
    this.isCurrent = X;
}

ou

// setter
public void makeCurrent(boolean X) {
    this.isCurrent = X;
}

Je ne suis pas sûr que ces noms aient du sens pour les anglophones.