File path: /emf-2.4.1/src/org/eclipse/emf/ecore/xml/type/internal/DataValue.java | File path: /emf-2.4.1/src/org/eclipse/emf/ecore/xml/type/internal/DataValue.java | |||
Method name: void initializePath(String, int)
|
Method name: void initializePath(String, int)
|
|||
Number of AST nodes: 10 | Number of AST nodes: 11 | |||
1 | while (index < end) {↵ | |||
1 | testChar = p_uriSpec.charAt(index);↵ | 2 | testChar = p_uriSpec.charAt(index);↵ | |
2 | ↵ | 3 | ↵ | |
3 | if (testChar == '?' || testChar == '#') {↵ | 4 | if (testChar == '#') {↵ | |
4 | break;↵ | 5 | break;↵ | |
5 | }↵ | 6 | }↵ | |
6 | ↵ | |||
7 | // check for valid escape sequence↵ | |||
8 | if (testChar == '%') {↵ | 7 | if (testChar == '%') {↵ | |
9 | if (index+2 >= end ||↵ | 8 | if (index+2 >= end ||↵ | |
10 | !isHex(p_uriSpec.charAt(index+1)) ||↵ | 9 | !isHex(p_uriSpec.charAt(index+1)) ||↵ | |
11 | !isHex(p_uriSpec.charAt(index+2))) {↵ | 10 | !isHex(p_uriSpec.charAt(index+2))) {↵ | |
12 | throw new MalformedURIException(↵ | 11 | throw new MalformedURIException(↵ | |
13 | "Opaque part contains invalid escape sequence!");↵ | 12 | "Query string contains invalid escape sequence!");↵ | |
14 | }↵ | 13 | }↵ | |
15 | index += 2;↵ | 14 | index += 2;↵ | |
16 | }↵ | 15 | ↵ | |
17 | // If the scheme specific part is opaque, it can contain '['↵ | |||
18 | // and ']'. uric_no_slash wasn't modified by RFC 2732, which↵ | |||
19 | // I've interpreted as an error in the spec, since the ↵ | |||
20 | // production should be equivalent to (uric - '/'), and uric↵ | |||
21 | // contains '[' and ']'. - mrglavas↵ | |||
22 | ↵ | 16 | }↵ | |
23 | else if (!isURICharacter(testChar)) {↵ | 17 | else if (!isURICharacter(testChar)) {↵ | |
24 | throw new MalformedURIException(↵ | 18 | throw new MalformedURIException(↵ | |
25 | "Opaque part contains invalid character: " + testChar);↵ | 19 | "Query string contains invalid character: " + testChar);↵ | |
26 | }↵ | 20 | ↵ | |
27 | ↵ | 21 | }↵ | |
28 | ++index; | 22 | index++;↵ | |
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) | 1.2 |
Clones location | Clones are in the same method |
Number of node comparisons | 19 |
Number of mapped statements | 7 |
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) | 2.6 |
Clone type | Type 3 |
ID | Statement | ID | Statement | |||||||||||
---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|
21 | testChar = p_uriSpec.charAt(index); | 36 | testChar = p_uriSpec.charAt(index); | |||||||||||
22 | if (testChar == '?' || testChar == '#') |
| | |||||||||||
23 | break; |
| | |||||||||||
|
| 37 | if (testChar == '#') | |||||||||||
|
| 38 | break; | |||||||||||
24 | if (testChar == '%') | 39 | if (testChar == '%') | |||||||||||
25 | if (index + 2 >= end || !isHex(p_uriSpec.charAt(index + 1)) || !isHex(p_uriSpec.charAt(index + 2))) | 40 | if (index + 2 >= end || !isHex(p_uriSpec.charAt(index + 1)) || !isHex(p_uriSpec.charAt(index + 2))) | |||||||||||
26 | throw new MalformedURIException("Opaque part contains invalid escape sequence!"); |
| 41 | throw new MalformedURIException("Query string contains invalid escape sequence!"); | ||||||||||
27 | index += 2; | 42 | index += 2; | |||||||||||
28 | else if (!isURICharacter(testChar)) | 43 | else if (!isURICharacter(testChar)) | |||||||||||
29 | throw new MalformedURIException("Opaque part contains invalid character: " + testChar); |
| 44 | throw new MalformedURIException("Query string contains invalid character: " + testChar); |
Row | Violation |
---|---|
1 | Unmatched statement if(testChar == '?' || testChar == '#') cannot be moved before or after the extracted code, because it has dependencies to/from statements that will be extracted |
2 | Unmatched break; |
3 | Unmatched statement if(testChar == '#') cannot be moved before or after the extracted code, because it has dependencies to/from statements that will be extracted |
4 | Unmatched break; |
5 | Clone fragment #1 returns variables testChar, index , while Clone fragment #2 returns variables testChar, index |