解决某个字段值为 null 导致中断后续正常返回值；解决 LEFT/RIGHT JOIN 副表关联主表外键的字段取别名导致 SQL 报错

Author: TommyLemon

APIJSONORM/src/main/java/apijson/orm/AbstractParser.java

@@ -432,6 +432,7 @@ public JSONObject parseResponse(JSONObject request) {
 			requestObject.put("sql:generate|cache|execute|maxExecute", getSQLExecutor().getGeneratedSQLCount() + "|" + getSQLExecutor().getCachedSQLCount() + "|" + getSQLExecutor().getExecutedSQLCount() + "|" + getMaxSQLCount());
 			requestObject.put("depth:count|max", queryDepth + "|" + getMaxQueryDepth());
 			requestObject.put("time:start|duration|end", startTime + "|" + duration + "|" + endTime);
+// TODO 放在 msg 中的调试和提示信息应该单独放一个字段，避免 APIAuto 异常分支不显示提示语或太长，以及 DEBUG 和非 DEBUG 模式下提示语不一致			requestObject.put("debug", debugStr);
 			if (error != null) {
 				requestObject.put("throw", error.getClass().getName());
 				requestObject.put("trace", error.getStackTrace());

APIJSONORM/src/main/java/apijson/orm/AbstractSQLConfig.java

@@ -83,7 +83,7 @@ public abstract class AbstractSQLConfig implements SQLConfig {
 	// * 和 / 不能同时出现，防止 /* */ 段注释！ # 和 -- 不能出现，防止行注释！ ; 不能出现，防止隔断SQL语句！空格不能出现，防止 CRUD,DROP,SHOW TABLES等语句！
 	private static final Pattern PATTERN_RANGE;
 	private static final Pattern PATTERN_FUNCTION;
-	private  static final Pattern PATTERN_STRING;
+	private static final Pattern PATTERN_STRING;
 
 	/**
 	 * 表名映射，隐藏真实表名，对安全要求很高的表可以这么做
@@ -1490,9 +1490,8 @@ public String getColumnString(boolean inSQLJoin) throws Exception {
 			return "(" + s + ")";
 		case GET:
 		case GETS:
-			boolean isQuery = RequestMethod.isQueryMethod(method); //TODO 这个有啥用？上面应是 getMethod 的值 GET 和 GETS 了。
 			String joinColumn = "";
-			if (isQuery && joinList != null) {
+			if (joinList != null) {
 				SQLConfig ecfg;
 				SQLConfig cfg;
 				String c;
@@ -1501,23 +1500,31 @@ public String getColumnString(boolean inSQLJoin) throws Exception {
 					if (j.isAppJoin()) {
 						continue;
 					}
-
-					ecfg = j.getOuterConfig();
-					if (ecfg != null && ecfg.getColumn() != null) { //优先级更高
-						cfg = ecfg;
-					}
-					else {
-						cfg = j.getJoinConfig();
-					}
-
-					if (StringUtil.isEmpty(cfg.getAlias(), true)) {
-						cfg.setAlias(cfg.getTable());
-					}
-
-					c = ((AbstractSQLConfig) cfg).getColumnString(true);
-					if (StringUtil.isEmpty(c, true) == false) {
-						joinColumn += (first ? "" : ", ") + c;
+					
+					if (j.isLeftOrRightJoin()) {
+						// 改为 SELECT  ViceTable.* 解决 SELECT sum(ViceTable.id) LEFT/RIGHT JOIN (SELECT sum(id) FROM ViceTable...) AS ViceTable
+						// 不仅导致 SQL 函数重复计算，还有时导致 SQL 报错或对应字段未返回
+						String quote = getQuote();
+						joinColumn += (first ? "" : ", ") + quote + (StringUtil.isEmpty(j.getAlias(), true) ? j.getTable() : j.getAlias()) + quote + ".*";
 						first = false;
+					} else {
+						ecfg = j.getOuterConfig();
+						if (ecfg != null && ecfg.getColumn() != null) { //优先级更高
+							cfg = ecfg;
+						}
+						else {
+							cfg = j.getJoinConfig();
+						}
+	
+						if (StringUtil.isEmpty(cfg.getAlias(), true)) {
+							cfg.setAlias(cfg.getTable());
+						}
+	
+						c = ((AbstractSQLConfig) cfg).getColumnString(true);
+						if (StringUtil.isEmpty(c, true) == false) {
+							joinColumn += (first ? "" : ", ") + c;
+							first = false;
+						}
 					}
 
 					inSQLJoin = true;

APIJSONORM/src/main/java/apijson/orm/AbstractSQLExecutor.java

@@ -542,7 +542,8 @@ protected JSONObject onPutColumn(@NotNull SQLConfig config, @NotNull ResultSet r
 			if (joinList != null) {
 				for (Join j : joinList) {
 					childConfig = j.isAppJoin() ? null : j.getCacheConfig(); //这里用config改了getSQL后再还原很麻烦，所以提前给一个config2更好
-
+					
+					// FIXME 副表的 SQL 函数，甚至普通字段都可能从 rsmd.getTableName(columnIndex) 拿到 ""
 					if (childConfig != null && childTable.equalsIgnoreCase(childConfig.getSQLTable())) {
 
 						childConfig.putWhere(j.getKey(), table.get(j.getTargetKey()), true);
@@ -561,13 +562,13 @@ protected JSONObject onPutColumn(@NotNull SQLConfig config, @NotNull ResultSet r
 		}
 
 		Object value = getValue(config, rs, rsmd, tablePosition, table, columnIndex, lable, childMap);
-		if (value != null) {
-			if (finalTable == null) {
-				finalTable = new JSONObject(true);
-				childMap.put(childSql, finalTable);
-			}
-			finalTable.put(lable, value);
+		//	必须 put 进去，否则某个字段为 null 可能导致中断后续正常返回值	if (value != null) {
+		if (finalTable == null) {
+			finalTable = new JSONObject(true);
+			childMap.put(childSql, finalTable);
 		}
+		finalTable.put(lable, value);
+		//		}
 
 		return table;
 	}