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: 20 | Number of AST nodes: 20 | |||
1 | s = s.trim();↵ | 1 | s = s.trim();↵ | |
2 | int i = Month.findSeparator(s);↵ | 2 | int i = Week.findSeparator(s);↵ | |
3 | if (i != -1) {↵ | 3 | if (i != -1) {↵ | |
4 | String s1 = s.substring(0, i).trim();↵ | 4 | String s1 = s.substring(0, i).trim();↵ | |
5 | String s2 = s.substring(i + 1, s.length()).trim();↵ | 5 | String s2 = s.substring(i + 1, s.length()).trim();↵ | |
6 | Year year = Month.evaluateAsYear(s1);↵ | 6 | Year y = Week.evaluateAsYear(s1);↵ | |
7 | int month;↵ | 7 | int w;↵ | |
8 | if (year != null) {↵ | 8 | if (y != null) {↵ | |
9 | month = SerialDate.stringToMonthCode(s2);↵ | 9 | w = Week.stringToWeek(s2);↵ | |
10 | if (month == -1) {↵ | 10 | if (w == -1) {↵ | |
11 | throw new TimePeriodFormatException(↵ | 11 | throw new TimePeriodFormatException(↵ | |
12 | "Can't evaluate the month."↵ | 12 | "Can't evaluate the ↵ | |
13 | );↵ | 13 | week.");↵ | |
14 | }↵ | 14 | }↵ | |
15 | result = new Month(month, year);↵ | 15 | result = new Week(w, y);↵ | |
16 | }↵ | 16 | }↵ | |
17 | else {↵ | 17 | else {↵ | |
18 | year = Month.evaluateAsYear(s2);↵ | 18 | y = Week.evaluateAsYear(s2);↵ | |
19 | if (year != null) {↵ | 19 | if (y != null) {↵ | |
20 | month = SerialDate.stringToMonthCode(s1);↵ | 20 | w = Week.stringToWeek(s1);↵ | |
21 | if (month == -1) {↵ | 21 | if (w == -1) {↵ | |
22 | throw new TimePeriodFormatException(↵ | 22 | throw new TimePeriodFormatException(↵ | |
23 | "Can't evaluate the month."↵ | 23 | "Can't evaluate the ↵ | |
24 | );↵ | 24 | week.");↵ | |
25 | }↵ | 25 | }↵ | |
26 | result = new Month(month, year);↵ | 26 | result = new Week(w, y);↵ | |
27 | }↵ | 27 | }↵ | |
28 | else {↵ | 28 | else {↵ | |
29 | throw new TimePeriodFormatException(↵ | 29 | throw new TimePeriodFormatException(↵ | |
30 | "Can't evaluate the year."↵ | 30 | "Can't evaluate the year."↵ | |
31 | );↵ | 31 | );↵ | |
32 | }↵ | 32 | }↵ | |
33 | }↵ | 33 | }↵ | |
34 | }↵ | 34 | }↵ | |
35 | else {↵ | 35 | else {↵ | |
36 | throw new TimePeriodFormatException(↵ | 36 | throw new TimePeriodFormatException(↵ | |
37 | "Could not find separator."↵ | 37 | "Could not find separator."↵ | |
38 | );↵ | 38 | );↵ | |
39 | } | 39 |
| |
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) | 1.3 |
Clones location | Clones are in different classes having the same super class |
Number of node comparisons | 84 |
Number of mapped statements | 18 |
Number of unmapped statements in the first code fragment | 2 |
Number of unmapped statements in the second code fragment | 2 |
Time elapsed for statement mapping (ms) | 9.1 |
Clone type | Type 2 |
ID | Statement | ID | Statement | ||||||||||||||||||||||
---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|
3 | s = s.trim(); | 3 | s = s.trim(); | ||||||||||||||||||||||
4 | int i = Month.findSeparator(s); |
| 4 | int i = Week.findSeparator(s); | |||||||||||||||||||||
5 | if (i != -1) | 5 | if (i != -1) | ||||||||||||||||||||||
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 y = Week.evaluateAsYear(s1); | |||||||||||||||||||||
9 | int month; |
| 9 | int w; | |||||||||||||||||||||
10 | if (year != null) |
| 10 | if (y != null) | |||||||||||||||||||||
|
| 11 | w = Week.stringToWeek(s2); | ||||||||||||||||||||||
11 | month = SerialDate.stringToMonthCode(s2); |
| | ||||||||||||||||||||||
12 | if (month == -1) |
| 12 | if (w == -1) | |||||||||||||||||||||
13 | throw new TimePeriodFormatException("Can't evaluate the month."); |
| 13 | throw new TimePeriodFormatException("Can't evaluate the week."); | |||||||||||||||||||||
14 | result = new Month(month, year); |
| 14 | result = new Week(w, y); | |||||||||||||||||||||
else | else | ||||||||||||||||||||||||
15 | year = Month.evaluateAsYear(s2); |
| 15 | y = Week.evaluateAsYear(s2); | |||||||||||||||||||||
16 | if (year != null) |
| 16 | if (y != null) | |||||||||||||||||||||
|
| 17 | w = Week.stringToWeek(s1); | ||||||||||||||||||||||
17 | month = SerialDate.stringToMonthCode(s1); |
| | ||||||||||||||||||||||
18 | if (month == -1) |
| 18 | if (w == -1) | |||||||||||||||||||||
19 | throw new TimePeriodFormatException("Can't evaluate the month."); |
| 19 | throw new TimePeriodFormatException("Can't evaluate the week."); | |||||||||||||||||||||
20 | result = new Month(month, year); |
| 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."); | ||||||||||||||||||||||
else | else | ||||||||||||||||||||||||
22 | throw new TimePeriodFormatException("Could not find separator."); | 22 | throw new TimePeriodFormatException("Could not find separator."); |
Row | Violation |
---|---|
1 | Expression Month cannot be unified with expression Week , because common superclass org.jfree.data.time.RegularTimePeriod does not declare member(s) private static int findSeparator(java.lang.String) |
2 | 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) |
3 | 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 |
4 | 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 |
5 | Expression new Month(month,year) cannot be parameterized, because it has dependencies to/from statements that will be extracted |
6 | Expression new Week(w,y) cannot be parameterized, because it has dependencies to/from statements that will be extracted |
7 | 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 | 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 |
9 | 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 |
10 | Expression new Month(month,year) cannot be parameterized, because it has dependencies to/from statements that will be extracted |
11 | Expression new Week(w,y) cannot be parameterized, because it has dependencies to/from statements that will be extracted |
12 | Clone fragment #1 returns variables s2, month, s1, result , while Clone fragment #2 returns variables s2, w, s1, result |