297-1-5

int result; // CASE 1 : Comparing to another Hour object // ----------------------------------------- if (o1 instanceof Hour) { Hour h = (Hour) o1; result = getDay().compareTo(h.getDay()); if (result == 0) { result = this.hour - h.getHour(); } } // CASE 2 : Comparing to another TimePeriod object // ----------------------------------------------- else if (o1 instanceof RegularTimePeriod) { // more difficult case - evaluate later... result = 0; } // CASE 3 : Comparing to a non-TimePeriod object // --------------------------------------------- else { // consider time periods to be ordered after general objects result = 1; } return result;
int result; // CASE 1 : Comparing to another Week object // -------------------------------------------- if (o1 instanceof Week) { Week w = (Week) o1; result = this.year - w.getYear().getYear(); if (result == 0) { result = this.week - w.getWeek(); } } // CASE 2 : Comparing to another TimePeriod object // ----------------------------------------------- else if (o1 instanceof RegularTimePeriod) { // more difficult case - evaluate later... result = 0; } // CASE 3 : Comparing to a non-TimePeriod object // --------------------------------------------- else { // consider time periods to be ordered after general objects result = 1; } return result;

Clone fragments detected by clone detection tool
File path: /jfreechart-1.0.10/src/org/jfree/data/time/Hour.java |  | File path: /jfreechart-1.0.10/src/org/jfree/data/time/Week.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 Hour object↵

2

            // CASE 1 : Comparing to another Week object↵

3

            // -----------------------------------------↵

3

            // --------------------------------------------↵

4

            if (o1 instanceof Hour) {↵

4

            if (o1 instanceof Week) {↵

5

                Hour h = (Hour) o1;↵

5

                Week w = (Week) o1;↵

6

                result = getDay().compareTo(h.getDay());↵

6

                result = this.year - w.getYear().getYear();↵

7

                if (result == 0) {↵

7

                if (result == 0) {↵

8

                    result = this.hour - h.getHour();↵

8

                    result = this.week - w.getWeek();↵

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

            return result;

See real code fragment |  | See real code fragment

Summary
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.2
Clones location | Clones are in different classes having the same super class
Number of node comparisons | 26

1. {Non-refactorable}

   Mapping Summary
   Number of mapped statements | 10
   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) | 1.4
   Clone type | Type 2

   Mapped Statements
   ID | Statement |  | ID | Statement
   --- | --- | --- | --- | ---

   1

       int result;

   1

       int result;

   2

       if (o1 instanceof Hour)

   2 | if (o1 instanceof Hour)
   2 | if (o1 instanceof Week)

   Differences
   Expression1 | Expression2 | Difference
   --- | --- | ---
   org.jfree.data.time.Hour | org.jfree.data.time.Week | SUBCLASS_TYPE_MISMATCH

   2

       if (o1 instanceof Week)

   3

       Hour h = (Hour)o1;

   3 | Hour h = (Hour)o1;
   3 | Week w = (Week)o1;

   Differences
   Expression1 | Expression2 | Difference
   --- | --- | ---
   org.jfree.data.time.Hour | org.jfree.data.time.Week | SUBCLASS_TYPE_MISMATCH
   h | w | VARIABLE_NAME_MISMATCH
   org.jfree.data.time.Hour | org.jfree.data.time.Week | SUBCLASS_TYPE_MISMATCH
   org.jfree.data.time.Hour | org.jfree.data.time.Week | SUBCLASS_TYPE_MISMATCH

   3

       Week w = (Week)o1;

   4

       result = getDay().compareTo(h.getDay());

   4 | result = getDay().compareTo(h.getDay());
   4 | result = this.year - w.getYear().getYear();

   Differences
   Expression1 | Expression2 | Difference
   --- | --- | ---
   getDay().compareTo(h.getDay()) | this.year - w.getYear().getYear() | TYPE_COMPATIBLE_REPLACEMENT

   Preondition Violations
   Expression getDay().compareTo(h.getDay()) cannot be parameterized, because it has dependencies to/from statements that will be extracted
   Expression this.year - w.getYear().getYear() cannot be parameterized, because it has dependencies to/from statements that will be extracted

   4

       result = this.year - w.getYear().getYear();

   5

       if (result == 0)

   5

       if (result == 0)

   6

       result = this.hour - h.getHour();

   6 | result = this.hour - h.getHour();
   6 | result = this.week - w.getWeek();

   Differences
   Expression1 | Expression2 | Difference
   --- | --- | ---
   hour | week | VARIABLE_NAME_MISMATCH
   getHour | getWeek | METHOD_INVOCATION_NAME_MISMATCH
   h | w | VARIABLE_NAME_MISMATCH
   org.jfree.data.time.Hour | org.jfree.data.time.Week | SUBCLASS_TYPE_MISMATCH

   Preondition Violations
   Expression h.getHour() cannot be parameterized, because it has dependencies to/from statements that will be extracted
   Expression w.getWeek() cannot be parameterized, because it has dependencies to/from statements that will be extracted

   6

       result = this.week - w.getWeek();

   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;

   Precondition Violations (4)
   Row | Violation
   --- | ---
   1 | Expression getDay().compareTo(h.getDay()) cannot be parameterized, because it has dependencies to/from statements that will be extracted
   2 | Expression this.year - w.getYear().getYear() cannot be parameterized, because it has dependencies to/from statements that will be extracted
   3 | Expression h.getHour() cannot be parameterized, because it has dependencies to/from statements that will be extracted
   4 | Expression w.getWeek() cannot be parameterized, because it has dependencies to/from statements that will be extracted