[BE-6] 3層アーキテクチャの設計、ゲーム開始 by yone-al · Pull Request #13 · kc3hack/2026_team10

yone-al · 2026-02-16T10:50:58Z

概要

3層アーキテクチャでの設計
ゲーム開始のレスポンス

動作確認

POST http://localhost:8080/solo へのリクエスト処理

変更内容・エビデンス

{
	"result": {
		"id": 21,
		"hints": [
			"ヒント1",
			"ヒント2",
			"ヒント3"
		]
	}
}

その他

略称一覧

LGTM : Looks Good To Me（レビューアーがPRの内容を確認し、問題がないと判断した際に使用する）
ask : 質問・確認事項（修正は任意、回答や説明を求める）
imo : In My Opinion（個人的な意見、修正は任意）
nits : 些細な指摘（修正は任意、気になった点の共有）
must : 必須の修正事項（マージ前に必ず対応が必要）
memo : メモ（参考情報として共有）

Copilot

Pull request overview

3層アーキテクチャ（controller/service/repository）を導入し、ゲーム開始用のエンドポイント（POST /solo）で「ID + ヒント一覧」を返す土台を追加するPRです。

Changes:

Controller/Service/Repository の各層を新規追加し、POST /solo からゲーム開始レスポンスを返す処理を実装
In-memory リポジトリでラウンド（Round）を生成・保存する処理を追加
モデル定義（Round / StartGameResult）を追加し、main.go でDI配線を実施

Reviewed changes

Copilot reviewed 7 out of 7 changed files in this pull request and generated 8 comments.

Show a summary per file

File	Description
backend/main.go	メモリリポジトリ/サービス/コントローラを生成し、`POST /solo` を追加
backend/controllers/hint_controller.go	`POST /solo` のハンドラでサービスを呼び出しレスポンス返却
backend/services/hint_service.go	固定のお題・ヒントを生成し、リポジトリへ保存して結果を返す
backend/repositories/hint_repository.go	Round の採番・保存を行う in-memory 実装を追加
backend/model/hint.go	Round/StartGameResult の型定義を追加
backend/.serena/project.yml	Serena 用プロジェクト設定を追加
backend/.serena/.gitignore	Serena の cache を無視する設定を追加

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

Copilot · 2026-02-16T11:06:17Z

backend/controllers/hint_controller.go

