From 208872fc3126c29cf817e7fe06873897135a8123 Mon Sep 17 00:00:00 2001 From: "zhang.pengcheng" Date: Mon, 16 Jun 2025 10:41:47 +0900 Subject: [PATCH] =?UTF-8?q?(review)=20stocks=E3=81=AE=E3=82=B3=E3=83=A1?= =?UTF-8?q?=E3=83=B3=E3=83=88=E3=82=92=E4=BF=AE=E6=AD=A3=E3=81=97=E3=81=BE?= =?UTF-8?q?=E3=81=97=E3=81=9F?= MIME-Version: 1.0 Content-Type: text/plain; charset=UTF-8 Content-Transfer-Encoding: 8bit --- .../todoapp/controller/StocksController.java | 9 ++++----- .../example/todoapp/dto/DeleteStockRequest.java | 15 --------------- ...ockRequest.java => UpdateStockRequestDTO.java} | 4 ++-- .../com/example/todoapp/model/RecipeStuffs.java | 2 -- .../java/com/example/todoapp/model/Recipes.java | 1 - .../java/com/example/todoapp/model/Stocks.java | 2 -- .../repository/RecipeStuffsRepository.java | 11 ----------- .../example/todoapp/service/StocksService.java | 6 +++--- 8 files changed, 9 insertions(+), 41 deletions(-) delete mode 100644 backend/src/main/java/com/example/todoapp/dto/DeleteStockRequest.java rename backend/src/main/java/com/example/todoapp/dto/{UpdateStockRequest.java => UpdateStockRequestDTO.java} (84%) diff --git a/backend/src/main/java/com/example/todoapp/controller/StocksController.java b/backend/src/main/java/com/example/todoapp/controller/StocksController.java index c419af8..b11844e 100644 --- a/backend/src/main/java/com/example/todoapp/controller/StocksController.java +++ b/backend/src/main/java/com/example/todoapp/controller/StocksController.java @@ -3,7 +3,7 @@ package com.example.todoapp.controller; import com.example.todoapp.dto.DeleteStockRequestDTO; import com.example.todoapp.dto.StockResponseDTO; import com.example.todoapp.dto.StockDTO; -import com.example.todoapp.dto.UpdateStockRequest; +import com.example.todoapp.dto.UpdateStockRequestDTO; import com.example.todoapp.model.Stocks; import com.example.todoapp.model.User; import com.example.todoapp.repository.UserRepository; @@ -88,14 +88,13 @@ public class StocksController { * 指定されたIDの在庫を更新する * * @param authentication 認証情報 - * @param stockId 更新する在庫のID - * @param stockDetails 更新内容 + * @param updateStockRequest 更新のリクエスト * @return 更新された在庫 */ @PutMapping("/update") public ResponseEntity> updateStock( Authentication authentication, - @Valid @RequestBody UpdateStockRequest updateStockRequest) { + @Valid @RequestBody UpdateStockRequestDTO updateStockRequest) { Stocks updatedStock = stockService.updateStocks(authentication.getName(), updateStockRequest); @@ -117,7 +116,7 @@ public class StocksController { * 指定されたIDの在庫を削除する * * @param authentication 認証情報 - * @param stockInfo 削除する在庫情報（stockIdをキーに持つ） + * @param request 削除のリクエスト * @return レスポンス */ @DeleteMapping("/delete") diff --git a/backend/src/main/java/com/example/todoapp/dto/DeleteStockRequest.java b/backend/src/main/java/com/example/todoapp/dto/DeleteStockRequest.java deleted file mode 100644 index 8beb478..0000000 --- a/backend/src/main/java/com/example/todoapp/dto/DeleteStockRequest.java +++ /dev/null @@ -1,15 +0,0 @@ -package com.example.todoapp.dto; -import lombok.Data; - -/** - * 在庫削除のリクエストオブジェクト（DTO） - *

- * このクラスはクライアントとサーバー間で在庫情報をやり取りするために使用されます。 - * エンティティとは異なり、必要な情報のみを含み、関連エンティティへの参照ではなくIDのみを保持します。 - *

- */ - -@Data -public class DeleteStockRequest { - private Long stockId; -} \ No newline at end of file diff --git a/backend/src/main/java/com/example/todoapp/dto/UpdateStockRequest.java b/backend/src/main/java/com/example/todoapp/dto/UpdateStockRequestDTO.java similarity index 84% rename from backend/src/main/java/com/example/todoapp/dto/UpdateStockRequest.java rename to backend/src/main/java/com/example/todoapp/dto/UpdateStockRequestDTO.java index 68a3105..3d3333a 100644 --- a/backend/src/main/java/com/example/todoapp/dto/UpdateStockRequest.java +++ b/backend/src/main/java/com/example/todoapp/dto/UpdateStockRequestDTO.java @@ -4,7 +4,7 @@ import java.time.LocalDate; import lombok.Data; /** - * 在庫削除のリクエストオブジェクト（DTO） + * 在庫更新のリクエストオブジェクト（DTO） *

* このクラスはクライアントとサーバー間で在庫情報をやり取りするために使用されます。 * エンティティとは異なり、必要な情報のみを含み、関連エンティティへの参照ではなくIDのみを保持します。 @@ -12,7 +12,7 @@ import lombok.Data; */ @Data -public class UpdateStockRequest { +public class UpdateStockRequestDTO { private Long stockId; private int amount; private int price; diff --git a/backend/src/main/java/com/example/todoapp/model/RecipeStuffs.java b/backend/src/main/java/com/example/todoapp/model/RecipeStuffs.java index c21eb9a..c5c930b 100644 --- a/backend/src/main/java/com/example/todoapp/model/RecipeStuffs.java +++ b/backend/src/main/java/com/example/todoapp/model/RecipeStuffs.java @@ -1,8 +1,6 @@ //-------------------------------- // RecipesStuffs.java // -// 分類：社員管理システムV2・ビジネスロジック層 -// // 更新履歴：2025/06/02 新規作成 // Copyright(c) 2025 IVIS All rights reserved. //-------------------------------------------- diff --git a/backend/src/main/java/com/example/todoapp/model/Recipes.java b/backend/src/main/java/com/example/todoapp/model/Recipes.java index ebe5f7e..25207af 100644 --- a/backend/src/main/java/com/example/todoapp/model/Recipes.java +++ b/backend/src/main/java/com/example/todoapp/model/Recipes.java @@ -1,7 +1,6 @@ //-------------------------------- // Recipes.java // -// 分類：社員管理システムV2・ビジネスロジック層 // // 更新履歴：2025/06/02 新規作成 // Copyright(c) 2025 IVIS All rights reserved. diff --git a/backend/src/main/java/com/example/todoapp/model/Stocks.java b/backend/src/main/java/com/example/todoapp/model/Stocks.java index c9c47b2..acc35df 100644 --- a/backend/src/main/java/com/example/todoapp/model/Stocks.java +++ b/backend/src/main/java/com/example/todoapp/model/Stocks.java @@ -1,8 +1,6 @@ //-------------------------------- // Stocks.java // -// 分類：社員管理システムV2・ビジネスロジック層 -// // 更新履歴：2025/06/03 新規作成 // Copyright(c) 2025 IVIS All rights reserved. //-------------------------------------------- diff --git a/backend/src/main/java/com/example/todoapp/repository/RecipeStuffsRepository.java b/backend/src/main/java/com/example/todoapp/repository/RecipeStuffsRepository.java index db92ff1..2a066e5 100644 --- a/backend/src/main/java/com/example/todoapp/repository/RecipeStuffsRepository.java +++ b/backend/src/main/java/com/example/todoapp/repository/RecipeStuffsRepository.java @@ -7,17 +7,6 @@ import org.springframework.data.jpa.repository.JpaRepository; import com.example.todoapp.model.RecipeStuffs; -import jakarta.transaction.Transactional; - -import org.springframework.data.jpa.repository.JpaRepository; -import org.springframework.data.jpa.repository.Modifying; -import org.springframework.data.jpa.repository.Query; -import org.springframework.data.repository.query.Param; -import org.springframework.stereotype.Repository; - -import java.util.List; -import java.util.Optional; - /** * レシピ食材関連データのリポジトリインターフェース *

diff --git a/backend/src/main/java/com/example/todoapp/service/StocksService.java b/backend/src/main/java/com/example/todoapp/service/StocksService.java index faeef4b..9fe9b9c 100644 --- a/backend/src/main/java/com/example/todoapp/service/StocksService.java +++ b/backend/src/main/java/com/example/todoapp/service/StocksService.java @@ -1,6 +1,6 @@ package com.example.todoapp.service; -import com.example.todoapp.dto.UpdateStockRequest; +import com.example.todoapp.dto.UpdateStockRequestDTO; import com.example.todoapp.model.Stocks; import com.example.todoapp.util.MessageUtils; import com.example.todoapp.model.User; @@ -72,11 +72,10 @@ public class StocksService { * 指定された在庫情報を編集する * * @param username ユーザー名 - * @param stockId 変数する在庫ID * @param stockDetails 編集内容(新しい情報) * @return 編集された在庫エンティティ */ - public Stocks updateStocks(String username, UpdateStockRequest stockDetails) { + public Stocks updateStocks(String username, UpdateStockRequestDTO stockDetails) { Stocks stock = getStockById(username, stockDetails.getStockId()); stock.setAmount(stockDetails.getAmount()); stock.setPrice(stockDetails.getPrice()); @@ -91,6 +90,7 @@ public class StocksService { * * @param userId ユーザー名 * @param stockId 削除する在庫のID + * @return 削除されたレコード数 */ public int deleteStockById(Long userId, Long stockId) { return stocksRepository.deleteByUserIdAndStockId(userId, stockId);