toString pour un enumRésolu

Question

bonjour a tous, j'ai fais une fonction toString() pour un enum, elle fonctionne mais je pense qu'elle peut être améliorée, cependant je ne vois pas comment.
Je compte sur vous...

public enum Symbole {
    maison,
    ballon,
    etoile,
    fanion,
    arbres;
    public static int NBR_SYMBOLE=Symbole.values().length;

    public static int getNBR_SYMBOLE() {
        return NBR_SYMBOLE;
    }

    public static void setNBR_SYMBOLE(int nbr_symbole) {
        NBR_SYMBOLE = nbr_symbole;
    }

    public String toString() {
        int num=0;
        String res="";
        Symbole[] tableauSymbole=Symbole.values();
        for(int i=0;i<NBR_SYMBOLE;i++){
            if(this==tableauSymbole[i]){
                num=i;
            }
        }
        switch(num){
        case 0: res+="maison";
        break;
        case 1: res+="ballon";
        break;
        case 2: res+="etoile";
        break;
        case 3: res+="fanion";
        break;
        case 4: res+="arbres";
        break;
        }
        return res;
        
    }

Depuis quelque temps, je fais beaucoup appel a ce forum pour me venir en aire, je vous remercie tous, en particulier DarkSidious qui a répondu a absolument toutes les questions que j'ai posé pour l'instant.

coucou747 · Accepted Answer

salut

cette fonction existe deja, tu n'es pas oblige de la definir toi meme.

si tu y tiens vraiment, alors tu peux utiliser :
num = this.ordinal();
a la place de :
for(int i=0;i<NBR_SYMBOLE;i++){
            if(this==tableauSymbole[i]){
                num=i;
            }
        }

cs_DARKSIDIOUS · Answer

Salut,

Améliorée en quel sens ?

Si c'est de stocker le nom des éléments :

public enum Symbole {
   private String nom;

    maison("maison"),
    ballon("ballon"),
    etoile("étoile"),
    fanion("fanion"),
    arbres("arbres");

    public Symbole(String nom) {
        this.nom =  nom;
    }

    public String toString() {
        return this.nom;
    }
}

Sinon, concernant ton enum :
public enum Symbole {
    maison,
    ballon,
    etoile,
    fanion,
    arbres;
    public static int NBR_SYMBOLE=Symbole.values().length;  // à quoi ca sert ? 

    public static int getNBR_SYMBOLE() { // à remplacer par un ordinal comme l'a indiqué notre ami coucou747
        return NBR_SYMBOLE;
    }

    public static void setNBR_SYMBOLE(int nbr_symbole) { // à quoi ca sert ???Le but n'est pas de savoir combien de symbole tu as ?
        NBR_SYMBOLE  = nbr_symbole;
    }

    public String toString() { // Quel est son rôle ? Retourner l'élément courant ou la liste des éléments ? Dans ton cas, elle ne renverra que l'élément courant, elle se serait plus simple en stockant le nom de l'élément directement dans l'enum...
        int num=0;
        String res="";
        Symbole[] tableauSymbole=Symbole.values();
        for(int i=0;i<NBR_SYMBOLE;i++){
            if(this==tableauSymbole[i]){
                num=i;
            }
        }
        switch(num){
        case 0: res+="maison";
        break;
        case 1: res+="ballon";
        break;
        case 2: res+="etoile";
        break;
        case 3: res+="fanion";
        break;
        case 4: res+="arbres";
        break;
        }
        return res;
        
    }

Si tu veux, tu peux aller voir ma source de la liste des départements pour voir un exemple concret d'une enum "qui fait papa maman"
______________________________________
DarK Sidious

ToString pour un enum

2 réponses

Votre réponse

Discussions similaires