File path: /jfreechart-1.0.10/src/org/jfree/data/time/Minute.java | File path: /jfreechart-1.0.10/src/org/jfree/data/time/Month.java | |||
Method name: int compareTo(Object)
|
Method name: int compareTo(Object)
|
|||
Number of AST nodes: 10 | Number of AST nodes: 10 | |||
1 | int result;↵ | 1 | int result;↵ | |
2 | // CASE 1 : Comparing to another Minute object↵ | 2 | // CASE 1 : Comparing to another Month object↵ | |
3 | // -------------------------------------------↵ | 3 | // --------------------------------------------↵ | |
4 | if (o1 instanceof Minute) {↵ | 4 | if (o1 instanceof Month) {↵ | |
5 | Minute m = (Minute) o1;↵ | 5 | Month m = (Month) o1;↵ | |
6 | result = getHour().compareTo(m.getHour());↵ | 6 | result = this.year - m.getYearValue();↵ | |
7 | if (result == 0) {↵ | 7 | if (result == 0) {↵ | |
8 | result = this.minute - m.getMinute();↵ | 8 | result = this.month - m.getMonth();↵ | |
9 | }↵ | 9 | }↵ | |
10 | }↵ | 10 | }↵ | |
11 | // CASE 2 : Comparing to another TimePeriod object↵ | 11 | // CASE 2 : Comparing to another TimePeriod object↵ | |
12 | // -----------------------------------------------↵ | 12 | // -----------------------------------------------↵ | |
13 | else if (o1 instanceof RegularTimePeriod) {↵ | 13 | else if (o1 instanceof RegularTimePeriod) {↵ | |
14 | // more difficult case - evaluate later...↵ | 14 | // more difficult case - evaluate later...↵ | |
15 | result = 0;↵ | 15 | result = 0;↵ | |
16 | }↵ | 16 | }↵ | |
17 | // CASE 3 : Comparing to a non-TimePeriod object↵ | 17 | // CASE 3 : Comparing to a non-TimePeriod object↵ | |
18 | // ---------------------------------------------↵ | 18 | // ---------------------------------------------↵ | |
19 | else {↵ | 19 | else {↵ | |
20 | // consider time periods to be ordered after general objects↵ | 20 | // consider time periods to be ordered after general objects↵ | |
21 | result = 1;↵ | 21 | result = 1;↵ | |
22 | }↵ | 22 | }↵ | |
23 | return result; | 23 |
| |
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.3 |
Clones location | Clones are in different classes having the same super class |
Number of node comparisons | 26 |
Number of mapped statements | 9 |
Number of unmapped statements in the first code fragment | 1 |
Number of unmapped statements in the second code fragment | 1 |
Time elapsed for statement mapping (ms) | 3.7 |
Clone type | Type 2 |
ID | Statement | ID | Statement | |||||||||||||||||||
---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|
1 | int result; | 1 | int result; | |||||||||||||||||||
2 | if (o1 instanceof Minute) |
| 2 | if (o1 instanceof Month) | ||||||||||||||||||
3 | Minute m = (Minute)o1; |
| 3 | Month m = (Month)o1; | ||||||||||||||||||
4 | result = getHour().compareTo(m.getHour()); |
| 4 | result = this.year - m.getYearValue(); | ||||||||||||||||||
5 | if (result == 0) | 5 | if (result == 0) | |||||||||||||||||||
|
| 6 | result = this.month - m.getMonth(); | |||||||||||||||||||
6 | result = this.minute - m.getMinute(); |
| | |||||||||||||||||||
7 | else if (o1 instanceof RegularTimePeriod) | 7 | else if (o1 instanceof RegularTimePeriod) | |||||||||||||||||||
8 | result = 0; | 8 | result = 0; | |||||||||||||||||||
else | else | |||||||||||||||||||||
9 | result = 1; | 9 | result = 1; | |||||||||||||||||||
10 | return result; | 10 | return result; |
Row | Violation |
---|---|
1 | Expression (Minute)o1 cannot be parameterized, because it has dependencies to/from statements that will be extracted |
2 | Expression (Month)o1 cannot be parameterized, because it has dependencies to/from statements that will be extracted |
3 | Expression getHour().compareTo(m.getHour()) cannot be parameterized, because it has dependencies to/from statements that will be extracted |
4 | Expression this.year - m.getYearValue() cannot be parameterized, because it has dependencies to/from statements that will be extracted |
5 | Unmatched statement result=this.month - m.getMonth(); cannot be moved before or after the extracted code, because it has dependencies to/from statements that will be extracted |
6 | Unmatched statement result=this.minute - m.getMinute(); cannot be moved before or after the extracted code, because it has dependencies to/from statements that will be extracted |
7 | Clone fragment #1 returns variables m, result , while Clone fragment #2 returns variables m, result |