Gas Optimizations

[code423n4]

File: Community.sol

Recommendation : Change storage to memory

line 184

line 230

line 648

line 738

Recommendation : Change code order

Line can be moved before line 184

Recommendation : Remove unnecessary variable

_community.memberCount can be used directly in lines 198 and 199

Recommendation : Remove repeated require block

This syntax gets used four times so it could be a modifier in all instances except in the escrow():

require(
_msgSender() == _communities[_communityID].owner,
"Community::!Owner"
);

Likewise on line 386it is not capitalised unlike the others

"Community::!owner"

Recommendation : Remove unecessary return declaration

type name of sender not necessary line 903

Recommendation : Remove unecessary asignment

Can be set by default to true instea of doing so explicitely

File : HomeFi.sol

Recommendation : Shorten shorten ipfs (CID v1) hash to 32 bytes
line 284

mintNFT function, parameter string memory _tokenURI can be shortened to create a bytes32 type, rather than a string that stores 128 bytes for IPFS CIDs version 0.