1

スライスを型宣言してデータベースから自身の情報を取得するメソッドを実装したのですが、メソッド内で確認するとArtistというフィールドの情報が取得できていますが、メソッドの呼び出し元で表示するとArtistフィールドを変更できていません。

type Track struct {
    gorm.Model
    Name     string `sql:"not null;unique" json:"name"`
    Phonetic string `json:"phonetic"`
    Artist   Artist `json:"artist"`
    ArtistID uint   `json:"artistId"`
}

type Tracks []Track

func (ts *Tracks) List(dba *gorm.DB, limit, offset int) error {
    err := dba.Order("created_at desc").Limit(limit).Offset(offset).Find(ts).Error
    if err != nil {
        return err
    }
    newSlice := make(Tracks, len(*ts))
    for i, t := range *ts {
        if err = dba.Model(&t).Related(&t.Artist).Error; err != nil {
            return err
        }
        newSlice[i] = t
    }
    ts = &newSlice
    // ここでTrack内のArtistに情報が入っている。
    log.Println(ts)
    return err
}

func someFunc(){
    var tracks Tracks
    tracks.List(dba, limit, offset)
    // ここで確認するとArtistが初期化されている。
    log.Println(tracks)
}

なぜこのような挙動になるのでしょうか。良い実装方法があれば教えてください。よろしくお願い致します。

1 件の回答 1

1

List メソッドのレシーバー ts のスコープは List 内に限られているので、今回のプログラムで外部変数の書き換えは起こっていません。 これは、レシーバーであるポインタが指し示す実体がスライスであるかどうかとは関係ありません。

詳細

以下の行でレシーバー ts にポインタの代入が行われています。

ts = &newSlice

しかしこれはメソッド内にスコープを持つ変数 ts を書き換えているだけで、ts が指し示している実体 *ts を書き換えてはいません。したがって、以下のように *ts を書き換えるようにすると上手くいきます。

*ts = newSlice

簡単な例

より問題を簡単にするために、以下のプログラムを用意しました。

package main

import "fmt"

type S struct {
    a int
}

func (s *S) update() {
    // s = &S{1}
    *s = S{1}
}

func main() {
    var x S
    fmt.Println(x)
    x.update()
    fmt.Println(x)
}

メソッド update() の実装を切り替えると、2回目の fmt.Println(x) で出力される内容が変わることが確認できます。Go Playground 上で動かせるようにしたので、お試しください。


註. Go の仕様の "Declarations and scope" にあるスコープの説明に書いてあります。

The scope of an identifier denoting a method receiver, function parameter, or result variable is the function body.

この質問に回答するには、ログインする必要があります。

求めていた回答ではありませんか? のタグが付いた他の質問を参照する。