ChatGPT解决这个技术问题 Extra ChatGPT

如何在返回 String 的 Spring MVC @ResponseBody 方法中响应 HTTP 400 错误

我将 Spring MVC 用于简单的 JSON API,使用基于 @ResponseBody 的方法,如下所示。 (我已经有一个直接生成 JSON 的服务层。)

@RequestMapping(value = "/matches/{matchId}", produces = "application/json")
@ResponseBody
public String match(@PathVariable String matchId) {
    String json = matchService.getMatchJson(matchId);
    if (json == null) {
        // TODO: how to respond with e.g. 400 "bad request"?
    }
    return json;
}

在给定的场景中,响应 HTTP 400 错误的最简单、最干净的方法是什么?

我确实遇到过以下方法:

return new ResponseEntity(HttpStatus.BAD_REQUEST);

...但我不能在这里使用它,因为我的方法的返回类型是字符串,而不是 ResponseEntity。


P
Peter Mortensen

将您的返回类型更改为 ResponseEntity<>,然后您可以将以下内容用于 400:

return new ResponseEntity<>(HttpStatus.BAD_REQUEST);

对于正确的请求:

return new ResponseEntity<>(json,HttpStatus.OK);

在 Spring 4.1 之后,ResponseEntity 中有一些辅助方法可以用作:

return ResponseEntity.status(HttpStatus.BAD_REQUEST).body(null);

return ResponseEntity.ok(json);

啊,所以你也可以像这样使用 ResponseEntity。这很好用,只是对原始代码的一个简单更改——谢谢!
欢迎您随时添加自定义标头检查 ResponseEntity 的所有构造函数
如果你传回字符串以外的东西怎么办?就像在 POJO 或其他对象中一样?
它将是 'ResponseEntity'
使用这种方法,您不再需要 @ResponseBody 注释
P
Peter Mortensen

像这样的东西应该可以工作,但我不确定是否有更简单的方法:

@RequestMapping(value = "/matches/{matchId}", produces = "application/json")
@ResponseBody
public String match(@PathVariable String matchId, @RequestBody String body,
            HttpServletRequest request, HttpServletResponse response) {
    String json = matchService.getMatchJson(matchId);
    if (json == null) {
        response.setStatus( HttpServletResponse.SC_BAD_REQUEST  );
    }
    return json;
}

谢谢!这很有效,也很简单。 (在这种情况下,可以通过删除未使用的 bodyrequest 参数来进一步简化。)
P
Peter Mortensen

这不一定是最紧凑的方式,但在我看来很干净:

if(json == null) {
    throw new BadThingException();
}
...

@ExceptionHandler(BadThingException.class)
@ResponseStatus(value = HttpStatus.BAD_REQUEST)
public @ResponseBody MyError handleException(BadThingException e) {
    return new MyError("That doesn’t work");
}

如果使用 Spring 3.1+,您可以在异常处理程序方法中使用 @ResponseBody,否则使用 ModelAndView 或其他东西。

@ResponseBody does not work with @ExceptionHandler [SPR-6902] #11567


抱歉,这似乎不起作用。它会在日志中产生带有长堆栈跟踪的 HTTP 500“服务器错误”:ERROR org.springframework.web.servlet.mvc.method.annotation.ExceptionHandlerExceptionResolver - Failed to invoke @ExceptionHandler method: public controller.TestController$MyError controller.TestController.handleException(controller.TestController$BadThingException) org.springframework.web.HttpMediaTypeNotAcceptableException: Could not find acceptable representation 答案中是否缺少某些内容?
另外,我没有完全理解定义另一个自定义类型(MyError)的意义。那有必要吗?我正在使用最新的 Spring (3.2.2)。
这个对我有用。我改用 javax.validation.ValidationException。 (春季 3.1.4)
这在你的服务和客户端之间有一个中间层的情况下非常有用,中间层有自己的错误处理能力。谢谢你这个例子@Zutty
这应该是公认的答案,因为它将异常处理代码移出正常流程并隐藏了 HttpServlet*
P
Peter Mortensen

