如何验证@RequestParams 不为空?
How do I validate that the @RequestParams are not empty?
我有一个计算器服务,可以从用户那里获取操作类型 num1 和 num2。我需要验证用户是否实际输入了这些值,而不是将其留空。
@RequestMapping(value = "/calculate")
@ResponseBody
public CalculationResult calculate(@RequestParam(name = "op") String operation, @RequestParam(name = "num1") Double num1, @RequestParam(name = "num2") Double num2) {
System.out.print("Operation:" + operation);
Double calculate = calculatorService.calculate(operation, num1, num2);
return new CalculationResult(calculate);
}
我有一个集成测试需要通过,因为它当前失败并出现错误:
{\"timestamp\":1488875777084,\"status\":400,\"error\":\"Bad
Request\",\"exception\":\"org.springframework.web.method.annotation.MethodArgumentTypeMismatchException\",\"message\":\"Failed
to convert value of type 'java.lang.String' to required type
'java.lang.Double';
下面是我的测试用例:
@Test
public void validates_all_parameters_are_set() throws Exception {
ResponseEntity<String> response = template.getForEntity( "/calculate?op=&num1=&num2=",
String.class);
assertThat(response.getStatusCode(), equalTo(HttpStatus.BAD_REQUEST));
assertThat(response.getBody(), equalTo("{\"error\":\"At least one parameter is invalid or not supplied\"}"));
}
我不知道如何验证这一点。
您目前没有检查数值;您可以将代码更改为:
import org.springframework.http.HttpStatus;
import org.springframework.http.ResponseEntity;
@RequestMapping(value = "/calculate")
@ResponseBody
public ResponseEntity<CalculationResult> calculate(@RequestParam(name = "op") String operation,
@RequestParam(name = "num1") Double num1,
@RequestParam(name = "num2") Double num2) {
if(null == op || null == num1 || null == num2) {
throw new IllegalArgumentException("{\"error\":\"At least one parameter is invalid or not supplied\"}")
}
System.out.print("Operation:" + operation);
Double calculate = calculatorService.calculate(operation, num1, num2);
return new ResponseEntity<>(new CalculationResult(calculate), HttpStatus.OK);
}
@ExceptionHandler(IllegalArgumentException.class)
@ResponseStatus(HttpStatus.BAD_REQUEST)
public final String exceptionHandlerIllegalArgumentException(final IllegalArgumentException e) {
return '"' + e.getMessage() + '"';
}
我早就回答过类似的问题,你也可以照着写你的测试,如下:
@Validated
public class CalculationController {
@RequestMapping(value = "/calculate")
@ResponseBody
public CalculationResult calculate(
@Valid @NotBlank @RequestParam(name = "op") String operation,
@Valid @NotNull @RequestParam(name = "num1") Double num1,
@Valid @NotNull @RequestParam(name = "num2") Double num2) {
System.out.print("Operation:" + operation);
Double calculate = calculatorService.calculate(operation, num1, num2);
return new CalculationResult(calculate);
}
}
对应的@Test应该修改为测试"may not be null"消息数组,如:
@Test
public void validates_all_parameters_are_set() throws Exception {
ResponseEntity<String> response = template.getForEntity( "/calculate?op=&num1=&num2=",
String.class);
assertThat(response.getStatusCode(), equalTo(HttpStatus.BAD_REQUEST));
assertThat(response.getBody(), equalTo("{\"error\":[\"may not be null\",\"may not be null\"]}"));
}
@RestController
@RequestMapping("/")
@Validated
public class Controller {
@GetMapping("/endpoint")
public String getEndpoint(@RequestParam @NotBlank String name) {}
}
我有一个计算器服务,可以从用户那里获取操作类型 num1 和 num2。我需要验证用户是否实际输入了这些值,而不是将其留空。
@RequestMapping(value = "/calculate")
@ResponseBody
public CalculationResult calculate(@RequestParam(name = "op") String operation, @RequestParam(name = "num1") Double num1, @RequestParam(name = "num2") Double num2) {
System.out.print("Operation:" + operation);
Double calculate = calculatorService.calculate(operation, num1, num2);
return new CalculationResult(calculate);
}
我有一个集成测试需要通过,因为它当前失败并出现错误:
{\"timestamp\":1488875777084,\"status\":400,\"error\":\"Bad Request\",\"exception\":\"org.springframework.web.method.annotation.MethodArgumentTypeMismatchException\",\"message\":\"Failed to convert value of type 'java.lang.String' to required type 'java.lang.Double';
下面是我的测试用例:
@Test
public void validates_all_parameters_are_set() throws Exception {
ResponseEntity<String> response = template.getForEntity( "/calculate?op=&num1=&num2=",
String.class);
assertThat(response.getStatusCode(), equalTo(HttpStatus.BAD_REQUEST));
assertThat(response.getBody(), equalTo("{\"error\":\"At least one parameter is invalid or not supplied\"}"));
}
我不知道如何验证这一点。
您目前没有检查数值;您可以将代码更改为:
import org.springframework.http.HttpStatus;
import org.springframework.http.ResponseEntity;
@RequestMapping(value = "/calculate")
@ResponseBody
public ResponseEntity<CalculationResult> calculate(@RequestParam(name = "op") String operation,
@RequestParam(name = "num1") Double num1,
@RequestParam(name = "num2") Double num2) {
if(null == op || null == num1 || null == num2) {
throw new IllegalArgumentException("{\"error\":\"At least one parameter is invalid or not supplied\"}")
}
System.out.print("Operation:" + operation);
Double calculate = calculatorService.calculate(operation, num1, num2);
return new ResponseEntity<>(new CalculationResult(calculate), HttpStatus.OK);
}
@ExceptionHandler(IllegalArgumentException.class)
@ResponseStatus(HttpStatus.BAD_REQUEST)
public final String exceptionHandlerIllegalArgumentException(final IllegalArgumentException e) {
return '"' + e.getMessage() + '"';
}
我早就回答过类似的问题
@Validated
public class CalculationController {
@RequestMapping(value = "/calculate")
@ResponseBody
public CalculationResult calculate(
@Valid @NotBlank @RequestParam(name = "op") String operation,
@Valid @NotNull @RequestParam(name = "num1") Double num1,
@Valid @NotNull @RequestParam(name = "num2") Double num2) {
System.out.print("Operation:" + operation);
Double calculate = calculatorService.calculate(operation, num1, num2);
return new CalculationResult(calculate);
}
}
对应的@Test应该修改为测试"may not be null"消息数组,如:
@Test
public void validates_all_parameters_are_set() throws Exception {
ResponseEntity<String> response = template.getForEntity( "/calculate?op=&num1=&num2=",
String.class);
assertThat(response.getStatusCode(), equalTo(HttpStatus.BAD_REQUEST));
assertThat(response.getBody(), equalTo("{\"error\":[\"may not be null\",\"may not be null\"]}"));
}
@RestController
@RequestMapping("/")
@Validated
public class Controller {
@GetMapping("/endpoint")
public String getEndpoint(@RequestParam @NotBlank String name) {}
}