Return not nil

proelbtn · web-flow · commit f5f8b157ebf8 · 2024-01-14T04:50:28.000+09:00
diff --git a/vmdb-api/repository.go b/vmdb-api/repository.go
@@ -29,7 +29,7 @@ func (r *Repository) findConfigBy(ctx context.Context, key string) (*Config, err
 }
 
 func (r *Repository) listProblemEnvironments(ctx context.Context) ([]ProblemEnvironment, error) {
-	var result []ProblemEnvironment
+	result := []ProblemEnvironment{}
 	if err := r.db.NewSelect().Model(&result).Scan(ctx); err != nil {
 		return nil, err
 	}
@@ -81,7 +81,7 @@ func (r *Repository) findAnswerBy(ctx context.Context, answerID uuid.UUID) (*Ans
 }
 
 func (r *Repository) listUnscoredAnswersFor(ctx context.Context, problemID uuid.UUID) ([]Answer, error) {
-	var result []Answer
+	result := []Answer{}
 	err := r.db.NewSelect().ColumnExpr("answers.*").
 		Table("answers").
 		Join("INNER JOIN scores").JoinOn("answers.id = scores.answer_id").

Original file line number	Diff line number	Diff line change
`@@ -29,7 +29,7 @@ func (r Repository) findConfigBy(ctx context.Context, key string) (Config, err`
`29`	`29`	`}`
`30`	`30`
`31`	`31`	`func (r *Repository) listProblemEnvironments(ctx context.Context) ([]ProblemEnvironment, error) {`
`32`		`- var result []ProblemEnvironment`
	`32`	`+ result := []ProblemEnvironment{}`
`33`	`33`	`if err := r.db.NewSelect().Model(&result).Scan(ctx); err != nil {`
`34`	`34`	`return nil, err`
`35`	`35`	`}`
`@@ -81,7 +81,7 @@ func (r Repository) findAnswerBy(ctx context.Context, answerID uuid.UUID) (Ans`
`81`	`81`	`}`
`82`	`82`
`83`	`83`	`func (r *Repository) listUnscoredAnswersFor(ctx context.Context, problemID uuid.UUID) ([]Answer, error) {`
`84`		`- var result []Answer`
	`84`	`+ result := []Answer{}`
`85`	`85`	`err := r.db.NewSelect().ColumnExpr("answers.*").`
`86`	`86`	`Table("answers").`
`87`	`87`	`Join("INNER JOIN scores").JoinOn("answers.id = scores.answer_id").`