Skip to content

Commit d120036

Browse files
authored
Merge pull request #886 from Isira-Seneviratne/toArray_improvements
Make improvements to methods using toArray().
2 parents 2906be2 + 7daca10 commit d120036

File tree

2 files changed

+9
-33
lines changed

2 files changed

+9
-33
lines changed

extractor/src/main/java/org/schabi/newpipe/extractor/kiosk/KioskList.java

Lines changed: 6 additions & 5 deletions
Original file line numberDiff line numberDiff line change
@@ -1,5 +1,7 @@
11
package org.schabi.newpipe.extractor.kiosk;
22

3+
import static org.schabi.newpipe.extractor.utils.Utils.isNullOrEmpty;
4+
35
import org.schabi.newpipe.extractor.NewPipe;
46
import org.schabi.newpipe.extractor.Page;
57
import org.schabi.newpipe.extractor.StreamingService;
@@ -8,13 +10,12 @@
810
import org.schabi.newpipe.extractor.localization.ContentCountry;
911
import org.schabi.newpipe.extractor.localization.Localization;
1012

11-
import javax.annotation.Nullable;
1213
import java.io.IOException;
1314
import java.util.HashMap;
1415
import java.util.Map;
1516
import java.util.Set;
1617

17-
import static org.schabi.newpipe.extractor.utils.Utils.isNullOrEmpty;
18+
import javax.annotation.Nullable;
1819

1920
public class KioskList {
2021

@@ -78,10 +79,10 @@ public KioskExtractor getDefaultKioskExtractor(final Page nextPage,
7879
if (!isNullOrEmpty(defaultKiosk)) {
7980
return getExtractorById(defaultKiosk, nextPage, localization);
8081
} else {
81-
if (!kioskList.isEmpty()) {
82+
final String first = kioskList.keySet().stream().findAny().orElse(null);
83+
if (first != null) {
8284
// if not set get any entry
83-
final Object[] keySet = kioskList.keySet().toArray();
84-
return getExtractorById(keySet[0].toString(), nextPage, localization);
85+
return getExtractorById(first, nextPage, localization);
8586
} else {
8687
return null;
8788
}

extractor/src/main/java/org/schabi/newpipe/extractor/utils/Parser.java

Lines changed: 3 additions & 28 deletions
Original file line numberDiff line numberDiff line change
@@ -20,23 +20,18 @@
2020

2121
package org.schabi.newpipe.extractor.utils;
2222

23-
import org.nibor.autolink.LinkExtractor;
24-
import org.nibor.autolink.LinkSpan;
25-
import org.nibor.autolink.LinkType;
23+
import static org.schabi.newpipe.extractor.utils.Utils.UTF_8;
24+
2625
import org.schabi.newpipe.extractor.exceptions.ParsingException;
2726

28-
import javax.annotation.Nonnull;
2927
import java.io.UnsupportedEncodingException;
3028
import java.net.URLDecoder;
31-
import java.util.ArrayList;
32-
import java.util.EnumSet;
3329
import java.util.HashMap;
34-
import java.util.List;
3530
import java.util.Map;
3631
import java.util.regex.Matcher;
3732
import java.util.regex.Pattern;
3833

39-
import static org.schabi.newpipe.extractor.utils.Utils.UTF_8;
34+
import javax.annotation.Nonnull;
4035

4136
/**
4237
* Avoid using regex !!!
@@ -111,24 +106,4 @@ public static Map<String, String> compatParseMap(@Nonnull final String input)
111106
}
112107
return map;
113108
}
114-
115-
@Nonnull
116-
public static String[] getLinksFromString(final String txt) throws ParsingException {
117-
try {
118-
final List<String> links = new ArrayList<>();
119-
final LinkExtractor linkExtractor = LinkExtractor.builder()
120-
.linkTypes(EnumSet.of(LinkType.URL, LinkType.WWW))
121-
.build();
122-
final Iterable<LinkSpan> linkSpans = linkExtractor.extractLinks(txt);
123-
for (final LinkSpan ls : linkSpans) {
124-
links.add(txt.substring(ls.getBeginIndex(), ls.getEndIndex()));
125-
}
126-
127-
String[] linksarray = new String[links.size()];
128-
linksarray = links.toArray(linksarray);
129-
return linksarray;
130-
} catch (final Exception e) {
131-
throw new ParsingException("Could not get links from string", e);
132-
}
133-
}
134109
}

0 commit comments

Comments
 (0)