Sonar fix too many method param 74/82574/1
authorArindam Mondal <arind.mondal@samsung.com>
Mon, 18 Mar 2019 09:54:39 +0000 (18:54 +0900)
committerarind.mondal <arind.mondal@samsung.com>
Mon, 18 Mar 2019 09:55:00 +0000 (18:55 +0900)
https://sonar.onap.org/issues?projectUuids=AV7CYEb_WvyQfiQiM1ZX&resolved=false&rules=squid%3AS00107

Issue-ID: VFC-1277
Change-Id: I73310edc9488b01a386d957ac4c9fd3d439cb021
Signed-off-by: arind.mondal <arind.mondal@samsung.com>
nokia/vnfmdriver/vfcadaptorservice/vfcadaptor/src/main/java/org/onap/vfc/nfvo/driver/vnfm/svnfm/adaptor/HealVnfContinueRunnable.java
nokia/vnfmdriver/vfcadaptorservice/vfcadaptor/src/main/java/org/onap/vfc/nfvo/driver/vnfm/svnfm/vnfmdriver/impl/VnfContinueProcessorImpl.java

index 63cb09b..ca51a1d 100644 (file)
@@ -1,5 +1,6 @@
 /**
  * Copyright 2016-2017, Nokia Corporation.
+ * Modifications Copyright (C) 2019 Samsung Electronics Co., Ltd.
  *
  * Licensed under the Apache License, Version 2.0 (the "License");
  * you may not use this file except in compliance with the License.
@@ -52,22 +53,79 @@ public class HealVnfContinueRunnable implements Runnable {
        private VnfmJobExecutionMapper jobDbMgmr;
        
        private Driver2CbamRequestConverter requestConverter;
+
+       ///
+
+       public static class HealVnfContinueRunnableBuilder {
+               private String inVnfmId;
+               private HealVnfRequest driverRequest;
+               private String vnfInstanceId;
+               private String jobId;
+               private NslcmMgmrInf nslcmMgmr;
+               private CbamMgmrInf cbamMgmr;
+               private Driver2CbamRequestConverter requestConverter;
+               private VnfmJobExecutionMapper dbManager;
+
+               public HealVnfContinueRunnableBuilder setInVnfmId(String inVnfmId) {
+                       this.inVnfmId = inVnfmId;
+                       return this;
+               }
+
+               public HealVnfContinueRunnableBuilder setDriverRequest(HealVnfRequest driverRequest) {
+                       this.driverRequest = driverRequest;
+                       return this;
+               }
+
+               public HealVnfContinueRunnableBuilder setVnfInstanceId(String vnfInstanceId) {
+                       this.vnfInstanceId = vnfInstanceId;
+                       return this;
+               }
+
+               public HealVnfContinueRunnableBuilder setJobId(String jobId) {
+                       this.jobId = jobId;
+                       return this;
+               }
+
+               public HealVnfContinueRunnableBuilder setNslcmMgmr(NslcmMgmrInf nslcmMgmr) {
+                       this.nslcmMgmr = nslcmMgmr;
+                       return this;
+               }
+
+               public HealVnfContinueRunnableBuilder setCbamMgmr(CbamMgmrInf cbamMgmr) {
+                       this.cbamMgmr = cbamMgmr;
+                       return this;
+               }
+
+               public HealVnfContinueRunnableBuilder setRequestConverter(Driver2CbamRequestConverter requestConverter) {
+                       this.requestConverter = requestConverter;
+                       return this;
+               }
+
+               public HealVnfContinueRunnableBuilder setDbManager(VnfmJobExecutionMapper dbManager) {
+                       this.dbManager = dbManager;
+                       return this;
+               }
+
+               public HealVnfContinueRunnable build() {
+                       return new HealVnfContinueRunnable(this);
+               }
+       }
+
+       ///
        
        
-       public HealVnfContinueRunnable(String inVnfmId, HealVnfRequest driverRequest, String vnfInstanceId, String jobId,
-                       NslcmMgmrInf nslcmMgmr, CbamMgmrInf cbamMgmr, Driver2CbamRequestConverter requestConverter, VnfmJobExecutionMapper dbManager)
-       {
-               this.driverRequest = driverRequest;
-               this.vnfInstanceId = vnfInstanceId;
-               this.nslcmMgmr = nslcmMgmr; 
-               this.cbamMgmr = cbamMgmr;
-               this.requestConverter = requestConverter;
-               this.jobId = jobId;
-               this.jobDbMgmr = dbManager;
-               this.vnfmId = inVnfmId;
+       public HealVnfContinueRunnable(HealVnfContinueRunnableBuilder builder) {
+           this.driverRequest = builder.driverRequest;
+        this.vnfInstanceId = builder.vnfInstanceId;
+        this.nslcmMgmr = builder.nslcmMgmr; 
+        this.cbamMgmr = builder.cbamMgmr;
+        this.requestConverter = builder.requestConverter;
+        this.jobId = builder.jobId;
+        this.jobDbMgmr = builder.dbManager;
+        this.vnfmId = builder.inVnfmId;
        }
-       
-       private void handleGrant(){
+
+    private void handleGrant(){
                try {
                        NslcmGrantVnfRequest grantRequest = buildNslcmGrantVnfRequest();
                        nslcmMgmr.grantVnf(grantRequest);
index 0749920..0108d46 100644 (file)
@@ -73,11 +73,12 @@ public class VnfContinueProcessorImpl implements VnfContinueProcessorInf{
        @Override
        public void continueHealVnf(String vnfmId, HealVnfRequest driverRequest, String vnfInstanceId, String jobId,
                        NslcmMgmrInf nslcmMgmr, CbamMgmrInf cbamMgmr, Driver2CbamRequestConverter requestConverter,
-                       VnfmJobExecutionMapper jobDbManager) {
-               HealVnfContinueRunnable task = new HealVnfContinueRunnable(vnfmId, driverRequest, vnfInstanceId, jobId,
-                               nslcmMgmr, cbamMgmr, requestConverter, jobDbManager);
-               Executors.newSingleThreadExecutor().submit(task);
-               
-       }
+            VnfmJobExecutionMapper jobDbManager) {
+        HealVnfContinueRunnable task = new HealVnfContinueRunnable.HealVnfContinueRunnableBuilder().setInVnfmId(vnfmId)
+                .setDriverRequest(driverRequest).setVnfInstanceId(vnfInstanceId).setJobId(jobId).setNslcmMgmr(nslcmMgmr)
+                .setCbamMgmr(cbamMgmr).setRequestConverter(requestConverter).setDbManager(jobDbManager).build();
+        Executors.newSingleThreadExecutor().submit(task);
+
+    }
 
 }