feat: optimize gas usage

[quiet-node]

function inputs and outputs should be appropriately allocate using the data location keywords

• calldata (most of the time inputs should be marked as calldata as they do not need to be modified): for input arguments (cheaper gas used compared to memory) that are read-only, non-modifiable.

• storage vs memory inside function body. Use storage in case of updating multiple fields in a struct will be cheaper in terms of gas usage compared to memory (example needs fix: https://github.com/syns-platform/contracts/blob/main/contracts/v1/SynsMarketplace.sol#L398)

[quiet-node]

When you keccak256 to hash the name of the role, the input of the keccak256 should have bytes as the input rather than string literal. Passing bytes to keccak256 will be more gas efficient as Solidity implicitly do extra processes to convert string literal to bytes for the keccak256 function.

@https://github.com/syns-platform/contracts/blob/main/contracts/v1/SynsClub.sol#L42 and more

[quiet-node]

• calldata: replace memory with calldata in function inputs | up to 1k gas
• load state vars to memory: in looping, load state vars in memory | up to 1k gas
• short circuit: instead of declaring upfront the condition, use it inside if statement | up to 300 gas
• loop increments: in for loop, replace i +=1 with ++i | up to 400 gas
• cache array length | up to 100 gas
• load array elements to memory | up to 100 gas

resource: https://www.youtube.com/watch?v=4r20M9Fr8lY&list=PLO5VPQH6OWdVQwpQfw9rZ67O6Pjfo6q-p&index=62