X-Git-Url: http://www2.svjatoslav.eu/gitweb/?a=blobdiff_plain;f=src%2Fmain%2Fjava%2Feu%2Fsvjatoslav%2Finspector%2Fjava%2Fstructure%2FMethodDescriptor.java;h=c31c9f62573b10a6ce50f637041c9f0da6b78f0f;hb=934aea34806546c732112258e5e98a482cf9d50e;hp=35e49d5071ababb2e4b57701e2e030efa23aed7e;hpb=d717e90f4c46e26f9f54ba5638aade6688527bf4;p=javainspect.git

diff --git a/src/main/java/eu/svjatoslav/inspector/java/structure/MethodDescriptor.java b/src/main/java/eu/svjatoslav/inspector/java/structure/MethodDescriptor.java
old mode 100644
new mode 100755
index 35e49d5..c31c9f6
--- a/src/main/java/eu/svjatoslav/inspector/java/structure/MethodDescriptor.java
+++ b/src/main/java/eu/svjatoslav/inspector/java/structure/MethodDescriptor.java
@@ -1,10 +1,10 @@
 /*
  * JavaInspect - Utility to visualize java software
- * Copyright (C) 2013, Svjatoslav Agejenko, svjatoslav@svjatoslav.eu
- * 
+ * Copyright (C) 2013-2015, Svjatoslav Agejenko, svjatoslav@svjatoslav.eu
+ *
  * This program is free software; you can redistribute it and/or
- * modify it under the terms of version 2 of the GNU General Public License
- * as published by the Free Software Foundation.
+ * modify it under the terms of version 3 of the GNU Lesser General Public License
+ * or later as published by the Free Software Foundation.
  */
 
 package eu.svjatoslav.inspector.java.structure;
@@ -15,29 +15,35 @@ import java.lang.reflect.Type;
 import java.util.ArrayList;
 import java.util.List;
 
