From e501a0bdca762a019aaebde94918f6893d695cb7 Mon Sep 17 00:00:00 2001 From: Ruben Dijkstra Date: Tue, 25 Mar 2014 17:44:28 +0100 Subject: [PATCH] invoke the toString method on the Class objects. this allows for more verbose exception strings (class and interface keywords, or no keywords when primitive) --- .../com/mysema/query/types/ConstructorExpression.java | 10 ++-------- 1 file changed, 2 insertions(+), 8 deletions(-) diff --git a/querydsl-core/src/main/java/com/mysema/query/types/ConstructorExpression.java b/querydsl-core/src/main/java/com/mysema/query/types/ConstructorExpression.java index 5764197e4..93dbf9f34 100644 --- a/querydsl-core/src/main/java/com/mysema/query/types/ConstructorExpression.java +++ b/querydsl-core/src/main/java/com/mysema/query/types/ConstructorExpression.java @@ -71,15 +71,9 @@ public class ConstructorExpression extends ExpressionBase implements Facto } } // prepare error message - StringBuilder formattedTypes = new StringBuilder(); - for (Class typ : givenTypes) { - if (formattedTypes.length() > 0) { - formattedTypes.append(", "); - } - formattedTypes.append(typ.getName()); - } + String parameters = Arrays.toString(givenTypes); throw new ExpressionException("Got no matching constructor. Class: " + - type.getName() +", parameters: " + formattedTypes.toString()); + type.getName() +", parameters: " + parameters); } public static ConstructorExpression create(Class type, Expression... args) {