if (theClass==Object.class || theClass==null) return null; Method method = getterMethod(theClass, propertyName); if (method!=null) { if ( !ReflectHelper.isPublic(theClass, method) ) method.setAccessible(true); return new BasicGetter(theClass, method, propertyName); } else { BasicGetter getter = getGetterOrNull( theClass.getSuperclass(), propertyName ); if (getter==null) { Class[] interfaces = theClass.getInterfaces(); for ( int i=0; getter==null && i<interfaces.length; i++ ) { getter=getGetterOrNull( interfaces[i], propertyName ); } } return getter; }
if (theClass==Object.class || theClass==null) return null; Method method = setterMethod(theClass, propertyName); if (method!=null) { if ( !ReflectHelper.isPublic(theClass, method) ) method.setAccessible(true); return new BasicSetter(theClass, method, propertyName); } else { BasicSetter setter = getSetterOrNull( theClass.getSuperclass(), propertyName ); if (setter==null) { Class[] interfaces = theClass.getInterfaces(); for ( int i=0; setter==null && i<interfaces.length; i++ ) { setter=getSetterOrNull( interfaces[i], propertyName ); } } return setter; }
Clone fragments detected by clone detection tool
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
		}
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.8
Clones locationClones are declared in the same class
Number of node comparisons36
  1. {Non-refactorable}
    Mapping Summary
    Number of mapped statements9
    Number of unmapped statements in the first code fragment4
    Number of unmapped statements in the second code fragment4
    Time elapsed for statement mapping (ms)1.9
    Clone typeType 2
    Mapped Statements
    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 = getterMethod(theClass, propertyName);
    3
    Method method = setterMethod(theClass, propertyName);
    Differences
    Expression1Expression2Difference
    getterMethodsetterMethodMETHOD_INVOCATION_NAME_MISMATCH
    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);
    Preondition Violations
    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
    Unmatched return new BasicSetter(theClass,method,propertyName);
    7
    return new BasicSetter(theClass, method, propertyName);
    7
    return new BasicGetter(theClass, method, propertyName);
    7
    return new BasicGetter(theClass, method, propertyName);
    Preondition Violations
    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
    Unmatched 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 (getter == null)
    9
    if (setter == null)
    Differences
    Expression1Expression2Difference
    gettersetterVARIABLE_NAME_MISMATCH
    org.hibernate.property.BasicPropertyAccessor.BasicGetterorg.hibernate.property.BasicPropertyAccessor.BasicSetterVARIABLE_TYPE_MISMATCH
    Preondition Violations
    Type org.hibernate.property.BasicPropertyAccessor.BasicGetter of variable getter does not match with type org.hibernate.property.BasicPropertyAccessor.BasicSetter of variable setter
    • Make classes org.hibernate.property.BasicPropertyAccessor.BasicGetter and org.hibernate.property.BasicPropertyAccessor.BasicSetter extend a common superclass
    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; getter == null && i < interfaces.length; i++)
    11
    for (int i = 0; setter == null && i < interfaces.length; i++)
    Differences
    Expression1Expression2Difference
    gettersetterVARIABLE_NAME_MISMATCH
    org.hibernate.property.BasicPropertyAccessor.BasicGetterorg.hibernate.property.BasicPropertyAccessor.BasicSetterVARIABLE_TYPE_MISMATCH
    Preondition Violations
    Type org.hibernate.property.BasicPropertyAccessor.BasicGetter of variable getter does not match with type org.hibernate.property.BasicPropertyAccessor.BasicSetter of variable setter
    • Make classes org.hibernate.property.BasicPropertyAccessor.BasicGetter and org.hibernate.property.BasicPropertyAccessor.BasicSetter extend a common superclass
    11
    for (int i = 0; setter == null && i < interfaces.length; i++)
                                                                                                                  
    12
    setter = getSetterOrNull(interfaces[i], propertyName);
    Preondition Violations
    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
    12
    setter = getSetterOrNull(interfaces[i], propertyName);
    12
    getter = getGetterOrNull(interfaces[i], propertyName);
    12
    getter = getGetterOrNull(interfaces[i], propertyName);
    Preondition Violations
    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
                                                                                                                  
                                        
    13
    return setter;
    Preondition Violations
    Unmatched return setter;
    13
    return setter;
    13
    return getter;
    13
    return getter;
    Preondition Violations
    Unmatched return getter;
                                        
    Precondition Violations (11)
    Row Violation
    1Unmatched 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
    2Unmatched return new BasicSetter(theClass,method,propertyName);
    3Unmatched 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
    4Unmatched return new BasicGetter(theClass,method,propertyName);
    5Type org.hibernate.property.BasicPropertyAccessor.BasicGetter of variable getter does not match with type org.hibernate.property.BasicPropertyAccessor.BasicSetter of variable setter
    6Type org.hibernate.property.BasicPropertyAccessor.BasicGetter of variable getter does not match with type org.hibernate.property.BasicPropertyAccessor.BasicSetter of variable setter
    7Unmatched 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
    8Unmatched 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
    9Unmatched return setter;
    10Unmatched return getter;
    11Clone fragment #1 returns variables method, interfaces, i , while Clone fragment #2 returns variables method, interfaces, i