为何循环中变量未被覆盖？GORM查询结果异常排查

问题场景 #

第一种写法（循环外声明card）：

func showAll(db *gorm.DB) {
   users := &[]models.User{}
   card := models.Card{}
   db.Find(users)
   for _, i := range *users {
      fmt.Println(i)

      db.Where("user_id = ?", i.ID).Find(&card)
      fmt.Println(card)
   }
}

fmt.Println(card)始终输出第一个查询到的值。

第二种写法（循环内声明card）：

func showAll(db *gorm.DB) {
   users := &[]models.User{}
   
   db.Find(users)
   for _, i := range *users {
      fmt.Println(i)
      card := models.Card{}
      db.Where("user_id = ?", i.ID).Find(&card)
      fmt.Println(card)
   }
}

能正确输出每个用户对应的卡片信息。

原因解析 #

• Gorm的字段覆盖逻辑：Gorm的Find方法接收结构体指针时，只会将查询到的字段值覆盖到指针指向的实例中，不会主动清空实例的原有字段。
• 循环外声明的问题：当card在循环外声明时，整个循环复用同一个内存地址的实例。如果某次查询没有匹配到数据（或者部分字段无值），Gorm不会修改该实例的已有值，导致旧数据残留；即使有匹配数据，未被查询结果覆盖的字段也会保留之前的旧值。
• 循环内声明的优势：每次循环都会创建一个全新的models.Card{}实例（初始化为零值），Gorm将查询结果写入新实例，完全避免了旧数据的干扰，自然能得到正确结果。

解决方案 #

1. 推荐方案：循环内声明结构体
   像第二种写法那样，每次循环创建新的Card实例，逻辑清晰，不会有残留问题。
2. 备选方案：手动重置结构体
   如果需要在循环外复用结构体，每次查询前手动将其重置为零值：

   func showAll(db *gorm.DB) {
      users := &[]models.User{}
      card := models.Card{}
      db.Find(users)
      for _, i := range *users {
         fmt.Println(i)
         // 手动重置为零值
         card = models.Card{}
         db.Where("user_id = ?", i.ID).Find(&card)
         fmt.Println(card)
      }
   }
   

内容的提问来源于stack exchange，提问作者RND