Solidity开发求助：基于Chainlink VRF的抽奖合约门票映射问题

问题场景 #

开发学习用抽奖小游戏，核心需求：

• 玩家可购买任意数量门票
• 每个ticketId（从1开始递增）与玩家地址绑定
• 所有ticketId存入数组，供后续Chainlink VRF随机抽取获奖者
• 示例：玩家1买3张、玩家2买4张，数组应为[1,2,3,4,5,6,7]，每个ID对应各自玩家地址

原代码核心错误 #

1. 固定数组长度变量失效：arrayLength是合约部署时初始化的固定值（0），不会随数组长度动态更新，导致每次调用set时循环起始值始终为1，重复生成重复的ticketId。
2. 映射索引逻辑错误：myMap[ticketId[uint(i)]]错误地用ticketId的数值作为数组索引（数组索引从0开始），会引发越界访问或关联错误地址。

修复后的合约代码 #

contract LotteryGame {
    // 映射ticketId到对应玩家地址
    mapping(uint => address payable) public ticketToPlayer;
    // 存储所有已售出的ticketId
    uint[] public allTicketIds;

    function buyTickets(uint _ticketCount) public {
        // 实时获取当前已售门票总数，作为新ticketId的起始基准
        uint currentTotalTickets = allTicketIds.length;
        // 循环生成对应数量的ticketId
        for(uint i = 1; i <= _ticketCount; i++) {
            uint newTicketId = currentTotalTickets + i;
            allTicketIds.push(newTicketId);
            // 直接用新生成的ticketId绑定玩家地址
            ticketToPlayer[newTicketId] = payable(msg.sender);
        }
    }
}

关键修改说明 #

• 移除固定的arrayLength变量，每次调用buyTickets时实时读取allTicketIds.length，确保新ticketId从正确数值开始递增。
• 直接使用计算出的newTicketId作为映射键，避免数组索引与ticketId数值的混淆，确保每个ticketId正确关联购买玩家。
• 优化函数和变量命名，提升代码语义性与可维护性。

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