Goのレスポンス生成、複雑にしすぎてませんか？YAGNI原則で劇的に改善する方法

// ❌ 悪い例：pkg/utils/response.go（243行の複雑な実装）
package utils

// ... (6つの複雑な構造体定義) ...
type EnhancedAPIResponse struct {
    Message   string                 `json:"message"`
    Data      interface{}            `json:"data,omitempty"`
    Error     string                 `json:"error,omitempty"`
    Metadata  ResponseMetadata       `json:"metadata"` // 誰が使う？
    // ...
}

type ResponseMetadata struct {
    StatusCode     int               `json:"status_code"`
    ProcessingTime float64           `json:"processing_time_ms"` // 本当に必要？
    ServerInfo     ServerInfo        `json:"server_info"`
    DataValidation ValidationInfo     `json:"data_validation"`
    Performance    PerformanceInfo   `json:"performance"`
}

// ... (さらに4つのメタデータ用構造体) ...

// メイン関数（100行以上の複雑な処理）
func CreateEnhancedResponse(statusCode int, message string, data interface{}, err string) events.APIGatewayProxyResponse {
    // 1. 複雑なメタデータ構築
    // 2. reflectパッケージを駆使した過剰なデータ検証
    // 3. 詳細すぎるログを6回も出力
    // 4. 複雑なフォールバック処理
    // 5. パフォーマンス情報を埋め込むためにレスポンスを再マーシャリング（!!）
    // 6. 10個以上のHTTPヘッダーを設定
    // ...
    return events.APIGatewayProxyResponse{ /* ... */ }
}

// ✅ 改善案：シンプルで必要十分な実装
package utils

import (
	"encoding/json"
	"net/http"

	"github.com/aws/aws-lambda-go/events"
)

// シンプルなレスポンス構造
type APIResponse struct {
	Message string      `json:"message"`
	Data    interface{} `json:"data,omitempty"`
	Error   string      `json:"error,omitempty"`
}

// CreateResponseはレスポンス生成の基本となるシンプルな関数
func CreateResponse(statusCode int, message string, data interface{}, errStr string) events.APIGatewayProxyResponse {
	response := APIResponse{
		Message: message,
		Data:    data,
		Error:   errStr,
	}

	body, marshalErr := json.Marshal(response)
	if marshalErr != nil {
		// シンプルなフォールバック
		statusCode = http.StatusInternalServerError
		body = []byte(`{"message":"Response serialization failed","error":"JSON marshal error"}`)
	}

	return events.APIGatewayProxyResponse{
		StatusCode: statusCode,
		Headers:    map[string]string{"Content-Type": "application/json"},
		Body:       string(body),
	}
}

// ✅ 使いやすいヘルパー関数群
func CreateSuccessResponse(message string, data interface{}) events.APIGatewayProxyResponse {
	return CreateResponse(http.StatusOK, message, data, "")
}

func CreateErrorResponse(statusCode int, message string, err string) events.APIGatewayProxyResponse {
	if statusCode < 400 {
		statusCode = http.StatusInternalServerError
	}
	return CreateResponse(statusCode, message, nil, err)
}

func CreateNotFoundResponse(message string) events.APIGatewayProxyResponse {
	return CreateErrorResponse(http.StatusNotFound, message, "Resource not found")
}

// ... 他にも `CreateValidationErrorResponse` など、用途に応じたヘルパーを用意

// ✅ 改善後のハンドラー：意図が明確で簡潔
func handleGetUser(userID string) events.APIGatewayProxyResponse {
	user, err := getUserByID(userID)
	if err != nil {
		if errors.Is(err, ErrUserNotFound) {
			return utils.CreateNotFoundResponse("User not found") // 1行でエラーレスポンス
		}
		return utils.CreateErrorResponse(500, "Failed to retrieve user", err.Error())
	}
	return utils.CreateSuccessResponse("User retrieved successfully", user) // 1行で成功レスポンス
}