我会稍微改变实现:

首先,我创建一个 UnknownMatchException

@ResponseStatus(HttpStatus.NOT_FOUND)
public class UnknownMatchException extends RuntimeException {
    public UnknownMatchException(String matchId) {
        super("Unknown match: " + matchId);
    }
}

注意 @ResponseStatus 的使用,它将被 Spring 的 ResponseStatusExceptionResolver 识别。如果抛出异常,它将创建一个具有相应响应状态的响应。 (我还冒昧地将状态代码更改为 404 - Not Found,我认为这更适合此用例,但如果您愿意,可以坚持使用 HttpStatus.BAD_REQUEST。)

接下来,我会将 MatchService 更改为具有以下签名:

interface MatchService {
    public Match findMatch(String matchId);
}

最后,我将更新控制器并委托给 Spring 的 MappingJackson2HttpMessageConverter 以自动处理 JSON 序列化(如果您将 Jackson 添加到类路径并将 @EnableWebMvc<mvc:annotation-driven /> 添加到配置中,则默认添加它。请参阅 { 1}):

@RequestMapping(value = "/matches/{matchId}", produces = MediaType.APPLICATION_JSON_VALUE)
@ResponseBody
public Match match(@PathVariable String matchId) {
    // Throws an UnknownMatchException if the matchId is not known
    return matchService.findMatch(matchId);
}

请注意,将域对象与视图对象或 DTO 对象分开是很常见的。这可以通过添加一个返回可序列化 JSON 对象的小型 DTO 工厂轻松实现:

@RequestMapping(value = "/matches/{matchId}", produces = MediaType.APPLICATION_JSON_VALUE)
@ResponseBody
public MatchDTO match(@PathVariable String matchId) {
    Match match = matchService.findMatch(matchId);
    return MatchDtoFactory.createDTO(match);
}

我有 500 个并且我记录:2015 年 5 月 28 日下午 5:23:31 org.apache.cxf.interceptor.AbstractFaultChainInitiatorObserver onMessage 严重:错误处理期间发生错误,放弃! org.apache.cxf.interceptor.Fault
完美的解决方案,我只想补充一点,我希望 DTO 是 Match 和其他一些对象的组合。
d
danidemi

这是一种不同的方法。创建一个带有 @ResponseStatus 注释的自定义 Exception,如下所示。

@ResponseStatus(code = HttpStatus.NOT_FOUND, reason = "Not Found")
public class NotFoundException extends Exception {

    public NotFoundException() {
    }
}

并在需要时扔掉。

@RequestMapping(value = "/matches/{matchId}", produces = "application/json")
@ResponseBody
public String match(@PathVariable String matchId) {
    String json = matchService.getMatchJson(matchId);
    if (json == null) {
        throw new NotFoundException();
    }
    return json;
}

这种方法允许您在堆栈跟踪中的任何位置终止执行,而不必返回一个“特殊值”,该值应该指定您希望返回的 HTTP 状态代码。
链接(有效地)断开了。上面没有任何关于异常的内容。
P
Peter Mortensen

最简单的方法是抛出 ResponseStatusException

@RequestMapping(value = "/matches/{matchId}", produces = "application/json")
@ResponseBody
public String match(@PathVariable String matchId, @RequestBody String body) {
    String json = matchService.getMatchJson(matchId);
    if (json == null) {
        throw new ResponseStatusException(HttpStatus.NOT_FOUND);
    }
    return json;
}

最佳答案:无需更改返回类型,也无需创建自己的异常。此外,如果需要,ResponseStatusException 允许添加原因消息。
需要注意的是 ResponseStatusException 仅在 Spring 5+ 版本中可用
这个答案应该在最上面
您不能将 JSON 正文作为响应。
这应该标记为答案,在 Web 控制器上很有用,无需更改返回类型。
N
Norris

