String s1 = s.substring(0, i).trim(); String s2 = s.substring(i + 1, s.length()).trim(); Year year = Month.evaluateAsYear(s1); int month; if (year != null) { month = SerialDate.stringToMonthCode(s2); if (month == -1) { throw new TimePeriodFormatException( "Can't evaluate the month." ); } result = new Month(month, year); } else { year = Month.evaluateAsYear(s2); if (year != null) { month = SerialDate.stringToMonthCode(s1); if (month == -1) { throw new TimePeriodFormatException( "Can't evaluate the month." ); } result = new Month(month, year); } else { throw new TimePeriodFormatException( "Can't evaluate the year." ); } }
if (i != -1) { String s1 = s.substring(0, i).trim(); String s2 = s.substring(i + 1, s.length()).trim(); Year y = Week.evaluateAsYear(s1); int w; if (y != null) { w = Week.stringToWeek(s2); if (w == -1) { throw new TimePeriodFormatException( "Can't evaluate the week."); } result = new Week(w, y); } else { y = Week.evaluateAsYear(s2); if (y != null) { w = Week.stringToWeek(s1); if (w == -1) { throw new TimePeriodFormatException( "Can't evaluate the week."); } result = new Week(w, y); } else { throw new TimePeriodFormatException( "Can't evaluate the year."); } } } else { throw new TimePeriodFormatException( "Could not find separator."); }
Clone fragments detected by clone detection tool
File path: /jfreechart-1.0.10/src/org/jfree/data/time/Month.java File path: /jfreechart-1.0.10/src/org/jfree/data/time/Week.java
Method name: Month parseMonth(String) Method name: Week parseWeek(String)
Number of AST nodes: 16 Number of AST nodes: 18
1
if (i != -1) {
1
String s1 = s.substring(0, i).trim();
2
                String s1 = s.substring(0, i).trim();
2
                String s2 = s.substring(i + 1, s.length()).trim();
3
                String s2 = s.substring(i + 1, s.length()).trim();
3
                Year year = Month.evaluateAsYear(s1);
4
                Year y = Week.evaluateAsYear(s1);
4
                int month;
5
                int w;
5
                if (year != null) {
6
                if (y != null) {
6
                    month = SerialDate.stringToMonthCode(s2);
7
                    w = Week.stringToWeek(s2);
7
                    if (month == -1) {
8
                    if (w == -1) {
8
                        throw new TimePeriodFormatException(
9
                        throw new TimePeriodFormatException(
9
                            "Can't evaluate the month."
10
                                "Can't evaluate the 
10
                        );
11
week.");
11
                    }
12
                    }
12
                    result = new Month(month, year);
13
                    result = new Week(w, y);
13
                }
14
                }
14
                else {
15
                else {
15
                    year = Month.evaluateAsYear(s2);
16
                    y = Week.evaluateAsYear(s2);
16
                    if (year != null) {
17
                    if (y != null) {
17
                        month = SerialDate.stringToMonthCode(s1);
18
                        w = Week.stringToWeek(s1);
18
                        if (month == -1) {
19
                        if (w == -1) {
19
                            throw new TimePeriodFormatException(
20
                            throw new TimePeriodFormatException(
20
                                "Can't evaluate the month."
21
                                    "Can't evaluate the 
21
                            );
22
week.");
22
                        }
23
                        }
23
                        result = new Month(month, year);
24
                        result = new Week(w, y);
24
                    }
25
                    }
25
                    else {
26
                    else {
26
                        throw new TimePeriodFormatException(
27
                        throw new TimePeriodFormatException(
27
                            "Can't evaluate the year."
28
                                "Can't evaluate the year.");
28
                 
29
                    }
29
       );
30
                }
30
            
31
            }
31
        }
32
        
32
    
33
    else {
34
                throw new TimePeriodFormatException(
35
                        "Could not find separator.");
33
            }
36
            }
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.5
Clones locationClones are in different classes having the same super class
Number of node comparisons64
  1. {Non-refactorable}
    Mapping Summary
    Number of mapped statements14
    Number of unmapped statements in the first code fragment2
    Number of unmapped statements in the second code fragment3
    Time elapsed for statement mapping (ms)10.7
    Clone typeType 3
    Mapped Statements
    ID Statement ID Statement
    6
    String s1 = s.substring(0, i).trim();
    6
    String s1 = s.substring(0, i).trim();
    7
    String s2 = s.substring(i + 1, s.length()).trim();
    7
    String s2 = s.substring(i + 1, s.length()).trim();
    8
    Year year = Month.evaluateAsYear(s1);
    8
    Year year = Month.evaluateAsYear(s1);
    8
    Year y = Week.evaluateAsYear(s1);
    Differences
    Expression1Expression2Difference
    yearyVARIABLE_NAME_MISMATCH
    org.jfree.data.time.Monthorg.jfree.data.time.WeekSUBCLASS_TYPE_MISMATCH
    Preondition Violations
    Expression Month cannot be unified with expression Week , because common superclass org.jfree.data.time.RegularTimePeriod does not declare member(s) private static org.jfree.data.time.Year evaluateAsYear(java.lang.String)
    8
    Year y = Week.evaluateAsYear(s1);
    9
    int month;
    9
    int month;
    9
    int w;
    Differences
    Expression1Expression2Difference
    monthwVARIABLE_NAME_MISMATCH
    9
    int w;
    10
    if (year != null)
    10
    if (year != null)
    10
    if (y != null)
    Differences
    Expression1Expression2Difference
    yearyVARIABLE_NAME_MISMATCH
    10
    if (y != null)
                                                            
    11
    w = Week.stringToWeek(s2);
    Preondition Violations
    Unmatched statement w=Week.stringToWeek(s2); cannot be moved before or after the extracted code, because it has dependencies to/from statements that will be extracted
    11
    w = Week.stringToWeek(s2);
    11
    month = SerialDate.stringToMonthCode(s2);
    11
    month = SerialDate.stringToMonthCode(s2);
    Preondition Violations
    Unmatched statement month=SerialDate.stringToMonthCode(s2); cannot be moved before or after the extracted code, because it has dependencies to/from statements that will be extracted
                                                                                          
    12
    if (month == -1)
    12
    if (month == -1)
    12
    if (w == -1)
    Differences
    Expression1Expression2Difference
    monthwVARIABLE_NAME_MISMATCH
    12
    if (w == -1)
    13
    throw new TimePeriodFormatException("Can't evaluate the month.");
    13
    throw new TimePeriodFormatException("Can't evaluate the month.");
    13
    throw new TimePeriodFormatException("Can't evaluate the week.");
    Differences
    Expression1Expression2Difference
    "Can't evaluate the month.""Can't evaluate the week."LITERAL_VALUE_MISMATCH
    13
    throw new TimePeriodFormatException("Can't evaluate the week.");
    14
    result = new Month(month, year);
    14
    result = new Month(month, year);
    14
    result = new Week(w, y);
    Differences
    Expression1Expression2Difference
    org.jfree.data.time.Monthorg.jfree.data.time.WeekSUBCLASS_TYPE_MISMATCH
    org.jfree.data.time.Monthorg.jfree.data.time.WeekSUBCLASS_TYPE_MISMATCH
    monthwVARIABLE_NAME_MISMATCH
    yearyVARIABLE_NAME_MISMATCH
    Preondition Violations
    Expression new Month(month,year) cannot be parameterized, because it has dependencies to/from statements that will be extracted
    Expression new Week(w,y) cannot be parameterized, because it has dependencies to/from statements that will be extracted
    14
    result = new Week(w, y);
    else
    else
    15
    year = Month.evaluateAsYear(s2);
    15
    year = Month.evaluateAsYear(s2);
    15
    y = Week.evaluateAsYear(s2);
    Differences
    Expression1Expression2Difference
    yearyVARIABLE_NAME_MISMATCH
    org.jfree.data.time.Monthorg.jfree.data.time.WeekSUBCLASS_TYPE_MISMATCH
    Preondition Violations
    Expression Month cannot be unified with expression Week , because common superclass org.jfree.data.time.RegularTimePeriod does not declare member(s) private static org.jfree.data.time.Year evaluateAsYear(java.lang.String)
    15
    y = Week.evaluateAsYear(s2);
    16
    if (year != null)
    16
    if (year != null)
    16
    if (y != null)
    Differences
    Expression1Expression2Difference
    yearyVARIABLE_NAME_MISMATCH
    16
    if (y != null)
                                                            
    17
    w = Week.stringToWeek(s1);
    Preondition Violations
    Unmatched statement w=Week.stringToWeek(s1); cannot be moved before or after the extracted code, because it has dependencies to/from statements that will be extracted
    17
    w = Week.stringToWeek(s1);
    17
    month = SerialDate.stringToMonthCode(s1);
    17
    month = SerialDate.stringToMonthCode(s1);
    Preondition Violations
    Unmatched statement month=SerialDate.stringToMonthCode(s1); cannot be moved before or after the extracted code, because it has dependencies to/from statements that will be extracted
                                                                                          
    18
    if (month == -1)
    18
    if (month == -1)
    18
    if (w == -1)
    Differences
    Expression1Expression2Difference
    monthwVARIABLE_NAME_MISMATCH
    18
    if (w == -1)
    19
    throw new TimePeriodFormatException("Can't evaluate the month.");
    19
    throw new TimePeriodFormatException("Can't evaluate the month.");
    19
    throw new TimePeriodFormatException("Can't evaluate the week.");
    Differences
    Expression1Expression2Difference
    "Can't evaluate the month.""Can't evaluate the week."LITERAL_VALUE_MISMATCH
    19
    throw new TimePeriodFormatException("Can't evaluate the week.");
    20
    result = new Month(month, year);
    20
    result = new Month(month, year);
    20
    result = new Week(w, y);
    Differences
    Expression1Expression2Difference
    org.jfree.data.time.Monthorg.jfree.data.time.WeekSUBCLASS_TYPE_MISMATCH
    org.jfree.data.time.Monthorg.jfree.data.time.WeekSUBCLASS_TYPE_MISMATCH
    monthwVARIABLE_NAME_MISMATCH
    yearyVARIABLE_NAME_MISMATCH
    Preondition Violations
    Expression new Month(month,year) cannot be parameterized, because it has dependencies to/from statements that will be extracted
    Expression new Week(w,y) cannot be parameterized, because it has dependencies to/from statements that will be extracted
    20
    result = new Week(w, y);
    else
    else
    21
    throw new TimePeriodFormatException("Can't evaluate the year.");
    21
    throw new TimePeriodFormatException("Can't evaluate the year.");
                                                                                                                                              
    22
    throw new TimePeriodFormatException("Could not find separator.");
    Preondition Violations
    Unmatched throw new TimePeriodFormatException("Could not find separator.");
    22
    throw new TimePeriodFormatException("Could not find separator.");
    Precondition Violations (12)
    Row Violation
    1Expression Month cannot be unified with expression Week , because common superclass org.jfree.data.time.RegularTimePeriod does not declare member(s) private static org.jfree.data.time.Year evaluateAsYear(java.lang.String)
    2Unmatched statement w=Week.stringToWeek(s2); cannot be moved before or after the extracted code, because it has dependencies to/from statements that will be extracted
    3Unmatched statement month=SerialDate.stringToMonthCode(s2); cannot be moved before or after the extracted code, because it has dependencies to/from statements that will be extracted
    4Expression new Month(month,year) cannot be parameterized, because it has dependencies to/from statements that will be extracted
    5Expression new Week(w,y) cannot be parameterized, because it has dependencies to/from statements that will be extracted
    6Expression Month cannot be unified with expression Week , because common superclass org.jfree.data.time.RegularTimePeriod does not declare member(s) private static org.jfree.data.time.Year evaluateAsYear(java.lang.String)
    7Unmatched statement w=Week.stringToWeek(s1); cannot be moved before or after the extracted code, because it has dependencies to/from statements that will be extracted
    8Unmatched statement month=SerialDate.stringToMonthCode(s1); cannot be moved before or after the extracted code, because it has dependencies to/from statements that will be extracted
    9Expression new Month(month,year) cannot be parameterized, because it has dependencies to/from statements that will be extracted
    10Expression new Week(w,y) cannot be parameterized, because it has dependencies to/from statements that will be extracted
    11Unmatched throw new TimePeriodFormatException("Could not find separator.");
    12Clone fragment #1 returns variable result with type org.jfree.data.time.Month , while Clone fragment #2 returns variable result with type org.jfree.data.time.Week