Discussion :

[WiZarD67]

Bonjour,

Je suis entrain de faire une application JAVA avec une API Rest et je rencontre un petit problème.

Le JSON de mon API n'est pas valide...

Code :

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

1
2
3
4
5
6
7
8
[
    [
        1,
        "FRJEADUP",
        "Jean",
        "DUPOND"
    ]
]

Je ne comprend pas vraiment pourquoi et comment cela se fait... si quelqu'un aurai une réponse ?

PS : J'utilise une architecture en oignon avec du CQRS et des ValueObjects.

Voici tout mon code :

CustomerController

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
@CrossOrigin("*")
@RestController
public class CustomerController {
 
    @Autowired
    private CustomerRepository repository;
 
    @ResponseBody
    @GetMapping(path = "/GetCustomers", produces = "application/json")
    public ResponseEntity<List<Customer>> getCustomers() {
        var GetAllCustomers = new GetAllCustomers(repository);
        return ResponseEntity.ok(GetAllCustomers.Do());
    }
 
}

ICustomerRepository

Code :

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

1
2
3
4
5
6
public interface ICustomerRepository {
 
    List<Customer> GetAll();
 
    void Add(Customer customer);
}

GetAllCustomers

Code :

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

1
2
3
4
5
6
7
8
9
10
11
12
public class GetAllCustomers {
 
    private ICustomerRepository _Repository;
 
    public GetAllCustomers(ICustomerRepository Repository) {
        _Repository = Repository;
    }
 
    public List<Customer> Do() {
        return _Repository.GetAll();
    }
}

CustomerRepository

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
20
21
22
@Repository
public class CustomerRepository implements ICustomerRepository {
 
    @Autowired
    EntityManager entityManager;
 
    @SuppressWarnings("unchecked")
    @Override
    public List<Customer> GetAll() {
        try {
            String SQL = "SELECT ID, LOGIN, FOR_NAME, LAST_NAME FROM CUSTOMER";
            Query query = entityManager.createNativeQuery(SQL);
 
            List<Customer> customers = query.getResultList();
 
            return customers;
        } catch (NoResultException e) {
            return null;
        }
 
    }
}

Customer

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
20
21
22
23
24
25
26
27
28
29
30
31
32
33
34
35
36
37
38
39
40
41
private int _ID;
 
    private Login _Login;
 
    private ForName _ForName;
 
    private LastName _LastName;
 
    public Customer(Login Login, ForName ForName, LastName LastName) {
        _Login = Login;
        _ForName = ForName;
        _LastName = LastName;
    }
 
    public Customer(int ID, Login Login, ForName ForName, LastName LastName) {
        _ID = ID;
        _Login = Login;
        _ForName = ForName;
        _LastName = LastName;
    }
 
    public int getID() {
        return _ID;
    }
 
    public String getLogin() {
        return _Login.getValue();
    }
 
    public String getForName() {
        return _ForName.getValue();
    }
 
    public String getLastName() {
        return _LastName.getValue();
    }
 
    @Override
    public String toString() {
        return getID() + " " + getLogin() + " " + getForName() + " " + getLastName();
    }

Login

Code :

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

1
2
3
4
5
6
7
8
9
10
11
12
public class Login {
 
    private String _Value;
 
    public Login(String Value) {
        _Value = Value;
    }
 
    public String getValue() {
        return _Value;
    }
}

ForName

Code :

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

1
2
3
4
5
6
7
8
9
10
11
12
public class ForName {
 
    private String _Value;
 
    public ForName(String Value) {
        _Value = Value;
    }
 
    public String getValue() {
        return _Value;
    }
}

LastName

Code :

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

1
2
3
4
5
6
7
8
9
10
11
12
public class LastName {
 
    private String _Value;
 
    public LastName(String Value) {
        _Value = Value;
    }
 
    public String getValue() {
        return _Value;
    }
}

Je sais qu'il y a beaucoup de classe et que ce n'est pas forcement simple à comprendre du premier regard..

En espérant que quelqu'un pourra m'aider.

WiZarD67

[licardentaistor]

c'est un JSON valide ... https://jsonformatter.curiousconcept.com/#

je ne vois pas où est le pb?

[JeitEmgie]

c'est un JSON valide ... https://jsonformatter.curiousconcept.com/#

je ne vois pas où est le pb?

Dans l'absolu, oui c'est du JSON...
Mais si la liste est censée représenter une List<Customer>, pas nécessairement...
cela pourrait être une array d'objets:

Code :

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

1
2
3
4
5
6
7
8
[
{
 "id" : 1,
 "login" : "FRJEADUP",
 "for_name" : "Jean",
 "last_name" : "DUPOND"
}
]

