File path: /hibernate-distribution-3.3.2.GA/project/core/src/main/java/org/hibernate/persister/collection/AbstractCollectionPersister.java | File path: /hibernate-distribution-3.3.2.GA/project/core/src/main/java/org/hibernate/persister/collection/BasicCollectionPersister.java | |||
Method name: void insertRows(PersistentCollection, Serializable, SessionImplementor)
|
Method name: int doUpdateRows(Serializable, PersistentCollection, SessionImplementor)
|
|||
Number of AST nodes: 22 | Number of AST nodes: 22 | |||
1 | if ( collection.needsInserting( entry, i, elementType ) ) {↵ | 1 | int offset = 1;↵ | |
2 | if ( useBatch ) {↵ | 2 | if ( useBatch ) {↵ | |
3 | if ( st == null ) {↵ | 3 | if ( st == null ) {↵ | |
4 | if ( callable ) {↵ | 4 | if ( callable ) {↵ | |
5 | st = session.getBatcher().prepareBatchCallableStatement( sql );↵ | 5 | st = session.getBatcher().prepareBatchCallableStatement( sql );↵ | |
6 | }↵ | 6 | }↵ | |
7 | else {↵ | 7 | else {↵ | |
8 | st = session.getBatcher().prepareBatchStatement( sql );↵ | 8 | st = session.getBatcher().prepareBatchStatement( sql );↵ | |
9 | }↵ | 9 | }↵ | |
10 | }↵ | 10 | }↵ | |
11 | }↵ | 11 | }↵ | |
12 | else {↵ | 12 | else {↵ | |
13 | if ( callable ) {↵ | 13 | if ( callable ) {↵ | |
14 | st = session.getBatcher().prepareCallableStatement( sql );↵ | 14 | st = session.getBatcher().prepareCallableStatement( sql );↵ | |
15 | }↵ | 15 | }↵ | |
16 | else {↵ | 16 | else {↵ | |
17 | st = session.getBatcher().prepareStatement( sql );↵ | 17 | st = session.getBatcher().prepareStatement( sql );↵ | |
18 | }↵ | 18 | }↵ | |
19 | }↵ | 19 | }↵ | |
20 | try {↵ | 20 | try {↵ | |
21 | offset += expectation.prepare( st );↵ | 21 | offset+= expectation.prepare( st );↵ | |
22 | //TODO: copy/paste from recreate()↵ | 22 | ↵ | |
23 | offset = writeKey( st, id, offset, session );↵ | 23 | int loc = writeElement( st, collection.getElement( entry ), offset, session );↵ | |
24 | if ( hasIdentifier ) {↵ | 24 | if ( hasIdentifier ) {↵ | |
25 | offset = writeIdentifier( st, collection.getIdentifier(entry, i), offset, session );↵ | 25 | writeIdentifier( st, collection.getIdentifier( entry, i ), loc, session );↵ | |
26 | }↵ | 26 | }↵ | |
27 | ↵ | 27 | else {↵ | |
28 | loc = writeKey( st, id, loc, session );↵ | |||
28 | if ( hasIndex /*&& !indexIsFormula*/ ) {↵ | 29 | if ( hasIndex && !indexContainsFormula ) {↵ | |
29 | offset = writeIndex( st, collection.getIndex(entry, i, this), offset, session );↵ | 30 | writeIndexToWhere( st, collection.getIndex( entry, i, this ), loc, session );↵ | |
30 | }↵ | 31 | }↵ | |
31 | ↵ | 32 | else {↵ | |
32 | writeElement(st, collection.getElement(entry), offset, session );↵ | 33 | writeElementToWhere( st, collection.getSnapshotElement( entry, i ), loc, session );↵ | |
33 | ↵ | 34 | }↵ | |
35 | }↵ | |||
34 | if ( useBatch ) {↵ | 36 | if ( useBatch ) {↵ | |
35 | session.getBatcher().addToBatch( expectation );↵ | 37 | session.getBatcher().addToBatch( expectation );↵ | |
36 | }↵ | 38 | }↵ | |
37 | else {↵ | 39 | else {↵ | |
38 | expectation.verifyOutcome( st.executeUpdate(), st, -1 );↵ | 40 | expectation.verifyOutcome( st.executeUpdate(), st, -1 );↵ | |
39 | }↵ | 41 | }↵ | |
40 | collection.afterRowInsert( this, entry, i );↵ | 42 | ↵ | |
41 | count++;↵ | |||
42 | }↵ | 43 | }↵ | |
43 | catch ( SQLException sqle ) {↵ | 44 | catch ( SQLException sqle ) {↵ | |
44 | if ( useBatch ) {↵ | 45 | if ( useBatch ) {↵ | |
45 | session.getBatcher().abortBatch( sqle );↵ | 46 | session.getBatcher().abortBatch( sqle );↵ | |
46 | }↵ | 47 | }↵ | |
47 | throw sqle;↵ | 48 | throw sqle;↵ | |
48 | }↵ | 49 | }↵ | |
49 | finally {↵ | 50 | finally {↵ | |
50 | if ( !useBatch ) {↵ | 51 | if ( !useBatch ) {↵ | |
51 | session.getBatcher().closeStatement( st );↵ | 52 | session.getBatcher().closeStatement( st );↵ | |
52 | }↵ | 53 | }↵ | |
53 | }↵ | 54 | }↵ | |
54 | } | 55 | count++; | |
See real code fragment | See real code fragment |
Number of common nesting structure subtrees | 2 |
Number of refactorable cases | 1 |
Number of non-refactorable cases | 1 |
Time elapsed for finding largest common nesting structure subtrees (ms) | 1.5 |
Clones location | Clones are in different classes having the same super class |
Number of node comparisons | 54 |
Number of mapped statements | 8 |
Number of unmapped statements in the first code fragment | 0 |
Number of unmapped statements in the second code fragment | 1 |
Time elapsed for statement mapping (ms) | 4.4 |
Clone type | Type 3 |
ID | Statement | ID | Statement | |
---|---|---|---|---|
| 15 | int offset = 1; | ||
18 | if (useBatch) | 16 | if (useBatch) | |
19 | if (st == null) | 17 | if (st == null) | |
20 | if (callable) | 18 | if (callable) | |
21 | st = session.getBatcher().prepareBatchCallableStatement(sql); | 19 | st = session.getBatcher().prepareBatchCallableStatement(sql); | |
else | else | |||
22 | st = session.getBatcher().prepareBatchStatement(sql); | 20 | st = session.getBatcher().prepareBatchStatement(sql); | |
else | else | |||
23 | if (callable) | 21 | if (callable) | |
24 | st = session.getBatcher().prepareCallableStatement(sql); | 22 | st = session.getBatcher().prepareCallableStatement(sql); | |
else | else | |||
25 | st = session.getBatcher().prepareStatement(sql); | 23 | st = session.getBatcher().prepareStatement(sql); |
Row | Violation |
---|
Number of mapped statements | 1 |
Number of unmapped statements in the first code fragment | 3 |
Number of unmapped statements in the second code fragment | 2 |
Time elapsed for statement mapping (ms) | 0.8 |
Clone type | Type 3 |
ID | Statement | ID | Statement | |||||||||||
---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|
29 | if (hasIdentifier) |
| 18 | if (callable) | ||||||||||
|
| 19 | st = session.getBatcher().prepareBatchCallableStatement(sql); | |||||||||||
| else | |||||||||||||
|
| 20 | st = session.getBatcher().prepareBatchStatement(sql); | |||||||||||
30 | offset = writeIdentifier(st, collection.getIdentifier(entry, i), offset, session); | |
Row | Violation |
---|---|
1 | Unmatched statement st=session.getBatcher().prepareBatchCallableStatement(sql); cannot be moved, because it updates a variable modified in other unmapped statements |
2 | Unmatched statement st=session.getBatcher().prepareBatchStatement(sql); cannot be moved, because it updates a variable modified in other unmapped statements |