Merge pull request #372 from DevFactory/release/Methods-should-not-be-empty-fix-1

squid:S1186 - Methods should not be empty

null-object/src/main/java/com/iluwatar/nullobject/NullNode.java

@@ -60,5 +60,7 @@ public class NullNode implements Node {
   }
 
   @Override
-  public void walk() {}
+  public void walk() {
+    // Do nothing
+  }
 }

visitor/src/main/java/com/iluwatar/visitor/CommanderVisitor.java

@@ -30,10 +30,14 @@ package com.iluwatar.visitor;
 public class CommanderVisitor implements UnitVisitor {
 
   @Override
-  public void visitSoldier(Soldier soldier) {}
+  public void visitSoldier(Soldier soldier) {
+    // Do nothing
+  }
 
   @Override
-  public void visitSergeant(Sergeant sergeant) {}
+  public void visitSergeant(Sergeant sergeant) {
+    // Do nothing
+  }
 
   @Override
   public void visitCommander(Commander commander) {

visitor/src/main/java/com/iluwatar/visitor/SergeantVisitor.java

@@ -30,7 +30,9 @@ package com.iluwatar.visitor;
 public class SergeantVisitor implements UnitVisitor {
 
   @Override
-  public void visitSoldier(Soldier soldier) {}
+  public void visitSoldier(Soldier soldier) {
+    // Do nothing
+  }
 
   @Override
   public void visitSergeant(Sergeant sergeant) {
@@ -38,5 +40,7 @@ public class SergeantVisitor implements UnitVisitor {
   }
 
   @Override
-  public void visitCommander(Commander commander) {}
+  public void visitCommander(Commander commander) {
+    // Do nothing
+  }
 }

visitor/src/main/java/com/iluwatar/visitor/SoldierVisitor.java

@@ -35,8 +35,12 @@ public class SoldierVisitor implements UnitVisitor {
   }
 
   @Override
-  public void visitSergeant(Sergeant sergeant) {}
+  public void visitSergeant(Sergeant sergeant) {
+    // Do nothing
+  }
 
   @Override
-  public void visitCommander(Commander commander) {}
+  public void visitCommander(Commander commander) {
+    // Do nothing
+  }
 }