Java: Как правильно обработать Json?

Question

[Михаил Иванов]

Добрый день!
Вопрос вот в чем:
Есть

//Основной класс
 class Client {
  private String login;
  private Set<Pet> pets;
}

//Обрабатывает запрос, и возвращает JSON объект 
@RequestMapping(value = "/ajax", method = RequestMethod.GET)
    public @ResponseBody String ajax() throws IOException {
        ObjectMapper mapper = new ObjectMapper();
        return mapper.writeValueAsString(clientDAO.getAll()); // тут List<> (список всех клиентов, а у каждого клиента Set животных)
    }
//В jsp описал обработку данных с помощью ajax

 $.ajax({
               url: '/ajax',
                type: 'GET',
               dataType: "json",
               contentType: "application/json ",
               success: function (data) {
                   alert("Данные" + data)
               },
                error: function () {
                    alert("Ошибка при получении данных!")
                }
})

При запуске кода выскакивает исключение и ссылается на поле Set pets, подскажете как правильно реализовать ajax запрос или я не правильно отправляю данные? Если подставить произвольный объект client без Set<> то все работает!

Comments

[Сергей Горностаев]

Покажите полный текст ошибки.

[Михаил Иванов]

Сергей Горностаев:
type Exception report

message Infinite recursion (StackOverflowError) (through reference chain: ru.misha.model.Pet["client"]->ru.misha.model.Client["pets"]->org.hibernate.collection.internal.PersistentSet[0]->ru.misha.model.Pet["client"]->ru.misha.model.Client["pets"]->org.hibernate.collection.internal.PersistentSet[0]->ru.misha.model.Pet["client"]->ru.misha.model.Client["pets"]->org.hibernate.collection.internal.PersistentSet[0]->ru.misha.model.Pet["client"]-

description The server encountered an internal error that prevented it from fulfilling this request.

exception
org.codehaus.jackson.map.JsonMappingException: Infinite recursion (StackOverflowError) (through reference chain: ru.misha.model.Pet["client"]->ru.misha.model.Client["pets"]->org.hibernate.collection.internal.PersistentSet[0]->ru.misha.model.Pet["client"]->ru.misha.model.Client["pets"]
org.codehaus.jackson.map.ser.BeanSerializer.serializeFields(BeanSerializer.java:156)
org.codehaus.jackson.map.ser.BeanSerializer.serialize(BeanSerializer.java:118)
org.codehaus.jackson.map.ser.ContainerSerializers$CollectionSerializer.serializeContents(ContainerSerializers.java:360)
org.codehaus.jackson.map.ser.ContainerSerializers$CollectionSerializer.serializeContents(ContainerSerializers.java:312)
org.codehaus.jackson.map.ser.ContainerSerializers$AsArraySerializer.serialize(ContainerSerializers.java:111)
org.codehaus.jackson.map.ser.BeanPropertyWriter.serializeAsField(BeanPropertyWriter.java:268)
org.codehaus.jackson.map.ser.BeanSerializer.serializeFields(BeanSerializer.java:146)
org.codehaus.jackson.map.ser.BeanSerializer.serialize(BeanSerializer.java:118)

Не много сократил, очень много раз дублируется текст ошибки.

[Сергей Горностаев]

Mishany: покажите класс Pet.

[Михаил Иванов]

Сергей Горностаев:
public class Pet extends Base {

private String petName;

private int age;

private Client client;

public Pet() {
}

public Pet(String petName, int age) {
this.petName = petName;
this.age = age;
}

public Pet(int id) {
this.id = id;
}

public Client getClient() {
return client;
}

public void setClient(Client client) {
this.client = client;
}

public String getPetName() {
return petName;
}

public void setPetName(String petName) {
this.petName = petName;
}

public int getAge() {
return age;
}

public void setAge(int age) {
this.age = age;
}
}

[Михаил Иванов]

Обычный класс, ничего в нем нет необычного.

[Михаил Иванов]

Вот весь проект целиком
https://github.com/Vl-Mikhail/Spring

[Сергей Горностаев]

Mishany: у вас циклическая зависимость между Pet и Client.

[Михаил Иванов]

Ну если я не ошибаюсь, то она должна быть, что бы корректно работать с бд через hibernate, или я не прав?
Тут смысл такой, я добавляю клиенту животное, а он автоматически проставляет внешний ключ в бд,

Answer 1

[Сергей Горностаев]

Как я уже писал в комментарии, у вас циклическая зависимость между Pet и Client. Выхода два - или убрать зависимость, или определить владельца связи, чтобы Hibernate мог правильно определить правило десириализации. Сделать это можно либо описав в xml правило мапинга и определив параметр inverse:

<hibernate-mapping>
    <class name="ru.misha.model.Client" table="clients" ...>
    ...
        <set name="pets" table="clients_pets" fetch="select" inverse="true">
            <key>
                <column name="client_id" not-null="true" />
            </key>
            <one-to-many class="ru.misha.model.Pet" />
        </set>
    ...
    </class>
...
</hibernate-mapping>

либо определить направление связи анотациями

@Entity
@Table(name = "clients")
public class Client extends Base {
    @OneToMany(mappedBy="client")
    @Column(name="clientId")
    private Set<Pet> pets;
}

Как-то так. Не копируйте бездумно, так как пишу по памяти и могу ошибаться в деталях. Почитайте документацию по Hibernate в заданом направлении.

Comments

[Михаил Иванов]

Спасибо, понял в каком направлении смотреть!

[Михаил Иванов]

Нашел решение:
Since Jackson 1.6 you can use two annotations to solve the infinite recursion problem without ignoring the getters/setters during serialization: @JsonManagedReference and @JsonBackReference

И добавил fetch="select" как вы рекомендовал, и того

еще рекомендуют использовать DTO, но с этим надо разбираться, а не очень хочется если честно!
Спасибо!

Answer 2

[mitekgrishkin]

> Infinite recursion (StackOverflowError) (through reference chain: ru.misha.model.Pet["client"]->ru.misha.model.Client["pets"]
Как я понимаю, прога тянет клиента, внутри клиента список животных. У животного вытягивается клиент, и идем по рекурсии заново. Память переполняется и приложение падает. Попробуйте либо хранить только id клиента в животном, либо добавьте OneToMany для списка животных, а в животном уберите ссылку на клиента

Comments

[Михаил Иванов]

Ясно почему так происходит, только вот решение как вы предлагаете, не будет работать. У меня есть OneToMany и ManyToOne, и если я уберу ссылку на клиента, связи через hibernate нарушатся.
Возможно есть какое то другое решение?

[mitekgrishkin]

Пока вижу 2 решения: 1) как я писал выше, тянуть не все, а только id. Потом нужного Client из дао грузить уже по id. 2) вытягивать нужного Client через дао, но уже передавая petId. В обоих придется избавиться от перекрестной зависимости.

[Михаил Иванов]

Вот все отлично работает без использования ajax, а тут прям засада

[Михаил Иванов]

Нашел решение:
Since Jackson 1.6 you can use two annotations to solve the infinite recursion problem without ignoring the getters/setters during serialization: @JsonManagedReference and @JsonBackReference
В xml добавил fetch="select", все заработало.

Еще рекомендуют использовать DTO, но с этим надо разбираться, а не очень хочется если честно!
Спасибо!

Answer 3

[Павел Гаев]

Можно глянуть вот сюда, а еще на мой ответ по схожему вопросу вот тут

Comments

[Михаил Иванов]

Спасибо, ваш ответ на тостере видел!