File path: /hibernate-distribution-3.3.2.GA/project/core/src/main/java/org/hibernate/collection/PersistentBag.java | File path: /hibernate-distribution-3.3.2.GA/project/core/src/main/java/org/hibernate/collection/PersistentList.java | |||
Method name: boolean addAll(Collection)
|
Method name: boolean addAll(Collection)
|
|||
Number of AST nodes: 9 | Number of AST nodes: 9 | |||
1 | if ( values.size()==0 ) ↵ | 1 | if ( values.size()==0 ) {↵ | |
2 | return false;↵ | 2 | return false;↵ | |
3 | ↵ | 3 | }↵ | |
4 | if ( !isOperationQueueEnabled() ) {↵ | 4 | if ( !isOperationQueueEnabled() ) {↵ | |
5 | write();↵ | 5 | write();↵ | |
6 | return bag.addAll(values);↵ | 6 | return list.addAll(values);↵ | |
7 | }↵ | 7 | }↵ | |
8 | else {↵ | 8 | else {↵ | |
9 | Iterator iter = values.iterator();↵ | 9 | Iterator iter = values.iterator();↵ | |
10 | while ( iter.hasNext() ) {↵ | 10 | while ( iter.hasNext() ) {↵ | |
11 | queueOperation( new SimpleAdd( iter.next() ) );↵ | 11 | queueOperation( new SimpleAdd( iter.next() ) );↵ | |
12 | }↵ | 12 | }↵ | |
13 | return values.size()>0;↵ | 13 | return values.size()>0;↵ | |
14 | } | 14 |
| |
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.4 |
Clones location | Clones are in different classes having the same super class |
Number of node comparisons | 34 |
Number of mapped statements | 9 |
Number of unmapped statements in the first code fragment | 0 |
Number of unmapped statements in the second code fragment | 0 |
Time elapsed for statement mapping (ms) | 20.6 |
Clone type | Type 2 |
ID | Statement | ID | Statement | ||||||||||||||
---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|
1 | if (values.size() == 0) | 1 | if (values.size() == 0) | ||||||||||||||
2 | return false; | 2 | return false; | ||||||||||||||
3 | if (!isOperationQueueEnabled()) | 3 | if (!isOperationQueueEnabled()) | ||||||||||||||
4 | write(); | 4 | write(); | ||||||||||||||
5 | return bag.addAll(values); |
| 5 | return list.addAll(values); | |||||||||||||
else | else | ||||||||||||||||
6 | Iterator iter = values.iterator(); | 6 | Iterator iter = values.iterator(); | ||||||||||||||
7 | while (iter.hasNext()) | 7 | while (iter.hasNext()) | ||||||||||||||
8 | queueOperation(new SimpleAdd(iter.next())); |
| 8 | queueOperation(new SimpleAdd(iter.next())); | |||||||||||||
9 | return values.size() > 0; | 9 | return values.size() > 0; |
Row | Violation |
---|---|
1 | Expression new SimpleAdd(iter.next()) cannot be parameterized, because it has dependencies to/from statements that will be extracted |
2 | Expression new SimpleAdd(iter.next()) cannot be parameterized, because it has dependencies to/from statements that will be extracted |
3 | Expression new SimpleAdd(iter.next()) cannot be unified with expression new SimpleAdd(iter.next()) , because common superclass type org.hibernate.collection.AbstractPersistentCollection.DelayedOperation cannot be passed as an argument to protected final void queueOperation(java.lang.Object) |