Hour result; if (this.hour != FIRST_HOUR_IN_DAY) { result = new Hour(this.hour - 1, this.day); } else { // we are at the first hour in the day... Day prevDay = (Day) this.day.previous(); if (prevDay != null) { result = new Hour(LAST_HOUR_IN_DAY, prevDay); } else { result = null; } } return result;
Second result = null; if (this.second != FIRST_SECOND_IN_MINUTE) { result = new Second(this.second - 1, getMinute()); } else { Minute previous = (Minute) getMinute().previous(); if (previous != null) { result = new Second(LAST_SECOND_IN_MINUTE, previous); } } 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/Second.java
Method name: RegularTimePeriod previous() Method name: RegularTimePeriod previous()
Number of AST nodes: 8 Number of AST nodes: 7
1
Hour result;
1
Second result = null;
2
        if (this.hour != FIRST_HOUR_IN_DAY) {
2
        if (this.second != FIRST_SECOND_IN_MINUTE) {
3
            result = new Hour(this.hour - 1, this.day);
3
            result = new 
4
        }
5
        else { // we are at the first hour in the day...
4
Second(this.second - 1, getMinute());
5
        }
6
        else {
6
            Day prevDay = (Day) this.day.previous();
7
            Minute previous = (Minute) getMinute().previous();
7
            if (prevDay != null) {
8
            if (previous != null) {
8
                result = new Hour(LAST_HOUR_IN_DAY, prevDay);
9
                result = new 
9
            }
10
            else {
11
                result = null;
10
Second(LAST_SECOND_IN_MINUTE, previous);
12
            }
11
            }
13
        }
12
        }
14
        return result;
13
        return result;
Summary
Number of common nesting structure subtrees1
Number of refactorable cases0
Number of non-refactorable cases1
Time elapsed for finding largest common nesting structure subtrees (ms)0.2
Clones locationClones are in different classes having the same super class
Number of node comparisons14
  1. {Non-refactorable}
    Mapping Summary
    Number of mapped statements4
    Number of unmapped statements in the first code fragment4
    Number of unmapped statements in the second code fragment3
    Time elapsed for statement mapping (ms)10.4
    Clone typeType 3
    Mapped Statements
    ID Statement ID Statement
                                                
    1
    Second result = null;
    1
    Hour result;
                                  
    2
    if (this.hour != FIRST_HOUR_IN_DAY)
    2
    if (this.hour != FIRST_HOUR_IN_DAY)
    2
    if (this.second != FIRST_SECOND_IN_MINUTE)
    Differences
    Expression1Expression2Difference
    hoursecondVARIABLE_NAME_MISMATCH
    FIRST_HOUR_IN_DAYFIRST_SECOND_IN_MINUTEVARIABLE_NAME_MISMATCH
    2
    if (this.second != FIRST_SECOND_IN_MINUTE)
    3
    result = new Hour(this.hour - 1, this.day);
    3
    result = new Hour(this.hour - 1, this.day);
    3
    result = new Second(this.second - 1, getMinute());
    Differences
    Expression1Expression2Difference
    org.jfree.data.time.Hourorg.jfree.data.time.SecondSUBCLASS_TYPE_MISMATCH
    org.jfree.data.time.Hourorg.jfree.data.time.SecondSUBCLASS_TYPE_MISMATCH
    hoursecondVARIABLE_NAME_MISMATCH
    org.jfree.data.time.Dayorg.jfree.data.time.MinuteSUBCLASS_TYPE_MISMATCH
    this.daygetMinute()TYPE_COMPATIBLE_REPLACEMENT
    Preondition Violations
    Expression new Hour(this.hour - 1,this.day) cannot be parameterized, because it has dependencies to/from statements that will be extracted
    Expression new Second(this.second - 1,getMinute()) cannot be parameterized, because it has dependencies to/from statements that will be extracted
    Expression getMinute() is a method call throwing exception(s) that should be caught by a try block that will be extracted
    3
    result = new Second(this.second - 1, getMinute());
    else
    else
    4
    Day prevDay = (Day)this.day.previous();
    4
    Day prevDay = (Day)this.day.previous();
    4
    Minute previous = (Minute)getMinute().previous();
    Differences
    Expression1Expression2Difference
    org.jfree.data.time.Dayorg.jfree.data.time.MinuteSUBCLASS_TYPE_MISMATCH
    prevDaypreviousVARIABLE_NAME_MISMATCH
    org.jfree.data.time.Dayorg.jfree.data.time.MinuteSUBCLASS_TYPE_MISMATCH
    org.jfree.data.time.Dayorg.jfree.data.time.MinuteSUBCLASS_TYPE_MISMATCH
    org.jfree.data.time.Dayorg.jfree.data.time.MinuteSUBCLASS_TYPE_MISMATCH
    this.daygetMinute()TYPE_COMPATIBLE_REPLACEMENT
    Preondition Violations
    Expression (Minute)getMinute().previous() is a method call throwing exception(s) that should be caught by a try block that will be extracted
    Expression getMinute() is a method call throwing exception(s) that should be caught by a try block that will be extracted
    4
    Minute previous = (Minute)getMinute().previous();
                                                  
    5
    if (previous != null)
    Preondition Violations
    Unmatched statement if(previous != null) cannot be moved before or after the extracted code, because it has dependencies to/from statements that will be extracted
    5
    if (previous != null)
                                                                                                              
    6
    result = new Second(LAST_SECOND_IN_MINUTE, previous);
    Preondition Violations
    Unmatched statement result=new Second(LAST_SECOND_IN_MINUTE,previous); cannot be moved before or after the extracted code, because it has dependencies to/from statements that will be extracted
    6
    result = new Second(LAST_SECOND_IN_MINUTE, previous);
    5
    if (prevDay != null)
    5
    if (prevDay != null)
    Preondition Violations
    Unmatched statement if(prevDay != null) cannot be moved before or after the extracted code, because it has dependencies to/from statements that will be extracted
                                                
    6
    result = new Hour(LAST_HOUR_IN_DAY, prevDay);
    6
    result = new Hour(LAST_HOUR_IN_DAY, prevDay);
    Preondition Violations
    Unmatched statement result=new Hour(LAST_HOUR_IN_DAY,prevDay); cannot be moved before or after the extracted code, because it has dependencies to/from statements that will be extracted
                                                                                              
    else
            
    7
    result = null;
                                  
    8
    return result;
    8
    return result;
    7
    return result;
    Differences
    Expression1Expression2Difference
    org.jfree.data.time.Hourorg.jfree.data.time.SecondSUBCLASS_TYPE_MISMATCH
    7
    return result;
    Precondition Violations (10)
    Row Violation
    1Expression new Hour(this.hour - 1,this.day) cannot be parameterized, because it has dependencies to/from statements that will be extracted
    2Expression new Second(this.second - 1,getMinute()) cannot be parameterized, because it has dependencies to/from statements that will be extracted
    3Expression getMinute() is a method call throwing exception(s) that should be caught by a try block that will be extracted
    4Expression (Minute)getMinute().previous() is a method call throwing exception(s) that should be caught by a try block that will be extracted
    5Expression getMinute() is a method call throwing exception(s) that should be caught by a try block that will be extracted
    6Unmatched statement if(previous != null) cannot be moved before or after the extracted code, because it has dependencies to/from statements that will be extracted
    7Unmatched statement result=new Second(LAST_SECOND_IN_MINUTE,previous); cannot be moved before or after the extracted code, because it has dependencies to/from statements that will be extracted
    8Unmatched statement if(prevDay != null) cannot be moved before or after the extracted code, because it has dependencies to/from statements that will be extracted
    9Unmatched statement result=new Hour(LAST_HOUR_IN_DAY,prevDay); cannot be moved before or after the extracted code, because it has dependencies to/from statements that will be extracted
    10Clone fragment #1 returns variable prevDay with type org.jfree.data.time.Day , while Clone fragment #2 returns variable previous with type org.jfree.data.time.Minute