MyCut

Summary

Storing players in i_players and rewards in i_rewards is unnecessary. Only i_players.length is used, and it can be stored as uint256 (Additionally, there’s no need to store players.length since it should be claimants.length).

Impact

This results in unnecessary storage use, and reading from storage is more gas-expensive than reading from memory (e.g., in the constructor's for loop).

Recommendation

Remove the i_players and i_rewards variables.