Avoid using loop as it can reach the gas limit.
If there is too much users, the transaction can revert when reaching the gas limit, and so the smart contract is not usable anymore...
Use a mapping, or an EnumerableMap from OpenZeppelin.
It can also simplify the getActivePlayerIndex function.
The contest is live. Earn rewards by submitting a finding.
This is your time to appeal against judgements on your submissions.
Appeals are being carefully reviewed by our judges.