shopizer-ecommerce / shopizer

Shopizer java e-commerce software
http://www.shopizer.com
Apache License 2.0
3.52k stars 2.98k forks source link

Refactored code to remove design and implementation smells #964

Open Hatim001 opened 10 months ago

Hatim001 commented 10 months ago

Things Done -

Implementation smells refactoring:

  1. Extract Method

    • File -> sm-shop/src/main/java/com/salesmanager/shop/utils/EnumValidator.java
    • Class -> EnumValidator
    • Method -> isValid
  2. Decompose Conditional

    • File -> sm-shop/src/main/java/com/salesmanager/shop/store/controller/shoppingCart/facade/ShoppingCartFacadeImpl.java
    • Class -> ShoppingCartFacadeImpl
    • Method -> createCartItems
  3. Rename method/variable

    • File -> sm-core/src/main/java/com/salesmanager/core/business/utils/DataUtils.java
    • Class -> DataUtils
    • Method -> getWeight

Design smells refactoring:

  1. Pull-up method

    • File -> sm-core-model/src/main/java/com/salesmanager/core/model/common/description/Description.java
    • Class -> Description
    • Method -> Description
  2. Extract Class

    • File -> sm-core/src/main/java/com/salesmanager/core/business/configuration/DataConfiguration.java
    • Class -> DataConfiguration
    • Method -> additionalProperties
  3. Push-down method

    • File -> sm-core-model/src/main/java/com/salesmanager/core/model/common/Criteria.java
    • Class -> Criteria
    • Method -> setLegacyPagination
sonarcloud[bot] commented 10 months ago

Kudos, SonarCloud Quality Gate passed!    Quality Gate passed

Bug A 0 Bugs
Vulnerability A 0 Vulnerabilities
Security Hotspot A 0 Security Hotspots
Code Smell A 0 Code Smells

No Coverage information No Coverage information
No Duplication information No Duplication information