1 min readMay 17, 2019
Hi Nico,
as you say, there is no right or wrong, all depends on your specific use case.
However, looking back at the blockchain history should be acceptable for audits, even if uncomfortable for administration. Since the code is also clearer by using a mapping and this article is intended to teach other developers, I changed RBAC.sol according to your suggestion. I even did some extra work to show the gas usage of hasRole.
Thanks again!