Donc le problème est plutôt les conventions entre le client et le serveur…

[WiZarD67]

C'est bien ce type de JSON que je voudrai

Code :

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

1
2
3
4
5
6
7
8
[
    {
         "id" : 1,
         "login" : "FRJEADUP",
         "for_name" : "Jean",
         "last_name" : "DUPOND"
    }
]

Si le problème vient d'une mauvaise conversion entre le serveur et le client il y a une solution ?

Ou en vue de ce que je viens d'écrire il faut que je change mon format de retour ?

[JeitEmgie]

Si le problème vient d'une mauvaise conversion entre le serveur et le client il y a une solution ?

Ou en vue de ce que je viens d'écrire il faut que je change mon format de retour ?

Justement : normalement avec Spring c'est automatique pour peu que les bons JAR soient dans le class path et que l'application context et le Java respectent les conventions...

Or côté Java cela me semble OK : @RestController qui retourne un @ResponseBody et produces = "application/json"...
reste à vérifier le reste...

[WiZarD67]

Je viens d'essayer avec une List instancié dans mon End point et tout fonctionne correctement

Code :

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

1
2
3
4
5
6
7
8
9
@GetMapping(path = "/GetCustomer", produces = "application/json")
    @ResponseBody
    public ResponseEntity<List<Customer>> getCustomer() {
        List<Customer> ls = new ArrayList<Customer>();
        ls.add(new Customer(new Random().nextInt(), new Login("FRJEADUP"), new ForName("Jean"), new LastName("Dupond")));
        ls.add(new Customer(new Random().nextInt(), new Login("FRJEADUP"), new ForName("Jean"), new LastName("Dupond")));
        ls.add(new Customer(new Random().nextInt(), new Login("FRJEADUP"), new ForName("Jean"), new LastName("Dupond")));
        return ResponseEntity.ok(ls);
    }

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
20
[
    {
        "id": -122011726,
        "lastName": "Dupond",
        "login": "FRJEADUP",
        "forName": "Jean"
    },
    {
        "id": -1732593737,
        "lastName": "Dupond",
        "login": "FRJEADUP",
        "forName": "Jean"
    },
    {
        "id": 1721364557,
        "lastName": "Dupond",
        "login": "FRJEADUP",
        "forName": "Jean"
    }
]

Je pense que ça doit venir de mon Repository... à ce niveau la

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
@SuppressWarnings("unchecked")
    @Override
    public List<Customer> GetAll() {
        try {
            String SQL = "SELECT ID, LOGIN, FOR_NAME, LAST_NAME FROM CUSTOMER";
            Query query = entityManager.createNativeQuery(SQL);
 
            List<Customer> customers = query.getResultList();
 
            return customers;
        } catch (NoResultException e) {
            return null;
        }
 
    }

[JeitEmgie]

Je pense que ça doit venir de mon Repository... à ce niveau la

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
@SuppressWarnings("unchecked")
    @Override
    public List<Customer> GetAll() {
        try {
            String SQL = "SELECT ID, LOGIN, FOR_NAME, LAST_NAME FROM CUSTOMER";
            Query query = entityManager.createNativeQuery(SQL);
 
            List<Customer> customers = query.getResultList();
 
            return customers;
        } catch (NoResultException e) {
            return null;
        }
 
    }

En effet : query.getResultList() d'un native query renvoit une liste d'Object...
ce n'est pas un TypeQuery de JPA que vous invoquez... donc vous devez faire la conversion vous-même ou spécifier un mapping ou une Class dans createNativeQuery.

Notez que si vous spécifiez une Class, il faut que le constructor correspondant aux types d'object de la List<Object> existe... donc dans votre cas un constructor (Integer, String, String, String).
Et si votre classe Customer est votre objet métier (@Entity), vous risquez des problèmes de validation des champs que vous n'initialisez pas... c'est pour cela qu'en général on utilise une classe spécifique (un DTO).

[WiZarD67]

En cherchant j'ai trouvé ce qui bloqué.

Pour ceux qui veulent :

Code :

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

1
2
3
4
5
6
7
8
9
10
11
12
@Override
    public List<Customer> GetAll() {
        String SQL = "SELECT * FROM CUSTOMER";
        Query query = entityManager.createNativeQuery(SQL, CustomerEntity.class);
 
        List<CustomerEntity> customersEntities = query.getResultList();
 
        List<Customer> customers = customersEntities.stream().map(x -> x.ToDomainEntity()).collect(Collectors.toList());
 
        return customers;
 
    }

La méthode ToDomainEntity convertie mes objets JPA en entité métier.