Start line:  
End line:  

Snippet Preview

Snippet HTML Code

Stack Overflow Questions
  /*
   * fb-contrib - Auxiliary detectors for Java programs
   * Copyright (C) 2005-2012 Dave Brosius
   * 
   * 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 com.mebigfatguy.fbcontrib.detect;
 
 import  org.apache.bcel.classfile.Code;
 
 import  edu.umd.cs.findbugs.BugInstance;
 import  edu.umd.cs.findbugs.BugReporter;
 import  edu.umd.cs.findbugs.BytecodeScanningDetector;
 import  edu.umd.cs.findbugs.ba.ClassContext;

looks for creation of arrays where the contents are constants, or static fields, and the array isn't further modified. These arrays should probably be defined as static fields so the method doesn't constantly recreate the array each time it is called.
 
 public class StaticArrayCreatedInMethod extends BytecodeScanningDetector 
 {
 	enum State {SEEN_NOTHING, SEEN_ARRAY_SIZE, SEEN_NEWARRAY, SEEN_DUP, SEEN_INDEX, SEEN_LDC, SEEN_INDEX_STORE}
 	
 	private BugReporter bugReporter;
 	private int arraySize;
 	private int storeCount;
 	private State state;
 	
 	public StaticArrayCreatedInMethod(BugReporter bugReporter) {
 		this. = bugReporter;
 	}
 	
 	public void visitClassContext(ClassContext classContext) {
 		try {
 			super.visitClassContext(classContext);
 		} finally {
 			
 		}
 	}

implements the visitor by forwarding calls for methods that are the static initializer

Parameters:
obj the context object of the currently parsed code block
 
 	public void visitCode(Code obj) {
 		if (!"<clinit>".equals(getMethodName())) {
 			super.visitCode(obj);
 		}
 	}

implements the visitor to look for creation of local arrays using constant values

Parameters:
seen the opcode of the currently parsed instruction
 
 	public void sawOpcode(int seen) {
 		int index;
 		
 		switch () {
 			case :
 				if (seen == BIPUSH) {
 					if ( > 0)
 				}else if ((seen >= ICONST_M1) && (seen <= ICONST_5)) {
 					 = seen - ICONST_M1 - 1;
 					if ( > 0)
 				}
 			break;
 			
 				if ((seen == ANEWARRAY) || (seen == NEWARRAY)) {
 					 = 0;
 				}
 				else
 			break;
 			
				if (seen == DUP)
				else
			break;
			case :
				if (seen == BIPUSH)
					index = getIntConstant();
				else if ((seen >= ICONST_M1) && (seen <= ICONST_5))
					index = seen - ICONST_M1 - 1;
				else {
					return;
				}
				if (index != )
				else
			break;
				if ((seen == LDC) || (seen == LDC_W))
				else
			break;
			case :
				if ((seen >= IASTORE) && (seen <= SASTORE)) {
					if ((++) == )
					else
				}
			break;
				if ((seen == ASTORE)
				||  ((seen >= ASTORE_0) && (seen <= ASTORE_3))) {
					.reportBug(new BugInstance(this"SACM_STATIC_ARRAY_CREATED_IN_METHOD", ( < 3) ? LOW_PRIORITY : (( < 10) ? NORMAL_PRIORITY : HIGH_PRIORITY))
							   .addClass(this)
							   .addMethod(this)
							   .addSourceLine(thisgetPC()));
				}
			break;
		}
	}
New to GrepCode? Check out our FAQ X