I don't really think this is an issue at all. This is a code style suggestion, and in my opinion, both are equally clear. Yes, just create a pull request and we can improve the style. Hi @iluwatar  
I can take this up Ok @kanwarpreet25 
Unnecesary return
https://github.com/iluwatar/java-design-patterns/blob/55c75799831716b6248ceefa1315881e7f90806b/spatial-partition/src/main/java/com/iluwatar/spatialpartition/QuadTree.java#L58

you can change this block
`
    if (!this.boundary.contains(p)) {
      return;
    } else {
`

to this other most clear:
`
    if (this.boundary.contains(p)) {
`


spatial-partition/src/main/java/com/iluwatar/spatialpartition/QuadTree.java
spatial-partition/src/main/java/com/iluwatar/spatialpartition/QuadTree.java
diff --git a/spatial-partition/src/main/java/com/iluwatar/spatialpartition/QuadTree.java b/spatial-partition/src/main/java/com/iluwatar/spatialpartition/QuadTree.java
index 072468ce80d4..815bf8acc74c 100644
--- a/spatial-partition/src/main/java/com/iluwatar/spatialpartition/QuadTree.java
+++ b/spatial-partition/src/main/java/com/iluwatar/spatialpartition/QuadTree.java
@@ -55,16 +55,13 @@ public class QuadTree {
   }
 
   void insert(Point p) {
-    if (!this.boundary.contains(p)) {
-      return;
-    } else {
+    if (this.boundary.contains(p)) {
       if (this.points.size() < this.capacity) {
         points.put(p.id, p);
       } else {
         if (!this.divided) {
           this.divide();
         }
-
         if (this.northwest.boundary.contains(p)) {
           this.northwest.insert(p);
         } else if (this.northeast.boundary.contains(p)) {
