/comments エンドポイントの実装

[yas-ako]

close #44

• まだ途中です

進捗

• domainの実装は完了
• Repositoryの実装は完了
• usecaseの実装の途中
• handlerはまだ

聞きたいこと

openapiの定義によれば、/comment エンドポイントにリクエストがあったとき、それが存在しないItemだった際は 404 エラーを返すことになっています。これを usecase/comment で実装しようとしたのですが、どのように実装すればいいのかがあまりよくわかっていないです。usecase/comment.goに次のように書いて依存先を追加する感じで良いのでしょうか？

type commentUseCase struct {
    CommentRepository domain.CommentRepository
    ItemRepository domain.ItemRepository // ← この行を追加する
}

そのうえで、

func NewCommentUseCase(commentRepository domain.CommentRepository, itemRepository domain.ItemRepository) CommentUseCase {
    return &commentUseCase{
        commentRepository: commentRepository,
        itemRepository:    itemRepository,
    }
}

func (u *commentUseCase) CreateComment(itemId int, userId string, text string) (*domain.Comment, error) {
    _, err := u.itemRepo.GetByID(itemId)
    // 省略
}

とする感じでしょうか？
書いたコードをまだ一度も実行していないため、正しく動くかはわかっていません。

[coderabbitai]

Important

Review skipped

Draft detected.

Please check the settings in the CodeRabbit UI or the .coderabbit.yaml file in this repository. To trigger a single review, invoke the @coderabbitai review command.

You can disable this status message by setting the reviews.review_status to false in the CodeRabbit configuration file.

Use the checkbox below for a quick retry:

• 🔍 Trigger review

✨ Finishing Touches

🧪 Generate unit tests (beta)

• Create PR with unit tests
• Post copyable unit tests in a comment
• Commit unit tests in branch feat/post-comment

Tip

Try Coding Plans. Let us write the prompt for your AI agent so you can ship faster (with fewer bugs).
Share your feedback on Discord.

---

Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

• X
• Mastodon
• Reddit
• LinkedIn

_{Comment @coderabbitai help to get the list of available commands and usage tips.}

[kavos113]

実装ありがとうございます。方針はとてもよいと思います！

質問については，その通りでusecaseにItemRepositoryの依存を加える形で実装を進めてください。
usecaseやhandlerはそれぞれ1つのrepo/usecaseに依存するのではなく，必要に応じて複数のrepo/usecaseに依存します。

go clean architecture などで調べると参考になりそうなコードが出てくるかもしれないので余裕があったら調べてみてください。