如何将两次查询结果合并为一个Ads结构体？附Go语言实现代码

看起来你现在的代码是典型的N+1查询问题——先查200条广告的ID和标题，再循环每条单独查询手机号，这会建立201次数据库连接，性能损耗极大，而且代码里还有没必要的反射操作。咱们直接改成一次查询就能搞定，分两种场景给你优化方案：

场景1：所有字段都在同一张表（ads表） #

既然phone字段本身就在ads表里，完全可以一次查询把所有需要的字段都取出来，直接映射到Ads结构体：

type Ads struct { 
    ID      int    `json:"id"` 
    Subject string `json:"subject"` 
    Phone   string `json:"phone"` 
}

func GetAdsPostgres() ([]Ads, error) { 
    // 提前指定切片容量，避免动态扩容的性能损耗
    ads := make([]Ads, 0, 200)
    
    // 一次查询所有需要的字段
    rows, err := db1.Query("SELECT ad_id, subject, phone FROM ads LIMIT 200")
    if err != nil {
        return nil, err
    }
    // 确保查询结束后释放数据库连接资源
    defer rows.Close()
    
    for rows.Next() {
        var ad Ads
        // 一次性把所有字段扫描到结构体中
        if err := rows.Scan(&ad.ID, &ad.Subject, &ad.Phone); err != nil {
            return nil, err
        }
        ads = append(ads, ad)
    }
    
    // 检查遍历过程中是否出现错误（比如数据截断、连接中断）
    if err := rows.Err(); err != nil {
        return nil, err
    }
    
    return ads, nil
}

这个方案的优势： #

• 只需要1次数据库查询，大幅减少连接开销，性能提升明显；
• 去掉了冗余的反射转换（原代码里的reflect.ValueOf(ad.ID)完全没必要，ad.ID本身就是int类型）；
• 完善了错误处理，不会漏掉任何查询或扫描过程中的错误；
• 提前初始化切片容量，避免多次扩容带来的性能损耗。

场景2：手机号在另一张关联表 #

如果phone字段不在ads表，而是在另一张关联表（比如ad_contacts，通过ad_id关联），那用SQL的JOIN语句合并两张表的数据即可：

假设关联表结构是ad_contacts(ad_id int, phone string)，优化后的代码如下：

func GetAdsPostgres() ([]Ads, error) { 
    ads := make([]Ads, 0, 200)
    
    // 使用LEFT JOIN确保即使没有对应手机号的广告也能被保留
    rows, err := db1.Query(`
        SELECT a.ad_id, a.subject, c.phone 
        FROM ads a
        LEFT JOIN ad_contacts c ON a.ad_id = c.ad_id
        LIMIT 200
    `)
    if err != nil {
        return nil, err
    }
    defer rows.Close()
    
    for rows.Next() {
        var ad Ads
        // 如果手机号可能为NULL，扫描到string类型会自动转为空字符串，无需额外处理
        if err := rows.Scan(&ad.ID, &ad.Subject, &ad.Phone); err != nil {
            return nil, err
        }
        ads = append(ads, ad)
    }
    
    if err := rows.Err(); err != nil {
        return nil, err
    }
    
    return ads, nil
}

这个方案的注意点： #

• 使用LEFT JOIN而不是INNER JOIN，这样即使某些广告没有对应的手机号记录，也能正常返回广告的ID和标题，手机号字段会是空字符串；
• 同样只需要一次查询，彻底解决N+1问题。

额外的小建议 #

• 永远记得用defer rows.Close()：不管查询成功还是失败，都要释放数据库连接，避免连接泄漏；
• 不要忽略任何错误：原代码里忽略了rows1的查询错误，这会导致隐藏的问题，一定要把错误返回给调用方处理；
• 结构体标签可以和数据库字段对应：如果用ORM工具（比如GORM），还可以通过标签直接映射字段，进一步简化代码，但原生SQL的话咱们上面的写法就足够高效了。

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