Add debug messages and toString

Treiblesschorle · gselzer · commit 053499544345 · 2019-04-12T12:54:02.000-05:00
diff --git a/src/main/java/org/scijava/ops/OpService.java b/src/main/java/org/scijava/ops/OpService.java
@@ -167,12 +167,16 @@ public <T> T findOpInstance(final String opName, final Nil<T> specialType, final
 			OpCandidate match = findTypeMatch(ref);
 			return (T) match.createOp(secondaryArgs);
 		} catch (OpMatchingException e) {
+			log.debug("No matching Op for request: " + ref + "\n");
+			log.debug("Attempting Op transformation...");
 			// If we can't find an op matching the original request, we try to find a transformation
 			OpTransformationCandidate transformation = transformer.findTransfromation(ref);
 			// If we found one, try to do transformation and return transformed op
 			if (transformation != null) {
+				log.debug("Matching Op transformation found:\n" + transformation + "\n");
 				return (T) transformation.exceute(this, secondaryArgs);
 			}
+			log.debug("No matching Op transformation found");
 			throw new IllegalArgumentException(e);
 		}
 	}
diff --git a/src/main/java/org/scijava/ops/transform/OpTransformation.java b/src/main/java/org/scijava/ops/transform/OpTransformation.java
@@ -98,7 +98,7 @@ public String toString() {
 		int i = 0;
 		OpTransformation c = this;
 		do {
-			s.append(i + ")\n");
+			s.append(i + ")");
 			s.append("\tFrom:\t");
 			s.append(Arrays.deepToString(c.getSource().getTypes()));
 			s.append("\n\tTo:\t\t");
diff --git a/src/main/java/org/scijava/ops/transform/OpTransformationCandidate.java b/src/main/java/org/scijava/ops/transform/OpTransformationCandidate.java
@@ -27,4 +27,12 @@ public Object exceute(OpService opService, Object... secondaryArgs) {
 			return null;
 		}
 	}
+	
+	@Override
+	public String toString() {
+		StringBuilder s = new StringBuilder();
+		s.append("Source Op:\n\n" + srcOp + "\n");
+		s.append("With transformation: \n" + transformation);
+		return s.toString();
+	}
 }

Original file line number	Diff line number	Diff line change
`@@ -167,12 +167,16 @@ public <T> T findOpInstance(final String opName, final Nil<T> specialType, final`
`167`	`167`	`OpCandidate match = findTypeMatch(ref);`
`168`	`168`	`return (T) match.createOp(secondaryArgs);`
`169`	`169`	`} catch (OpMatchingException e) {`
	`170`	`+ log.debug("No matching Op for request: " + ref + "\n");`
	`171`	`+ log.debug("Attempting Op transformation...");`
`170`	`172`	`// If we can't find an op matching the original request, we try to find a transformation`
`171`	`173`	`OpTransformationCandidate transformation = transformer.findTransfromation(ref);`
`172`	`174`	`// If we found one, try to do transformation and return transformed op`
`173`	`175`	`if (transformation != null) {`
	`176`	`+ log.debug("Matching Op transformation found:\n" + transformation + "\n");`
`174`	`177`	`return (T) transformation.exceute(this, secondaryArgs);`
`175`	`178`	`}`
	`179`	`+ log.debug("No matching Op transformation found");`
`176`	`180`	`throw new IllegalArgumentException(e);`
`177`	`181`	`}`
`178`	`182`	`}`
Original file line number	Diff line number	Diff line change
`@@ -27,4 +27,12 @@ public Object exceute(OpService opService, Object... secondaryArgs) {`
`27`	`27`	`return null;`
`28`	`28`	`}`
`29`	`29`	`}`
	`30`	`+`
	`31`	`+ @Override`
	`32`	`+ public String toString() {`
	`33`	`+ StringBuilder s = new StringBuilder();`
	`34`	`+ s.append("Source Op:\n\n" + srcOp + "\n");`
	`35`	`+ s.append("With transformation: \n" + transformation);`
	`36`	`+ return s.toString();`
	`37`	`+ }`
`30`	`38`	`}`