File path: /hibernate-distribution-3.3.2.GA/project/core/src/main/java/org/hibernate/id/TableGenerator.java | File path: /hibernate-distribution-3.3.2.GA/project/core/src/main/java/org/hibernate/id/enhanced/TableStructure.java | |||
Method name: Serializable doWorkInCurrentTransaction(Connection, String)
|
Method name: Serializable doWorkInCurrentTransaction(Connection, String)
|
|||
Number of AST nodes: 19 | Number of AST nodes: 18 | |||
1 | do {↵ | 1 | do {↵ | |
2 | // The loop ensures atomicity of the↵ | |||
3 | // select + update even for no transaction↵ | |||
4 | // or read committed isolation level↵ | |||
5 | sql = query;↵ | 2 | ↵ | |
6 | SQL_STATEMENT_LOGGER.logStatement( sql, FormatStyle.BASIC );↵ | 3 | SQL_STATEMENT_LOGGER.logStatement( selectQuery, FormatStyle.BASIC ); ↵ | |
7 | PreparedStatement qps = conn.prepareStatement(query);↵ | 4 | PreparedStatement selectPS = conn.prepareStatement( selectQuery ); ↵ | |
8 | try {↵ | 5 | try { ↵ | |
9 | ResultSet rs = qps.executeQuery();↵ | 6 | ResultSet selectRS = selectPS.executeQuery(); ↵ | |
10 | if ( !rs.next() ) {↵ | 7 | if ( !selectRS.next() ) { ↵ | |
11 | String err = "could not read a hi value - you need to populate the table: " + tableName;↵ | 8 | String err = "could not read a hi value - you need to populate the table: " + tableName; ↵ | |
12 | log.error(err);↵ | 9 | log.error( err ); ↵ | |
13 | throw new IdentifierGenerationException(err);↵ | 10 | throw new IdentifierGenerationException( err ); ↵ | |
14 | }↵ | 11 | } ↵ | |
15 | result = rs.getInt(1);↵ | 12 | result = selectRS.getLong( 1 ); ↵ | |
16 | rs.close();↵ | 13 | selectRS.close(); ↵ | |
17 | }↵ | 14 | } ↵ | |
18 | catch (SQLException sqle) {↵ | 15 | catch ( SQLException sqle ) { ↵ | |
19 | log.error("could not read a hi value", sqle);↵ | 16 | log.error( "could not read a hi value", sqle ); ↵ | |
20 | throw sqle;↵ | 17 | throw sqle; ↵ | |
21 | }↵ | 18 | } ↵ | |
22 | finally {↵ | 19 | finally { ↵ | |
23 | qps.close();↵ | 20 | selectPS.close(); ↵ | |
24 | }↵ | 21 | }↵ | |
25 | sql = update;↵ | |||
22 | ↵ | |||
26 | SQL_STATEMENT_LOGGER.logStatement( sql, FormatStyle.BASIC );↵ | 23 | SQL_STATEMENT_LOGGER.logStatement( updateQuery, FormatStyle.BASIC ); ↵ | |
27 | PreparedStatement ups = conn.prepareStatement(update);↵ | 24 | PreparedStatement updatePS = conn.prepareStatement( updateQuery ); ↵ | |
28 | try {↵ | 25 | try { ↵ | |
29 | ups.setInt↵ | 26 | int increment = applyIncrementSizeToSourceValues ? incrementSize : 1;↵ | |
30 | ( 1, result + 1 );↵ | 27 | updatePS.setLong( 1, result + increment ); ↵ | |
31 | ups.setInt( 2, result );↵ | 28 | updatePS.setLong( 2, result ); ↵ | |
32 | rows = ups.executeUpdate();↵ | 29 | rows = updatePS.executeUpdate(); ↵ | |
33 | }↵ | 30 | } ↵ | |
34 | catch (SQLException sqle) {↵ | 31 | catch ( SQLException sqle ) { ↵ | |
35 | log.error("could not update hi value in: " + tableName, sqle);↵ | 32 | log.error( "could not updateQuery hi value in: " + tableName, sqle ); ↵ | |
36 | throw sqle;↵ | 33 | throw sqle; ↵ | |
37 | }↵ | 34 | } ↵ | |
38 | finally {↵ | 35 | finally { ↵ | |
39 | ups.close();↵ | 36 | updatePS.close(); ↵ | |
40 | }↵ | 37 | } ↵ | |
41 | }↵ | 38 | }↵ | |
42 | while (rows==0); | 39 | while ( rows == 0 ); | |
See real code fragment | See real code fragment |
Number of common nesting structure subtrees | 1 |
Number of refactorable cases | 0 |
Number of non-refactorable cases | 1 |
Time elapsed for finding largest common nesting structure subtrees (ms) | 0.6 |
Clones location | Clones are in different classes having the same super class |
Number of node comparisons | 87 |
Number of mapped statements | 14 |
Number of unmapped statements in the first code fragment | 5 |
Number of unmapped statements in the second code fragment | 4 |
Time elapsed for statement mapping (ms) | 10.3 |
Clone type | Type 3 |
ID | Statement | ID | Statement | ||||||||||||||
---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|
21 | do while(rows == 0) | 20 | do while(rows == 0) | ||||||||||||||
3 | sql = query; |
| | ||||||||||||||
4 | SQL_STATEMENT_LOGGER.logStatement(sql, FormatStyle.BASIC); |
| 3 | SQL_STATEMENT_LOGGER.logStatement(selectQuery, FormatStyle.BASIC); | |||||||||||||
5 | PreparedStatement qps = conn.prepareStatement(query); |
| 4 | PreparedStatement selectPS = conn.prepareStatement(selectQuery); | |||||||||||||
6 | try |
| 5 | try | |||||||||||||
7 | ResultSet rs = qps.executeQuery(); |
| 6 | ResultSet selectRS = selectPS.executeQuery(); | |||||||||||||
8 | if (!rs.next()) |
| 7 | if (!selectRS.next()) | |||||||||||||
9 | String err = "could not read a hi value - you need to populate the table: " + tableName; | 8 | String err = "could not read a hi value - you need to populate the table: " + tableName; | ||||||||||||||
10 | log.error(err); | 9 | log.error(err); | ||||||||||||||
11 | throw new IdentifierGenerationException(err); | 10 | throw new IdentifierGenerationException(err); | ||||||||||||||
|
| 11 | result = selectRS.getLong(1); | ||||||||||||||
12 | result = rs.getInt(1); |
| | ||||||||||||||
13 | rs.close(); |
| 12 | selectRS.close(); | |||||||||||||
14 | sql = update; |
| | ||||||||||||||
15 | SQL_STATEMENT_LOGGER.logStatement(sql, FormatStyle.BASIC); |
| 13 | SQL_STATEMENT_LOGGER.logStatement(updateQuery, FormatStyle.BASIC); | |||||||||||||
16 | PreparedStatement ups = conn.prepareStatement(update); |
| 14 | PreparedStatement updatePS = conn.prepareStatement(updateQuery); | |||||||||||||
17 | try |
| 15 | try | |||||||||||||
|
| 16 | int increment = applyIncrementSizeToSourceValues ? incrementSize : 1; | ||||||||||||||
|
| 17 | updatePS.setLong(1, result + increment); | ||||||||||||||
|
| 18 | updatePS.setLong(2, result); | ||||||||||||||
18 | ups.setInt(1, result + 1); |
| | ||||||||||||||
19 | ups.setInt(2, result); |
| | ||||||||||||||
20 | rows = ups.executeUpdate(); |
| 19 | rows = updatePS.executeUpdate(); |
Row | Violation |
---|---|
1 | Unmatched statement sql=query; cannot be moved before or after the extracted code, because it has dependencies to/from statements that will be extracted |
2 | Expression sql cannot be parameterized, because it has dependencies to/from statements that will be extracted |
3 | Unmatched statement result=selectRS.getLong(1); cannot be moved before or after the extracted code, because it has dependencies to/from statements that will be extracted |
4 | Unmatched statement result=selectRS.getLong(1); cannot be moved before or after the extracted code, because it throws exception(s) that should be caught by a try block that will be extracted |
5 | Unmatched statement result=rs.getInt(1); cannot be moved before or after the extracted code, because it has dependencies to/from statements that will be extracted |
6 | Unmatched statement result=rs.getInt(1); cannot be moved before or after the extracted code, because it throws exception(s) that should be caught by a try block that will be extracted |
7 | Unmatched statement sql=update; cannot be moved before or after the extracted code, because it has dependencies to/from statements that will be extracted |
8 | Expression sql cannot be parameterized, because it has dependencies to/from statements that will be extracted |
9 | Unmatched statement int increment=applyIncrementSizeToSourceValues ? incrementSize : 1; cannot be moved before or after the extracted code, because it has dependencies to/from statements that will be extracted |
10 | Unmatched statement updatePS.setLong(1,result + increment); cannot be moved before or after the extracted code, because it has dependencies to/from statements that will be extracted |
11 | Unmatched statement updatePS.setLong(1,result + increment); cannot be moved before or after the extracted code, because it throws exception(s) that should be caught by a try block that will be extracted |
12 | Unmatched statement updatePS.setLong(2,result); cannot be moved before or after the extracted code, because it has dependencies to/from statements that will be extracted |
13 | Unmatched statement updatePS.setLong(2,result); cannot be moved before or after the extracted code, because it throws exception(s) that should be caught by a try block that will be extracted |
14 | Unmatched statement ups.setInt(1,result + 1); cannot be moved before or after the extracted code, because it has dependencies to/from statements that will be extracted |
15 | Unmatched statement ups.setInt(1,result + 1); cannot be moved before or after the extracted code, because it throws exception(s) that should be caught by a try block that will be extracted |
16 | Unmatched statement ups.setInt(2,result); cannot be moved before or after the extracted code, because it has dependencies to/from statements that will be extracted |
17 | Unmatched statement ups.setInt(2,result); cannot be moved before or after the extracted code, because it throws exception(s) that should be caught by a try block that will be extracted |
18 | Clone fragment #1 returns variables rs, ups , while Clone fragment #2 returns variables selectRS, updatePS |