Solidity随机数组函数返回长度为0，求获取正确数组长度方法

问题根源 #

你的代码存在几个关键问题，直接导致返回数组异常：

• result和len变量未声明，编译无法通过，更无法正常返回结果
• return result;之后的代码永远不会执行，len = result.length;属于无效代码
• 如果调用时传入的population本身是空数组，函数自然会返回长度为0的数组

修复后的可行代码 #

function randomArray(uint[] memory population) public view returns(uint[] memory) {
    // 提前处理空数组情况
    if (population.length == 0) {
        return population;
    }

    uint a = population.length; 
    uint b = population.length; 

    for(uint i = 0; i < b ; i++){
        // 注意：block.timestamp作为随机源不安全，链上可预测，仅用于演示
        uint randNumber = (uint(keccak256(abi.encodePacked(block.timestamp, population[i]))) % a) + 1;
        uint interim = population[randNumber - 1];
        population[randNumber - 1] = population[a - 1];
        population[a - 1] = interim;
        a = a - 1;
    }

    // 直接返回处理后的memory数组
    return population;
}

核心修复说明 #

1. 清理无效代码：移除未声明的result和len相关语句，直接返回处理后的population数组
2. 空数组防护：增加判断逻辑，避免传入空数组时执行无意义的循环
3. 安全提醒：block.timestamp作为随机数生成源存在可预测性风险，生产环境建议使用Chainlink VRF这类专业安全的随机数方案

验证方法 #

调用函数时传入非空数组（比如[1,2,3]），返回的数组长度会和传入的数组长度一致，不会出现长度为0的情况。

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