Discussion :

[Daïmanu]

Bonjour.

J'ai une application qui fait appel à une API externe PersonAPI de cette façon :

Code java :

Sélectionner tout - Visualiser dans une fenêtre à part

1
2
3
4
5
6
7
8
9
10
public void modifierPersonne() {
    //Traider input
    PersonDto output = personApi.modify(input);
    //Traiter output
}
public void creerPersonne() {
    //Traider input
    PersonDto output = personApi.create(input);
    //Traiter output
}

L'API retourne ce genre de JSON :

Code JSON :

Sélectionner tout - Visualiser dans une fenêtre à part

1
2
3
4
5
6
7
{
    code: 200,
    messages: [],
    data: {
        person: {name: "name", age: 100, ...}
    }
}

Problème, la gestion des erreurs est mal conçue et l'API peut retourner un code HTTP 200 avec des données tel que :

Code JSON :

Sélectionner tout - Visualiser dans une fenêtre à part

1
2
3
4
5
6
7
8
9
10
{
    code: 400,
    messages: [
        severity: "ERROR",
        content: "Invalid character in 'name'"
    ],
    data: {
        person: {name: "א", age: 100, ...}
    }
}

Ce comportement ne pouvant pas changer, je souhaite m'adapter, mais si possible sans toucher au code existant (base de code assez grande).
Les Aspects semblent être une bonne solution, j'en utilise déjà pour log des appels de fonctions très précises.
Je pense intercepter les données de retour et envoyer une exception si le champ severity est "ERROR".

Je voudrais savoir si il est possible d'intercepter le processus juste après que la fonction ait retourné le JSON (retour de personApi.create()) mais avant que la valeur soit assignée à p output ?
Sinon si il y existe une option plus pertinente ?

[thelvin]

Hello,

il est pas clair quels sont les partis en présence.

Qu'est-ce qui est un programme serveur d'API qui tourne quelque part dans le monde, et qu'est-ce qui est un programme qui appelle cette API ? Y a-t-il d'autres programmes ? Quel est le programme que tu as besoin de modifier ? Les bouts de code que tu as montré font partie de quel programme ?

C'est quoi, p ?

[Daïmanu]

L'API est est une API REST qui tourne sur un serveur, qui se matérialise dans le code par le service personApi.

J'aimerais modifier mon programme, mais si possible sans toucher au code que j'ai présenté.

p est un mauvais-copier-coller, c'est output.

[thelvin]

?

Bah. Peut-être et peut-être pas. Si tu as déjà utilisé des aspects tu dois avoir une idée de comment ça marche n'est-ce pas ?

Alors comment diable quelqu'un qui n'est pas toi, pourrait-il avoir la moindre idée de si c'est possible, avec ce que tu nous montres ? Je trouve étrange qu'il y ait besoin de te dire ça, sans que ça te paraisse évident dès le départ.

Je veux dire, que ce soit possible, c'est pratiquement garanti, surtout qu'il existe des moyens de coller des aspects sur les méthodes privées. Difficile d'imaginer que la manière dont personApi matérialise l'API ne rende pas ça possible d'une manière ou d'une autre.

Mais concernant comment s'y prendre en pratique, il faut bien entendu regarder de quelle manière exacte cette API est matérialisée, bref il faut regarder exactement ce que tu ne montres pas. Et c'est assez surprenant que ça ne t'ait pas sauté aux yeux je dois dire.

Je voudrais savoir si il est possible d'intercepter le processus juste après que la fonction ait retourné le JSON (retour de personApi.create()) mais avant que la valeur soit assignée à p output ?

On rappellera qu'avec le code montré, personApi.create() ne renvoie absolument pas du JSON, mais un objet PersonDto. Objet qui donc, vraisemblablement, ne contient pas les champs code ni message, ni d'ailleurs le status HTTP qui avait réellement été répondu.

[PhilippeGibault]

Le problème, c'est qu'il y a deux écoles sur les exceptions, celle de Java de base, qui est la mienne, et décrite dans "Effective Java" (https://www.amazon.fr/Effective-Java.../dp/0134685997 ), et celle de Spring.

Je pense que la vision Spring n'est pas bonne, mais c'est un avis perso.
Effectivement, ça peut toujours mal se passer, soit parce que l'utilisateur envoie des données incomplètes, soit parce que la BDD tombe en panne ou que sais-je!

Si on prend Spring, il n'y a pas d'exception dans la signature des méthodes (exemple ici: https://docs.spring.io/spring-data/c...epository.html ).

Or, ce n'est pas vrai. En réalité, si la BDD crash, alors Spring va renvoyer une RuntimeException ( https://docs.oracle.com/javase/8/doc...Exception.html ).

Dans "Effective Java", au contraire (et dans le Java de base), on différentie les exceptions.
Il y a l'exception de base et la Runtime (celle envoyé par Spring) qui peut toujours être envoyé, n'importe quand.

Il est conseillé de mettre dans la signature des méthodes les exceptions qui peuvent être envoyé suite à une erreur "prévisible" (Erreur de l'utilisateur, plantage de la BDD ...) et laisser la Runtime pour les erreurs du développeur (NullPointerException étant la plus connue).

Dans ton cas, je crains que l'on ai pas mis en avant l'exception dans la signature de la méthode.
Dans ce cas, on peut faire de la façon suivante:

Code :

Sélectionner tout - Visualiser dans une fenêtre à part

1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
17
18
19
 
public void modifierPersonne() throws ServiceException{
    try{  
      //Traider input
      PersonDto output = personApi.modify(input);
      //Traiter output
    }catch(Exception e){
        throw new MyException(e);
    }
}
public void creerPersonne() throws ServiceException{
    try{
      //Traider input
      PersonDto output = personApi.create(input);
     //Traiter output
    }catch(Exception e){
       throw new MyException(e);
    }
}

Ainsi, tu te laisse le soin d'avoir une gestion centralisée des exceptions (généralement dans la couche controller).

Tu peux aussi traiter l'éventuelle exception dirrectement (en logant ...).

A noter que l'"Exception" est vague. Dans la réalité; le personApi jette une RuntimeException. Elle est dans une logique Spring.

[thelvin]

C'est pas du tout le problème présenté -_-°.

Que tu n'aimes pas les RuntimeException ça te regarde et ça se défend, mais on est plus que probablement dans un cas où la personne s'en tamponne si ça lui permet d'appliquer son idée.