Clean up the code to make it cleaner

Signed-off-by: longtanfei <[email protected]>

Author: lucky8987

Diff for: spring-cloud-gateway-server/pom.xml

@@ -212,11 +212,6 @@
 			<artifactId>spring-cloud-test-support</artifactId>
 			<scope>test</scope>
 		</dependency>
-		<dependency>
-			<groupId>org.springframework.cloud</groupId>
-			<artifactId>spring-cloud-starter-stream-rabbit</artifactId>
-			<scope>test</scope>
-		</dependency>
 		<dependency>
 			<groupId>io.projectreactor</groupId>
 			<artifactId>reactor-test</artifactId>

Diff for: spring-cloud-gateway-server/src/main/java/org/springframework/cloud/gateway/handler/AsyncPredicate.java

@@ -77,8 +77,7 @@ public String toString() {
 
 		@Override
 		public void accept(Visitor visitor) {
-			if (delegate instanceof GatewayPredicate) {
-				GatewayPredicate gatewayPredicate = (GatewayPredicate) delegate;
+			if (delegate instanceof GatewayPredicate gatewayPredicate) {
 				gatewayPredicate.accept(visitor);
 			}
 		}

Diff for: spring-cloud-gateway-server/src/main/java/org/springframework/cloud/gateway/handler/predicate/CookieRoutePredicateFactory.java

@@ -58,12 +58,7 @@ public boolean test(ServerWebExchange exchange) {
 				if (cookies == null) {
 					return false;
 				}
-				for (HttpCookie cookie : cookies) {
-					if (cookie.getValue().matches(config.regexp)) {
-						return true;
-					}
-				}
-				return false;
+				return cookies.stream().anyMatch(cookie -> cookie.getValue().matches(config.regexp));
 			}
 
 			@Override

Diff for: spring-cloud-gateway-server/src/main/java/org/springframework/cloud/gateway/handler/predicate/GatewayPredicate.java

@@ -47,8 +47,8 @@ default void accept(Visitor visitor) {
 	static GatewayPredicate wrapIfNeeded(Predicate<? super ServerWebExchange> other) {
 		GatewayPredicate right;
 
-		if (other instanceof GatewayPredicate) {
-			right = (GatewayPredicate) other;
+		if (other instanceof GatewayPredicate gatewayPredicate) {
+			right = gatewayPredicate;
 		}
 		else {
 			right = new GatewayPredicateWrapper(other);
@@ -72,8 +72,7 @@ public boolean test(ServerWebExchange exchange) {
 
 		@Override
 		public void accept(Visitor visitor) {
-			if (delegate instanceof GatewayPredicate) {
-				GatewayPredicate gatewayPredicate = (GatewayPredicate) delegate;
+			if (delegate instanceof GatewayPredicate gatewayPredicate) {
 				gatewayPredicate.accept(visitor);
 			}
 		}

Diff for: spring-cloud-gateway-server/src/main/java/org/springframework/cloud/gateway/route/RouteDefinitionRouteLocator.java

@@ -201,14 +201,10 @@ private AsyncPredicate<ServerWebExchange> combinePredicates(RouteDefinition rout
 			// this is a very rare case, but possible, just match all
 			return AsyncPredicate.from(exchange -> true);
 		}
-		AsyncPredicate<ServerWebExchange> predicate = lookup(routeDefinition, predicates.get(0));
 
-		for (PredicateDefinition andPredicate : predicates.subList(1, predicates.size())) {
-			AsyncPredicate<ServerWebExchange> found = lookup(routeDefinition, andPredicate);
-			predicate = predicate.and(found);
-		}
-
-		return predicate;
+		return predicates.stream()
+				.map(nextPredicate -> lookup(routeDefinition, nextPredicate))
+				.reduce(AsyncPredicate.from(exchange -> true), AsyncPredicate::and);
 	}
 
 	@SuppressWarnings("unchecked")