PHP函数返回数组问题：调用后var_dump返回NULL求助

首先，你遇到的核心问题很直观：调用函数时没有接收它的返回值。你的函数内部确实生成并返回了$counter，但外部代码没有把这个返回值赋值给变量，导致全局作用域里的$counter根本没被定义，var_dump()自然输出NULL啦。

第一步：修复返回值接收问题 #

只要在调用函数时把返回值赋值给$counter就行：

$type = 'RIDDOR - Major Injury';
// 将函数返回值赋值给外部的$counter变量
$counter = riddor_dates($type,$id,$id1,$datatable,$date_from,$date_to);
var_dump($counter);

第二步：优化代码的其他问题 #

作为PHP MySQLi新手，你的代码还有几个可以改进的地方，能让它更安全、高效且易维护：

1. 彻底杜绝SQL注入风险 #

直接把变量拼进SQL语句是非常危险的操作，恶意用户很容易通过构造参数攻击你的数据库。一定要用MySQLi的预处理语句来处理动态参数。

2. 用SQL直接计数，替代PHP循环遍历 #

你现在是把结果集全部取出来再用PHP循环计数，其实可以直接让数据库用COUNT()函数统计，效率会高很多，代码也更简洁。

3. 避免使用全局变量 #

global $connection会让代码的耦合性变高，不利于后续维护和复用，建议把数据库连接作为参数传入函数。

优化后的完整代码 #

function riddor_dates($type, $id, $id1, $datatable, $date_from, $date_to, $connection) {
    // 用COUNT()直接在SQL中统计目标数量
    $sql = "SELECT COUNT(`" . $id . "`) AS total 
            FROM `" . $datatable . "` 
            WHERE `" . $id1 . "` BETWEEN ? AND ? 
            AND `" . $id . "` = ?";
    
    // 初始化预处理语句
    if ($stmt = mysqli_prepare($connection, $sql)) {
        // 绑定参数：三个字符串类型参数（日期和类型用s，数字类型可改用i）
        mysqli_stmt_bind_param($stmt, "sss", $date_from, $date_to, $type);
        // 执行SQL语句
        mysqli_stmt_execute($stmt);
        // 将查询结果绑定到变量
        mysqli_stmt_bind_result($stmt, $count);
        // 获取结果值
        mysqli_stmt_fetch($stmt);
        
        echo "<br>" . $type . ": " . $count;
        $counter[$type] = $count;
        
        // 关闭预处理语句
        mysqli_stmt_close($stmt);
        return $counter;
    } else {
        // 记录SQL错误信息，方便调试
        error_log("SQL Prepare Error: " . mysqli_error($connection));
        return [];
    }
}

// 调用函数时传入数据库连接，并接收返回值
$type = 'RIDDOR - Major Injury';
$counter = riddor_dates($type, $id, $id1, $datatable, $date_from, $date_to, $connection);
var_dump($counter);

这样修改后，不仅解决了返回值的问题，还让代码更安全、高效，也更符合PHP数据库开发的最佳实践～

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