Clean

microsoft · Jul 28, 2022 · 572d3ff · 572d3ff
1 parent 4e7bd34
commit 572d3ff
Show file tree

Hide file tree

Showing 14 changed files with 241 additions and 235 deletions.
diff --git a/...e/src/main/java/com/microsoft/applicationinsights/extensibility/context/CloudContext.java b/...e/src/main/java/com/microsoft/applicationinsights/extensibility/context/CloudContext.java
@@ -24,7 +24,7 @@
 import com.microsoft.applicationinsights.internal.util.MapUtil;
 import java.util.concurrent.ConcurrentMap;
 
-public class CloudContext {
+public final class CloudContext {
 
   private final ConcurrentMap<String, String> tags;
 

diff --git a/.../src/main/java/com/microsoft/applicationinsights/extensibility/context/DeviceContext.java b/.../src/main/java/com/microsoft/applicationinsights/extensibility/context/DeviceContext.java
@@ -28,6 +28,10 @@ public final class DeviceContext {
 
   private final ConcurrentMap<String, String> tags;
 
+  public DeviceContext(ConcurrentMap<String, String> tags) {
+    this.tags = tags;
+  }
+
   public void setType(String type) {
     MapUtil.setStringValueOrRemove(tags, ContextTagKeys.getKeys().getDeviceType(), type);
   }
@@ -69,8 +73,4 @@ public void setLocale(String locale) {
   public void setLanguage(String language) {
     MapUtil.setStringValueOrRemove(tags, ContextTagKeys.getKeys().getDeviceLanguage(), language);
   }
-
-  public DeviceContext(ConcurrentMap<String, String> tags) {
-    this.tags = tags;
-  }
 }
diff --git a/...e/src/main/java/com/microsoft/applicationinsights/internal/schemav2/AvailabilityData.java b/...e/src/main/java/com/microsoft/applicationinsights/internal/schemav2/AvailabilityData.java
@@ -40,72 +40,72 @@ public class AvailabilityData {
   public AvailabilityData() {}
 
   public String getId() {
-    return this.id;
+    return id;
   }
 
-  public void setId(String value) {
-    this.id = value;
+  public void setId(String id) {
+    this.id = id;
   }
 
   public String getName() {
-    return this.name;
+    return name;
   }
 
-  public void setName(String value) {
-    this.name = value;
+  public void setName(String name) {
+    this.name = name;
   }
 
   public Duration getDuration() {
-    return this.duration;
+    return duration;
   }
 
-  public void setDuration(Duration value) {
-    this.duration = value;
+  public void setDuration(Duration duration) {
+    this.duration = duration;
   }
 
   public boolean getSuccess() {
-    return this.success;
+    return success;
   }
 
-  public void setSuccess(boolean value) {
-    this.success = value;
+  public void setSuccess(boolean success) {
+    this.success = success;
   }
 
   public String getRunLocation() {
-    return this.runLocation;
+    return runLocation;
   }
 
-  public void setRunLocation(String value) {
-    this.runLocation = value;
+  public void setRunLocation(String runLocation) {
+    this.runLocation = runLocation;
   }
 
   public String getMessage() {
-    return this.message;
+    return message;
   }
 
-  public void setMessage(String value) {
-    this.message = value;
+  public void setMessage(String message) {
+    this.message = message;
   }
 
   public ConcurrentMap<String, String> getProperties() {
-    if (this.properties == null) {
-      this.properties = new ConcurrentHashMap<>();
+    if (properties == null) {
+      properties = new ConcurrentHashMap<>();
     }
-    return this.properties;
+    return properties;
   }
 
-  public void setProperties(ConcurrentMap<String, String> value) {
-    this.properties = value;
+  public void setProperties(ConcurrentMap<String, String> properties) {
+    this.properties = properties;
   }
 
   public ConcurrentMap<String, Double> getMeasurements() {
-    if (this.measurements == null) {
-      this.measurements = new ConcurrentHashMap<>();
+    if (measurements == null) {
+      measurements = new ConcurrentHashMap<>();
     }
-    return this.measurements;
+    return measurements;
   }
 
-  public void setMeasurements(ConcurrentMap<String, Double> value) {
-    this.measurements = value;
+  public void setMeasurements(ConcurrentMap<String, Double> measurements) {
+    this.measurements = measurements;
   }
 }
diff --git a/...sdk/core/src/main/java/com/microsoft/applicationinsights/internal/schemav2/DataPoint.java b/...sdk/core/src/main/java/com/microsoft/applicationinsights/internal/schemav2/DataPoint.java
@@ -21,71 +21,77 @@
 
 package com.microsoft.applicationinsights.internal.schemav2;
 
+import javax.annotation.Nullable;
+
 public class DataPoint {
 
   private String name;
   private DataPointType kind = DataPointType.Measurement;
   private double value;
-  private Integer count;
-  private Double min;
-  private Double max;
-  private Double stdDev;
+  @Nullable private Integer count;
+  @Nullable private Double min;
+  @Nullable private Double max;
+  @Nullable private Double stdDev;
 
   public DataPoint() {}
 
   public String getName() {
-    return this.name;
+    return name;
   }
 
-  public void setName(String value) {
-    this.name = value;
+  public void setName(String name) {
+    this.name = name;
   }
 
   public DataPointType getKind() {
-    return this.kind;
+    return kind;
   }
 
-  public void setKind(DataPointType value) {
-    this.kind = value;
+  public void setKind(DataPointType kind) {
+    this.kind = kind;
   }
 
   public double getValue() {
-    return this.value;
+    return value;
   }
 
   public void setValue(double value) {
     this.value = value;
   }
 
+  @Nullable
   public Integer getCount() {
-    return this.count;
+    return count;
   }
 
-  public void setCount(Integer value) {
-    this.count = value;
+  public void setCount(@Nullable Integer count) {
+    this.count = count;
   }
 
+  @Nullable
   public Double getMin() {
-    return this.min;
+    return min;
   }
 
-  public void setMin(Double value) {
-    this.min = value;
+  public void setMin(@Nullable Double min) {
+    this.min = min;
   }
 
+  @Nullable
   public Double getMax() {
-    return this.max;
+    return max;
   }
 
-  public void setMax(Double value) {
-    this.max = value;
+  public void setMax(@Nullable Double max) {
+    this.max = max;
   }
 
+  @Nullable
   public Double getStdDev() {
-    return this.stdDev;
+    return stdDev;
   }
 
-  public void setStdDev(Double value) {
-    this.stdDev = value;
+  public void setStdDev(@Nullable Double stdDev) {
+    this.stdDev = stdDev;
   }
 }
diff --git a/...sdk/core/src/main/java/com/microsoft/applicationinsights/internal/schemav2/EventData.java b/...sdk/core/src/main/java/com/microsoft/applicationinsights/internal/schemav2/EventData.java
@@ -34,32 +34,32 @@ public class EventData {
   public EventData() {}
 
   public String getName() {
-    return this.name;
+    return name;
   }
 
-  public void setName(String value) {
-    this.name = value;
+  public void setName(String name) {
+    this.name = name;
   }
 
   public ConcurrentMap<String, String> getProperties() {
-    if (this.properties == null) {
-      this.properties = new ConcurrentHashMap<>();
+    if (properties == null) {
+      properties = new ConcurrentHashMap<>();
     }
-    return this.properties;
+    return properties;
   }
 
-  public void setProperties(ConcurrentMap<String, String> value) {
-    this.properties = value;
+  public void setProperties(ConcurrentMap<String, String> properties) {
+    this.properties = properties;
   }
 
   public ConcurrentMap<String, Double> getMeasurements() {
-    if (this.measurements == null) {
-      this.measurements = new ConcurrentHashMap<>();
+    if (measurements == null) {
+      measurements = new ConcurrentHashMap<>();
     }
-    return this.measurements;
+    return measurements;
   }
 
-  public void setMeasurements(ConcurrentMap<String, Double> value) {
-    this.measurements = value;
+  public void setMeasurements(ConcurrentMap<String, Double> measurements) {
+    this.measurements = measurements;
   }
 }
diff --git a/...core/src/main/java/com/microsoft/applicationinsights/internal/schemav2/ExceptionData.java b/...core/src/main/java/com/microsoft/applicationinsights/internal/schemav2/ExceptionData.java
@@ -38,51 +38,51 @@ public class ExceptionData {
   public ExceptionData() {}
 
   public List<ExceptionDetails> getExceptions() {
-    if (this.exceptions == null) {
-      this.exceptions = new ArrayList<>();
+    if (exceptions == null) {
+      exceptions = new ArrayList<>();
     }
-    return this.exceptions;
+    return exceptions;
   }
 
-  public void setExceptions(List<ExceptionDetails> value) {
-    this.exceptions = value;
+  public void setExceptions(List<ExceptionDetails> exceptions) {
+    this.exceptions = exceptions;
   }
 
   public SeverityLevel getSeverityLevel() {
-    return this.severityLevel;
+    return severityLevel;
   }
 
-  public void setSeverityLevel(SeverityLevel value) {
-    this.severityLevel = value;
+  public void setSeverityLevel(SeverityLevel severityLevel) {
+    this.severityLevel = severityLevel;
   }
 
   public String getProblemId() {
-    return this.problemId;
+    return problemId;
   }
 
-  public void setProblemId(String value) {
-    this.problemId = value;
+  public void setProblemId(String problemId) {
+    this.problemId = problemId;
   }
 
   public ConcurrentMap<String, String> getProperties() {
-    if (this.properties == null) {
-      this.properties = new ConcurrentHashMap<>();
+    if (properties == null) {
+      properties = new ConcurrentHashMap<>();
     }
-    return this.properties;
+    return properties;
   }
 
-  public void setProperties(ConcurrentMap<String, String> value) {
-    this.properties = value;
+  public void setProperties(ConcurrentMap<String, String> properties) {
+    this.properties = properties;
   }
 
   public ConcurrentMap<String, Double> getMeasurements() {
-    if (this.measurements == null) {
-      this.measurements = new ConcurrentHashMap<>();
+    if (measurements == null) {
+      measurements = new ConcurrentHashMap<>();
     }
-    return this.measurements;
+    return measurements;
   }
 
-  public void setMeasurements(ConcurrentMap<String, Double> value) {
-    this.measurements = value;
+  public void setMeasurements(ConcurrentMap<String, Double> measurements) {
+    this.measurements = measurements;
   }
 }