Skip to content

Commit 1c0d64f

Browse files
committed
SeriesInfoExtractorServiceImpl: refactor code to use Stream API for getting unique candidates.
Prerequsite to #821 No functional changes.
1 parent bd40a90 commit 1c0d64f

File tree

1 file changed

+8
-5
lines changed

1 file changed

+8
-5
lines changed

src/main/java/ru/mystamps/web/service/SeriesInfoExtractorServiceImpl.java

Lines changed: 8 additions & 5 deletions
Original file line numberDiff line numberDiff line change
@@ -17,12 +17,13 @@
1717
*/
1818
package ru.mystamps.web.service;
1919

20+
import java.util.Arrays;
2021
import java.util.Collections;
21-
import java.util.HashSet;
2222
import java.util.List;
2323
import java.util.Set;
2424
import java.util.regex.Matcher;
2525
import java.util.regex.Pattern;
26+
import java.util.stream.Collectors;
2627

2728
import org.apache.commons.lang3.StringUtils;
2829

@@ -82,8 +83,9 @@ protected List<Integer> extractCategory(String fragment) {
8283
log.debug("Determining category from a fragment: '{}'", fragment);
8384

8485
String[] candidates = StringUtils.split(fragment, "\n\t ,");
85-
Set<String> uniqueCandidates = new HashSet<>();
86-
Collections.addAll(uniqueCandidates, candidates);
86+
Set<String> uniqueCandidates = Arrays.stream(candidates)
87+
.distinct()
88+
.collect(Collectors.toSet());
8789

8890
log.debug("Possible candidates: {}", uniqueCandidates);
8991

@@ -115,8 +117,9 @@ protected List<Integer> extractCountry(String fragment) {
115117
log.debug("Determining country from a fragment: '{}'", fragment);
116118

117119
String[] candidates = StringUtils.split(fragment, "\n\t ,");
118-
Set<String> uniqueCandidates = new HashSet<>();
119-
Collections.addAll(uniqueCandidates, candidates);
120+
Set<String> uniqueCandidates = Arrays.stream(candidates)
121+
.distinct()
122+
.collect(Collectors.toSet());
120123

121124
log.debug("Possible candidates: {}", uniqueCandidates);
122125

0 commit comments

Comments
 (0)