File path: /hibernate-distribution-3.3.2.GA/project/core/src/main/java/org/hibernate/property/BasicPropertyAccessor.java | File path: /hibernate-distribution-3.3.2.GA/project/core/src/main/java/org/hibernate/property/BasicPropertyAccessor.java | |||
Method name: BasicGetter getGetterOrNull(Class, String)
|
Method name: BasicSetter getSetterOrNull(Class, String)
|
|||
Number of AST nodes: 13 | Number of AST nodes: 13 | |||
1 | if (theClass==Object.class || theClass==null) return null;↵ | 1 | if (theClass==Object.class || theClass==null) return null;↵ | |
2 | Method method = getterMethod(theClass, propertyName);↵ | 2 | Method method = setterMethod(theClass, propertyName);↵ | |
3 | if (method!=null) {↵ | 3 | if (method!=null) {↵ | |
4 | if ( !ReflectHelper.isPublic(theClass, method) ) method.setAccessible(true);↵ | 4 | if ( !ReflectHelper.isPublic(theClass, method) ) method.setAccessible(true);↵ | |
5 | return new BasicGetter(theClass, method, propertyName);↵ | 5 | return new BasicSetter(theClass, method, propertyName);↵ | |
6 | }↵ | 6 | }↵ | |
7 | else {↵ | 7 | else {↵ | |
8 | BasicGetter getter = getGetterOrNull( theClass.getSuperclass(), propertyName );↵ | 8 | BasicSetter setter = getSetterOrNull( theClass.getSuperclass(), propertyName );↵ | |
9 | if (getter==null) {↵ | 9 | if (setter==null) {↵ | |
10 | Class[] interfaces = theClass.getInterfaces();↵ | 10 | Class[] interfaces = theClass.getInterfaces();↵ | |
11 | for ( int i=0; getter==null && i<interfaces.length; i++ ) {↵ | 11 | for ( int i=0; setter==null && i<interfaces.length; i++ ) {↵ | |
12 | getter=getGetterOrNull( interfaces[i], propertyName );↵ | 12 | setter=getSetterOrNull( interfaces[i], propertyName );↵ | |
13 | }↵ | 13 | }↵ | |
14 | }↵ | 14 | }↵ | |
15 | return getter;↵ | 15 | return setter;↵ | |
16 | } | 16 |
| |
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) | 0.8 |
Clones location | Clones are declared in the same class |
Number of node comparisons | 36 |
Number of mapped statements | 9 |
Number of unmapped statements in the first code fragment | 4 |
Number of unmapped statements in the second code fragment | 4 |
Time elapsed for statement mapping (ms) | 1.9 |
Clone type | Type 2 |
ID | Statement | ID | Statement | |||||||||||||||
---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|
1 | if (theClass == Object.class || theClass == null) | 1 | if (theClass == Object.class || theClass == null) | |||||||||||||||
2 | return null; | 2 | return null; | |||||||||||||||
3 | Method method = getterMethod(theClass, propertyName); |
| 3 | Method method = setterMethod(theClass, propertyName); | ||||||||||||||
4 | if (method != null) | 4 | if (method != null) | |||||||||||||||
5 | if (!ReflectHelper.isPublic(theClass, method)) | 5 | if (!ReflectHelper.isPublic(theClass, method)) | |||||||||||||||
6 | method.setAccessible(true); | 6 | method.setAccessible(true); | |||||||||||||||
|
| 7 | return new BasicSetter(theClass, method, propertyName); | |||||||||||||||
7 | return new BasicGetter(theClass, method, propertyName); |
| | |||||||||||||||
else | else | |||||||||||||||||
| 8 | BasicSetter setter = getSetterOrNull(theClass.getSuperclass(), propertyName); | ||||||||||||||||
8 | BasicGetter getter = getGetterOrNull(theClass.getSuperclass(), propertyName); | | ||||||||||||||||
9 | if (getter == null) |
| 9 | if (setter == null) | ||||||||||||||
10 | Class[] interfaces = theClass.getInterfaces(); | 10 | Class[] interfaces = theClass.getInterfaces(); | |||||||||||||||
11 | for (int i = 0; getter == null && i < interfaces.length; i++) |
| 11 | for (int i = 0; setter == null && i < interfaces.length; i++) | ||||||||||||||
|
| 12 | setter = getSetterOrNull(interfaces[i], propertyName); | |||||||||||||||
12 | getter = getGetterOrNull(interfaces[i], propertyName); |
| | |||||||||||||||
|
| 13 | return setter; | |||||||||||||||
13 | return getter; |
| |
Row | Violation |
---|---|
1 | Unmatched statement return new BasicSetter(theClass,method,propertyName); cannot be moved before the extracted code, because it has control dependencies from statements that will be extracted |
2 | Unmatched return new BasicSetter(theClass,method,propertyName); |
3 | Unmatched statement return new BasicGetter(theClass,method,propertyName); cannot be moved before the extracted code, because it has control dependencies from statements that will be extracted |
4 | Unmatched return new BasicGetter(theClass,method,propertyName); |
5 | Type org.hibernate.property.BasicPropertyAccessor.BasicGetter of variable getter does not match with type org.hibernate.property.BasicPropertyAccessor.BasicSetter of variable setter |
6 | Type org.hibernate.property.BasicPropertyAccessor.BasicGetter of variable getter does not match with type org.hibernate.property.BasicPropertyAccessor.BasicSetter of variable setter |
7 | Unmatched statement setter=getSetterOrNull(interfaces[i],propertyName); cannot be moved before or after the extracted code, because it has dependencies to/from statements that will be extracted |
8 | Unmatched statement getter=getGetterOrNull(interfaces[i],propertyName); cannot be moved before or after the extracted code, because it has dependencies to/from statements that will be extracted |
9 | Unmatched return setter; |
10 | Unmatched return getter; |
11 | Clone fragment #1 returns variables method, interfaces, i , while Clone fragment #2 returns variables method, interfaces, i |