+
+func (c *HintController) GetHints(ctx *gin.Context) {
+	result, err := c.service.GetHints()
+	if err != nil {


エラー発生時に err を一切利用せず固定メッセージだけ返しているため、サーバ側での原因追跡が困難です。少なくともサーバログへ err を出力する（あるいは gin.Context.Error で付与する）など、運用上追える形にしてください。

Suggested change

if err != nil {

if err != nil {

ctx.Error(err)

Copilot · 2026-02-16T11:06:17Z

backend/main.go

+	rounds := []model.Round{
+		{
+			ID:     1,
+			Answer: "お題",
+			Hints:  []string{"ヒント1", "ヒント2", "ヒント3"},
+		},
+	}
+
+	hintRepository := repositories.NewHintMemoryRepository(rounds)


main.go 側で rounds に固定のお題/ヒントを初期投入している一方、service 側でも同じ固定値を生成しているため、データ定義が二重化しています（また最初の POST /solo の採番が 2 から始まります）。初期データは repository のみで管理する/サービス側に寄せる等、どこが真実のソースかを一本化してください。

Suggested change

rounds := []model.Round{

{

ID: 1,

Answer: "お題",

Hints: []string{"ヒント1", "ヒント2", "ヒント3"},

},

}

hintRepository := repositories.NewHintMemoryRepository(rounds)

hintRepository := repositories.NewHintMemoryRepository([]model.Round{})

@copilot
rounds := []model.Round{}

hintRepository := repositories.NewHintMemoryRepository(rounds)

これでどう？

Copilot · 2026-02-16T11:06:18Z

backend/repositories/hint_repository.go

+func (r *HintMemoryRepository) CreateRound(answer string, hints []string) (*model.Round, error) {
+	newID := len(r.rounds) + 1
+	round := model.Round{
+		ID:     newID,
+		Answer: answer,
+		Hints:  hints,
+	}
+	r.rounds = append(r.rounds, round)
+	return &round, nil


HintMemoryRepository は Gin のハンドラから同時に呼ばれる可能性があり、rounds への append/len がデータレースになります（ID の重複や panic の原因）。mutex 等で保護するか、永続層に切り替えるなどしてスレッドセーフにしてください。

Copilot · 2026-02-16T11:06:18Z

backend/repositories/hint_repository.go

+	newID := len(r.rounds) + 1
+	round := model.Round{
+		ID:     newID,
+		Answer: answer,
+		Hints:  hints,


newID を len(rounds)+1 で作ると、初期データの ID が連番でない/欠番がある場合に ID 衝突が起きます。既存の最大 ID を走査するか、ID 生成を別コンポーネントに寄せるなどして一意性を担保してください。

Copilot · 2026-02-16T11:06:18Z

backend/repositories/hint_repository.go

+		Hints:  hints,
+	}
+	r.rounds = append(r.rounds, round)
+	return &round, nil


CreateRound が返している *Round は、append した要素ではなくローカル変数 round のアドレスです。呼び出し側が返値を保持して更新するケース等で、リポジトリ内のデータと不整合になります。append 後のスライス要素（末尾）を指すポインタを返すようにしてください。

Suggested change

return &round, nil

return &r.rounds[len(r.rounds)-1], nil

Copilot · 2026-02-16T11:06:18Z

backend/services/hint_service.go

+	result, err := s.repository.CreateRound(answer, hints)
+	if err != nil {
+		return nil, errors.New("failed to save hint data")
+	}


repository.CreateRound の err を握りつぶして固定文言に置き換えているため、原因追跡が難しくなります。呼び出し元で扱えるように err をラップして返す（%w）など、元のエラー情報を保持してください。

Copilot · 2026-02-16T11:06:18Z

backend/services/hint_service.go

+	// IDとヒントを返す
+	response := &model.StartGameResult{
+		ID:    result.ID,
+		Hints: hints,


レスポンスの Hints が repository から返された result.Hints ではなくローカル変数 hints を参照しています。将来 repository 側で加工/コピーした場合にレスポンスと保存内容がズレるので、返却値（result）由来のデータを使うようにしてください。

Suggested change

Hints: hints,

Hints: result.Hints,

Copilot · 2026-02-16T11:06:19Z

backend/controllers/hint_controller.go

+type IHintController interface {
+	GetHints(ctx *gin.Context)
+}


POST /solo のハンドラ名が GetHints になっており、処理内容（ゲーム開始/ラウンド作成）と名前が一致していません。用途が伝わるように StartGame / CreateRound などの名前に揃えることを検討してください。

Copilot · 2026-02-16T11:40:45Z

@yone-al I've opened a new pull request, #14, to work on those changes. Once the pull request is ready, I'll request review from you.

bearl27

まぁいいんじゃない

Copilot

Pull request overview

Copilot reviewed 6 out of 6 changed files in this pull request and generated 2 comments.

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

Copilot · 2026-02-16T12:21:56Z

backend/repositories/hint_repository.go

+func (r *HintMemoryRepository) CreateRound(answer string, hints []string) (*model.Round, error) {
+	maxID := 0
+	for _, rd := range r.rounds {
+		if rd.ID > maxID {
+			maxID = rd.ID


HintMemoryRepository.CreateRound が r.rounds の走査/更新をロック無しで行っています。Gin はリクエストを並列に処理するため、複数の POST /solo が同時に来るとデータレースや slice 破損が起き得ます。mutex 等で CreateRound 全体を保護するか、スレッドセーフなストレージ実装に切り替えてください。

Copilot · 2026-02-16T12:21:57Z

backend/repositories/hint_repository.go

+		Hints:  hints,
+	}
+	r.rounds = append(r.rounds, round)
+	return &r.rounds[len(r.rounds)-1], nil


append 後に slice 要素へのポインタ（&r.rounds[len-1]）を返していますが、後続の append で再確保が起きると呼び出し側が保持しているポインタが無効になり得ます。返却用に新しい Round を確保して返すか、ポインタではなく値を返す形にすると安全です。

Suggested change

return &r.rounds[len(r.rounds)-1], nil

retRound := round

return &retRound, nil

[BE-6] 3層アーキテクチャの設計、ゲーム開始

d1bf3cb

yone-al requested a review from kubodayo as a code owner February 16, 2026 10:50

github-actions bot assigned yone-al Feb 16, 2026

github-actions bot added the back label Feb 16, 2026

yone-al requested review from bearl27 and Copilot February 16, 2026 11:01

Copilot started reviewing on behalf of yone-al February 16, 2026 11:02 View session

Copilot AI reviewed Feb 16, 2026

View reviewed changes

yone-al marked this pull request as draft February 16, 2026 11:14

Copilot AI mentioned this pull request Feb 16, 2026

Fix data duplication in HintMemoryRepository initialization #14

Closed

2 tasks

[BE-6] fix: 3層アーキテクチャの設計、ゲーム開始

2feadf4

yone-al marked this pull request as ready for review February 16, 2026 12:10

[BE-6] fix: .serenaを削除

66a8701

yone-al requested a review from Copilot February 16, 2026 12:18

bearl27 approved these changes Feb 16, 2026

View reviewed changes

Copilot started reviewing on behalf of yone-al February 16, 2026 12:19 View session

Copilot AI reviewed Feb 16, 2026

View reviewed changes

yone-al merged commit 1b88b57 into develop Feb 16, 2026
8 checks passed

yone-al deleted the BE-6 branch February 16, 2026 12:28

	return &r.rounds[len(r.rounds)-1], nil
	retRound := round
	return &retRound, nil

Conversation

yone-al commented Feb 16, 2026 • edited Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

概要

動作確認

変更内容・エビデンス

その他

Uh oh!

Copilot AI left a comment

Choose a reason for hiding this comment

Pull request overview

Reviewed changes

Uh oh!

Copilot AI Feb 16, 2026

Choose a reason for hiding this comment

Uh oh!

Copilot AI Feb 16, 2026

Choose a reason for hiding this comment

Uh oh!

yone-al Feb 16, 2026 • edited Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

Choose a reason for hiding this comment

Uh oh!

yone-al Feb 16, 2026

Choose a reason for hiding this comment

Uh oh!

Copilot AI Feb 16, 2026

Choose a reason for hiding this comment

Uh oh!

Copilot AI Feb 16, 2026

Choose a reason for hiding this comment

Uh oh!

Copilot AI Feb 16, 2026

Choose a reason for hiding this comment

Uh oh!

Copilot AI Feb 16, 2026

Choose a reason for hiding this comment

Uh oh!

Copilot AI Feb 16, 2026

Choose a reason for hiding this comment

Uh oh!

Copilot AI Feb 16, 2026

Choose a reason for hiding this comment

Uh oh!

Copilot AI commented Feb 16, 2026

Uh oh!

bearl27 left a comment

Choose a reason for hiding this comment

Uh oh!

Copilot AI left a comment

Choose a reason for hiding this comment

Pull request overview

Uh oh!

Copilot AI Feb 16, 2026

Choose a reason for hiding this comment

Uh oh!

Copilot AI Feb 16, 2026

Choose a reason for hiding this comment

Uh oh!

Uh oh!

Reviewers

Assignees

Labels

Projects

Milestone

Development

Uh oh!

4 participants

yone-al commented Feb 16, 2026 •

edited

Loading

yone-al Feb 16, 2026 •

edited

Loading