--- a/java/sql-dk/src/info/globalcode/sql/dk/SQLCommandNamed.java Tue Dec 24 01:42:24 2013 +0100
+++ b/java/sql-dk/src/info/globalcode/sql/dk/SQLCommandNamed.java Tue Dec 24 11:54:58 2013 +0100
@@ -18,13 +18,14 @@
package info.globalcode.sql.dk;
import static info.globalcode.sql.dk.Functions.notNull;
-import static info.globalcode.sql.dk.Functions.escapeRegEx;
import static info.globalcode.sql.dk.Functions.findByName;
import java.sql.Connection;
import java.sql.PreparedStatement;
import java.sql.SQLException;
import java.util.ArrayList;
import java.util.List;
+import java.util.logging.Level;
+import java.util.logging.Logger;
import java.util.regex.Matcher;
import java.util.regex.Pattern;
@@ -34,7 +35,7 @@
*/
public class SQLCommandNamed extends SQLCommand {
- private static final String PROBLEM_MARK = "<OMG>";
+ private static final Logger log = Logger.getLogger(SQLCommandNamed.class.getName());
private String namePrefix;
private String nameSuffix;
private List<NamedParameter> parameters;
@@ -54,6 +55,7 @@
public PreparedStatement prepareStatement(Connection c) throws SQLException {
buildPattern();
placeParametersAndUpdateQuery();
+ logPossiblyMissingParameters();
return c.prepareStatement(updatedQuery.toString());
}
@@ -72,16 +74,16 @@
private void buildPattern() {
StringBuilder patternString = new StringBuilder();
- patternString.append(escapeRegEx(namePrefix));
+ patternString.append(Pattern.quote(namePrefix));
patternString.append("(");
for (int i = 0; i < parameters.size(); i++) {
- patternString.append(escapeRegEx(parameters.get(i).getName()));
- if (i < parameters.size()) {
+ patternString.append(Pattern.quote(parameters.get(i).getName()));
+ if (i < parameters.size() - 1) {
patternString.append("|");
}
}
patternString.append(")");
- patternString.append(escapeRegEx(nameSuffix));
+ patternString.append(Pattern.quote(nameSuffix));
pattern = Pattern.compile(patternString.toString());
}
@@ -95,13 +97,6 @@
String name = m.group(1);
updatedQuery.append(originalQuery.substring(lastPosition, m.start()));
-
- if (name.isEmpty()) {
- updatedQuery.append(PROBLEM_MARK);
- updatedQuery.append(originalQuery.substring(m.end(), originalQuery.length()));
- throw new SQLException("Named parameter (near " + PROBLEM_MARK + ") is not defined: „" + updatedQuery + "“");
- }
-
updatedQuery.append("?");
parametersUsed.add(findByName(parameters, name));
@@ -117,6 +112,17 @@
}
}
+ private void logPossiblyMissingParameters() {
+ Pattern p = Pattern.compile(Pattern.quote(namePrefix) + ".*?" + Pattern.quote(nameSuffix));
+ Matcher m = p.matcher(updatedQuery);
+ int lastPosition = 0;
+ while (m.find(lastPosition)) {
+
+ log.log(Level.WARNING, "Possibly missing parameter: {0}", m.group());
+ lastPosition = m.end();
+ }
+ }
+
@Override
public List<NamedParameter> getParameters() {
return parameters;