Как сопоставить неустановленное свойство со значением, не получая исключение NullPointerException в Dozer - PullRequest
Новая
       14

Как сопоставить неустановленное свойство со значением, не получая исключение NullPointerException в Dozer

1 голос
/ 10 сентября 2009

Используя Dozer для отображения двух объектов, я имею: 

/**
/* This first class uses the GXT (ExtJS) framework
**/
Class1 extends BaseModelData
{
    public int getId()
    {
        return (Integer)get("id");
    }

    public void setId(int id)
    {
        set("id", id);
    }

    // more properties
}

Class2
{
    public int getId()
    {
        return id;
    }

    public void setId(int id)
    {
        this.id = id;
    }

    // more properties
}

Если я не установил Id в первом классе (вызывая class1.setId ()), результатом будет NullPointerException от Dozer. Я понимаю, что это правильно, так как get ("id") будет нулевым.

Конечно, я могу решить эту проблему, установив проверку на ноль и вернув -1 или 0, или что-то еще.

Проблема в том, что это становится ошибкой времени выполнения, а не ошибкой времени компиляции. Я бы предпочел решить эту проблему правильно.

Теперь я прочитал в документации Dozer, что вы можете пропустить его, выполнив map-null = "false" , но я не смог заставить это работать ...

Есть предложения?

1 Ответ

1 голос
/ 11 сентября 2009

Я считаю, что проблема не в dozer, а в скрытой автоматической распаковке в вашем геттере: 

   public int getId()
{
    return (Integer)get("id");
}

(Integer) get ("id") неявно приводится в тип int, потому что тип возвращаемого значения вашего метода - "int".

Это будет работать в большинстве случаев ... ЗА ИСКЛЮЧЕНИЕМ, когда результат равен нулю, и в этом случае вы получаете исключение NullPointerException, поскольку int никогда не может быть нулевым.

Это приводит к скрытым исключениям NullPointerExceptions ... Подробнее здесь: http://www.theserverside.com/blogs/thread.tss?thread_id=41731

Чтобы решить эту проблему, у вас есть несколько вариантов:

  • Если Class1 и Class2 могут фактически содержать нулевой идентификатор, вы хотите изменить ваши получатели / установщики, чтобы получать / устанавливать целые числа вместо примитивных целых.

  • Если и Class1, и Class2 никогда не должны содержать нулевой идентификатор, и вы считаете, что это инвариант класса, вы можете оставить тип примитива int в getter / setter и либо:

    • Убедитесь, что get ("id") никогда не будет нулевым, инициализируя его каким-либо конкретным значением (например, 0) в конструкторе) и убедившись, что ничто не может установить его в null.
    • Или решите, что getId () вернет значение по умолчанию, если оно пустое, и добавьте проверку нуля в получателе, как вы сказали.

  • Если Class1 может иметь нулевой идентификатор, а Class2 может и не иметь, у вас должны быть получатели и установщики Class1, использующие тип Integer вместо примитива int, и вы должны создать CustomConverter dozer, который возвращает значение по умолчанию, когда поле источника равно нулю.

Привет

[EDIT] Вот тестовый код, который показывает, что Dozer игнорирует отображение нулей при запросе:

src / com / test / dozer / Class1.java: 

package com.test.dozer;

import com.extjs.gxt.ui.client.data.BaseModelData;

public class Class1 extends BaseModelData {

    // Notice the return type here: "Integer" and *not* int
    // Returning int throws a NullPointerException when get("id") is null!
    public Integer getId() {
        return (Integer) get("id");
    }

    public void setId(Integer id) {
        set("id", id);
    }

}

src / com / test / dozer / Class2.java: 

package com.test.dozer;

public class Class2 {

    private int id;

    public void setId(int id) {
        this.id = id;
    }

    public int getId() {
        return id;
    }
}

src / dozerMappingFile.xml: 

<?xml version="1.0" encoding="UTF-8"?>
<mappings xmlns="http://dozer.sourceforge.net"
          xmlns:xsi="http://www.w3.org/2001/XMLSchema-instance"
          xsi:schemaLocation="http://dozer.sourceforge.net http://dozer.sourceforge.net/schema/beanmapping.xsd">

  <configuration>
    <stop-on-errors>true</stop-on-errors>
    <date-format>MM/dd/yyyy HH:mm</date-format><!-- default dateformat will apply to all class maps unless the class mapping explicitly overrides it -->
    <wildcard>true</wildcard><!-- default wildcard policy that will apply to all class maps unless the class mapping explicitly overrides it -->
  </configuration>

  <mapping map-null="false">
    <class-a>com.test.dozer.Class1</class-a>
    <class-b>com.test.dozer.Class2</class-b>
  </mapping>

</mappings>

src / com / test / dozer / DozerTest.java: 

package com.test.dozer;

import java.util.Arrays;

import junit.framework.Assert;

import org.dozer.DozerBeanMapper;
import org.junit.Before;
import org.junit.Test;

public class DozerTest {

    private DozerBeanMapper mapper;

    @Before
    public void setUp() {
        mapper = new DozerBeanMapper(Arrays.asList("dozerMappingFile.xml"));
    }

    /**
     * Verifies that class1's id is mapped into class2's id when not null.
     */
    @Test
    public void testMappingWhenIdNotNull() {
        Class1 class1 = new Class1();
        class1.setId(1);
        Class2 class2 = new Class2();
        class2.setId(2);

        mapper.map(class1, class2);

        Assert.assertEquals(1, class2.getId());
    }

    /**
     * Verifies that class2's id is not set to null when class1's id is null.
     */
    @Test
    public void testMappingWhenIdIsNull() {
        Class1 class1 = new Class1();
        Class2 class2 = new Class2();
        class2.setId(2);

        mapper.map(class1, class2);

        Assert.assertEquals(2, class2.getId());
    }

}
...