[Go]UserとTweetが1対多の関係でTweetにUserを追加する

Question

前提・実現したいこと

GoでTwitterのようなアプリを作ろうとしています。
UserとTweetが1対多の関係です。

Tweetの一覧ページでTweetにUserの情報を追加しようとした際に、うまくできず詰まってしまいました。

発生している問題・エラーメッセージ

ソースコードは以下の通りです。

models/tweet.go

package models

type Tweet struct {
	ID        int
	Content   string
	UserID    int
	User      User
	CreatedAt time.Time
}

func GetTweets() (tweets []Tweet, err error) {
	cmd := `select * from tweets`
	rows, err := db.Db.Query(cmd)
	if err != nil {
		log.Fatalln(err)
	}
	for rows.Next() {
		var t Tweet
		err = rows.Scan(&t.ID,
			&t.Content,
			&t.UserID,
			&t.CreatedAt)
		if err != nil {
			log.Fatalln(err)
		}
		tweets = append(tweets, t)
	}
	rows.Close()

	return tweets, err
}

models/user.go

package models

type User struct {
	ID         int
	UUID       string
	Name       string
	Email      string
	PassWord   string
	CreatedAt  time.Time
	Tweets     []Tweet
}

func GetUser(id int) (u User, err error) {
	u = User{}
	cmd := `select * from users where id = ?`
	err = db.Db.QueryRow(cmd, id).Scan(
		&u.ID,
		&u.UUID,
		&u.Name,
		&u.Email,
		&u.PassWord,
		&u.CreatedAt,
	)
	return u, err
}

controllers/tweets_controller.go

package controllers

func TweetIndex(w http.ResponseWriter, r *http.Request) {
	tweets, _ := models.GetTweets()
	t := models.Tweet{}
	for _, t = range tweets {
		t.User, err = models.GetUser(t.UserID)
		if err != nil {
			fmt.Println(err)
		}
		fmt.Println(t) //①for文の中
	}
	fmt.Println(tweets) //②for文の外
	generateHTML(w, tweets, "layout", "common/navbar_priv", "tweets/index")
}

①for文の中では、
Tweetの持つUserがきちんと取得できたのですが、

②for文の外では、
Tweetの持つUserが初期値になってしまいます。

どのようにすれば、Tweetストラクトの持つUserにデータを反映できるのかがわかりません。

補足情報

ひとつひとつのTweetを表示するページでは、以下のようにすることでviewから呼び出せました。

models/tweet.go

/*
    略
*/

func GetTweet(id int) (t Tweet, err error) {
	cmd := `select * from tweets where id = ?`
	t = Tweet{}

	err = db.Db.QueryRow(cmd, id).Scan(
		&t.ID,
		&t.Content,
		&t.UserID,
		&t.CreatedAt)
	return t, err
}

controllers/tweets_controller.go

/*
    略
*/

func TweetShow(w http.ResponseWriter, r *http.Request, id int) {
	t, err := models.GetTweet(id)
	if err != nil {
		log.Println(err)
	}
	t.User, err = models.GetUser(t.UserID)
	if err != nil {
		fmt.Println(err)
	}
	generateHTML(w, t, "layout", "common/navbar_priv", "tweets/show")
}

Comments

[hoshi-takanori]

range は各要素のコピーを返すので、要素を更新したいのであれば index を使って処理する必要があります。
https://inabajunmr.hatenablog.com/entry/2019/11/07/093217

[kyo._.]

ありがとうございます。
「range は各要素のコピーを返す」からできていなかったんですね。納得しました。

for文の部分を以下のようにするとできました。

```go
for i := 0; i < len(tweets); i++ {
tweets[i].User, err = models.GetUser(tweets[i].UserID)
if err != nil {
fmt.Println(err)
}
fmt.Println(tweets[i])
}
```

[kyo._.]

回答の部分に投稿していただけるとベストアンサーにして解決済みにできるですが、、
お手数ですがお願いできますか？

Answer 1

range は各要素のコピーを返すので、要素を更新したいのであれば index を使って処理する必要があります。
参考: sliceをrangeでループして取得した値のフィールドを更新しても、元のsliceは更新されない（rangeで取得した値はコピー） - チョキチョキかにさん

    for i := range tweets {
        tweets[i].User, err = models.GetUser(tweets[i].UserID)
        // 略
    }

Comments

[kyo._.]

ありがとうございます。