正如一些答案中提到的,可以为要返回的每个 HTTP 状态创建一个异常类。我不喜欢必须为每个项目的每个状态创建一个类的想法。这是我想出的。

创建一个接受 HTTP 状态的通用异常

创建 Controller Advice 异常处理程序

让我们来看看代码

package com.javaninja.cam.exception;

import org.springframework.http.HttpStatus;


/**
 * The exception used to return a status and a message to the calling system.
 * @author norrisshelton
 */
@SuppressWarnings("ClassWithoutNoArgConstructor")
public class ResourceException extends RuntimeException {

    private HttpStatus httpStatus = HttpStatus.INTERNAL_SERVER_ERROR;

    /**
     * Gets the HTTP status code to be returned to the calling system.
     * @return http status code.  Defaults to HttpStatus.INTERNAL_SERVER_ERROR (500).
     * @see HttpStatus
     */
    public HttpStatus getHttpStatus() {
        return httpStatus;
    }

    /**
     * Constructs a new runtime exception with the specified HttpStatus code and detail message.
     * The cause is not initialized, and may subsequently be initialized by a call to {@link #initCause}.
     * @param httpStatus the http status.  The detail message is saved for later retrieval by the {@link
     *                   #getHttpStatus()} method.
     * @param message    the detail message. The detail message is saved for later retrieval by the {@link
     *                   #getMessage()} method.
     * @see HttpStatus
     */
    public ResourceException(HttpStatus httpStatus, String message) {
        super(message);
        this.httpStatus = httpStatus;
    }
}

然后我创建一个控制器建议类

package com.javaninja.cam.spring;


import com.javaninja.cam.exception.ResourceException;

import org.springframework.http.ResponseEntity;
import org.springframework.web.bind.annotation.ExceptionHandler;


/**
 * Exception handler advice class for all SpringMVC controllers.
 * @author norrisshelton
 * @see org.springframework.web.bind.annotation.ControllerAdvice
 */
@org.springframework.web.bind.annotation.ControllerAdvice
public class ControllerAdvice {

    /**
     * Handles ResourceExceptions for the SpringMVC controllers.
     * @param e SpringMVC controller exception.
     * @return http response entity
     * @see ExceptionHandler
     */
    @ExceptionHandler(ResourceException.class)
    public ResponseEntity handleException(ResourceException e) {
        return ResponseEntity.status(e.getHttpStatus()).body(e.getMessage());
    }
}

使用它

throw new ResourceException(HttpStatus.BAD_REQUEST, "My message");

http://javaninja.net/2016/06/throwing-exceptions-messages-spring-mvc-controller/


非常好的方法.. 而不是一个简单的字符串,我更喜欢返回一个带有 errorCode 和消息字段的 JSON..
这应该是正确的答案,具有自定义状态代码和消息的通用和全局异常处理程序:D
浏览器不链接链接:“警告:潜在的安全风险……javaninja.net 的证书已于 2021 年 11 月 18 日过期。”
P
Peter Mortensen

我在我的 Spring Boot 应用程序中使用它:

@RequestMapping(value = "/matches/{matchId}", produces = "application/json")
@ResponseBody
public ResponseEntity<?> match(@PathVariable String matchId, @RequestBody String body,
            HttpServletRequest request, HttpServletResponse response) {

    Product p;
    try {
      p = service.getProduct(request.getProductId());
    } catch(Exception ex) {
       return new ResponseEntity<String>(HttpStatus.BAD_REQUEST);
    }

    return new ResponseEntity(p, HttpStatus.OK);
}

一个解释将是有序的。例如,想法/要点是什么?来自 the Help Center“...总是解释为什么你提出的解决方案是合适的以及它是如何工作的”。请通过 editing (changing) your answer 回复,而不是在评论中(没有“编辑:”、“更新:”或类似的 - 答案应该看起来好像是今天写的)。
E
EpicPandaForce

