Omniscia Arcade XYZ Audit

Code Style

Code Style

During the manual portion of the audit, we identified 27 optimizations that can be applied to the codebase that will decrease the operational cost associated with the execution of a particular function and generally ensure that the project complies with the latest best practices and standards in Solidity.

Additionally, this section of the audit contains any opinionated adjustments we believe the code should make to make it more legible as well as truer to its purpose.

These optimizations are enumerated below:

IDSeverityAddressedTitle
AIV-01CLoop Iterator Optimization
AIV-02COptimization of if Structure
ABV-01CIneffectual Usage of Safe Arithmetics
ABV-02CLoop Iterator Optimizations
AVT-01CLoop Iterator Optimization
AVT-02CMisconception of Re-Entrancy Attack Vectors
AVT-03COptimization of if-else Structure
AVT-04CPotential Enhancement of Functionality
AVT-05CPotential Optimization of Project Structure
CWT-01CInefficient mapping Lookups
FCR-01CRedundant Parenthesis Statements
FCR-02CSuboptimal Struct Declaration Styles
LCE-01CIneffectual Usage of Safe Arithmetics
LCE-02CInefficient Loop Limit Evaluation
LCE-03CInefficient mapping Lookups
LCE-04CLoop Iterator Optimizations
LCE-05COptimization of Assignments
LCE-06CSuboptimal Struct Declaration Style
LLY-01CInefficient Tight Packing of Structs
OCR-01CIneffectual Usage of Safe Arithmetics
OCR-02CLoop Iterator Optimizations
OER-01CPotential Optimization of Project Structure
PNE-01CInefficient Renunciation of Role
PVR-01CLoop Iterator Optimization
PVR-02CRedundant Usage of SafeCast
PVR-03CUndocumented Predicate Feature
RCR-01CRedundant Conditional Evaluation