防止“持久对象异常”

我有一个非常基本的 JAX-RS 服务（BookService下面的类）允许创建类型的实体Book（也见下文）。POST有效负载

{
    "acquisitionDate": 1418849700000,
    "name": "Funny Title",
    "numberOfPages": 100
}

成功地坚持了Book并返回201 CREATED。然而，包括一个id有效负载上具有任何非空值的属性都会触发org.hibernate.PersistentObjectException与消息detached entity passed to persist. 我明白这意味着什么，并包括一个id创建对象时（在本例中）在有效负载上没有任何意义。但是，我更愿意防止此异常一直冒泡，并向我的用户展示，例如400 BAD REQUEST在这种情况下（或者至少完全忽略该属性）。然而，有两个主要问题：

1. 到达的异常create is an EJBTransactionRolledbackException我必须沿着堆栈跟踪一路爬行才能发现根本原因；
2. 根本原因是org.hibernate.PersistentObjectException- 我正在部署到使用 Hibernate 的 Wildfly，但我想保持我的代码可移植，所以我真的不想捕获这个特定的异常。

据我了解，有两种可能的解决方案：

1. Use book.setId(null) before bookRepo.create(book)。这会忽略这样一个事实：id属性携带一个值并继续请求。
2. 检查是否book.getId() != null并抛出类似的东西IllegalArgumentException可以映射到400状态码。似乎是更好的解决方案。

然而，来自其他框架（例如 Django Rest Framework），我真的更喜欢由框架本身来处理这个问题...我的问题是，是否有任何内置方法可以实现我可以实现的这种行为失踪了吗？

这是BookService class:

@Stateless
@Path("/books")
public class BookService {
    @Inject
    private BookRepo bookRepo;

    @Context
    UriInfo uriInfo;

    @Consumes(MediaType.APPLICATION_JSON)
    @Path("/")
    @POST
    @Produces(MediaType.APPLICATION_JSON)
    public Response create(@Valid Book book) {
        bookRepo.create(book);
        return Response.created(getBookUri(book)).build();
    }

    private URI getBookUri(Book book) {
        return uriInfo.getAbsolutePathBuilder()
                .path(book.getId().toString()).build();
    }
}

这是Book class:

@Entity
@Table(name = "books")
public class Book {
    @Column(nullable = false)
    @NotNull
    @Temporal(TemporalType.TIMESTAMP)
    private Date acquisitionDate;

    @Column(nullable = false, updatable = false)
    @GeneratedValue(strategy = GenerationType.IDENTITY)
    @Id
    private Integer id;

    @Column(nullable = false)
    @NotNull
    @Size(max = 255, min = 1)
    private String name;

    @Column(nullable = false)
    @Min(value = 1)
    @NotNull
    private Integer numberOfPages;

    (getters/setters/...)
}

这是BookRepo class:

@Stateless
public class BookRepo {
    @PersistenceContext(unitName = "book-repo")
    protected EntityManager em;

    public void create(Book book) {
        em.persist(book);
    }
}

我不知道这是否真的是您正在寻找的答案，但我只是在尝试这个想法并实现了一些东西。

JAX-RS 2 规范定义了 bean 验证的模型，所以我想也许您可以利用它。所有错误的验证都会映射到 400。您说过“我希望防止此异常一直冒泡，并向我的用户显示，例如 400 BAD REQUEST”，但是如果验证不好，你无论如何都会得到它。因此，无论您计划如何处理验证异常（如果有的话），您都可以在此处执行相同的操作。

基本上我只是创建了一个约束注释来验证 id 字段中的空值。您可以通过以下方式在注释中定义 id 字段的名称idField注释属性，所以你不限于id。此外，这也可以用于其他对象，因此您不必像第二个解决方案中所建议的那样重复检查该值。

你可以玩玩它。只是想我会把这个选项扔掉。

import java.lang.annotation.ElementType;
import java.lang.annotation.Retention;
import static java.lang.annotation.RetentionPolicy.RUNTIME;
import java.lang.annotation.Target;
import java.lang.reflect.Field;
import java.util.logging.Level;
import java.util.logging.Logger;
import javax.validation.Constraint;
import javax.validation.ConstraintValidator;
import javax.validation.ConstraintValidatorContext;
import javax.validation.Payload;

@Constraint(validatedBy = NoId.NoIdValidator.class)
@Target({ElementType.PARAMETER})
@Retention(RUNTIME)
public @interface NoId {

    String message() default "Cannot have value for id attribute";

    Class<?>[] groups() default {};

    Class<? extends Payload>[] payload() default {};

    String idField() default "id";

    public static class NoIdValidator implements ConstraintValidator<NoId, Object> {
        private String idField;

        @Override
        public void initialize(NoId annotation) {
            idField = annotation.idField();
        }

        @Override
        public boolean isValid(Object bean, ConstraintValidatorContext cvc) {

            boolean isValid = false;
            try {
                Field field = bean.getClass().getDeclaredField(idField);
                if (field == null) {
                    isValid = true;
                } else {
                    field.setAccessible(true);
                    Object value = field.get(bean);
                    if (value == null) {
                        isValid = true;
                    }
                }
            } catch (NoSuchFieldException 
                    | SecurityException 
                    | IllegalArgumentException 
                    | IllegalAccessException ex) {
                Logger.getLogger(NoId.class.getName()).log(Level.SEVERE, null, ex);
            }
            return isValid;
        }
    }
}

Usage:

@POST
@Consumes(MediaType.APPLICATION_JSON)
public Response createBook(@Valid @NoId(idField = "id") Book book) {
    book.setId(1);
    return Response.created(URI.create("http://blah.com/books/1"))
            .entity(book).build();
}

注意默认的idField is id，所以如果你不指定它，它会寻找id对象类中的字段。您还可以指定message就像任何其他约束注释一样：

@NoId(idField = "bookId", message = "bookId must not be specified")
                          // default "Cannot have value for id attribute"