使用 Spring Boot,我不完全确定为什么这是必要的(即使在 @ExceptionHandler 上定义了 @ResponseBody,我也得到了 /error 回退),但以下内容本身不起作用:

@ResponseBody
@ResponseStatus(HttpStatus.BAD_REQUEST)
@ExceptionHandler(IllegalArgumentException.class)
public ErrorMessage handleIllegalArguments(HttpServletRequest httpServletRequest, IllegalArgumentException e) {
    log.error("Illegal arguments received.", e);
    ErrorMessage errorMessage = new ErrorMessage();
    errorMessage.code = 400;
    errorMessage.message = e.getMessage();
    return errorMessage;
}

它仍然抛出异常,显然是因为没有可生产的媒体类型被定义为请求属性:

// AbstractMessageConverterMethodProcessor
@SuppressWarnings("unchecked")
protected <T> void writeWithMessageConverters(T value, MethodParameter returnType,
        ServletServerHttpRequest inputMessage, ServletServerHttpResponse outputMessage)
        throws IOException, HttpMediaTypeNotAcceptableException, HttpMessageNotWritableException {

    Class<?> valueType = getReturnValueType(value, returnType);
    Type declaredType = getGenericType(returnType);
    HttpServletRequest request = inputMessage.getServletRequest();
    List<MediaType> requestedMediaTypes = getAcceptableMediaTypes(request);
    List<MediaType> producibleMediaTypes = getProducibleMediaTypes(request, valueType, declaredType);
if (value != null && producibleMediaTypes.isEmpty()) {
        throw new IllegalArgumentException("No converter found for return value of type: " + valueType);   // <-- throws
    }

// ....

@SuppressWarnings("unchecked")
protected List<MediaType> getProducibleMediaTypes(HttpServletRequest request, Class<?> valueClass, Type declaredType) {
    Set<MediaType> mediaTypes = (Set<MediaType>) request.getAttribute(HandlerMapping.PRODUCIBLE_MEDIA_TYPES_ATTRIBUTE);
    if (!CollectionUtils.isEmpty(mediaTypes)) {
        return new ArrayList<MediaType>(mediaTypes);

所以我添加了它们。

@ResponseBody
@ResponseStatus(HttpStatus.BAD_REQUEST)
@ExceptionHandler(IllegalArgumentException.class)
public ErrorMessage handleIllegalArguments(HttpServletRequest httpServletRequest, IllegalArgumentException e) {
    Set<MediaType> mediaTypes = new HashSet<>();
    mediaTypes.add(MediaType.APPLICATION_JSON_UTF8);
    httpServletRequest.setAttribute(HandlerMapping.PRODUCIBLE_MEDIA_TYPES_ATTRIBUTE, mediaTypes);
    log.error("Illegal arguments received.", e);
    ErrorMessage errorMessage = new ErrorMessage();
    errorMessage.code = 400;
    errorMessage.message = e.getMessage();
    return errorMessage;
}

这让我通过了“支持的兼容媒体类型”,但它仍然无法正常工作,因为我的 ErrorMessage 有问题:

public class ErrorMessage {
    int code;

    String message;
}

JacksonMapper 没有将其处理为“可转换”,所以我不得不添加 getter/setter,并且我还添加了 @JsonProperty 注释

public class ErrorMessage {
    @JsonProperty("code")
    private int code;

    @JsonProperty("message")
    private String message;

    public int getCode() {
        return code;
    }

    public void setCode(int code) {
        this.code = code;
    }

    public String getMessage() {
        return message;
    }

    public void setMessage(String message) {
        this.message = message;
    }
}

然后我按预期收到了我的消息

{"code":400,"message":"An \"url\" parameter must be defined."}

P
Peter Mortensen

另一种方法是使用 @ExceptionHandler@ControllerAdvice 将所有处理程序集中在同一个类中。如果没有,您必须将处理程序方法放在要为其管理异常的每个控制器中。

您的处理程序类:

@ControllerAdvice
public class MyExceptionHandler extends ResponseEntityExceptionHandler {

  @ExceptionHandler(MyBadRequestException.class)
  public ResponseEntity<MyError> handleException(MyBadRequestException e) {
    return ResponseEntity
        .badRequest()
        .body(new MyError(HttpStatus.BAD_REQUEST, e.getDescription()));
  }
}

您的自定义异常:

public class MyBadRequestException extends RuntimeException {

  private String description;

  public MyBadRequestException(String description) {
    this.description = description;
  }

  public String getDescription() {
    return this.description;
  }
}

现在您可以从任何控制器抛出异常,并且可以在您的建议类中定义其他处理程序。


P
Peter Mortensen

无需显式返回 ResponseEntity 即可在控制器中处理异常的最简单、最简洁的方法是添加 @ExceptionHandler 方法。

使用 Spring Boot 2.0.3.RELEASE 的示例片段:

// Prefer static import of HttpStatus constants as it's cleaner IMHO

// Handle with no content returned
@ExceptionHandler(IllegalArgumentException.class)
@ResponseStatus(BAD_REQUEST)
void onIllegalArgumentException() {}

// Return 404 when JdbcTemplate does not return a single row
@ExceptionHandler(IncorrectResultSizeDataAccessException.class)
@ResponseStatus(NOT_FOUND)
void onIncorrectResultSizeDataAccessException() {}

// Catch all handler with the exception as content
@ExceptionHandler(Exception.class)
@ResponseStatus(I_AM_A_TEAPOT)
@ResponseBody Exception onException(Exception e) {
  return e;
}

作为旁白:

如果在所有上下文/用法中,matchService.getMatchJson(matchId) == null 无效,那么我的建议是让 getMatchJson 抛出异常,例如 IllegalArgumentException 而不是返回 null 并让它冒泡到控制器的 @ExceptionHandler。

如果 null 用于测试其他条件,那么我将有一个特定的方法,例如 matchService.hasMatchJson(matchId)。一般来说,如果可能,我会避免 null 以避免意外的 NullPointerException。


P
Peter Mortensen

您也可以只throw new HttpMessageNotReadableException("error description")从 Spring 的 default error handling 中受益。

但是,就像那些默认错误一样,不会设置响应正文。

我发现这些在拒绝可能仅是手工制作的请求时很有用,这可能表明存在恶意意图,因为它们掩盖了基于更深层次的自定义验证及其标准拒绝请求的事实。


HttpMessageNotReadableException("error description") 已弃用。
P
Peter Mortensen

使用带有状态代码的自定义响应。

像这样:

class Response<T>(
    val timestamp: String = DateTimeFormatter
            .ofPattern("yyyy-MM-dd HH:mm:ss.SSSSSS")
            .withZone(ZoneOffset.UTC)
            .format(Instant.now()),
    val code: Int = ResultCode.SUCCESS.code,
    val message: String? = ResultCode.SUCCESS.message,
    val status: HttpStatus = HttpStatus.OK,
    val error: String? = "",
    val token: String? = null,
    val data: T? = null
) : : ResponseEntity<Response.CustomResponseBody>(status) {

data class CustomResponseBody(
    val timestamp: String = DateTimeFormatter
            .ofPattern("yyyy-MM-dd HH:mm:ss.SSSSSS")
            .withZone(ZoneOffset.UTC)
            .format(Instant.now()),
    val code: Int = ResultCode.SUCCESS.code,
    val message: String? = ResultCode.SUCCESS.message,
    val error: String? = "",
    val token: String? = null,
    val data: Any? = null
)

override fun getBody(): CustomResponseBody? = CustomResponseBody(timestamp, code, message, error, token, data)