実装例にオリジナルの仕様を追加 #1
Conversation
| public class DemoController { | ||
|
|
||
| @GetMapping("/names") | ||
| public List<String> getName(){ | ||
| return List.of("Ichiro","Jiro", "Saburo"); | ||
| public String getName( @RequestParam("id") @Pattern (regexp = "^[1-3]$" ) String id) { |
There was a problem hiding this comment.
たいしたことではないのですが、フォーマットはこのようにするのがbetterです。
| public String getName( @RequestParam("id") @Pattern (regexp = "^[1-3]$" ) String id) { | |
| public String getName(@RequestParam("id") @Pattern(regexp = "^[1-3]$") String id) { |
| public List<String> getName(){ | ||
| return List.of("Ichiro","Jiro", "Saburo"); | ||
| public String getName( @RequestParam("id") @Pattern (regexp = "^[1-3]$" ) String id) { | ||
| Map<Integer, String> namesById = Map.of(1,"Ichiro",2, "Jiro",3,"Saburo"); |
There was a problem hiding this comment.
super
| Map<Integer, String> namesById = Map.of(1,"Ichiro",2, "Jiro",3,"Saburo"); | |
| Map<Integer, String> namesById = Map.of(1, "Ichiro", 2, "Jiro", 3, "Saburo"); |
|
|
||
| @GetMapping("/fruit") | ||
| public List<String> getFruitList(@RequestParam("priceMin") @Pattern (regexp = "^[0-9]{1,4}$" ) String priceMin) { | ||
| Map<String, Integer> fruits = Map.of("apple",100,"orange", 150,"grape",1000); |
There was a problem hiding this comment.
Mapの変数名は
keyToValue
valueByKey
のような形式がよいですね。
fruitsToPriceとかでしょうか。
| } | ||
| }); | ||
| if (fruitsFilterByPrice.isEmpty()){ | ||
| return null; |
There was a problem hiding this comment.
空ならば[]を返すようにしてもいいかと思います。
Collections.emptyList()的なものがあったはず。
もしくはfilterした結果をそのまま返せばよいですね。
| Integer priceMinToInt = Integer.valueOf(priceMin); | ||
| List<String> fruitsFilterByPrice = new ArrayList<>(); | ||
|
|
||
| fruits.forEach((fruitName, fruitPrice) -> { |
|
こちら対応あとでよいです↓
こちらですがレスポンスのステータスコードが500になります。 なんでこうなっているの？と思うはずですが、Spring FrameworkのGitHubのIssueでも話が上がっていますね。 じゃあ、どう実装するかですが、このあたりが参考になると思います。 https://www.baeldung.com/spring-boot-bean-validation ただし、こちら優先度は落としてもらっていいです。 |
| return namesById.get(Integer.valueOf(id)); | ||
| } | ||
|
|
||
| @GetMapping("/fruit") |
指摘事項修正(バリデーションのステータスコード対応除く)
|
ステータスコード対応を除き、いただいた指摘事項対応しました。
|
アレンジした内容
以下2の実行結果を抜粋。