This is an automated email from the ASF dual-hosted git repository.
ggregory pushed a commit to branch master
in repository https://gitbox.apache.org/repos/asf/commons-rdf.git
The following commit(s) were added to refs/heads/master by this push:
new 58959f6 No need to nest in else.
58959f6 is described below
commit 58959f6fe7d5f78cc22207b36833e3f39b9f247b
Author: Gary Gregory <garydgreg...@gmail.com>
AuthorDate: Fri Mar 5 14:53:41 2021 -0500
No need to nest in else.
---
.../apache/commons/rdf/jsonldjava/JsonLdRDF.java | 43 +++++++++++-----------
.../java/org/apache/commons/rdf/rdf4j/RDF4J.java | 3 +-
.../org/apache/commons/rdf/simple/DatasetImpl.java | 17 +++++----
.../org/apache/commons/rdf/simple/GraphImpl.java | 17 +++++----
4 files changed, 42 insertions(+), 38 deletions(-)
diff --git
a/commons-rdf-jsonld-java/src/main/java/org/apache/commons/rdf/jsonldjava/JsonLdRDF.java
b/commons-rdf-jsonld-java/src/main/java/org/apache/commons/rdf/jsonldjava/JsonLdRDF.java
index fc26046..f01251c 100644
---
a/commons-rdf-jsonld-java/src/main/java/org/apache/commons/rdf/jsonldjava/JsonLdRDF.java
+++
b/commons-rdf-jsonld-java/src/main/java/org/apache/commons/rdf/jsonldjava/JsonLdRDF.java
@@ -273,21 +273,21 @@ public final class JsonLdRDF implements RDF {
}
if (blankNodeOrIRI instanceof IRI) {
return ((IRI) blankNodeOrIRI).getIRIString();
- } else if (blankNodeOrIRI instanceof BlankNode) {
- final BlankNode blankNode = (BlankNode) blankNodeOrIRI;
- final String ref = blankNode.uniqueReference();
- if (ref.startsWith(bnodePrefix)) {
- // One of ours (but possibly not a JsonLdBlankNode) -
- // we can use the suffix directly
- return ref.replace(bnodePrefix, "");
- }
- // Map to unique bnode identifier, e.g.
- // _:0dbd92ee-ab1a-45e7-bba2-7ade54f87ec5
- final UUID uuid =
UUID.nameUUIDFromBytes(ref.getBytes(StandardCharsets.UTF_8));
- return "_:" + uuid;
- } else {
+ }
+ if (!(blankNodeOrIRI instanceof BlankNode)) {
throw new IllegalArgumentException("Expected a BlankNode or IRI,
not: " + blankNodeOrIRI);
}
+ final BlankNode blankNode = (BlankNode) blankNodeOrIRI;
+ final String ref = blankNode.uniqueReference();
+ if (ref.startsWith(bnodePrefix)) {
+ // One of ours (but possibly not a JsonLdBlankNode) -
+ // we can use the suffix directly
+ return ref.replace(bnodePrefix, "");
+ }
+ // Map to unique bnode identifier, e.g.
+ // _:0dbd92ee-ab1a-45e7-bba2-7ade54f87ec5
+ final UUID uuid =
UUID.nameUUIDFromBytes(ref.getBytes(StandardCharsets.UTF_8));
+ return "_:" + uuid;
}
JsonLdTerm asRDFTerm(final Node node, final String blankNodePrefix) {
@@ -296,17 +296,18 @@ public final class JsonLdRDF implements RDF {
}
if (node.isIRI()) {
return new JsonLdIRIImpl(node);
- } else if (node.isBlankNode()) {
+ }
+ if (node.isBlankNode()) {
return new JsonLdBlankNodeImpl(node, blankNodePrefix);
- } else if (node.isLiteral()) {
- // TODO: Our own JsonLdLiteral
- if (node.getLanguage() != null) {
- return createLiteral(node.getValue(), node.getLanguage());
- }
- return createLiteral(node.getValue(),
createIRI(node.getDatatype()));
- } else {
+ }
+ if (!node.isLiteral()) {
throw new IllegalArgumentException("Node is neither IRI, BlankNode
nor Literal: " + node);
}
+ // TODO: Our own JsonLdLiteral
+ if (node.getLanguage() != null) {
+ return createLiteral(node.getValue(), node.getLanguage());
+ }
+ return createLiteral(node.getValue(), createIRI(node.getDatatype()));
}
RDFDataset.Quad createJsonLdQuad(final BlankNodeOrIRI graphName, final
BlankNodeOrIRI subject, final IRI predicate, final RDFTerm object) {
diff --git
a/commons-rdf-rdf4j/src/main/java/org/apache/commons/rdf/rdf4j/RDF4J.java
b/commons-rdf-rdf4j/src/main/java/org/apache/commons/rdf/rdf4j/RDF4J.java
index 6b5ec8c..e99b239 100644
--- a/commons-rdf-rdf4j/src/main/java/org/apache/commons/rdf/rdf4j/RDF4J.java
+++ b/commons-rdf-rdf4j/src/main/java/org/apache/commons/rdf/rdf4j/RDF4J.java
@@ -300,7 +300,8 @@ public final class RDF4J implements RDF {
public RDF4JBlankNodeOrIRI asRDFTerm(final
org.eclipse.rdf4j.model.Resource value) {
if(value instanceof IRI){
return asRDFTerm((IRI)value);
- } else if (value instanceof BNode){
+ }
+ if (value instanceof BNode){
return asRDFTerm((BNode)value);
}
throw new IllegalArgumentException("Value is not a BNode or IRI: " +
value.getClass());
diff --git
a/commons-rdf-simple/src/main/java/org/apache/commons/rdf/simple/DatasetImpl.java
b/commons-rdf-simple/src/main/java/org/apache/commons/rdf/simple/DatasetImpl.java
index 94d57e3..a8c0455 100644
---
a/commons-rdf-simple/src/main/java/org/apache/commons/rdf/simple/DatasetImpl.java
+++
b/commons-rdf-simple/src/main/java/org/apache/commons/rdf/simple/DatasetImpl.java
@@ -91,18 +91,19 @@ final class DatasetImpl implements Dataset {
// equivalent object, based on the code in the package private
// BlankNodeImpl class
return factory.createBlankNode(blankNode.uniqueReference());
- } else if (object instanceof IRI && !(object instanceof IRIImpl)) {
+ }
+ if (object instanceof IRI && !(object instanceof IRIImpl)) {
final IRI iri = (IRI) object;
return factory.createIRI(iri.getIRIString());
- } else if (object instanceof Literal && !(object instanceof
LiteralImpl)) {
- final Literal literal = (Literal) object;
- if (literal.getLanguageTag().isPresent()) {
- return factory.createLiteral(literal.getLexicalForm(),
literal.getLanguageTag().get());
- }
- return factory.createLiteral(literal.getLexicalForm(), (IRI)
internallyMap(literal.getDatatype()));
- } else {
+ }
+ if (!(object instanceof Literal) || (object instanceof LiteralImpl)) {
throw new IllegalArgumentException("Not a BlankNode, IRI or
Literal: " + object);
}
+ final Literal literal = (Literal) object;
+ if (literal.getLanguageTag().isPresent()) {
+ return factory.createLiteral(literal.getLexicalForm(),
literal.getLanguageTag().get());
+ }
+ return factory.createLiteral(literal.getLexicalForm(), (IRI)
internallyMap(literal.getDatatype()));
}
@Override
diff --git
a/commons-rdf-simple/src/main/java/org/apache/commons/rdf/simple/GraphImpl.java
b/commons-rdf-simple/src/main/java/org/apache/commons/rdf/simple/GraphImpl.java
index 92307ce..7ec7589 100644
---
a/commons-rdf-simple/src/main/java/org/apache/commons/rdf/simple/GraphImpl.java
+++
b/commons-rdf-simple/src/main/java/org/apache/commons/rdf/simple/GraphImpl.java
@@ -72,18 +72,19 @@ final class GraphImpl implements Graph {
// equivalent object, based on the code in the package private
// BlankNodeImpl class
return factory.createBlankNode(blankNode.uniqueReference());
- } else if (object instanceof IRI) {
+ }
+ if (object instanceof IRI) {
final IRI iri = (IRI) object;
return factory.createIRI(iri.getIRIString());
- } else if (object instanceof Literal) {
- final Literal literal = (Literal) object;
- if (literal.getLanguageTag().isPresent()) {
- return factory.createLiteral(literal.getLexicalForm(),
literal.getLanguageTag().get());
- }
- return factory.createLiteral(literal.getLexicalForm(), (IRI)
internallyMap(literal.getDatatype()));
- } else {
+ }
+ if (!(object instanceof Literal)) {
throw new IllegalArgumentException("RDFTerm was neither a
BlankNode, IRI nor Literal: " + object);
}
+ final Literal literal = (Literal) object;
+ if (literal.getLanguageTag().isPresent()) {
+ return factory.createLiteral(literal.getLexicalForm(),
literal.getLanguageTag().get());
+ }
+ return factory.createLiteral(literal.getLexicalForm(), (IRI)
internallyMap(literal.getDatatype()));
}
private Triple internallyMap(final Triple triple) {
