Golang 无法覆盖指向切片的指针

Question

尝试在切片指针上进行范围时，我不断收到此错误。

app/domain/repositories/class_repository.go:24: cannot range over classes (type *[]entities.Class)

我做错了什么？

这是结构：

 package repositories

import (
    "mobifit/app/domain/entities"
)

type ClassRepository struct {
    *Repository
}

func (c *ClassRepository) ClassesForLastNDays(days int) *[]entities.Class {
    classes := new([]entities.Class)
    query := Select("*").
        From("Class").
        Where("VisibleAt > CURRENT_TIMESTAMP() - INTERVAL ? DAY").
        OrderBy("ClassTypeId").
        Sql()
    c.Repository.Select(classes, query, days)
    c.populateClassRelationships(classes)
    return classes
}

func (c *ClassRepository) populateClassRelationships(classes *[]entities.Class) {
    for i := range classes {  <<<<<<<<<<< Here is the problem
        class := classes[i]

        // ClassType
        c.Repository.GetById(class.ClassType, class.ClassTypeId)

        //Instructor
        c.Repository.GetById(class.Instructor, class.ClassType.InstructorId)

        // Equipment
        query := Select("E.*").
            From("Equipment E").
            Join("ClassEquipment CE on E.Id = CE.EquipmentId").
            Where("CE.ClassId = ?").
            Sql()
        c.Repository.Select(class.Equipment, query, class.Id)
    }
}

这是类结构：

package entities

import (
    "time"
)

    type Class struct {
        Id                int
        ClassTypeId       int
        VideoPath         string
        VideoSize         int
        Duration          float64
        CreatedAt         time.Time
        VisibleAt         time.Time
        NoLongerVisibleAt time.Time

        // Relationships
        ClassType  ClassType
        Instructor User
        Equipment  []Equipment
    }

Answer 1

您假设指向切片的指针将在迭代中自动取消引用。

事实并非如此，也没有理由这样做，因为切片已经是一种指针，使指向切片的指针完全无用。

来自Effective Go：

如果函数接受切片参数，则对元素进行更改 切片对调用者可见，类似于传递一个 指向底层数组的指针。

在内部，切片是由

• 指向底层数组中切片第一个元素的指针
• 切片的长度
• 切片的容量（切片通常可以延长到数组的末尾）

这个结构很小，导致指针没用。

Answer 2

来自Effective Go：

如果您正在循环遍历数组、切片、字符串或映射，或者正在读取 从一个通道，一个范围子句可以管理循环。

您正在尝试将 指针 迭代到一个切片，该切片是单个值，而不是集合，因此是不可能的。

将populateClassRelationships 的参数更改为切片，而不是指向切片的指针。或者你可以取消引用指针：

func (c *ClassRepository) populateClassRelationships(classes *[]entities.Class) {
    for i := range *classes { // dereferencing the pointer to get the actual slice
        class := classes[i]

        // ClassType
        c.Repository.GetById(class.ClassType, class.ClassTypeId)

        //Instructor
        c.Repository.GetById(class.Instructor, class.ClassType.InstructorId)

        // Equipment
        query := Select("E.*").
            From("Equipment E").
            Join("ClassEquipment CE on E.Id = CE.EquipmentId").
            Where("CE.ClassId = ?").
            Sql()
        c.Repository.Select(class.Equipment, query, class.Id)
    }
}

Answer 3

如果你需要从 *slice 中提取一个单独的元素，你必须首先像这样取消引用它：(*slice)[0]。在我意识到这一点之前，我用头撞了*slice[0] 大约 6 个小时。它与操作顺序有关，IMO 并不是一个非常优雅的结果。

我最终编写了一些指针接收器方法来进行就地修改，例如追加和弹出，在我看来，这是一种合理的方式 - 可以在此处找到一个示例：https://play.golang.org/p/qZEYMcPHl4

Answer 4

你可以取消引用指针：

func (c *ClassRepository) populateClassRelationships(classes *[]entities.Class) {
    for _, class := range *classes { // NOTE the * dereference
    // ClassType
    c.Repository.GetById(class.ClassType, class.ClassTypeId)

    //Instructor
    c.Repository.GetById(class.Instructor, class.ClassType.InstructorId)

    // Equipment
    query := Select("E.*").
        From("Equipment E").
        Join("ClassEquipment CE on E.Id = CE.EquipmentId").
        Where("CE.ClassId = ?").
        Sql()
    c.Repository.Select(class.Equipment, query, class.Id)
    }
}

我还更改了 range 子句，因为我认为您没有修改 classes。