Skip to content

Commit 739e1dd

Browse files
authored
Merge pull request #2121 from lonecloud/code_optimization
code optimization
2 parents 5e2a954 + 67771db commit 739e1dd

File tree

1 file changed

+6
-9
lines changed

1 file changed

+6
-9
lines changed

src/main/java/org/apache/ibatis/session/defaults/DefaultSqlSession.java

Lines changed: 6 additions & 9 deletions
Original file line numberDiff line numberDiff line change
@@ -142,9 +142,13 @@ public <E> List<E> selectList(String statement, Object parameter) {
142142

143143
@Override
144144
public <E> List<E> selectList(String statement, Object parameter, RowBounds rowBounds) {
145+
return selectList(statement, parameter, rowBounds, Executor.NO_RESULT_HANDLER);
146+
}
147+
148+
private <E> List<E> selectList(String statement, Object parameter, RowBounds rowBounds, ResultHandler handler) {
145149
try {
146150
MappedStatement ms = configuration.getMappedStatement(statement);
147-
return executor.query(ms, wrapCollection(parameter), rowBounds, Executor.NO_RESULT_HANDLER);
151+
return executor.query(ms, wrapCollection(parameter), rowBounds, handler);
148152
} catch (Exception e) {
149153
throw ExceptionFactory.wrapException("Error querying database. Cause: " + e, e);
150154
} finally {
@@ -164,14 +168,7 @@ public void select(String statement, ResultHandler handler) {
164168

165169
@Override
166170
public void select(String statement, Object parameter, RowBounds rowBounds, ResultHandler handler) {
167-
try {
168-
MappedStatement ms = configuration.getMappedStatement(statement);
169-
executor.query(ms, wrapCollection(parameter), rowBounds, handler);
170-
} catch (Exception e) {
171-
throw ExceptionFactory.wrapException("Error querying database. Cause: " + e, e);
172-
} finally {
173-
ErrorContext.instance().reset();
174-
}
171+
selectList(statement, parameter, rowBounds, handler);
175172
}
176173

177174
@Override

0 commit comments

Comments
 (0)