foreach 和 for 循环没有从 sql 查询中获取所有数据答案

【问题标题】：foreach and for loop not getting all data from sql queryforeach 和 for 循环没有从 sql 查询中获取所有数据
【发布时间】：2020-12-28 11:24:20
【问题描述】：

**已编辑

我将 sql 查询分成 3 个，现在查询看起来像这样。

$sql_2 = "SELECT DISTINCT cr.id AS courseid,
        cr.fullname AS coursename,
        cr.idnumber AS idnumber,
        COUNT(DISTINCT ra.id) AS enrols, 
        COUNT(DISTINCT cc.timecompleted) AS completed 
        FROM {course} cr JOIN {context} ct ON ( ct.instanceid = cr.id ) 
        LEFT JOIN {role_assignments} ra ON ( ra.contextid = ct.id ) and ra.roleid = 5 
        LEFT JOIN {course_completions} cc ON (cc.course = cr.id) 
        WHERE cr.id = :course_id GROUP BY cr.fullname, cr.id";

$sql_3 = "SELECT DISTINCT 
        u.id AS userid,
        u.email AS email,
        u.lastaccess,
        c.id AS courseid
        FROM {user} u
        JOIN {role_assignments} ra ON ra.userid = u.id
        LEFT JOIN {context} ct ON ct.id = ra.contextid AND ra.roleid = 5
        LEFT JOIN {course} c ON c.id = ct.instanceid AND ct.contextlevel = 50
        WHERE c.id = :course_id";

$sql_4 = "SELECT DISTINCT 
        u.id AS userid,
        sc.name AS item,
        m.name AS module,
        cm.id AS cmid,
        cm.idnumber AS itemidnumber,
        sc.launch AS launch,
        cmc.completionstate AS completed,
        cmc.timemodified AS timecompleted
        FROM {user} u
        JOIN {role_assignments} ra ON ra.userid = u.id
        LEFT JOIN {context} ct ON ct.id = ra.contextid AND ra.roleid = 5
        LEFT JOIN {course} c ON c.id = ct.instanceid AND ct.contextlevel = 50
        LEFT JOIN {course_modules} cm ON cm.course = c.id
        LEFT JOIN {course_modules_completion} cmc ON cmc.userid = u.id AND cmc.coursemoduleid = cm.id
        LEFT JOIN {modules} m ON m.id = cm.module
        LEFT JOIN {scorm} sc ON sc.id = cm.instance
        WHERE c.id = :course_id and m.id = 18";

$courses = $DB->get_records_sql($sql_2, ["course_id" => $requestedcourseids]);
$users = $DB->get_records_sql($sql_3, ["course_id" => $requestedcourseids]);
$modules = $DB->get_records_sql($sql_4, ["course_id" => $requestedcourseids]);

