duane9322 2016-01-13 07:40
浏览 23
已采纳

分配的指针字段变为<nil>

I have a struct:

type user struct {
Id string
..
data_ptr *userData
}

And I store slice of users in global scope:

type Hall struct {
    users []user
}

var hall = Hall{}    //global

Finally, http handler:

func dataHandler(response http.ResponseWriter, request *http.Request) {
    userExist, user_ptr := hall.haveUserId()    //works fine
    switch requestType {
    case "load":    
        user_ptr.loadData()   //data loaded and user_ptr.data_ptr is set
    case "newData":
        user_ptr.data_ptr = newData  // <-- this is it, now previously set data_ptr == nil

So, why the heck, I mean I send "load" request, it loads data, sets data_ptr for user_ptr. But on next call, "newData" request, user_ptr.data_ptr is nil?

Just in case, here is loadData():

func (p *user) loadData(userId) {
    ..
    data := userData {}
    p.data_ptr = &data
}

EDIT: where user_ptr comes from:

func (h *Hall) haveUserId(id string) (bool, *user) {
    for _, u := range h.users {
        if u.Id == id {
            fmt.Println("UID found")
            return true, &u
        }
    }
    return false, nil
}
  • 写回答

1条回答 默认 最新

  • duanjuan3931 2016-01-13 08:04
    关注

    This is because you operate on a copy and not on the slice element itself.

    In your haveUserId() function the for ... range makes a copy of the elements it loops over, and you return the address of this copy. And so later you will modify this copy which is independent from the value in the slice. So if later you check the address in the slice element, it will still be unchanged (nil).

    Possible fix: return the address of the slice element: &h.users[i]

    func (h *Hall) haveUserId(id string) (bool, *user) {
        for i := range h.users {
            if h.users[i].Id == id {
                fmt.Println("UID found")
                return true, &h.users[i]
            }
        }
        return false, nil
    }
    

    To demonstrate this, see this example:

    type Point struct{ x, y int }
    ps := []Point{{1, 2}, {3, 4}}
    fmt.Println(ps) // Output: [{1 2} {3 4}]
    
    for _, v := range ps {
        v.x += 10 // Modifies just the copy
    }
    fmt.Println(ps) // Output (unchanged): [{1 2} {3 4}]
    
    for i := range ps {
        ps[i].x += 10 // Modifies value in slice
    }
    fmt.Println(ps) // Output (changed): [{11 2} {13 4}]
    

    Try it on the Go Playground.

    本回答被题主选为最佳回答 , 对您是否有帮助呢?
    评论

报告相同问题?

悬赏问题

  • ¥15 DIFY API Endpoint 问题。
  • ¥20 sub地址DHCP问题
  • ¥15 delta降尺度计算的一些细节,有偿
  • ¥15 Arduino红外遥控代码有问题
  • ¥15 数值计算离散正交多项式
  • ¥30 数值计算均差系数编程
  • ¥15 redis-full-check比较 两个集群的数据出错
  • ¥15 Matlab编程问题
  • ¥15 训练的多模态特征融合模型准确度很低怎么办
  • ¥15 kylin启动报错log4j类冲突