X-Git-Url: http://git.phpeclipse.com

diff --git a/net.sourceforge.phpeclipse/src/net/sourceforge/phpdt/internal/compiler/ast/MethodDeclaration.java b/net.sourceforge.phpeclipse/src/net/sourceforge/phpdt/internal/compiler/ast/MethodDeclaration.java
index 93097f0..615fc04 100644
--- a/net.sourceforge.phpeclipse/src/net/sourceforge/phpdt/internal/compiler/ast/MethodDeclaration.java
+++ b/net.sourceforge.phpeclipse/src/net/sourceforge/phpdt/internal/compiler/ast/MethodDeclaration.java
@@ -1,15 +1,20 @@
 package net.sourceforge.phpdt.internal.compiler.ast;
 
-import net.sourceforge.phpdt.internal.compiler.ast.Block;
-import net.sourceforge.phpdt.internal.compiler.ast.ArgumentDeclaration;
 import net.sourceforge.phpdt.internal.compiler.parser.OutlineableWithChildren;
 import net.sourceforge.phpdt.internal.compiler.parser.Outlineable;
+import net.sourceforge.phpdt.internal.compiler.ast.declarations.VariableUsage;
 import net.sourceforge.phpdt.internal.ui.PHPUiImages;
+import net.sourceforge.phpeclipse.PHPeclipsePlugin;
 import org.eclipse.jface.resource.ImageDescriptor;
+import org.eclipse.jface.text.Position;
+import org.eclipse.core.runtime.CoreException;
 
 import java.util.Hashtable;
 import java.util.Enumeration;
 import java.util.ArrayList;
+import java.util.List;
+
+import test.PHPParserSuperclass;
 
 /**
  * A Method declaration.
@@ -27,23 +32,29 @@ public class MethodDeclaration extends Statement implements OutlineableWithChild
   public int bodyEnd = -1;
   /** Tell if the method is a class constructor. */
   public boolean isConstructor;
+
+  /** The parent object. */
   private Object parent;
   /** The outlineable children (those will be in the node array too. */
   private ArrayList children = new ArrayList();
 
+  /** Tell if the method returns a reference. */
   public boolean reference;
 
