Bless inheritance! Use YoutubeDL or YoutubeDLProchains, in a static way!

Author: memento

src/main/java/com/sapher/youtubedl/YoutubeDL.java

@@ -141,18 +141,8 @@ public static VideoInfo getVideoInfo(String url) throws YoutubeDLException  {
         ObjectMapper objectMapper = new ObjectMapper();
         VideoInfo videoInfo;
 
-        String output;
-        if(executablePath.contains("proxychains")) {
-        	//Proxychains : remove proxychains header ex: "ProxyChains-3.1 (http://proxychains.sf.net)\n"
-            output = response.getOut();
-            output = output.substring(output.indexOf("{"));
-        }else {
-        	output = response.getOut();
-        }
-        
-
         try {
-            videoInfo = objectMapper.readValue(output, VideoInfo.class);
+            videoInfo = objectMapper.readValue(response.getOut(), VideoInfo.class);
         } catch (IOException e) {
             throw new YoutubeDLException("Unable to parse video information: " + e.getMessage());
         }

src/main/java/com/sapher/youtubedl/YoutubeDLProchains.java

@@ -0,0 +1,43 @@
+package com.sapher.youtubedl;
+
+import java.io.IOException;
+
+import com.fasterxml.jackson.databind.ObjectMapper;
+import com.sapher.youtubedl.mapper.VideoInfo;
+
+public class YoutubeDLProchains extends YoutubeDL{
+	
+	/**
+     * Retrieve all information available on a video
+     * @param url Video url
+     * @return Video info
+     * @throws YoutubeDLException
+     */
+    public static VideoInfo getVideoInfo(String url) throws YoutubeDLException  {
+    	
+    	//We use proxychains, here
+    	setExecutablePath("proxychains youtube-dl");
+    	
+        // Build request
+        YoutubeDLRequest request = new YoutubeDLRequest(url);
+        request.setOption("dump-json");
+        request.setOption("no-playlist");
+        YoutubeDLResponse response = YoutubeDL.execute(request);
+
+        // Parse result
+        ObjectMapper objectMapper = new ObjectMapper();
+        VideoInfo videoInfo;
+        
+        //Proxychains : remove proxychains header ex: "ProxyChains-3.1 (http://proxychains.sf.net)\n"
+    	String output = response.getOut();
+        output = output.substring(output.indexOf("{"));
+        
+        try {
+            videoInfo = objectMapper.readValue(output, VideoInfo.class);
+        } catch (IOException e) {
+            throw new YoutubeDLException("Unable to parse video information: " + e.getMessage());
+        }
+
+        return videoInfo;
+    }
+}

src/test/java/com/sapher/youtubedl/YoutubeDLTest.java

@@ -95,10 +95,7 @@ public void testGetVideoInfoInDetails() throws YoutubeDLException {
 
     @Test
     public void testGetVideoInfoInDetailsWithProxychains() throws YoutubeDLException {
-        YoutubeDL youtubeDL = new YoutubeDL();
-        youtubeDL.setExecutablePath("proxychains youtube-dl");
-        
-    	VideoInfo videoInfo = youtubeDL.getVideoInfo(VIDEO_URL);
+        VideoInfo videoInfo = YoutubeDLProchains.getVideoInfo(VIDEO_URL);
         Assert.assertNotNull(videoInfo);
         //Let's check we can access key elements from video info
         Assert.assertNotNull(videoInfo.getId());