[Code review: Contract Bugs]: Duplicate code can be reused using modifier or function

[Zena-park]

Describe the bug

• The same code is used in multiple places. https://github.com/tokamak-network/gem-nft-contract/blob/a47ef0b91cb5d1ab3c19149619d3e1b8c9995225/src/L2/Treasury.sol#L90-L92 https://github.com/tokamak-network/gem-nft-contract/blob/a47ef0b91cb5d1ab3c19149619d3e1b8c9995225/src/L2/Treasury.sol#L97-L99 https://github.com/tokamak-network/gem-nft-contract/blob/a47ef0b91cb5d1ab3c19149619d3e1b8c9995225/src/L2/Treasury.sol#L104-L106 https://github.com/tokamak-network/gem-nft-contract/blob/a47ef0b91cb5d1ab3c19149619d3e1b8c9995225/src/L2/Treasury.sol#L111-L113 https://github.com/tokamak-network/gem-nft-contract/blob/a47ef0b91cb5d1ab3c19149619d3e1b8c9995225/src/L2/Treasury.sol#L118-L120 https://github.com/tokamak-network/gem-nft-contract/blob/a47ef0b91cb5d1ab3c19149619d3e1b8c9995225/src/L2/Treasury.sol#L125-L127 https://github.com/tokamak-network/gem-nft-contract/blob/a47ef0b91cb5d1ab3c19149619d3e1b8c9995225/src/L2/Treasury.sol#L133-L135 https://github.com/tokamak-network/gem-nft-contract/blob/a47ef0b91cb5d1ab3c19149619d3e1b8c9995225/src/L2/Treasury.sol#L140-L142 https://github.com/tokamak-network/gem-nft-contract/blob/a47ef0b91cb5d1ab3c19149619d3e1b8c9995225/src/L2/Treasury.sol#L147-L149 https://github.com/tokamak-network/gem-nft-contract/blob/a47ef0b91cb5d1ab3c19149619d3e1b8c9995225/src/L2/Treasury.sol#L162-L164 https://github.com/tokamak-network/gem-nft-contract/blob/a47ef0b91cb5d1ab3c19149619d3e1b8c9995225/src/L2/Treasury.sol#L175-L177

Impact

It can reduce the code byte size.

It makes code management easier.

Exploit Scenario

No response

Recommendation

Replace duplicate above code with _checkNonAddress functions.

  function _checkNonAddress(address account) internal  {
          if(account == address(0))   revert InvalidAddress();
   }

[mehdi-defiesta]

@Zena-park Thank you. Here is the update.

[Zena-park]

I confirmed. Thank you.