PatchesAndForks/rome
1
0
Fork 0
Code Issues Pull requests Projects Releases Packages Wiki Activity

Replaced Collection.size() > 0 through !Collection.isEmpty()

Browse source
This commit is contained in:
Patrick Gotthard 2014-04-13 20:29:39 +02:00
parent 489c1f4b5c
commit e63c849766
7 changed files with 8 additions and 8 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

2
src/main/java/org/rometools/feed/module/cc/io/ModuleParserRSS1.java
View file

@ -120,7 +120,7 @@ public class ModuleParserRSS1 implements ModuleParser {
licenses.add(License.findByValue(licenseTag.getAttributeValue("resource", RDF))); licenses.add(License.findByValue(licenseTag.getAttributeValue("resource", RDF)));
} }
if (licenses.size() > 0) { if (!licenses.isEmpty()) {
module.setLicenses(licenses.toArray(new License[licenses.size()])); module.setLicenses(licenses.toArray(new License[licenses.size()]));
} }

4
src/main/java/org/rometools/feed/module/cc/io/ModuleParserRSS2.java
View file

@ -97,7 +97,7 @@ public class ModuleParserRSS2 implements ModuleParser {
licenses.add(license); licenses.add(license);
} }
} }
if (licenses.size() > 0) { if (!licenses.isEmpty()) {
module.setAllLicenses(licenses.toArray(new License[0])); module.setAllLicenses(licenses.toArray(new License[0]));
} }
} }
@ -109,7 +109,7 @@ public class ModuleParserRSS2 implements ModuleParser {
final Element licenseTag = it.next(); final Element licenseTag = it.next();
licenses.add(License.findByValue(licenseTag.getTextTrim())); licenses.add(License.findByValue(licenseTag.getTextTrim()));
} }
if (licenses.size() > 0) { if (!licenses.isEmpty()) {
module.setLicenses(licenses.toArray(new License[0])); module.setLicenses(licenses.toArray(new License[0]));
} }

2
src/main/java/org/rometools/feed/module/content/io/ContentModuleGenerator.java
View file

@ -103,7 +103,7 @@ public class ContentModuleGenerator implements com.sun.syndication.io.ModuleGene
final List<ContentItem> contentItems = cm.getContentItems(); final List<ContentItem> contentItems = cm.getContentItems();
if (contentItems != null && contentItems.size() > 0) { if (contentItems != null && !contentItems.isEmpty()) {
final Element items = new Element("items", CONTENT_NS); final Element items = new Element("items", CONTENT_NS);
final Element bag = new Element("Bag", RDF_NS); final Element bag = new Element("Bag", RDF_NS);
items.addContent(bag); items.addContent(bag);

2
src/main/java/org/rometools/feed/module/content/io/ContentModuleParser.java
View file

@ -81,7 +81,7 @@ public class ContentModuleParser implements com.sun.syndication.io.ModuleParser
final ArrayList<String> contentStrings = new ArrayList<String>(); final ArrayList<String> contentStrings = new ArrayList<String>();
final ArrayList<String> encodedStrings = new ArrayList<String>(); final ArrayList<String> encodedStrings = new ArrayList<String>();
if (encodeds.size() > 0) { if (!encodeds.isEmpty()) {
foundSomething = true; foundSomething = true;
for (int i = 0; i < encodeds.size(); i++) { for (int i = 0; i < encodeds.size(); i++) {

2
src/main/java/org/rometools/feed/module/opensearch/impl/OpenSearchModuleParser.java
View file

@ -89,7 +89,7 @@ public class OpenSearchModuleParser implements ModuleParser {
final List<Element> queries = dcRoot.getChildren("Query", OS_NS); final List<Element> queries = dcRoot.getChildren("Query", OS_NS);
if (queries != null && queries.size() > 0) { if (queries != null && !queries.isEmpty()) {
// Create the OSQuery list // Create the OSQuery list
final List<OSQuery> osqList = new LinkedList<OSQuery>(); final List<OSQuery> osqList = new LinkedList<OSQuery>();

2
src/main/java/org/rometools/feed/module/sle/io/ModuleGenerator.java
View file

@ -112,7 +112,7 @@ public class ModuleGenerator implements com.sun.syndication.io.ModuleGenerator {
listInfo.addContent(sort); listInfo.addContent(sort);
} }
if (listInfo.getChildren().size() > 0) { if (!listInfo.getChildren().isEmpty()) {
element.addContent(listInfo); element.addContent(listInfo);
} }
} }

2
src/main/java/org/rometools/feed/module/sse/SSE091Parser.java
View file

@ -172,7 +172,7 @@ public class SSE091Parser implements DelegatingModuleParser {
private Element getFirstContent(final Element element, final String name) { private Element getFirstContent(final Element element, final String name) {
final List<Element> filterList = element.getContent(new ContentFilter(name)); final List<Element> filterList = element.getContent(new ContentFilter(name));
Element firstContent = null; Element firstContent = null;
if (filterList != null && filterList.size() > 0) { if (filterList != null && !filterList.isEmpty()) {
firstContent = filterList.get(0); firstContent = filterList.get(0);
} }
return firstContent; return firstContent;