-  public MethodDeclaration(Object parent,
-                           char[] name,
-                           Hashtable arguments,
-                           boolean reference,
-                           int sourceStart,
-                           int sourceEnd) {
+  private Position position;
+
+  public MethodDeclaration(final Object parent,
+                           final char[] name,
+                           final Hashtable arguments,
+                           final boolean reference,
+                           final int sourceStart,
+                           final int sourceEnd) {
     super(sourceStart, sourceEnd);
     this.name = name;
     this.arguments = arguments;
     this.parent = parent;
     this.reference = reference;
+    position = new Position(sourceStart, sourceEnd);
   }
 
   /**
@@ -51,39 +62,24 @@ public class MethodDeclaration extends Statement implements OutlineableWithChild
    * @param tab the number of tabs
    * @return the String containing the method
    */
-  public String toString(int tab) {
-    StringBuffer buff = new StringBuffer(tabString(tab));
-    buff.append("function ");//$NON-NLS-1$
-    if (reference) {
-      buff.append('&');//$NON-NLS-1$
-    }
-    buff.append(name).append("(");//$NON-NLS-1$
-
-    if (arguments != null) {
-      Enumeration values = arguments.elements();
-      int i = 0;
-      while (values.hasMoreElements()) {
-        ArgumentDeclaration o = (ArgumentDeclaration) values.nextElement();
-        buff.append(o.toString(0));
-        if (i != (arguments.size() - 1)) {
-          buff.append(", "); //$NON-NLS-1$
-        }
-        i++;
-      }
-    }
-    buff.append(")"); //$NON-NLS-1$
-
-   buff.append(toStringStatements(tab + 1));
+  public String toString(final int tab) {
+    final StringBuffer buff = new StringBuffer(tabString(tab));
+    buff.append(toStringHeader());
+    buff.append(toStringStatements(tab + 1));
     return buff.toString();
   }
 
+  public String toStringHeader() {
+    return "function " + toString();
+  }
+
   /**
    * Return the statements of the method into Strings
    * @param tab the number of tabs
    * @return the String containing the statements
    */
-  public String toStringStatements(int tab) {
-    StringBuffer buff = new StringBuffer(" {"); //$NON-NLS-1$
+  public String toStringStatements(final int tab) {
+    final StringBuffer buff = new StringBuffer(" {"); //$NON-NLS-1$
     if (statements != null) {
       for (int i = 0; i < statements.length; i++) {
         buff.append("\n").append(statements[i].toString(tab)); //$NON-NLS-1$
@@ -104,19 +100,169 @@ public class MethodDeclaration extends Statement implements OutlineableWithChild
     return PHPUiImages.DESC_FUN;
   }
 
+  public void setParent(final Object parent) {
+    this.parent = parent;
+  }
+
   public Object getParent() {
     return parent;
   }
 
-  public boolean add(Outlineable o) {
+  public boolean add(final Outlineable o) {
     return children.add(o);
   }
 
-  public Outlineable get(int index) {
+  public Outlineable get(final int index) {
     return (Outlineable) children.get(index);
   }
 
   public int size() {
     return children.size();
   }
+
+  public String toString() {
+    final StringBuffer buff = new StringBuffer();
+    if (reference) {
+      buff.append("&");//$NON-NLS-1$
+    }
+    buff.append(name).append("(");//$NON-NLS-1$
+
+    if (arguments != null) {
+      final Enumeration values = arguments.elements();
+      int i = 0;
+      while (values.hasMoreElements()) {
+        final VariableDeclaration o = (VariableDeclaration) values.nextElement();
+        buff.append(o.toStringExpression());
+        if (i != (arguments.size() - 1)) {
+          buff.append(", "); //$NON-NLS-1$
+        }
+        i++;
+      }
+    }
+    buff.append(")"); //$NON-NLS-1$
+    return buff.toString();
+  }
+
+  public Position getPosition() {
+    return position;
+  }
+
+  public List getList() {
+    return children;
+  }
+
+  /**
+   * Get the variables from outside (parameters, globals ...)
+   * @return the variables from outside
+   */
+  public List getOutsideVariable() {
+    final ArrayList list = new ArrayList();
+    if (arguments != null) {
+      final Enumeration vars = arguments.keys();
+      while (vars.hasMoreElements()) {
+        list.add(new VariableUsage((String) vars.nextElement(), getSourceStart()));
+      }
+    }
+
+    if (statements != null) {
+      for (int i = 0; i < statements.length; i++) {
+        list.addAll(statements[i].getOutsideVariable());
+      }
+    }
+    return list;
+  }
+
+  /**
+   * get the modified variables.
+   * @return the variables from we change value
+   */
+  public List getModifiedVariable() {
+    final ArrayList list = new ArrayList();
+    if (statements != null) {
+      for (int i = 0; i < statements.length; i++) {
+        list.addAll(statements[i].getModifiedVariable());
+      }
+    }
+    return list;
+  }
+
+  /**
+   * Get the variables used.
+   * @return the variables used
+   */
+  public List getUsedVariable() {
+    final ArrayList list = new ArrayList();
+    if (statements != null) {
+      for (int i = 0; i < statements.length; i++) {
+        list.addAll(statements[i].getUsedVariable());
+      }
+    }
+    return list;
+  }
+
+  private boolean isVariableDeclaredBefore(List list, VariableUsage var) {
+    final String name = var.getName();
+    final int pos = var.getStartOffset();
+    for (int i = 0; i < list.size(); i++) {
+      VariableUsage variableUsage = (VariableUsage) list.get(i);
+      if (variableUsage.getName().equals(name) && variableUsage.getStartOffset() < pos) {
+        return true;
+      }
+    }
+    return false;
+  }
+
+  private void dumpList(List list, String name) {
+    StringBuffer buff = new StringBuffer(name).append("\n");
+    for (int i = 0; i < list.size(); i++) {
+      buff.append(list.get(i).toString()).append("\n");
+    }
+    if (PHPeclipsePlugin.DEBUG) {
+      PHPeclipsePlugin.log(1, buff.toString());
+    }
+  }
+
+  /**
+   * This method will analyze the code.
+   */
+  public void analyzeCode() {
+    final List outsideVars = getOutsideVariable();
+    final List modifiedVars = getModifiedVariable();
+
+    final List declaredVars = new ArrayList(outsideVars.size() + modifiedVars.size());
+    declaredVars.addAll(outsideVars);
+    declaredVars.addAll(modifiedVars);
+
+    final List usedVars = getUsedVariable();
+
+/*    dumpList(outsideVars, "outside");
+    dumpList(modifiedVars, "modified");
+    dumpList(usedVars, "used");  */
+
+
+    //look for used variables that were not declared before
+    findUnknownUsedVars(usedVars, declaredVars);
+  }
+
+  /**
+   * This method will add a warning on all used variables in a method that aren't declared before.
+   * @param usedVars the used variable list
+   * @param declaredVars the declared variable list
+   */
+  private void findUnknownUsedVars(final List usedVars, final List declaredVars) {
+    for (int i = 0; i < usedVars.size(); i++) {
+      VariableUsage variableUsage = (VariableUsage) usedVars.get(i);
+      if (!isVariableDeclaredBefore(declaredVars, variableUsage)) {
+        try {
+          PHPParserSuperclass.setMarker("warning, usage of an unknown variable : " + variableUsage.getName(),
+                                        variableUsage.getStartOffset(),
+                                        variableUsage.getStartOffset() + variableUsage.getName().length(),
+                                        PHPParserSuperclass.WARNING,
+                                        "");
+        } catch (CoreException e) {
+          PHPeclipsePlugin.log(e);
+        }
+      }
+    }
+  }
 }