Reprise du message précédent :

Je vais tenter ça

 
oui c'est pour avoir du nullsafe sinon OSEF.
 

 
Alors, là il n'y a pas d'Enum, mais des constantes String.
 
Du coup j'aurais fait :
 

public enum MesCodes {
 
  INGENIEUR("0003" ),
  EMPLOYE("0005" ),
 
...
 
  private final String value;
 
  MesCodes(String value) {
      this.value = value:
  }
 
  public static MesCode getValue(String name) { 
    try {
       return MesCodes.valueOf(name.toUpperCase()).value;
    } catch (Exception e) {
       return ""
 }
 
 
}

 
Et à la place du Switch :
 

public String getEmploi(String codeTypeProfession) {
    return MesCodes.getValue(codeTypeProfession);

 
 
Après il manque un truc aux Enums, c'est un genre de "getOrDefault', mais ça peut se construire facilement avec une classe utilitaire.
 
J'avais fait ça pour un projet où on en avait beaucoup, tu avais des méthodes du genre :
 

public static <E extends Enum, T> E valueOf(T value, Function<E, T> valueProvider, E defaultValue);

Chuis d'accord, c'est aussi ce que je dis.

'tin je pensais que c'était des constantes entières mais en fait ça convertit des constantes string en constantes string?

 
.. ouais..
 
Non, mais le code est WTFesque du début à la fin en fait.

https://semianalysis.com/the-semico [...] ir-own-ip/

La remarque initiale est correcte, le motif est mauvais. On utilise une fonction ultra générique, le switch, pour faire quelque chose de très précis : faire correspondre une valeur à une autre. À vue de nez je pense que le mieux serait d'utiliser une Map immutable ; je ne connais pas assez bien java pour savoir si ça se fait bien.

Quant à l'alternative proposée, elle ne résout en rien le problème initial : elle répond à la forme sans rien changer  au fond. Je pense comme d'autres ici que c'est plutôt la responsabilité du reviewer qui n'a pas su expliquer la raison de sa remarque, mais c'est vrai qu'à sa place je me moquerais aussi (mais gentiment).

(et si jamais ses codes sont des constantes à usage hyper restreint, alors il casse pas les noix et il crée un Map<String, String>, et c'est marre.

C’est trivial mais c’est con, ça augmente le coût en mémoire et cpu sans rien apporter.

Bah ça donne une intention au code, c'est en général plus important que les 12 octets d'économisés  

T'es nouveau ici ? Masklinn n'a jamais tort et ne sodomise jamais de drosophile pour tenter de sauver la face  

MACHINE MODO C4EST NOUS FAIRE 0 TOUS UN CADEAU § /FOU/

Ton intention au code tu l’as en mettant le switch dans une méthode de conversion, c’est autrement plus clair

 
Non c'est pas un anti-pattern. Un switch où les seuls statements sont des return, c'est lisible, ça ne pose pas de problème.
 
 

 
Mettre une constante pour ça n'est pas totalement inutile, ça montre que t'avais vraiment l'intention de mettre une chaîne vide, pas que t'as oublié une valeur, mais on est d'accord qu'en termes d'utilité sémantique c'est très faible

Je fais plus de Java depuis un moment mais c'est probablement la seule et unique utilisation réelle que j'avais d'un switch, vu que coder des automates finis n'est pas vraiment le truc que je fais tous les jours.
Illustration dans la doc officielle: https://docs.oracle.com/javase/tuto [...] witch.html

 
ACK ?
 
je vais surement me faire jeter, mais je dirais qu'on a pas assez de contexte pour juger. Si les classes métiers ont plein de valeurs avec des mapping fixes, ce serait pas plus propre d'avoir une table de config en BDD / un fichier de conf ? L'autre question que je me poserais c'est à quelle fréquence ces codes changent (ou des nouveaux sont introduits), et si tu dois comprendre les valeurs passées ou pas.

Un pauvre switch -> une Map immuable -> une BDD ou un fichier de conf.
   
 
En fait vous vous foutez de notre gueule, ou vous préparez un sketch pour le prochain Montreux comedy les deux?

En tout cas en PHP j'aurais pas laissé passer une PR avec du code similaire

Il faut prévoir une interface pour que le métier puisse changer les données simplement, faudra donc rajouter quelques écrans, je pense qu'on peut partir sur cette architecture un peu basique pour le MVP :

jump table  

 
c'est du PIC ? c'est pas plutôt

ou

si tu veux copier "machin" dans W

On parlait de prise de notes la dernière fois. Moi qui adore tout noter dans Sublime text, j'ai découvert : https://zettlr.com/

C'est vraiment pas mal. Y'a beaucoup d'options dispo, différent espace de travail. On peut rendre le truc cloudable en utilisant un dossier de google drive par exemple. Par contre c'est du électron derrière et pas hyper opti quoi, donc c'est moins fluide qu'un sublime texte.

Et c'est gratos.

 
ouais c'est du pic, j'ai fait ça de tête
edit : je viens d'aller voir la doc MPASM , W ou , 0 ça sera traité pareil
movfw le risque c'est de taper movwf

Tu noteras que j'ai pas dit qu'il fallait le faire, j'ai dit que selon le contexte c'est pas deviner de demander comment la config change. Si la réponse est jamais, j'ai aucun problème a coder ça dans un switch, même si je trouve une énumération plus expressive à l'usage (completion des valeurs possibles déjà).
En python je ferais ça avec un dict parce que c'est assez idiomatique, et dict a des méthodes pratiques pour gérer les valeurs par défaut.

Bonne rentrée

Déjà repris moi et ma fille aussi mais merci  

Retour des bouchons  

ça peut s'empêcher -> https://twitter.com/wdormann/status/1432703702079508480

Jai découvert Obsidian y'a quelques mois et je pense que je vais l'utiliser a mon nouveau job (qui commence dans 2 semaines.. je serai bien resté en congé encore environ 4 ans).

Non, un switch ça peut faire n'importe quoi il faut lire toutes les lignes pour être sûr de ce que ça fait. Une map, ça mappe, l'intention est fournie avec la structure.

+1
Ici c'est clairement un tableau de correspondance qui est attendu, avec une valeur par défaut si non trouvée.  
Le switch n'est pas l'idéal pour ça.

 
Ici, rentrée pour ma fille ce matin. Mais première année où elle va toute seule à l’école.  
 
Galère en moins !
 
Mais bon, numéro 2 va à la nurserie cette année… du coup…. Il remplace numéro 1.

 
+1
 
Autre chose : ces valeurs étant issues du métier, et ayant une signification fonctionnelle précise, il est plus que probable qu'on en aura besoin ailleurs.
 
Et il est aussi assez probable qu'elles existent déjà dans le code, sous une forme ou une autre, ce qui pour le coup serait très emmerdant.
 
Donc à mon sens, la création des Enums pour les listes de valeurs métier est quasi systématiquement à privilégier.  
 
A faire dès l'apparition desdites listes, et à documenter.

Je pense qu'il faut faire un atelier avec le métier pour qu'il définisse mieux son besoin sur les workflows de validation des changements de valeur. En parallèle je lance une consultation pour avoir un consultant qui nous aide sur le dossier de choix pour l'outil de moteur de workflow, pendant ce temps Plam définit l'infra on prem, Jubi a bien demandé en CODIR la semaine dernière si ça scalait, donc on partira probablement sur 2 exadata X8M-8 reparti sur 2 sites

 
.. et le switch passera en prod pour des années...

 
Au moins ton troll m'a fait découvrir les X8M-8

Si t'en achètes 4 dans le même bon de commande, Oracle t'offre un gode moulé sur la mhÿte d'ellison à l'échelle 4:1

Ça me fait penser que ta rien à envier à Larry sur sa faculté à troller

C'est ça qu'il faut pour Pinterest

 
tu ne voudrai pas me faire une étude comparative par rapport à une solution multi tenant ?  
 
(en vrai, je suis d'accord avec g5 : les tables de correspondances entre codes métiers DOIVENT être définis dans une map en constantes, il y a zéro chance pour qu'il ne soient utilisés qu'à un endroit.)

 
Vraiment pas mal. Il charge vite et bien en plus.

Dans le RACI c'est Plam qui s'occupe de l'archi