/* * FindBugs - Find bugs in Java programs * Copyright (C) 2004 Dave Brosius <dbrosius@users.sourceforge.net> * Copyright (C) 2004 University of Maryland * * This library is free software; you can redistribute it and/or * modify it under the terms of the GNU Lesser General Public * License as published by the Free Software Foundation; either * version 2.1 of the License, or (at your option) any later version. * * This library is distributed in the hope that it will be useful, * but WITHOUT ANY WARRANTY; without even the implied warranty of * MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE. See the GNU * Lesser General Public License for more details. * * You should have received a copy of the GNU Lesser General Public * License along with this library; if not, write to the Free Software * Foundation, Inc., 59 Temple Place, Suite 330, Boston, MA 02111-1307 USA */ package edu.umd.cs.findbugs.detect; import org.apache.bcel.Repository; import org.apache.bcel.classfile.Code; import org.apache.bcel.classfile.JavaClass; import org.apache.bcel.classfile.Method; import edu.umd.cs.findbugs.BugAccumulator; import edu.umd.cs.findbugs.BugInstance; import edu.umd.cs.findbugs.BugReporter; import edu.umd.cs.findbugs.BytecodeScanningDetector; import edu.umd.cs.findbugs.StatelessDetector; import edu.umd.cs.findbugs.SystemProperties; import edu.umd.cs.findbugs.ba.AnalysisContext; import edu.umd.cs.findbugs.ba.ClassContext; /** * Find occurrences of collection.toArray( new Foo[0] ); This causes another * memory allocation through reflection Much better to do collection.toArray( * new Foo[collection.size()] ); * * @author Dave Brosius */ public class InefficientToArray extends BytecodeScanningDetector implements StatelessDetector { private static final boolean DEBUG = SystemProperties.getBoolean("ita.debug"); static final int SEEN_NOTHING = 0; static final int SEEN_ICONST_0 = 1; static final int SEEN_ANEWARRAY = 2; private final static JavaClass collectionClass; private BugReporter bugReporter; private BugAccumulator bugAccumulator; private int state = SEEN_NOTHING; static { JavaClass tmp = null; try { tmp = AnalysisContext.lookupSystemClass("java.util.Collection"); } catch (ClassNotFoundException cnfe) { AnalysisContext.reportMissingClass(cnfe); } collectionClass = tmp; } public InefficientToArray(BugReporter bugReporter) { this.bugReporter = bugReporter; this.bugAccumulator = new BugAccumulator(bugReporter); } @Override public void visitClassContext(ClassContext classContext) { if (collectionClass != null) classContext.getJavaClass().accept(this); } @Override public void visit(Method obj) { if (DEBUG) System.out.println("------------------- Analyzing " + obj.getName() + " ----------------"); state = SEEN_NOTHING; super.visit(obj); } @Override public void visit(Code obj) { super.visit(obj); bugAccumulator.reportAccumulatedBugs(); } @Override public void sawOpcode(int seen) { if (DEBUG) System.out.println("State: " + state + " Opcode: " + OPCODE_NAMES[seen]); switch (state) { case SEEN_NOTHING: if (seen == ICONST_0) state = SEEN_ICONST_0; break; case SEEN_ICONST_0: if (seen == ANEWARRAY) { state = SEEN_ANEWARRAY; } else state = SEEN_NOTHING; break; case SEEN_ANEWARRAY: if (((seen == INVOKEVIRTUAL) || (seen == INVOKEINTERFACE)) && (getNameConstantOperand().equals("toArray")) && (getSigConstantOperand().equals("([Ljava/lang/Object;)[Ljava/lang/Object;"))) { try { String clsName = getDottedClassConstantOperand(); JavaClass cls = Repository.lookupClass(clsName); if (cls.implementationOf(collectionClass)) bugAccumulator.accumulateBug( new BugInstance(this, "ITA_INEFFICIENT_TO_ARRAY", LOW_PRIORITY).addClassAndMethod(this), this); } catch (ClassNotFoundException cnfe) { bugReporter.reportMissingClass(cnfe); } } state = SEEN_NOTHING; break; default: state = SEEN_NOTHING; break; } } } // vim:ts=4