replaced streams with for loop for serviceprovider lookup to resolves null type safety warning from Collectors.toMap
SonarQube Analysis / Build and Analyze (pull_request) Successful in 3m51s

This commit is contained in:
2026-06-27 01:07:36 +02:00
parent fffdd2a575
commit 9c2cbd1608
2 changed files with 13 additions and 6 deletions
@@ -1,9 +1,9 @@
package com.vaessl.app.connection;
import java.time.Instant;
import java.util.EnumMap;
import java.util.List;
import java.util.Map;
import java.util.stream.Collectors;
import org.springframework.stereotype.Service;
@@ -18,8 +18,11 @@ public class ConnectionService {
private final ConnectionRepository cRepository;
public ConnectionService(List<ConnectionProvider> providers, ConnectionRepository cRepository) {
this.providerRegistry = providers.stream()
.collect(Collectors.toMap(ConnectionProvider::getServiceType, p -> p));
Map<ServiceType, ConnectionProvider> registry = new EnumMap<>(ServiceType.class);
for (ConnectionProvider provider : providers) {
registry.put(provider.getServiceType(), provider);
}
this.providerRegistry = registry;
this.cRepository = cRepository;
}
@@ -1,8 +1,8 @@
package com.vaessl.app.search;
import java.util.EnumMap;
import java.util.List;
import java.util.Map;
import java.util.stream.Collectors;
import org.springframework.data.domain.Page;
import org.springframework.data.domain.Pageable;
@@ -16,8 +16,12 @@ public class SearchService {
private final Map<ServiceType, SearchProvider> providerRegistry;
public SearchService(List<SearchProvider> providers) {
this.providerRegistry = Map.copyOf(providers.stream()
.collect(Collectors.toMap(SearchProvider::getServiceType, p -> p)));
Map<ServiceType, SearchProvider> registry = new EnumMap<>(ServiceType.class);
for (SearchProvider provider : providers) {
registry.put(provider.getServiceType(), provider);
}
this.providerRegistry = registry;
}
/**