我在每个结果下添加了foreach和for循环，得到多维数组结果，得到这样的模块结果

   "course": [
{
"courseid": 4,
"coursename": "Activities",
"idnumber": "",
"enrols": 6,
"completed": 1,
"users": [
    {
        "userid": 3,
        "email": "test1@hotmail.com",
        "lastaccess": 1599379804,
        "modules": [
            {
                "userid": 3,
                "module": "scorm",
                "item": "Vitamin",
                "itemidnumber": "",
                "cmid": 24,
                "launch": 28,
                "completed": 1,
                "timecompleted": 1595575023
            },
            {
                "userid": 2,
                "module": "scorm",
                "item": "Vitamin",
                "itemidnumber": "",
                "cmid": 24,
                "launch": 28,
                "completed": 0,
                "timecompleted": 1593135761
            },
            {
                "userid": 4,
                "module": "scorm",
                "item": "Vitamin",
                "itemidnumber": "",
                "cmid": 24,
                "launch": 28,
                "completed": 1,
                "timecompleted": 1589359945
            },
            {
                "userid": 8,
                "module": "scorm",
                "item": "Vitamin",
                "itemidnumber": "",
                "cmid": 24,
                "launch": 28,
                "completed": null,
                "timecompleted": null
            },
            {
                "userid": 7,
                "module": "scorm",
                "item": "Vitamin",
                "itemidnumber": "",
                "cmid": 24,
                "launch": 28,
                "completed": null,
                "timecompleted": null
            }
        ]
    },
    {
        "userid": 2,
        "email": "test2@gmail.com",
        "lastaccess": 1599635370,
        "modules": [
            {
                "userid": 9,
                "module": "scorm",
                "item": "Vitamin",
                "itemidnumber": "",
                "cmid": 24,
                "launch": 28,
                "completed": 1,
                "timecompleted": 1595576987
            },
            {
                "userid": 3,
                "module": "scorm",
                "item": "Vitamin",
                "itemidnumber": "",
                "cmid": 24,
                "launch": 28,
                "completed": 1,
                "timecompleted": 1595575023
            },,,,

如您所见，模块列表仅显示一个具有每个用户 ID 的模块。

它应该列出每个用户 8 个不同的模块，但它只循环 6 次，这只是用户计数的数量。我想让用户使用用户尝试过的所有模块。

这里是代码

foreach($courses as $completion) {

if(!is_null($course)) {
    $coursesdata[] = $course;
}
$course = array();
$course['courseid'] = $completion->courseid;
$course['coursename'] = $completion->coursename;
$course['idnumber'] = $completion->idnumber;
$course['enrols'] = $completion->enrols;
$course['completed'] = $completion->completed;
$course['users'] = [];

if(!empty($users)) {
    foreach($users as $userinfo) {
        for($i=0; $i<count($users); $i++) {
            $user = array();
            $user['userid'] = $userinfo->userid;
            $user['email'] = $userinfo->email;
            $user['lastaccess'] = $userinfo->lastaccess;
            $user['modules'] = array();

            if(!empty($modules)) {
                foreach($modules as $details) {
                    for($i=0; $i<count($modules); $i++) {
                        $module = array();
                        $module['module'] = $details->module;
                        $module['item'] = $details->item;
                        $module['itemidnumber'] = $details->itemidnumber;
                        $module['cmid'] = $details->cmid;
                        $module['launch'] = $details->launch;
                        $module['completed'] = $details->completed;
                        $module['timecompleted'] = $details->timecompleted;
                    }
                    if(!is_null($module)) {
                        $user['modules'][] = $module;
                    }
                }
            }
        }
        if(!is_null($user)) {
            $course['users'][] = $user;
        }
    }
}
}

问题：如何将 $modules 中的所有数据放在 $users 数组下？目前它只循环 $modules 查询中的第一个元素。

我已经尝试了 cmets and answer 中建议的方法，但这是迄今为止我得到的最接近的方法。

任何帮助或提示将不胜感激。

提前致谢！

【问题讨论】：

(1)什么是“注册”？ (2)你能用相关表及其关系更新问题吗？
对于想法：geeksforgeeks.org/how-to-create-an-array-for-json-using-php 和 stackoverflow.com/questions/19454208/… 和许多其他人。
@ariefbayu 感谢您的评论。我已经编辑了一点，希望现在对你有意义。
好长的问题，你要我们看这么多行代码。找到问题的一个重要方法是一次又一次地将问题分成两半，将其减少到几行不起作用。

标签： php arrays moodle

【解决方案1】：

首先你当前的代码容易受到sql注入，你真的应该使用参数Moodle Documentation。我还将模块分解为另一个查询。

$sql_2 = "SELECT DISTINCT cr.id AS courseid,
    cr.fullname AS coursename,
    cr.idnumber AS idnumber,
    COUNT(DISTINCT ra.id) AS enrols, 
    COUNT(DISTINCT cc.timecompleted) AS completed 
    FROM {course} cr JOIN {context} ct ON ( ct.instanceid = cr.id ) 
    LEFT JOIN {role_assignments} ra ON ( ra.contextid = ct.id ) and ra.roleid = 5 
    LEFT JOIN {course_completions} cc ON (cc.course = cr.id) 
    WHERE cr.id = :course_id GROUP BY cr.fullname, cr.id";

 $sql_3 = "SELECT DISTINCT 
        u.id AS userid,
        u.email AS email,
        c.id AS courseid
        FROM {user} u
        JOIN {role_assignments} ra ON ra.userid = u.id
        LEFT JOIN {context} ct ON ct.id = ra.contextid AND ra.roleid = 5
        LEFT JOIN {course} c ON c.id = ct.instanceid AND ct.contextlevel = 50
        WHERE c.id = :course_id and m.id = 18 ORDER BY u.email";
        
$sql_4 = "SELECT DISTINCT 
        u.id AS userid,
        m.name AS module
        FROM {user} u
        JOIN {role_assignments} ra ON ra.userid = u.id
        LEFT JOIN {context} ct ON ct.id = ra.contextid AND ra.roleid = 5
        LEFT JOIN {course} c ON c.id = ct.instanceid AND ct.contextlevel = 50
        LEFT JOIN {course_modules} cm ON cm.course = c.id
        LEFT JOIN {course_modules_completion} cmc ON cmc.userid = u.id AND cmc.coursemoduleid = cm.id
        LEFT JOIN {modules} m ON m.id = cm.module
        LEFT JOIN {scorm} sc ON sc.id = cm.instance
        WHERE c.id = :course_id and m.id = 18 ORDER BY u.email";

$courses = $DB->get_records_sql($sql_2, ["course_id" => $requestedcourseids]);
$users = $DB->get_records_sql($sql_3, ["course_id" => $requestedcourseids]);
$modules = $DB->get_records_sql($sql_4, ["course_id" => $requestedcourseids]);

$course_user_modules = [];

foreach($courses as $course){
    $course["users"] = [];
    foreach($users as $course_user){
        if($course["courseid"]==$course_user["courseid"]){
            unset($course_user["courseid"]);
            $course_user["modules"] = [];
            foreach($modules as $user_module){
                if($user_module["userid"]==$course_user["userid"]){
                    unset($user_module["userid"]);
                    array_push($course_user["modules"], $user_module);
                }
            }
            array_push($course["users"], $course_user);
        }
    }
    array_push($course_user_modules, $course);
}

【讨论】：

非常感谢您的回答。我已将查询分成两部分并测试了代码，但我得到了"message": "Cannot use object of type stdClass as array"。我查看了stackoverflow.com/questions/6815520/… 来解决这个问题，但我做不到。还有什么我应该尝试的吗？
您在对象上使用数组运算符。请向我们展示与您的错误对应的代码行。

【解决方案2】：

我在调用 get_records_sql 的 Moodle 数据库中发现了问题

$modules = $DB->get_records_sql($sql_4, ["course_id" => $requestedcourseids]);

这将返回一个由结果中的第一个字段索引的数组。这意味着在我的情况下，每个用户 ID 只返回一组数组。在我将 moduleid(unique value) 的 sql 查询更改为先出现后，它现在按预期工作。

$sql_4 = "SELECT DISTINCT   cm.id AS moduleid,
                            u.id AS userid,
                            sc.name AS item,
                            m.name AS module,
                            cm.idnumber AS itemidnumber,
                            sc.launch AS launch,
                            cmc.completionstate AS completed,
                            cmc.timemodified AS timecompleted
                            FROM {user} u
                            JOIN {role_assignments} ra ON ra.userid = u.id
                            LEFT JOIN {context} ct ON ct.id = ra.contextid AND ra.roleid = 5
                            LEFT JOIN {course} c ON c.id = ct.instanceid AND ct.contextlevel = 50
                            LEFT JOIN {course_modules} cm ON cm.course = c.id
                            LEFT JOIN {course_modules_completion} cmc ON cmc.userid = u.id AND cmc.coursemoduleid = cm.id
                            LEFT JOIN {modules} m ON m.id = cm.module
                            LEFT JOIN {scorm} sc ON sc.id = cm.instance
                            WHERE c.id = :course_id and m.id = 18 and u.id = :userid";

之前是这样的，

users:
    userid: 2,
    module:
      moduleid: 12,
    userid: 8,
    module:
      moduleid: 12
    userid: 9,
    module:
      moduleid: 12,

现在我明白了，

users:
    userid: 2,
    module:
      moduleid: 12,
      moduleid: 14,
      moduleid: 17,
    userid: 8,
    module:
      moduleid: 12,
      moduleid: 14,
      moduleid: 17,
    userid: 9,
    module:
      moduleid: 12,
      moduleid: 14,
      moduleid: 17,

这篇文章帮助我解决了这个问题。 get_records_sql returns only one result using inner join in moodle

【讨论】：