Règles d'optimisation du code source

Version imprimable

29/05/2021, 11h55
favdb

Règles d'optimisation du code source

Bonjour,

Le message est un peu long en fonction des éléments de contexte.

Je développe depuis plusieurs années un logiciel qui comporte plusieurs milliers de lignes. Suite à un récent incident à répétition j'ai mis en cause ma manière de coder mon application (ce qui n'était pas le cas). Celle-ci gère une base de données (h2database) et lorsque j'ai installé le plugin SonarLint pour repérer les optimisations à faire je me suis rendu compte que beaucoup de signalements concernaient la duplication de variables statiques (genre intitulé d'un champ de la BDD). J'ai bien compris qu'il me faudrait regrouper ces variables statiques quelque part. Mais je m'interroge sur le fichier où mettre ça: fichier unique, ou déclarations dans les fichiers concernés?

Un exemple concret: l'une des entités de la BDD concerne la description de Lieux. Cette description est utilisée dans plusieurs fichiers: Editeur.java (pour modifier/créer), Panel.java (pour afficher en mode tableau), Export.java (pour exporter les informations dans différents formats), etc... Donc où mettre les variables statiques? Dans chacun des fichiers? Dans un fichier unique commun?

À savoir que ma BDD comporte pas moins de 59 éléments distincts.

J'espère avoir été assez clair.

PS: petite info complémentaire, le logiciel comporte 1929 fichiers, dont 1874 fichiers Java pour un total de 214982 lignes de code.
31/05/2021, 09h10
Cincinnatus

Citation:

Envoyé par favdb

Un exemple concret: l'une des entités de la BDD concerne la description de Lieux. Cette description est utilisée dans plusieurs fichiers: Editeur.java (pour modifier/créer), Panel.java (pour afficher en mode tableau), Export.java (pour exporter les informations dans différents formats), etc... Donc où mettre les variables statiques? Dans chacun des fichiers? Dans un fichier unique commun?

Bonjour,

La réponse me semble être dans la question... Est-ce qu'il existe une classe pour les Lieux ? Si oui, c'est peut-être l'endroit où déclarer les constantes (ou variables ?) concernant ces entités, indépendamment de leur utilisation. Ou une classe de description spécifique, mais en fonction de leur modélisation, pas de leur usage (qui peut varier, comme indiqué ci dessus (dans des classes plutôt que des fichiers .java, d'ailleurs)).

Il vaut mieux penser modélisation objet, pas usage des données. ça me semble plus clair et plus maintenable.
04/06/2021, 11h09
favdb

Merci pour ta réponse qui me conforte dans mes conclusions: il n'y a pas de règle précise. En revanche je retiens la suggestion de mettre les données statiques dans la classe de description des objets. Ça me paraît logique, c'est pas forcément ce que j'ai mis en oeuvre jusqu'à présent. Je vais donc entreprendre la réécriture de ces quelques 40 objets.
04/06/2021, 13h05
foetus
Citation:

Envoyé par favdb

Merci pour ta réponse qui me conforte dans mes conclusions: il n'y a pas de règle précise.

Enfin je pense que si :mrgreen: on peut dire que le problème c'est de déterminer qui est/ sont le(s) propriétaire(s) de la donnée et que va en faire la variable qui va l'utiliser (lecture, écriture, lecture et écriture)

Lorsque tu fais 1 variable globale/ statique, toutes les autres variables sont propriétaires et toutes peuvent la lire/ modifier.
1 technique (surtout avec 1 IHM) est de faire 1 god object (<- lien wikipedia en français) (en gros de regrouper toutes les variables/ fonctions statiques/ globales dans 1 seul objet)

Mais comme le dit @Cincinnatus, le mieux est
- de définir le propriétaire - s'il en a plusieurs c'est + compliqué
- de déterminer si ce propriétaire a la possibilité de créer/ détruire ou pas - association vs agrégation en UML
- d'exposer l'interface nécessaire pour lire/ modifier - souvent via les accesseurs et les mutateurs
L'injection de dépendance est aussi très utilisée pour transmettre des données entres 2 classes.

En gros, cela s'appelle de la conception :zoubi: