CodeIgniter中多活动记录导致个人信息重复显示的求助

你的问题核心在于视图的循环逻辑错误：内层foreach每次都输出会员个人信息，导致有多少条活动记录，会员信息就重复多少次。结合你现有的数据结构，我们可以从调整视图逻辑入手快速解决，也可以优化数据结构让代码更清晰。

方案一：直接调整视图代码（最快解决） #

基于你当前的$SActivity数据结构（二维数组，每个子数组对应一个会员的所有活动记录），我们把会员个人信息移到外层循环，只输出一次，内层循环仅渲染活动表格行：

<?php foreach ($SActivity as $memberActivities) {
    // 同一个会员的所有活动记录里，个人信息是一致的，取第一条即可
    $member = $memberActivities[0];
?>
    <h2><?php echo $member->first_name; ?> <?php echo $member->last_name; ?></h2>
    <p><?php echo $member->email; ?></p>
    <table>
        <thead>
            <tr>
                <th>ActivityName</th>
                <th>Activity_startTime</th>
                <th>Activity_endTime</th>
            </tr>
        </thead>
        <tbody>
            <?php foreach ($memberActivities as $activity) { ?>
            <tr>
                <td><?php echo $activity->activity_name; ?></td>
                <td><?php echo $activity->activity_startTime; ?></td>
                <td><?php echo $activity->activity_endTime; ?></td>
            </tr>
            <?php } ?>
        </tbody>
    </table>
<?php } ?>

逻辑说明： #

1. 外层循环遍历每个会员的活动集合$memberActivities
2. 从集合的第一条记录中提取会员个人信息（因为同一个会员的所有活动记录里，个人字段完全相同）
3. 内层循环仅遍历活动记录，渲染表格的每一行活动数据

方案二：优化数据结构（更优雅的写法） #

如果希望数据结构更清晰，避免依赖“同一会员记录信息一致”的隐含条件，可以调整控制器和模型，让返回的数据明确区分会员信息和活动列表：

步骤1：修改控制器代码 #

将会员基础信息和其活动列表关联起来，组装成更清晰的数组结构：

// 获取所有会员基础信息
$allMembers = $this->Member_model->getAlldata();
$memberWithActivities = [];

foreach ($allMembers as $member) {
    // 获取当前会员的所有活动
    $activities = $this->Member_model->getSecondaryMemberActivity($member->member_id);
    // 组装数据：会员信息 + 活动列表（空活动则用空数组）
    $memberWithActivities[] = [
        'member_info' => $member,
        'activities' => $activities ?: []
    ];
}

// 传递给视图的变量
$getLoginMemberDetails['member_data'] = $memberWithActivities;

步骤2：修改视图代码 #

基于新的清晰数据结构，视图逻辑会更直观：

<?php foreach ($member_data as $item) {
    $member = $item['member_info'];
    $activities = $item['activities'];
?>
    <h2><?php echo $member->first_name; ?> <?php echo $member->last_name; ?></h2>
    <p><?php echo $member->email; ?></p>
    
    <?php if (!empty($activities)) { ?>
    <table>
        <thead>
            <tr>
                <th>ActivityName</th>
                <th>Activity_startTime</th>
                <th>Activity_endTime</th>
            </tr>
        </thead>
        <tbody>
            <?php foreach ($activities as $activity) { ?>
            <tr>
                <td><?php echo $activity->activity_name; ?></td>
                <td><?php echo $activity->activity_startTime; ?></td>
                <td><?php echo $activity->activity_endTime; ?></td>
            </tr>
            <?php } ?>
        </tbody>
    </table>
    <?php } else { ?>
    <p>暂无活动记录</p>
    <?php } ?>
<?php } ?>

步骤3：优化模型函数（可选） #

调整模型的getSecondaryMemberActivity函数，确保没有活动时返回空数组而不是0，让数据处理更统一：

public function getSecondaryMemberActivity($gotMemberId) {
    $result = $this->db->where('members.member_id', $gotMemberId)
        ->from('members')
        ->join('member_activity', 'members.member_id = member_activity.member_id','LEFT')
        ->join('activity_name', 'activity_name.activity_name_id = member_activity.activity_id','LEFT')
        ->get()
        ->result();
    
    // 无论是否有结果，都返回数组（空数组或活动数组）
    return $result ?? [];
}

这样处理后，代码的可读性和可维护性都会提升，也避免了因数据结构变化导致的潜在问题。

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