Rock Paper Scissors

First Flight #38
Beginner FriendlySolidity
100 EXP
View results
Submission Details
Severity: low
Invalid

GameState mis-management & unused enum value

Description: The enum 'GameState' has a 'Revealed' state which is never used. Both commit and reveal phases share the same Committed state, making it easy to mix up allowed actions.

Impact: Increases risk of logic mistakes.

Recommended Mitigation: Use distinct states:

enum GameState { Created, Joined, Committed, Revealing, Finished, Cancelled }

Transition explicitly:
Created -> (join) -> Joined -> (both commit) -> Revealing -> (both reveal/timeout) -> Committed (next turn) or Finished/Cancelled.

Updates

Appeal created

m3dython Lead Judge 5 months ago
Submission Judgement Published
Invalidated
Reason: Non-acceptable severity

Support

FAQs

Can't find an answer? Chat with us on Discord, Twitter or Linkedin.