Project

Vulnerability Details

_whitelistedCurrencies.length in currencymanager.sol is read from storage when trying to access function viewWhitelistedCurrencies . we can save gas by assigning it to memory

https://github.com/Cyfrin/2024-11-one-world/blob/1e872c7ab393c380010a507398d4b4caca1ae32b/contracts/dao/CurrencyManager.sol#L92

Impact

more gas cost

Tools Used

manual view

Recommendations

assign _whitelistedCurrencies.length to a local variable
example
uint256 totalCurrencies = _whitelistedCurrencies.length();