[YouTube] Use Optional chaining in YoutubePlaylistExtractor

Author: Isira-Seneviratne

extractor/src/main/java/org/schabi/newpipe/extractor/services/youtube/YoutubeParsingHelper.java

@@ -826,7 +826,7 @@ public static Optional<String> getTextFromObject(@Nonnull final JsonObject textO
 
                     final String string;
                     if (html) {
-                        string = text.replaceAll("\\n", "<br>")
+                        string = text.replace("\\n", "<br>")
                                 .replaceAll(" {2}", " &nbsp;");
                     } else {
                         string = text;

extractor/src/main/java/org/schabi/newpipe/extractor/services/youtube/extractors/YoutubePlaylistExtractor.java

@@ -37,6 +37,7 @@
 import java.nio.charset.StandardCharsets;
 import java.util.Base64;
 import java.util.List;
+import java.util.Optional;
 
 import javax.annotation.Nonnull;
 import javax.annotation.Nullable;
@@ -266,52 +267,25 @@ public boolean isUploaderVerified() throws ParsingException {
 
     @Override
     public long getStreamCount() throws ParsingException {
-        if (isNewPlaylistInterface) {
-            final String numVideosText =
-                    getTextFromObject(getPlaylistHeader().getObject("numVideosText"))
-                            .orElse(null);
-            if (numVideosText != null) {
-                try {
-                    return Long.parseLong(Utils.removeNonDigitCharacters(numVideosText));
-                } catch (final NumberFormatException ignored) {
-                }
-            }
-
-            final String firstByLineRendererText = getTextFromObject(
-                    getPlaylistHeader().getArray("byline")
-                            .getObject(0)
-                            .getObject("text"))
-                    .orElse(null);
-
-            if (firstByLineRendererText != null) {
-                try {
-                    return Long.parseLong(Utils.removeNonDigitCharacters(firstByLineRendererText));
-                } catch (final NumberFormatException ignored) {
-                }
-            }
-        }
-
-        // These data structures are returned in both layouts
-        final JsonArray briefStats =
-                (isNewPlaylistInterface ? getPlaylistHeader() : getPlaylistInfo())
-                        .getArray("briefStats");
-        if (!briefStats.isEmpty()) {
-            final var briefsStatsText = getTextFromObject(briefStats.getObject(0)).orElse(null);
-            if (briefsStatsText != null) {
-                return Long.parseLong(Utils.removeNonDigitCharacters(briefsStatsText));
-            }
-        }
-
-        final JsonArray stats = (isNewPlaylistInterface ? getPlaylistHeader() : getPlaylistInfo())
-                .getArray("stats");
-        if (!stats.isEmpty()) {
-            final var statsText = getTextFromObject(stats.getObject(0)).orElse(null);
-            if (statsText != null) {
-                return Long.parseLong(Utils.removeNonDigitCharacters(statsText));
-            }
-        }
-
-        return ITEM_COUNT_UNKNOWN;
+        final var header = getPlaylistHeader();
+        final Optional<String> count = isNewPlaylistInterface
+                ? getTextFromObject(header.getObject("numVideosText"))
+                .or(() -> getTextFromObject(header.getArray("byline")
+                        .getObject(0).getObject("text")))
+                : Optional.empty();
+        final var playlist = isNewPlaylistInterface ? header : getPlaylistInfo();
+
+        // "briefStats" and "stats" are returned in both layouts
+        return count.or(() -> getTextFromObject(playlist.getArray("briefStats").getObject(0)))
+                .or(() -> getTextFromObject(playlist.getArray("stats").getObject(0)))
+                .map(numText -> {
+                    try {
+                        return Long.parseLong(Utils.removeNonDigitCharacters(numText));
+                    } catch (final NumberFormatException e) {
+                        return null;
+                    }
+                })
+                .orElse(ITEM_COUNT_UNKNOWN);
     }
 
     @Nonnull