Merge pull request #643 from Gaboso/master
Replaced the if-else statement by a single return statement. Thanks for all these style changes
This commit is contained in:
commit
a02933f0d2
@ -57,10 +57,7 @@ public class BatoRipper extends AbstractHTMLRipper {
|
||||
public boolean pageContainsAlbums(URL url) {
|
||||
Pattern p = Pattern.compile("https?://bato.to/series/([\\d]+)/?");
|
||||
Matcher m = p.matcher(url.toExternalForm());
|
||||
if (m.matches()) {
|
||||
return true;
|
||||
}
|
||||
return false;
|
||||
return m.matches();
|
||||
}
|
||||
|
||||
@Override
|
||||
@ -94,10 +91,7 @@ public class BatoRipper extends AbstractHTMLRipper {
|
||||
|
||||
p = Pattern.compile("https?://bato.to/chapter/([\\d]+)/?");
|
||||
m = p.matcher(url.toExternalForm());
|
||||
if (m.matches()) {
|
||||
return true;
|
||||
}
|
||||
return false;
|
||||
return m.matches();
|
||||
}
|
||||
|
||||
@Override
|
||||
|
@ -35,10 +35,7 @@ public class NhentaiRipper extends AbstractHTMLRipper {
|
||||
public boolean pageContainsAlbums(URL url) {
|
||||
Pattern pa = Pattern.compile("^https?://nhentai\\.net/tag/([a-zA-Z0-9_\\-]+)/?");
|
||||
Matcher ma = pa.matcher(url.toExternalForm());
|
||||
if (ma.matches()) {
|
||||
return true;
|
||||
}
|
||||
return false;
|
||||
return ma.matches();
|
||||
}
|
||||
|
||||
@Override
|
||||
|
@ -60,10 +60,7 @@ public class PichunterRipper extends AbstractHTMLRipper {
|
||||
private boolean isPhotoSet(URL url) {
|
||||
Pattern p = Pattern.compile("https?://www.pichunter.com/gallery/\\d+/(\\S*)/?");
|
||||
Matcher m = p.matcher(url.toExternalForm());
|
||||
if (m.matches()) {
|
||||
return true;
|
||||
}
|
||||
return false;
|
||||
return m.matches();
|
||||
}
|
||||
|
||||
@Override
|
||||
|
@ -37,10 +37,7 @@ public class Rule34Ripper extends AbstractHTMLRipper {
|
||||
public boolean canRip(URL url){
|
||||
Pattern p = Pattern.compile("https?://rule34.xxx/index.php\\?page=post&s=list&tags=([\\S]+)");
|
||||
Matcher m = p.matcher(url.toExternalForm());
|
||||
if (m.matches()) {
|
||||
return true;
|
||||
}
|
||||
return false;
|
||||
return m.matches();
|
||||
}
|
||||
|
||||
@Override
|
||||
|
@ -38,10 +38,7 @@ public class WebtoonsRipper extends AbstractHTMLRipper {
|
||||
public boolean canRip(URL url) {
|
||||
Pattern pat = Pattern.compile("https?://www.webtoons.com/[a-zA-Z-_]+/[a-zA-Z_-]+/([a-zA-Z0-9_-]*)/[a-zA-Z0-9_-]+/\\S*");
|
||||
Matcher mat = pat.matcher(url.toExternalForm());
|
||||
if (mat.matches()) {
|
||||
return true;
|
||||
}
|
||||
return false;
|
||||
return mat.matches();
|
||||
}
|
||||
|
||||
|
||||
|
@ -64,10 +64,7 @@ public class XhamsterRipper extends AbstractHTMLRipper {
|
||||
public boolean canRip(URL url) {
|
||||
Pattern p = Pattern.compile("^https?://[wmde.]*xhamster\\.com/photos/gallery/.*?(\\d+)$");
|
||||
Matcher m = p.matcher(url.toExternalForm());
|
||||
if (m.matches()) {
|
||||
return true;
|
||||
}
|
||||
return false;
|
||||
return m.matches();
|
||||
}
|
||||
|
||||
@Override
|
||||
|
@ -56,7 +56,7 @@ public class ViddmeRipper extends VideoRipper {
|
||||
logger.info(" Retrieving " + this.url.toExternalForm());
|
||||
Document doc = Http.url(this.url).get();
|
||||
Elements videos = doc.select("meta[name=twitter:player:stream]");
|
||||
if (videos.size() == 0) {
|
||||
if (videos.isEmpty()) {
|
||||
throw new IOException("Could not find twitter:player:stream at " + url);
|
||||
}
|
||||
String vidUrl = videos.first().attr("content");
|
||||
|
Loading…
Reference in New Issue
Block a user