如何將具有相似主體的幾種方法簡化為一個？

我想簡化我的代碼并將幾種方法的公共部分合并到一個新的方法中。你能給我一條建議嗎——最好的方法是什么？我的方法是：

@ExceptionHandler(value = {ProhibitedScimTypeException.class})
public ResponseEntity<ErrorDto> policyConflict(final ProhibitedScimTypeException exception) {
    final var errorDto = new ErrorDto();
    errorDto.setDetail(exception.getMessage());
    errorDto.setStatus(BAD_REQUEST.toString());
    errorDto.setScimType("prohibited");
    return new ResponseEntity<>(errorDto, HttpStatus.BAD_REQUEST);
}

@ExceptionHandler(value = {UserAlreadyExistsException.class})
public ResponseEntity<ErrorDto> userNameExistsConflict(final UserAlreadyExistsException exception) {
    final var errorDto = new ErrorDto();
    errorDto.setDetail(exception.getMessage());
    errorDto.setStatus(CONFLICT.toString());
    errorDto.setScimType("uniqueness");
    return new ResponseEntity<>(errorDto, HttpStatus.CONFLICT);
}

@ExceptionHandler(value = {UserNotFoundException.class})
public ResponseEntity<ErrorDto> userNameNotFoundConflict(final UserNotFoundException exception) {
    final var errorDto = new ErrorDto();
    errorDto.setDetail(exception.getMessage());
    errorDto.setStatus(NOT_FOUND.toString());
    errorDto.setScimType("prohibited");
    return new ResponseEntity<>(errorDto, HttpStatus.NOT_FOUND);
}

我想將共同部分分開：

final var errorDto = new ErrorDto();
errorDto.setDetail(exception.getMessage());
errorDto.setStatus(MEHTOD.toString());
errorDto.setScimType("something");

uj5u.com熱心網友回復：

您可以將公共部分提取到這樣的方法中嗎？

private ResponseEntity<ErrorDto> conflict(final Throwable exception, HttpStatus status, String scrimType) {
    final var errorDto = new ErrorDto();
    errorDto.setDetail(exception.getMessage());
    errorDto.setStatus(status.toString());
    errorDto.setScimType(scrimType);
    return new ResponseEntity<>(errorDto, status);
}

并從您的方法中呼叫它，

@ExceptionHandler(value = {ProhibitedScimTypeException.class})
public ResponseEntity<ErrorDto> policyConflict(final ProhibitedScimTypeException exception) {
    return conflict(exception, HttpStatus.BAD_REQUEST, "prohibited");
}

@ExceptionHandler(value = {UserAlreadyExistsException.class})
public ResponseEntity<ErrorDto> userNameExistsConflict(final UserAlreadyExistsException exception) {
    return conflict(exception, HttpStatus.CONFLICT, "uniqueness");
}

@ExceptionHandler(value = {UserNotFoundException.class})
public ResponseEntity<ErrorDto> userNameNotFoundConflict(final UserNotFoundException exception) {
    return conflict(exception, HttpStatus.NOT_FOUND, "prohibited");
}

uj5u.com熱心網友回復：

創建一個方法：

    private ResponseEntity<ErrorDto> conflict(final Throwable exception, final Object status, final String scimType, final HttpStatus httpStatus) {
      final var errorDto = new ErrorDto();
      errorDto.setDetail(exception.getMessage());
      errorDto.setStatus(status.toString());
      errorDto.setScimType(scimType);
      return new ResponseEntity<>(errorDto, httpStatus);
    }

并像使用它一樣

    @ExceptionHandler(value = {ProhibitedScimTypeException.class})
    public ResponseEntity<ErrorDto> policyConflict(final ProhibitedScimTypeException exception) {
      return this.conflict(exception, BAD_REQUEST, "prohibited", HttpStatus.BAD_REQUEST);
    }

標籤：爪哇休息例外压倒一切

上一篇：使用日期的lt>比較器搜索mongodb資料

下一篇：當資源具有計算屬性時設計RESTPOST端點時的模式