refactor: fixing codesmells from sonarcloud

cardano-foundation · May 7, 2024 · 61b3158 · 61b3158
1 parent cf30a02
commit 61b3158
Show file tree

Hide file tree

Showing 14 changed files with 37 additions and 49 deletions.
diff --git a/api/src/main/java/org/cardanofoundation/rosetta/api/block/model/domain/BlockTx.java b/api/src/main/java/org/cardanofoundation/rosetta/api/block/model/domain/BlockTx.java
@@ -21,7 +21,7 @@ public class BlockTx {
   protected String hash;
   protected String blockHash;
   protected Long blockNo;
-  protected String fee; // TODO can be removed if we found another way to calculate the size
+  protected String fee;
   protected Long size;
   protected Long scriptSize;
   protected List<Utxo> inputs;

diff --git a/api/src/main/java/org/cardanofoundation/rosetta/api/block/model/domain/ProtocolParams.java b/api/src/main/java/org/cardanofoundation/rosetta/api/block/model/domain/ProtocolParams.java
@@ -38,7 +38,7 @@ public class ProtocolParams {
   private ExtraEntropy extraEntropy; //13
   private ProtocolVersion protocolVersion; //14
   @JsonProperty("minUTxOValue")
-  private BigInteger minUtxo; //TODO //15
+  private BigInteger minUtxo; //15
 
   private BigInteger minPoolCost; //16
   private BigInteger adaPerUtxoByte; //17

diff --git a/.../main/java/org/cardanofoundation/rosetta/api/block/model/entity/ProtocolParamsEntity.java b/.../main/java/org/cardanofoundation/rosetta/api/block/model/entity/ProtocolParamsEntity.java
@@ -40,7 +40,7 @@ public class ProtocolParamsEntity {
   private String extraEntropy; //13
   private Integer protocolMajorVer; //14
   private Integer protocolMinorVer; //14
-  private BigInteger minUtxo; //TODO //15
+  private BigInteger minUtxo; //15
 
   private BigInteger minPoolCost; //16
   private BigInteger adaPerUtxoByte; //17

diff --git a/...rc/main/java/org/cardanofoundation/rosetta/api/network/controller/NetworkApiDelegate.java b/...rc/main/java/org/cardanofoundation/rosetta/api/network/controller/NetworkApiDelegate.java
@@ -3,6 +3,7 @@
 import java.io.IOException;
 
 import org.springframework.http.ResponseEntity;
+import org.springframework.web.bind.annotation.PostMapping;
 import org.springframework.web.bind.annotation.RequestMapping;
 import org.springframework.web.bind.annotation.RequestMethod;
 import io.swagger.v3.oas.annotations.Operation;
@@ -28,8 +29,7 @@ public interface NetworkApiDelegate {
                   @ApiResponse(responseCode = "400", description = "Invalid `body`")
           }
   )
-  @RequestMapping(
-          method = RequestMethod.POST,
+  @PostMapping(
           value = "/network/list",
           produces = { "application/json;charset=utf-8" },
           consumes = { "application/json;charset=utf-8" }
@@ -45,8 +45,7 @@ ResponseEntity<NetworkListResponse> networkList(MetadataRequest metadataRequest)
                   @ApiResponse(responseCode = "400", description = "Invalid `body`")
           }
   )
-  @RequestMapping(
-          method = RequestMethod.POST,
+  @PostMapping(
           value = "/network/options",
           produces = { "application/json;charset=utf-8" },
           consumes = { "application/json;charset=utf-8" }
@@ -62,8 +61,7 @@ ResponseEntity<NetworkOptionsResponse> networkOptions(NetworkRequest networkRequ
                   @ApiResponse(responseCode = "400", description = "Invalid `body`")
           }
   )
-  @RequestMapping(
-          method = RequestMethod.POST,
+  @PostMapping(
           value = "/network/status",
           produces = { "application/json;charset=utf-8" },
           consumes = { "application/json;charset=utf-8" }

diff --git a/.../main/java/org/cardanofoundation/rosetta/api/network/service/impl/NetworkServiceImpl.java b/.../main/java/org/cardanofoundation/rosetta/api/network/service/impl/NetworkServiceImpl.java
@@ -64,13 +64,12 @@ public class NetworkServiceImpl implements NetworkService {
   private final ResourceLoader resourceLoader;
 
   @PostConstruct
-  public void filePathExistingValidator() throws ServerException {
+  public void filePathExistingValidator() {
     validator(topologyFilepath);
     validator(genesisPath);
-//    validator(cardanoNodeVersion);
   }
 
-  private void validator( String path) throws ServerException {
+  private void validator( String path) {
     if(!new File(path).exists()) {
       throw ExceptionFactory.configNotFoundException();
     }
@@ -124,7 +123,6 @@ public NetworkOptionsResponse getNetworkOptions(NetworkRequest networkRequest)
                 .toList())
             .historicalBalanceLookup(true)
             .callMethods(new ArrayList<>())
-//            .balanceExemptions(loadExemptionsFile()) // TODO Removed to get it working clean - add balance exemptions
             .mempoolCoins(false))
         .build();
   }

diff --git a/api/src/main/java/org/cardanofoundation/rosetta/common/exception/GlobalExceptionHandler.java b/api/src/main/java/org/cardanofoundation/rosetta/common/exception/GlobalExceptionHandler.java
@@ -29,24 +29,22 @@ public ResponseEntity<Error> handleGlobalException(Exception exception,
     }
 
     Error errorResponse;
-
-    if (exception instanceof ApiException apiException) {
-
-      errorResponse = apiException.getError();
-
-    } else if ( exception instanceof MethodArgumentNotValidException methodArgumentNotValidException){
-      List<String> errors = methodArgumentNotValidException.getBindingResult().getFieldErrors()
-          .stream().map(FieldError::getDefaultMessage).toList();
-      errorResponse = RosettaErrorType.UNSPECIFIED_ERROR.toRosettaError(true,
-          Details.builder().message(
-              "An error occurred for request " + request.getRequestId() + ": "
-                  + errors ).build());
-    }
-    else {
-      errorResponse = RosettaErrorType.UNSPECIFIED_ERROR.toRosettaError(true,
-          Details.builder().message(
-              "An error occurred for request " + request.getRequestId() + ": "
-                  + exception.getMessage()).build());
+    switch (exception) {
+      case ApiException apiException:
+        errorResponse = apiException.getError();
+        break;
+      case MethodArgumentNotValidException methodArgumentNotValidException:
+        List<String> errors = methodArgumentNotValidException.getBindingResult().getFieldErrors()
+            .stream().map(FieldError::getDefaultMessage).toList();
+        errorResponse = RosettaErrorType.UNSPECIFIED_ERROR.toRosettaError(true,
+            Details.builder().message(
+                "An error occurred for request " + request.getRequestId() + ": "
+                    + errors ).build());
+      default:
+        errorResponse = RosettaErrorType.UNSPECIFIED_ERROR.toRosettaError(true,
+            Details.builder().message(
+                "An error occurred for request " + request.getRequestId() + ": "
+                    + exception.getMessage()).build());
     }
 
     return new ResponseEntity<>(errorResponse, HttpStatus.INTERNAL_SERVER_ERROR);

diff --git a/api/src/main/java/org/cardanofoundation/rosetta/common/interceptor/LogInterceptor.java b/api/src/main/java/org/cardanofoundation/rosetta/common/interceptor/LogInterceptor.java
@@ -28,14 +28,4 @@ public boolean preHandle(HttpServletRequest request, HttpServletResponse respons
 
         return true;
     }
-
-    @Override
-    public void postHandle(HttpServletRequest httpServletRequest, HttpServletResponse httpServletResponse, Object o, ModelAndView modelAndView) throws Exception {
-
-    }
-
-    @Override
-    public void afterCompletion(HttpServletRequest httpServletRequest, HttpServletResponse httpServletResponse, Object o, Exception e) throws Exception {
-
-    }
 }
diff --git a/api/src/main/java/org/cardanofoundation/rosetta/common/mapper/CborMapToOperation.java b/api/src/main/java/org/cardanofoundation/rosetta/common/mapper/CborMapToOperation.java
@@ -37,6 +37,8 @@
 
 public class CborMapToOperation {
 
+  private CborMapToOperation() {}
+
   public static Operation cborMapToOperation(Map operationMap) {
     Operation operation = new Operation();
 

diff --git a/.../main/java/org/cardanofoundation/rosetta/common/mapper/CborMapToTransactionExtraData.java b/.../main/java/org/cardanofoundation/rosetta/common/mapper/CborMapToTransactionExtraData.java
@@ -14,6 +14,8 @@
 
 public class CborMapToTransactionExtraData {
 
+  private CborMapToTransactionExtraData() {}
+
   public static TransactionExtraData convertCborMapToTransactionExtraData(Map map) {
     String transactionMetadataHex = getTransactionMetadataHexFromMap(map);
 

diff --git a/api/src/main/java/org/cardanofoundation/rosetta/common/mapper/DataMapper.java b/api/src/main/java/org/cardanofoundation/rosetta/common/mapper/DataMapper.java
@@ -141,7 +141,7 @@ public static Amount mapAmount(String value, String symbol, Integer decimals,
     amount.setCurrency(Currency.builder()
                             .symbol(symbol)
                             .decimals(decimals)
-                            .metadata(metadata) // TODO check metadata for Amount
+                            .metadata(metadata)
                             .build());
     return amount;
   }

diff --git a/api/src/main/java/org/cardanofoundation/rosetta/common/mapper/OperationToCborMap.java b/api/src/main/java/org/cardanofoundation/rosetta/common/mapper/OperationToCborMap.java
@@ -33,6 +33,8 @@
 
 public class OperationToCborMap {
 
+  private OperationToCborMap() {}
+
   public static Map convertToCborMap(Operation operation) {
     Map operationMap = new Map();
     // fill with nested objects
@@ -441,8 +443,8 @@ private static DataItem objectToDataItem(Object metadata) {
     Map map = new Map();
     HashMap<String, Object> metadataMap = (HashMap<String, Object>) metadata;
     metadataMap.forEach((key, value) -> {
-      if (value instanceof String) {
-        map.put(new UnicodeString(key), new UnicodeString((String) value));
+      if (value instanceof String valueString) {
+        map.put(new UnicodeString(key), new UnicodeString(valueString));
       } else {
         map.put(new UnicodeString(key), objectToDataItem(value));
       }

diff --git a/api/src/main/java/org/cardanofoundation/rosetta/common/model/cardano/ProtocolParameters.java b/api/src/main/java/org/cardanofoundation/rosetta/common/model/cardano/ProtocolParameters.java
@@ -81,7 +81,7 @@ private String toIndentedString(Object o) {
 
   public ProtocolParameters fromEntity(ProtocolParams entity) {
     return ProtocolParameters.builder()
-        .coinsPerUtxoSize(entity.getAdaPerUtxoByte().toString()) // TODO check if it's the right value
+        .coinsPerUtxoSize(entity.getAdaPerUtxoByte().toString())
         .maxTxSize(entity.getMaxTxSize())
         .maxValSize(BigInteger.valueOf(entity.getMaxValSize()))
         .keyDeposit(entity.getKeyDeposit().toString())

diff --git a/api/src/main/java/org/cardanofoundation/rosetta/common/services/impl/LoggingServiceImpl.java b/api/src/main/java/org/cardanofoundation/rosetta/common/services/impl/LoggingServiceImpl.java
@@ -69,7 +69,7 @@ private Map<String, String> buildParametersMap(HttpServletRequest httpServletReq
   private Map<String, String> buildHeadersMap(HttpServletRequest request) {
     Map<String, String> map = new HashMap<>();
 
-    Enumeration headerNames = request.getHeaderNames();
+    Enumeration<String> headerNames = request.getHeaderNames();
     while (headerNames.hasMoreElements()) {
       String key = (String) headerNames.nextElement();
       String value = request.getHeader(key);

diff --git a/api/src/main/java/org/cardanofoundation/rosetta/common/util/ValidateParseUtil.java b/api/src/main/java/org/cardanofoundation/rosetta/common/util/ValidateParseUtil.java
@@ -233,13 +233,11 @@ public static PoolRegistationParametersReturn validateAndParsePoolRegistationPar
             }
         }
         try {
-            PoolRegistationParametersReturn poolRegistationParametersReturnDto = new PoolRegistationParametersReturn(
+            return new PoolRegistationParametersReturn(
                 valueOf(Long.parseLong(poolRegistrationParameters.getCost())),
                 valueOf(Long.parseLong(poolRegistrationParameters.getPledge())),
                 valueOf(Long.parseLong(numerator)),
                 valueOf(Long.parseLong(denominator)));
-
-            return poolRegistationParametersReturnDto;
         } catch (Exception error) {
             log.error("[validateAndParsePoolRegistationParameters] Given pool parameters are invalid");
             throw ExceptionFactory.invalidPoolRegistrationParameters(error.getMessage());