smell 'Long Parameter List' added
diff --git a/org.eclipse.emf.refactor.smells.uml24.metricbased/bin/org/eclipse/emf/refactor/smells/uml24/metricbased/LongParameterList.class b/org.eclipse.emf.refactor.smells.uml24.metricbased/bin/org/eclipse/emf/refactor/smells/uml24/metricbased/LongParameterList.class
new file mode 100644
index 0000000..60a46f3
--- /dev/null
+++ b/org.eclipse.emf.refactor.smells.uml24.metricbased/bin/org/eclipse/emf/refactor/smells/uml24/metricbased/LongParameterList.class
Binary files differ
diff --git a/org.eclipse.emf.refactor.smells.uml24.metricbased/plugin.xml b/org.eclipse.emf.refactor.smells.uml24.metricbased/plugin.xml
index e68a7cc..44ecb54 100644
--- a/org.eclipse.emf.refactor.smells.uml24.metricbased/plugin.xml
+++ b/org.eclipse.emf.refactor.smells.uml24.metricbased/plugin.xml
@@ -31,6 +31,13 @@
modelsmell_metamodel="http://www.eclipse.org/uml2/4.0.0/UML"
modelsmell_name="Large Class">
</modelsmell>
+ <modelsmell
+ id="org.eclipse.emf.refactor.smells.uml24.longparameterlist"
+ modelsmell_description="The model contains an operation with more input parameters than the specified limit."
+ modelsmell_finderclass="org.eclipse.emf.refactor.smells.uml24.metricbased.LongParameterList"
+ modelsmell_metamodel="http://www.eclipse.org/uml2/4.0.0/UML"
+ modelsmell_name="Long Parameter List">
+ </modelsmell>
</extension>
</plugin>
diff --git a/org.eclipse.emf.refactor.smells.uml24.metricbased/src/org/eclipse/emf/refactor/smells/uml24/metricbased/LongParameterList.java b/org.eclipse.emf.refactor.smells.uml24.metricbased/src/org/eclipse/emf/refactor/smells/uml24/metricbased/LongParameterList.java
new file mode 100644
index 0000000..4686946
--- /dev/null
+++ b/org.eclipse.emf.refactor.smells.uml24.metricbased/src/org/eclipse/emf/refactor/smells/uml24/metricbased/LongParameterList.java
@@ -0,0 +1,52 @@
+package org.eclipse.emf.refactor.smells.uml24.metricbased;
+
+import java.util.LinkedList;
+import java.util.List;
+
+import org.eclipse.emf.ecore.EObject;
+import org.eclipse.emf.refactor.metrics.core.Metric;
+import org.eclipse.emf.refactor.metrics.interfaces.IMetricCalculator;
+import org.eclipse.emf.refactor.smells.core.MetricBasedModelSmellFinderClass;
+
+public class LongParameterList extends MetricBasedModelSmellFinderClass {
+
+ private String metricId = "org.eclipse.emf.refactor.metrics.uml24.ninppo";
+ private Metric localMetric = Metric.getMetricInstanceFromId(metricId);
+
+ @Override
+ public LinkedList<LinkedList<EObject>> findSmell(EObject root) {
+ LinkedList<EObject> rootList = new LinkedList<EObject>();
+ rootList.add(root);
+ IMetricCalculator localCalculateClass = localMetric.getCalculateClass();
+ double globalLimit = this.getLimit();
+ return findSmellyObjectGroups(root, globalLimit, localCalculateClass);
+ }
+
+ private LinkedList<LinkedList<EObject>> findSmellyObjectGroups(EObject root, double globalLimit,
+ IMetricCalculator localCalculateClass) {
+ String context = localMetric.getContext();
+ LinkedList<LinkedList<EObject>> smellyEObjects = new LinkedList<LinkedList<EObject>>();
+ List<EObject> containedEObjects = root.eContents();
+ for(EObject object : containedEObjects){
+ String objectType = object.eClass().getInstanceClass().getSimpleName();
+ if(objectType.equals(context)){
+ LinkedList<EObject> rootList = new LinkedList<EObject>();
+ rootList.add(object);
+ localCalculateClass.setContext(rootList);
+ double localValue = localCalculateClass.calculate();
+ if(limitReached(localValue, globalLimit)) {
+ LinkedList<EObject> currentObjects = new LinkedList<EObject>();
+ currentObjects.add(object);
+ smellyEObjects.add((currentObjects));
+ }
+ } else {
+ smellyEObjects.addAll(findSmellyObjectGroups(object, globalLimit, localCalculateClass));
+ }
+ }
+ return smellyEObjects;
+ }
+
+ private boolean limitReached(double localValue, double globalLimit) {
+ return (localValue > globalLimit);
+ }
+}
