File path: /hibernate-distribution-3.3.2.GA/project/core/src/main/java/org/hibernate/collection/PersistentList.java | File path: /hibernate-distribution-3.3.2.GA/project/core/src/main/java/org/hibernate/collection/PersistentBag.java | |||
Method name: Serializable getSnapshot(CollectionPersister)
|
Method name: Serializable getSnapshot(CollectionPersister)
|
|||
Number of AST nodes: 7 | Number of AST nodes: 6 | |||
1 | EntityMode entityMode = getSession().getEntityMode();↵ | 1 | EntityMode entityMode = getSession().getEntityMode();↵ | |
2 | ArrayList clonedList = new ArrayList( list.size() );↵ | 2 | ArrayList clonedList = new ArrayList( bag.size() );↵ | |
3 | Iterator iter = list.iterator();↵ | 3 | Iterator iter = bag.iterator();↵ | |
4 | while ( iter.hasNext() ) {↵ | 4 | while ( iter.hasNext() ) {↵ | |
5 | Object deepCopy = persister.getElementType()↵ | 5 | clonedList.add( persister.getElementType()↵ | |
6 | .deepCopy( iter.next(), entityMode, persister.getFactory() );↵ | 6 | .deepCopy( iter.next(), entityMode, persister.getFactory() )↵ | |
7 | clonedList.add( deepCopy );↵ | 7 | );↵ | |
8 | }↵ | 8 | }↵ | |
9 | return clonedList; | 9 |
| |
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.1 |
Clones location | Clones are in different classes having the same super class |
Number of node comparisons | 16 |
Number of mapped statements | 6 |
Number of unmapped statements in the first code fragment | 1 |
Number of unmapped statements in the second code fragment | 0 |
Time elapsed for statement mapping (ms) | 1.2 |
Clone type | Type 3 |
ID | Statement | ID | Statement | ||||||||||||
---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|
1 | EntityMode entityMode = getSession().getEntityMode(); | 1 | EntityMode entityMode = getSession().getEntityMode(); | ||||||||||||
2 | ArrayList clonedList = new ArrayList(list.size()); |
| 2 | ArrayList clonedList = new ArrayList(bag.size()); | |||||||||||
3 | Iterator iter = list.iterator(); |
| 3 | Iterator iter = bag.iterator(); | |||||||||||
4 | while (iter.hasNext()) | 4 | while (iter.hasNext()) | ||||||||||||
5 | Object deepCopy = persister.getElementType().deepCopy(iter.next(), entityMode, persister.getFactory()); |
| | ||||||||||||
6 | clonedList.add(deepCopy); |
| 5 | clonedList.add(persister.getElementType().deepCopy(iter.next(), entityMode, persister.getFactory())); | |||||||||||
7 | return clonedList; | 6 | return clonedList; |
Row | Violation |
---|---|
1 | Unmatched statement Object deepCopy=persister.getElementType().deepCopy(iter.next(),entityMode,persister.getFactory()); cannot be moved before or after the extracted code, because it has dependencies to/from statements that will be extracted |
2 | Expression persister.getElementType().deepCopy(iter.next(),entityMode,persister.getFactory()) cannot be parameterized, because it has dependencies to/from statements that will be extracted |
3 | Clone fragment #1 returns variables entityMode, iter , while Clone fragment #2 returns variables |