Refactor

src/main/java/com/wechi/adweb/bridge/google/ads/controller/GoogleAdsController.java

@@ -145,7 +145,9 @@ public class GoogleAdsController extends BaseController {
 
         // 2. Executes the API request.
         Map<String, MetricsDTO> placementStats = googleAdsService.getPlacementStats(reportRequest);
-        log.info("****** getKeywordStats() ****** duration = {} seconds", getElapsedSeconds(start));
+        log.info(
+                "****** getPlacementStats() ****** duration = {} seconds",
+                getElapsedSeconds(start));
         return OpenAPIResponse.<Map<String, MetricsDTO>>builder()
                 .status(APIStatus.SUCCESS)
                 .data(placementStats)

src/main/java/com/wechi/adweb/bridge/google/ads/dto/MetricsDTO.java

@@ -24,9 +24,9 @@ public class MetricsDTO {
         this.impressions += metricsDTO.impressions;
         this.clicks += metricsDTO.clicks;
 
-        this.ctr += metricsDTO.ctr;
-        this.averageCpc += metricsDTO.averageCpc;
-        this.averageCpm += metricsDTO.averageCpm;
+        this.ctr = Math.max(this.ctr, metricsDTO.ctr);
+        this.averageCpc = Math.max(this.averageCpc, metricsDTO.averageCpc);
+        this.averageCpm = Math.max(this.averageCpm, metricsDTO.averageCpm);
 
         this.conversions += metricsDTO.conversions;