style: enable RedundantModifier in checkstyle (TheAlgorithms#5140)

vil02 · web-flow · commit b3903f57686e · 2024-05-03T22:10:49.000+03:00
diff --git a/checkstyle.xml b/checkstyle.xml
@@ -148,7 +148,7 @@
     <!-- Modifier Checks                                    -->
     <!-- See https://checkstyle.org/checks/modifier/index.html -->
     <module name="ModifierOrder"/>
-    <!-- TODO <module name="RedundantModifier"/> -->
+    <module name="RedundantModifier"/>
 
     <!-- Checks for blocks. You know, those {}'s         -->
     <!-- See https://checkstyle.org/checks/blocks/index.html -->
diff --git a/src/main/java/com/thealgorithms/datastructures/bags/Bag.java b/src/main/java/com/thealgorithms/datastructures/bags/Bag.java
@@ -80,7 +80,7 @@ private class ListIterator<Element> implements Iterator<Element> {
 
         private Node<Element> currentElement;
 
-        public ListIterator(Node<Element> firstElement) {
+        ListIterator(Node<Element> firstElement) {
             currentElement = firstElement;
         }
 
diff --git a/src/main/java/com/thealgorithms/datastructures/bloomfilter/BloomFilter.java b/src/main/java/com/thealgorithms/datastructures/bloomfilter/BloomFilter.java
@@ -42,7 +42,7 @@ private class Hash<T> {
 
         int index;
 
-        public Hash(int index) {
+        Hash(int index) {
             this.index = index;
         }
 
diff --git a/src/main/java/com/thealgorithms/datastructures/buffers/CircularBuffer.java b/src/main/java/com/thealgorithms/datastructures/buffers/CircularBuffer.java
@@ -43,7 +43,7 @@ private static class CircularPointer {
         private int pointer;
         private final int max;
 
-        public CircularPointer(int pointer, int max) {
+        CircularPointer(int pointer, int max) {
             this.pointer = pointer;
             this.max = max;
         }
diff --git a/src/main/java/com/thealgorithms/datastructures/caches/LFUCache.java b/src/main/java/com/thealgorithms/datastructures/caches/LFUCache.java
@@ -17,7 +17,7 @@ private class Node {
         private Node previous;
         private Node next;
 
-        public Node(K key, V value, int frequency) {
+        Node(K key, V value, int frequency) {
             this.key = key;
             this.value = value;
             this.frequency = frequency;
diff --git a/src/main/java/com/thealgorithms/datastructures/caches/LRUCache.java b/src/main/java/com/thealgorithms/datastructures/caches/LRUCache.java
@@ -126,10 +126,10 @@ static final class Entry<I, J> {
         private I key;
         private J value;
 
-        public Entry() {
+        Entry() {
         }
 
-        public Entry(Entry<I, J> preEntry, Entry<I, J> nextEntry, I key, J value) {
+        Entry(Entry<I, J> preEntry, Entry<I, J> nextEntry, I key, J value) {
             this.preEntry = preEntry;
             this.nextEntry = nextEntry;
             this.key = key;
diff --git a/src/main/java/com/thealgorithms/datastructures/caches/MRUCache.java b/src/main/java/com/thealgorithms/datastructures/caches/MRUCache.java
@@ -124,10 +124,10 @@ static final class Entry<I, J> {
         private I key;
         private J value;
 
-        public Entry() {
+        Entry() {
         }
 
-        public Entry(Entry<I, J> preEntry, Entry<I, J> nextEntry, I key, J value) {
+        Entry(Entry<I, J> preEntry, Entry<I, J> nextEntry, I key, J value) {
             this.preEntry = preEntry;
             this.nextEntry = nextEntry;
             this.key = key;
diff --git a/src/main/java/com/thealgorithms/datastructures/crdt/GCounter.java b/src/main/java/com/thealgorithms/datastructures/crdt/GCounter.java
@@ -26,7 +26,7 @@ class GCounter {
      *
      * @param n The number of nodes in the cluster.
      */
-    public GCounter(int myId, int n) {
+    GCounter(int myId, int n) {
         this.myId = myId;
         this.n = n;
         this.P = new HashMap<>();
diff --git a/src/main/java/com/thealgorithms/datastructures/crdt/LWWElementSet.java b/src/main/java/com/thealgorithms/datastructures/crdt/LWWElementSet.java
@@ -26,7 +26,7 @@ class Element {
      * @param timestamp The timestamp associated with the element.
      * @param bias      The bias of the element (ADDS or REMOVALS).
      */
-    public Element(String key, int timestamp, Bias bias) {
+    Element(String key, int timestamp, Bias bias) {
         this.key = key;
         this.timestamp = timestamp;
         this.bias = bias;
@@ -49,7 +49,7 @@ class LWWElementSet {
     /**
      * Constructs an empty LWWElementSet.
      */
-    public LWWElementSet() {
+    LWWElementSet() {
         this.addSet = new HashMap<>();
         this.removeSet = new HashMap<>();
     }
diff --git a/src/main/java/com/thealgorithms/datastructures/crdt/PNCounter.java b/src/main/java/com/thealgorithms/datastructures/crdt/PNCounter.java
@@ -28,7 +28,7 @@ class PNCounter {
      * @param myId The identifier of the current node.
      * @param n    The number of nodes in the cluster.
      */
-    public PNCounter(int myId, int n) {
+    PNCounter(int myId, int n) {
         this.myId = myId;
         this.n = n;
         this.P = new HashMap<>();
diff --git a/src/main/java/com/thealgorithms/datastructures/graphs/A_Star.java b/src/main/java/com/thealgorithms/datastructures/graphs/A_Star.java
@@ -14,7 +14,7 @@ private static class Graph {
         private ArrayList<ArrayList<Edge>> graph;
 
         // Initialise ArrayLists in Constructor
-        public Graph(int size) {
+        Graph(int size) {
             this.graph = new ArrayList<>();
             for (int i = 0; i < size; i++) {
                 this.graph.add(new ArrayList<>());
@@ -38,7 +38,7 @@ private static class Edge {
         private int to;
         private int weight;
 
-        public Edge(int from, int to, int weight) {
+        Edge(int from, int to, int weight) {
             this.from = from;
             this.to = to;
             this.weight = weight;
@@ -64,7 +64,7 @@ private static class PathAndDistance {
         private ArrayList<Integer> path; // list of visited nodes in this path.
         private int estimated; // heuristic value associated to the last node od the path (current node).
 
-        public PathAndDistance(int distance, ArrayList<Integer> path, int estimated) {
+        PathAndDistance(int distance, ArrayList<Integer> path, int estimated) {
             this.distance = distance;
             this.path = path;
             this.estimated = estimated;
diff --git a/src/main/java/com/thealgorithms/datastructures/graphs/BellmanFord.java b/src/main/java/com/thealgorithms/datastructures/graphs/BellmanFord.java
@@ -31,7 +31,7 @@ class Edge {
          * @param v End vertex
          * @param c Weight
          */
-        public Edge(int a, int b, int c) {
+        Edge(int a, int b, int c) {
             u = a;
             v = b;
             w = c;
diff --git a/src/main/java/com/thealgorithms/datastructures/graphs/ConnectedComponent.java b/src/main/java/com/thealgorithms/datastructures/graphs/ConnectedComponent.java
@@ -15,7 +15,7 @@ class Node {
 
         E name;
 
-        public Node(E name) {
+        Node(E name) {
             this.name = name;
         }
     }
@@ -24,7 +24,7 @@ class Edge {
 
         Node startNode, endNode;
 
-        public Edge(Node startNode, Node endNode) {
+        Edge(Node startNode, Node endNode) {
             this.startNode = startNode;
             this.endNode = endNode;
         }
@@ -33,7 +33,7 @@ public Edge(Node startNode, Node endNode) {
     ArrayList<Edge> edgeList;
     ArrayList<Node> nodeList;
 
-    public Graph() {
+    Graph() {
         edgeList = new ArrayList<Edge>();
         nodeList = new ArrayList<Node>();
     }
diff --git a/src/main/java/com/thealgorithms/datastructures/graphs/Cycles.java b/src/main/java/com/thealgorithms/datastructures/graphs/Cycles.java
@@ -10,7 +10,7 @@ class Cycle {
     private boolean[] visited;
     ArrayList<ArrayList<Integer>> cycles = new ArrayList<ArrayList<Integer>>();
 
-    public Cycle() {
+    Cycle() {
         Scanner in = new Scanner(System.in);
         System.out.print("Enter the no. of nodes: ");
         nodes = in.nextInt();
diff --git a/src/main/java/com/thealgorithms/datastructures/graphs/Graphs.java b/src/main/java/com/thealgorithms/datastructures/graphs/Graphs.java
@@ -6,7 +6,7 @@ class AdjacencyListGraph<E extends Comparable<E>> {
 
     ArrayList<Vertex> vertices;
 
-    public AdjacencyListGraph() {
+    AdjacencyListGraph() {
         vertices = new ArrayList<>();
     }
 
@@ -15,7 +15,7 @@ private class Vertex {
         E data;
         ArrayList<Vertex> adjacentVertices;
 
-        public Vertex(E data) {
+        Vertex(E data) {
             adjacentVertices = new ArrayList<>();
             this.data = data;
         }
diff --git a/src/main/java/com/thealgorithms/datastructures/graphs/Kruskal.java b/src/main/java/com/thealgorithms/datastructures/graphs/Kruskal.java
@@ -23,7 +23,7 @@ private static class Edge {
         private int to;
         private int weight;
 
-        public Edge(int from, int to, int weight) {
+        Edge(int from, int to, int weight) {
             this.from = from;
             this.to = to;
             this.weight = weight;
diff --git a/src/main/java/com/thealgorithms/datastructures/graphs/MatrixGraphs.java b/src/main/java/com/thealgorithms/datastructures/graphs/MatrixGraphs.java
@@ -68,7 +68,7 @@ class AdjacencyMatrixGraph {
     /**
      * Constructor
      */
-    public AdjacencyMatrixGraph(int givenNumberOfVertices) {
+    AdjacencyMatrixGraph(int givenNumberOfVertices) {
         this.setNumberOfVertices(givenNumberOfVertices);
         this.setNumberOfEdges(0);
         this.setAdjacency(new int[givenNumberOfVertices][givenNumberOfVertices]);
diff --git a/src/main/java/com/thealgorithms/datastructures/hashmap/hashing/GenericHashMapUsingArrayList.java b/src/main/java/com/thealgorithms/datastructures/hashmap/hashing/GenericHashMapUsingArrayList.java
@@ -105,7 +105,7 @@ private class Node {
         K key;
         V val;
 
-        public Node(K key, V val) {
+        Node(K key, V val) {
             this.key = key;
             this.val = val;
         }
diff --git a/src/main/java/com/thealgorithms/datastructures/lists/DoublyLinkedList.java b/src/main/java/com/thealgorithms/datastructures/lists/DoublyLinkedList.java
@@ -110,7 +110,7 @@ class Link {
      *
      * @param value Value of node
      */
-    public Link(int value) {
+    Link(int value) {
         this.value = value;
     }
 
diff --git a/src/main/java/com/thealgorithms/datastructures/lists/SkipList.java b/src/main/java/com/thealgorithms/datastructures/lists/SkipList.java
@@ -222,7 +222,7 @@ private static class Node<E> {
         private final List<Node<E>> backward;
 
         @SuppressWarnings("unchecked")
-        public Node(E value, int height) {
+        Node(E value, int height) {
             this.value = value;
             this.height = height;
 
diff --git a/src/main/java/com/thealgorithms/datastructures/queues/LinkedQueue.java b/src/main/java/com/thealgorithms/datastructures/queues/LinkedQueue.java
@@ -11,15 +11,15 @@ static class Node<T> {
         T data;
         Node<T> next;
 
-        public Node() {
+        Node() {
             this(null);
         }
 
-        public Node(T data) {
+        Node(T data) {
             this(data, null);
         }
 
-        public Node(T data, Node<T> next) {
+        Node(T data, Node<T> next) {
             this.data = data;
             this.next = next;
         }
diff --git a/src/main/java/com/thealgorithms/datastructures/queues/PriorityQueues.java b/src/main/java/com/thealgorithms/datastructures/queues/PriorityQueues.java
@@ -30,7 +30,7 @@ class PriorityQueue {
      * Default Constructor
      */
 
-    public PriorityQueue() {
+    PriorityQueue() {
         /* If capacity is not defined, default size of 11 would be used
          *  capacity=max+1 because we cant access 0th element of PQ, and to
          *  accomodate (max)th elements we need capacity to be max+1.
@@ -50,7 +50,7 @@ public PriorityQueue() {
      * @param size Size of the queue
      */
 
-    public PriorityQueue(int size) {
+    PriorityQueue(int size) {
         maxSize = size + 1;
         queueArray = new int[maxSize];
         nItems = 0;
diff --git a/src/main/java/com/thealgorithms/datastructures/queues/Queues.java b/src/main/java/com/thealgorithms/datastructures/queues/Queues.java
@@ -38,7 +38,7 @@ class Queue {
     /**
      * init with DEFAULT_CAPACITY
      */
-    public Queue() {
+    Queue() {
         this(DEFAULT_CAPACITY);
     }
 
@@ -47,7 +47,7 @@ public Queue() {
      *
      * @param size Size of the new queue
      */
-    public Queue(int size) {
+    Queue(int size) {
         maxSize = size;
         queueArray = new int[size];
         front = 0;
diff --git a/src/main/java/com/thealgorithms/datastructures/stacks/StackOfLinkedList.java b/src/main/java/com/thealgorithms/datastructures/stacks/StackOfLinkedList.java
@@ -33,7 +33,7 @@ class Node {
     public int data;
     public Node next;
 
-    public Node(int data) {
+    Node(int data) {
         this.data = data;
         this.next = null;
     }
@@ -60,7 +60,7 @@ class LinkedListStack {
     /**
      * Init properties
      */
-    public LinkedListStack() {
+    LinkedListStack() {
         head = null;
         size = 0;
     }
diff --git a/src/main/java/com/thealgorithms/datastructures/trees/BinaryTree.java b/src/main/java/com/thealgorithms/datastructures/trees/BinaryTree.java
@@ -47,7 +47,7 @@ static class Node {
          *
          * @param value Value to put in the node
          */
-        public Node(int value) {
+        Node(int value) {
             data = value;
             left = null;
             right = null;
diff --git a/src/main/java/com/thealgorithms/datastructures/trees/KDTree.java b/src/main/java/com/thealgorithms/datastructures/trees/KDTree.java
@@ -68,7 +68,7 @@ public int getDimension() {
             return coordinates.length;
         }
 
-        public Point(int[] coordinates) {
+        Point(int[] coordinates) {
             this.coordinates = coordinates;
         }
 
diff --git a/src/main/java/com/thealgorithms/datastructures/trees/LazySegmentTree.java b/src/main/java/com/thealgorithms/datastructures/trees/LazySegmentTree.java
@@ -15,7 +15,7 @@ static class Node {
         private int lazy; // lazied value that should be added to children nodes
         Node left, right; // left and right children
 
-        public Node(int start, int end, int value) {
+        Node(int start, int end, int value) {
             this.start = start;
             this.end = end;
             this.value = value;
diff --git a/src/main/java/com/thealgorithms/datastructures/trees/PrintTopViewofTree.java b/src/main/java/com/thealgorithms/datastructures/trees/PrintTopViewofTree.java
@@ -13,7 +13,7 @@ class TreeNode {
     TreeNode left, right;
 
     // Constructor
-    public TreeNode(int key) {
+    TreeNode(int key) {
         this.key = key;
         left = right = null;
     }
@@ -27,7 +27,7 @@ class QItem {
     TreeNode node;
     int hd;
 
-    public QItem(TreeNode n, int h) {
+    QItem(TreeNode n, int h) {
         node = n;
         hd = h;
     }
@@ -39,11 +39,11 @@ class Tree {
     TreeNode root;
 
     // Constructors
-    public Tree() {
+    Tree() {
         root = null;
     }
 
-    public Tree(TreeNode n) {
+    Tree(TreeNode n) {
         root = n;
     }
 
diff --git a/src/main/java/com/thealgorithms/datastructures/trees/nearestRightKey.java b/src/main/java/com/thealgorithms/datastructures/trees/nearestRightKey.java
@@ -53,13 +53,13 @@ class NRKTree {
     public NRKTree right;
     public int data;
 
-    public NRKTree(int x) {
+    NRKTree(int x) {
         this.left = null;
         this.right = null;
         this.data = x;
     }
 
-    public NRKTree(NRKTree right, NRKTree left, int x) {
+    NRKTree(NRKTree right, NRKTree left, int x) {
         this.left = left;
         this.right = right;
         this.data = x;
diff --git a/src/main/java/com/thealgorithms/greedyalgorithms/JobSequencing.java b/src/main/java/com/thealgorithms/greedyalgorithms/JobSequencing.java
@@ -19,7 +19,7 @@ public int compareTo(Job otherJob) {
             return otherJob.profit - this.profit;
         }
 
-        public Job(char id, int deadline, int profit) {
+        Job(char id, int deadline, int profit) {
             this.id = id;
             this.deadline = deadline;
             this.profit = profit;
diff --git a/src/main/java/com/thealgorithms/maths/FFT.java b/src/main/java/com/thealgorithms/maths/FFT.java
diff --git a/src/main/java/com/thealgorithms/misc/WordBoggle.java b/src/main/java/com/thealgorithms/misc/WordBoggle.java
diff --git a/src/main/java/com/thealgorithms/others/Dijkstra.java b/src/main/java/com/thealgorithms/others/Dijkstra.java
diff --git a/src/main/java/com/thealgorithms/others/KochSnowflake.java b/src/main/java/com/thealgorithms/others/KochSnowflake.java
diff --git a/src/main/java/com/thealgorithms/others/QueueUsingTwoStacks.java b/src/main/java/com/thealgorithms/others/QueueUsingTwoStacks.java
diff --git a/src/main/java/com/thealgorithms/others/TopKWords.java b/src/main/java/com/thealgorithms/others/TopKWords.java
diff --git a/src/main/java/com/thealgorithms/scheduling/PreemptivePriorityScheduling.java b/src/main/java/com/thealgorithms/scheduling/PreemptivePriorityScheduling.java
diff --git a/src/main/java/com/thealgorithms/sorts/TopologicalSort.java b/src/main/java/com/thealgorithms/sorts/TopologicalSort.java

Original file line number	Diff line number	Diff line change
`@@ -80,7 +80,7 @@ private class ListIterator<Element> implements Iterator<Element> {`
`80`	`80`
`81`	`81`	`private Node<Element> currentElement;`
`82`	`82`
`83`		`- public ListIterator(Node<Element> firstElement) {`
	`83`	`+ ListIterator(Node<Element> firstElement) {`
`84`	`84`	`currentElement = firstElement;`
`85`	`85`	`}`
`86`	`86`
Original file line number	Diff line number	Diff line change
`@@ -42,7 +42,7 @@ private class Hash<T> {`
`42`	`42`
`43`	`43`	`int index;`
`44`	`44`
`45`		`- public Hash(int index) {`
	`45`	`+ Hash(int index) {`
`46`	`46`	`this.index = index;`
`47`	`47`	`}`
`48`	`48`
Original file line number	Diff line number	Diff line change
`@@ -43,7 +43,7 @@ private static class CircularPointer {`
`43`	`43`	`private int pointer;`
`44`	`44`	`private final int max;`
`45`	`45`
`46`		`- public CircularPointer(int pointer, int max) {`
	`46`	`+ CircularPointer(int pointer, int max) {`
`47`	`47`	`this.pointer = pointer;`
`48`	`48`	`this.max = max;`
`49`	`49`	`}`
Original file line number	Diff line number	Diff line change
`@@ -26,7 +26,7 @@ class GCounter {`
`26`	`26`	`*`
`27`	`27`	`* @param n The number of nodes in the cluster.`
`28`	`28`	`*/`
`29`		`- public GCounter(int myId, int n) {`
	`29`	`+ GCounter(int myId, int n) {`
`30`	`30`	`this.myId = myId;`
`31`	`31`	`this.n = n;`
`32`	`32`	`this.P = new HashMap<>();`