Sepolia测试网Solidity合约getAllUsers返回空数组求助

核心问题：注册函数逻辑完全写反 #

你的register函数条件判断逻辑错误，导致从未真正执行注册逻辑，userIDs数组始终为空，因此getAllUsers必然返回空数组。

原代码中的判断：

if (bytes(users[id].email).length == 0 || bytes(users[id].password).length == 0){
    return false;
}

这个逻辑的意思是：当用户不存在时（mapping默认返回空结构体，email/password均为空字符串，长度为0），直接返回false，跳过后续注册步骤。也就是说，只有当用户已经存在时，才会执行注册逻辑——这完全和预期相反。

修正方案 #

把判断条件反转，改为如果用户已存在（email/password不为空），则返回false拒绝注册，否则执行注册：

function register(string memory id, string memory email, string memory password) public returns (bool) {
    // 检查用户是否已存在，存在则返回false
    if (bytes(users[id].email).length != 0 || bytes(users[id].password).length != 0){
        return false;
    }
    UserData memory newUser = UserData(email, password, false);
    users[id] = newUser;
    userIDs.push(id);
    return true;
}

更严谨的写法可以结合deleted字段判断（避免重复添加已删除用户的ID）：

function register(string memory id, string memory email, string memory password) public returns (bool) {
    // 已存在且未删除的用户不能重复注册
    if (!users[id].deleted && bytes(users[id].email).length != 0) {
        return false;
    }
    UserData memory newUser = UserData(email, password, false);
    users[id] = newUser;
    // 如果是恢复已删除用户，不需要重复添加id到数组
    if (users[id].deleted) {
        users[id].deleted = false;
    } else {
        userIDs.push(id);
    }
    return true;
}

其他注意事项 #

1. 交易执行结果的误解：你提到“注册功能运行正常”，但实际上原函数在用户不存在时会返回false，但以太坊交易只要没有revert就会被打包，所以你可能误以为交易成功就是注册成功，实际上合约内部并未执行注册逻辑。建议用require替代返回false，这样注册失败时交易会回滚，更容易发现问题：

require(bytes(users[id].email).length == 0 && bytes(users[id].password).length == 0, "User already exists");

2. view函数调用方式：getAllUsers是view函数，不需要用send()调用，send()用于发送修改状态的交易。直接用call()或合约实例直接调用即可，send()调用view函数不会改变状态，也无法得到正确的查询结果。

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