Organisation et optimisation de mes classes?

Version imprimable

29/11/2007, 16h31
Sylk

Organisation et optimisation de mes classes?

Bonjour,
Je dois préparer une classe en JAVA capable de transformer du XML en un protocole propre a mon entreprise qui est du type

Code:

141=5;321=COUCOU;980=user1; etc…

Ou le code 141 serait la version du protocole utilisé,
le code 321 le message de bienvenue,
le code 980 l'utilisateur qui envoie ce message, etc…

Mon idée est de faire 2 classes :
- une classe MonProtocole qui dispose des 1000 codes existants pour ce protocole,
- une classe MonProtocoleConstante, qui est composé de constantes contenant le numéro de code dans le protocole (exemple : public static final int VERSION = 141) pour rendre le code plus lisible dans ma classe MonProtocole

Il me semble que cette manière n'est pas très optimal : En effet, à chaque fois qu'on instancie la classe MonProtocole, on instancie 1000 attributs, alors que l'on ne va en utiliser peut être que 50…

Y'a t'il une solution plus optimale ?

Merci d'avance !
29/11/2007, 23h17
Sanguko

Les constantes ne sont créées qu'une seule fois, au chargement de la classe MonProtocoleConstante.

Au lieu de créer une classe MonProtocoleConstante, tu peux aussi en faire une interface, implémentée par MonProtocole.
30/11/2007, 09h02
LGnord

Les champs statics de MonProtocoleConstante sont initialisés une unique fois.

Par contre, je n'ai pas compris si les 1000 attributs de MonProtocole sont les constantes définies dans MonProtocoleConstante. S'agit-il d'autres choses?

Tu pourrais utiliser un enum plutôt que de faire des "public static final int VARIABLE = valeur".

Code:

1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
17
18
19
20
21
22
public enum Const {
    VERSION(141),
    ERREUR(400),
    OK(200);
 
    private int value;
 
    Const(int value) {
        this.value = value;
    }
 
    public int getValue() {
         return value;
    }
 
    // si tu veux que l'affichage de l'enum n'affiche que la valeur
    @Override
    public String toString() {
        return value;
    }
 
}

30/11/2007, 10h22
Sylk
Je ne vais pas utiliser le type enum parce que la classe MonProtocoleConstante sera utilisée par d’autres classes dans le futur, et sachant qu’elle contient plus de 1000 constantes, cela sera plus facile à maintenir si cette information est présentes dans une classe seulement.

Au début, je pensais créer le même nombre d'attributs pour l'objet MonProtocole qu'il y a de constantes dans la classe MonProtocoleConstante, mais en reflechissant un peu, je me suis rendu compte qu'il était plus judicieux de créer un simple attibut de type String et d'ajouter a cette chaine les differents codes de mon protocole :

Exemple
Code:

1 2 3 4 5 6 7 8 9 10 public class MonProtocole { String message; (...) Public void addField( int field, String value ) { message = message + ";" + field + "=" + value) } }
L'utilisateur appele addField avec comme premier argument (field) une constante de la classe MonProtocoleConstante
Code:

1 2 myProtocol.addField(MonProtocoleConstante.VERSION, "5") ;
Bien sur, dans la classe MonProtocoleConstante on a :
Code:

1 2 public static final int VERSION = 141;