Issue #1487: reuse Random instance (#1488)

* Issue #1487: reuse Random instance

* Issue #1487: reuse Random instance

Author: thboileau

changes.md

@@ -1,8 +1,9 @@
 Changes log
 ===========
 
-- 2.6.1 (??-??-2025)
-
+- 2.6.1 (??-??-2026)
+    - Bugs fixed
+        - Reuse an instance of Random class in RandomUtils. Issue #1487.
 - 2.6.0 (29-06-2025)
 
 - 2.6 Release Candidate 2 (21-06-2025)

org.restlet.java/org.restlet/src/main/java/org/restlet/routing/Router.java

@@ -94,7 +94,7 @@ public class Router extends Restlet {
 
 	/**
 	 * Each call will be randomly routed to one of the routes that reached the
-	 * required score. If the random route selected is not a match then the
+	 * required score. If the random route selected is not a match, then the
 	 * immediate next route is evaluated until one matching route is found. If we
 	 * get back to the initial random route selected with no match, then we return
 	 * null. Unless all the routes score above the required score, this mode will

org.restlet.java/org.restlet/src/main/java/org/restlet/util/RouteList.java

@@ -14,9 +14,9 @@
 import org.restlet.Restlet;
 import org.restlet.routing.Route;
 
+import java.security.SecureRandom;
 import java.util.Collections;
 import java.util.List;
-import java.util.Random;
 import java.util.concurrent.CopyOnWriteArrayList;
 
 /**
@@ -34,14 +34,16 @@
  * @see java.util.List
  */
 public final class RouteList extends WrapperList<Route> {
-	/** The index of the last route used in the round robin mode. */
+	/** The index of the last route used in the round-robin mode. */
 	private volatile int lastIndex;
+	/** Used when asked to return a random route. */
+	private final SecureRandom random = new SecureRandom();
 
 	/**
 	 * Constructor.
 	 */
 	public RouteList() {
-		super(new CopyOnWriteArrayList<Route>());
+		super(new CopyOnWriteArrayList<>());
 		this.lastIndex = -1;
 	}
 
@@ -51,7 +53,7 @@ public RouteList() {
 	 * @param delegate The delegate list.
 	 */
 	public RouteList(List<Route> delegate) {
-		super(new CopyOnWriteArrayList<Route>(delegate));
+		super(new CopyOnWriteArrayList<>(delegate));
 		this.lastIndex = -1;
 	}
 
@@ -120,7 +122,7 @@ public synchronized Route getLast(Request request, Response response, float requ
 	}
 
 	/**
-	 * Returns a next route match in a round robin mode for a given call.
+	 * Returns a next route match in a round-robin mode for a given call.
 	 * 
 	 * @param request       The request to score.
 	 * @param response      The response to score.
@@ -159,7 +161,7 @@ public synchronized Route getRandom(Request request, Response response, float re
 		int length = size();
 
 		if (length > 0) {
-			int j = new Random().nextInt(length);
+			int j = random.nextInt(length);
 			Route route = get(j);
 
 			if (route.score(request, response) >= requiredScore) {
@@ -169,7 +171,7 @@ public synchronized Route getRandom(Request request, Response response, float re
 			boolean loopedAround = false;
 
 			do {
-				if ((j == length) && (loopedAround == false)) {
+				if ((j == length) && !loopedAround) {
 					j = 0;
 					loopedAround = true;
 				}
@@ -187,7 +189,7 @@ public synchronized Route getRandom(Request request, Response response, float re
 	}
 
 	/**
-	 * Removes all routes routing to a given target.
+	 * Removes all routes to a given target.
 	 * 
 	 * @param target The target Restlet to detach.
 	 */