-public class MethodDescriptor implements GraphElement {
+public class MethodDescriptor implements GraphElement,
+		Comparable<MethodDescriptor> {
 
-	public String name;
+	/**
+	 * This class corresponds to single method within a java class.
+	 */
+
+	public String methodName;
 	public ClassDescriptor returnType;
-	private final ClassDescriptor parent;
+	private final ClassDescriptor parentClass;
 
-	List<ClassDescriptor> typeArguments = new ArrayList<ClassDescriptor>();
+	List<ClassDescriptor> argumentTypes = new ArrayList<ClassDescriptor>();
 
-	public MethodDescriptor(final Method method, final ClassDescriptor parent,
-			final ClassGraph dump) {
+	boolean isInherited;
 
-		this.parent = parent;
+	public MethodDescriptor(final ClassDescriptor parent,
+			final String methodName) {
+		parentClass = parent;
+		this.methodName = methodName;
+	}
 
-		name = method.getName();
+	public void analyze(final Method method) {
 
 		if (!method.getDeclaringClass().getName()
-				.equals(parent.fullyQualifiedName))
-			// do not index inherited methods
-			return;
-
-		parent.methods.add(this);
+				.equals(parentClass.getFullyQualifiedName()))
+			isInherited = true;
 
-		returnType = dump.addClass(method.getReturnType());
+		returnType = parentClass.getClassGraph().getOrCreateClassDescriptor(
+				method.getReturnType());
 		returnType.registerReference();
 
 		final Type genericType = method.getGenericReturnType();
@@ -46,13 +52,23 @@ public class MethodDescriptor implements GraphElement {
 			for (final Type t : pt.getActualTypeArguments())
 				if (t instanceof Class) {
 					final Class cl = (Class) t;
-					final ClassDescriptor classDescriptor = dump.addClass(cl);
+					final ClassDescriptor classDescriptor = parentClass
+							.getClassGraph().getOrCreateClassDescriptor(cl);
 					classDescriptor.registerReference();
-					typeArguments.add(classDescriptor);
+					argumentTypes.add(classDescriptor);
 				}
 
 		}
+	}
 
+	@Override
+	public int compareTo(final MethodDescriptor o) {
+
+		final int nameComparisonResult = methodName.compareTo(o.methodName);
+		if (nameComparisonResult != 0)
+			return nameComparisonResult;
+
+		return toString().compareTo(o.toString());
 	}
 
 	@Override
@@ -64,12 +80,13 @@ public class MethodDescriptor implements GraphElement {
 		final StringBuffer result = new StringBuffer();
 
 		// describe associated types
-		for (final ClassDescriptor classDescriptor : typeArguments)
+		for (final ClassDescriptor classDescriptor : argumentTypes)
 			if (classDescriptor.isVisible())
 				if (classDescriptor.areReferencesShown())
 					result.append("    " + getGraphId() + " -> "
-							+ classDescriptor.getGraphId() + "[label=\"" + name
-							+ "\", color=\"" + classDescriptor.getColor()
+							+ classDescriptor.getGraphId() + "[label=\""
+							+ methodName + "\", color=\""
+							+ classDescriptor.getColor()
 							+ "\", style=\"dotted, bold\"];\n");
 
 		if (!returnType.isVisible())
@@ -78,8 +95,8 @@ public class MethodDescriptor implements GraphElement {
 		// main type
 		if (returnType.areReferencesShown())
 			result.append("    " + getGraphId() + " -> "
-					+ returnType.getGraphId() + "[label=\"" + name + "\","
-					+ " color=\"" + returnType.getColor()
+					+ returnType.getGraphId() + "[label=\"" + methodName
+					+ "\"," + " color=\"" + returnType.getColor()
 					+ "\", style=\"dotted, bold\"];\n");
 
 		return result.toString();
@@ -92,7 +109,7 @@ public class MethodDescriptor implements GraphElement {
 
 		final StringBuffer result = new StringBuffer();
 
-		result.append("        // " + name + "\n");
+		result.append("        // " + methodName + "\n");
 
 		result.append("        <TR><td ALIGN=\"right\">"
 				+ "<FONT POINT-SIZE=\"8.0\">" + returnType.getClassName(true)
@@ -105,36 +122,57 @@ public class MethodDescriptor implements GraphElement {
 
 	@Override
 	public String getGraphId() {
-		return parent.getGraphId() + ":" + name;
+		return parentClass.getGraphId() + ":" + methodName;
 	}
 
 	public String getMethodLabel() {
-		return name;
+		return methodName;
+	}
+
+	public int getOutsideVisibleReferencesCount() {
+		int result = 0;
+
+		if (returnType.isVisible())
+			result++;
+
+		for (final ClassDescriptor classDescriptor : argumentTypes)
+			if (classDescriptor.isVisible())
+				result++;
+
+		return result;
 	}
 
 	@Override
 	public boolean isVisible() {
 
-		if (Utils.isSystemMethod(name))
+		// hide inherited methods
+		if (isInherited)
 			return false;
 
-		if (parent.isEnum && Utils.isEnumMethod(name))
+		// hide common object methods
+		if (Utils.isCommonObjectMethod(methodName))
 			return false;
 
-		if (!(name.startsWith("get") || name.startsWith("set")))
-			return true;
-
-		final String upprCaseName = name.substring(3).toUpperCase();
-
-		for (String parentField : parent.nameToFieldMap.keySet()) {
-			parentField = parentField.toUpperCase();
+		// hide common Enumeration methods
+		if (parentClass.isEnum && Utils.isEnumMethod(methodName))
+			return false;
 
-			if (upprCaseName.equals(parentField))
+		// hide get/set methods for the field of the same name
+		if (methodName.startsWith("get") || methodName.startsWith("set"))
+			if (parentClass.hasFieldIgnoreCase(methodName.substring(3)))
 				return false;
 
+		// hide is methods for the boolean field of the same name
+		if (methodName.startsWith("is")) {
+			final FieldDescriptor field = parentClass
+					.getFieldIgnoreCase(methodName.substring(2));
+			if (field != null)
+				if ("boolean".equals(field.getType().getFullyQualifiedName()))
+					return false;
 		}
 
 		return true;
+
 	}
 
 }