[Fb-contrib-commit] fb-contrib/htdocs flip1.gif,NONE,1.1 flip2.gif,NONE,1.1 index.html,1.24,1.25
Brought to you by:
dbrosius
Menu
▾
▴
[Fb-contrib-commit] fb-contrib/htdocs flip1.gif,NONE,1.1 flip2.gif,NONE,1.1 index.html,1.24,1.25
|
From: Dave B. <dbr...@us...> - 2005-11-10 07:18:22
|
Update of /cvsroot/fb-contrib/fb-contrib/htdocs In directory sc8-pr-cvs1.sourceforge.net:/tmp/cvs-serv4752/htdocs Modified Files: index.html Added Files: flip1.gif flip2.gif Log Message: proof that i'm only a programmer Index: index.html =================================================================== RCS file: /cvsroot/fb-contrib/fb-contrib/htdocs/index.html,v retrieving revision 1.24 retrieving revision 1.25 diff -u -d -r1.24 -r1.25 --- index.html 10 Nov 2005 06:36:45 -0000 1.24 +++ index.html 10 Nov 2005 07:18:11 -0000 1.25 @@ -2,6 +2,24 @@ <head> <meta name="keywords" content="findbugs,plugin,detector,bug,static,jar"/> <title>fb-contrib: A FindBugs auxiliary detector plugin</title> +<script language="javascript"> +function toggleBlock(divId, imgId) +{ + var el = document.getElementById(divId); + if (el.style.display=="block") + { + el.style.display="none"; + el = document.getElementById(imgId); + el.src="flip1.gif"; + } + else + { + el.style.display="block"; + el = document.getElementById(imgId); + el.src="flip2.gif"; + } +} +</script> </head> <body background> <div style="position:absolute;top:0;left:0;width:256;height:100%;z-index:1;background-image:url(blend.jpg);"> @@ -19,98 +37,117 @@ <a href="http://www.sourceforge.net/projects/fb-contrib">Project Page</a> <hr/> + <img id="cvs_image" src="flip2.gif" onClick="toggleBlock('cvs', 'cvs_image');"/> Detectors added in CVS<br/> - Note: these detectors are still in testing - <ul> - <li><b>Abnormal Finally Block Return</b><br/> - Finds methods that return or throw an exception from a finally block. Since a finally block is - executed after any return or throw statements that are present in the try or catch block, these exit - values are swallowed by the finally block's actions.</li> - <li><b>Non Collections Method Use</b><br/> - Finds calls to collections objects using methods that are not defined in the Collections interfaces, - but that have equivalent methods defined in such. By using the new methods, it allows for the - replacement of concrete classes with interfaces, and simplifies changing collection types if - desired.</li> - <li><b>Confusing Autoboxed Overloading</b><br/> - Finds classes that define overloaded methods where the only difference is a parameter being - of type java.lang.Character, and int, long, float or double. Due to autoboxing, one might conclude - that a parameter of 'a' would autobox to Character, but would instead be cast to a double.</li> - </ul> + <div id="cvs" style="display:block;"> + Note: these detectors are still in testing + <ul> + <li><b>Abnormal Finally Block Return</b><br/> + Finds methods that return or throw an exception from a finally block. Since a finally block is + executed after any return or throw statements that are present in the try or catch block, these exit + values are swallowed by the finally block's actions.</li> + <li><b>Non Collections Method Use</b><br/> + Finds calls to collections objects using methods that are not defined in the Collections interfaces, + but that have equivalent methods defined in such. By using the new methods, it allows for the + replacement of concrete classes with interfaces, and simplifies changing collection types if + desired.</li> + <li><b>Confusing Autoboxed Overloading</b><br/> + Finds classes that define overloaded methods where the only difference is a parameter being + of type java.lang.Character, and int, long, float or double. Due to autoboxing, one might conclude + that a parameter of 'a' would autobox to Character, but would instead be cast to a double.</li> + </ul> + </div> + <hr/> + <img id="v1_4_0_image" src="flip1.gif" onClick="toggleBlock('v1_4_0', 'v1_4_0_image');"/> Detectors added in v1.4.0<br/> - <ul> - <li><b>Final Parameters</b><br/> - Finds parameters that could be marked as final, but aren't. Doing so helps document the method, keeps - unwanted changes to creep in, and may help the jvm jit compiler to optimize better.</li> - <li><b>Abstract Class Empty Methods</b><br/> - Finds empty methods, or methods that just throw an exception in abstract classes. In these cases, it - probably is more correct to just define the method to be abstract as well, so that proper subclass - implementation is enforced.</li> - <li><b>Manual Array Copy</b><br/> - Finds methods that copy elements from one array to another manually using a loop. It is better - performing to use System.arraycopy, as this method is native.</li> - <li><b>Floating Point Loops</b><br/> - Finds methods that use floating point variables as the index to loops. Since floating point math is inprecise, - errors will accumulate each time through the loop, making the logic suspect. It is better to use an - integer index, and calculate the desired floating point value from this integer.</li> - </ul> + <div id="v1_4_0" style="display:none;"> + <ul> + <li><b>Final Parameters</b><br/> + Finds parameters that could be marked as final, but aren't. Doing so helps document the method, keeps + unwanted changes to creep in, and may help the jvm jit compiler to optimize better.</li> + <li><b>Abstract Class Empty Methods</b><br/> + Finds empty methods, or methods that just throw an exception in abstract classes. In these cases, it + probably is more correct to just define the method to be abstract as well, so that proper subclass + implementation is enforced.</li> + <li><b>Manual Array Copy</b><br/> + Finds methods that copy elements from one array to another manually using a loop. It is better + performing to use System.arraycopy, as this method is native.</li> + <li><b>Floating Point Loops</b><br/> + Finds methods that use floating point variables as the index to loops. Since floating point math is inprecise, + errors will accumulate each time through the loop, making the logic suspect. It is better to use an + integer index, and calculate the desired floating point value from this integer.</li> + </ul> + </div> + <hr/> + <img id="v1_2_0_image" src="flip1.gif" onClick="toggleBlock('v1_2_0', 'v1_2_0_image');"/> Detectors added in v1.2.0<br/> - <ul> - <li><b>Parallel Lists</b><br/> - Finds classes that maintain two or more lists or arrays that appear to share a one-to-one relationship - through a common index. It is usually better to create a separate class that holds all corresponding attributes, - and add instances of this class to just one list or array.</li> - <li><b>Dubious List Collection</b><br/> - Finds fields that are implementations of java.util.List, but that are used in a set-like fashion. - Since lookup type operations are performed using a linear search for Lists, the performance for large - Lists will be poor. Consideration should be made as to whether these fields should be sets.</li> - <li><b>Partially Constructed Object Access</b><br/> - Finds constructors that call non-final methods in non-final classes. If this class is derived from, and the - method is overridden, then that method will be executing on an object that hasn't been constructed in regards - to the subclass implementation. These methods should probably be defined as final.</li> - <li><b>Literal String Comparison</b><br/> - Finds methods that call the equals or compareTo methods on a String variable passing in a String literal. - A NullPointerException may occur if the string variable is null. If instead the method was called on - the string literal, and the variable was passed as an argument, this exception could never happen</li> - </ul> + <div id="v1_2_0" style="display:none;"> + <ul> + <li><b>Parallel Lists</b><br/> + Finds classes that maintain two or more lists or arrays that appear to share a one-to-one relationship + through a common index. It is usually better to create a separate class that holds all corresponding attributes, + and add instances of this class to just one list or array.</li> + <li><b>Dubious List Collection</b><br/> + Finds fields that are implementations of java.util.List, but that are used in a set-like fashion. + Since lookup type operations are performed using a linear search for Lists, the performance for large + Lists will be poor. Consideration should be made as to whether these fields should be sets.</li> + <li><b>Partially Constructed Object Access</b><br/> + Finds constructors that call non-final methods in non-final classes. If this class is derived from, and the + method is overridden, then that method will be executing on an object that hasn't been constructed in regards + to the subclass implementation. These methods should probably be defined as final.</li> + <li><b>Literal String Comparison</b><br/> + Finds methods that call the equals or compareTo methods on a String variable passing in a String literal. + A NullPointerException may occur if the string variable is null. If instead the method was called on + the string literal, and the variable was passed as an argument, this exception could never happen</li> + </ul> + </div> + <hr/> + <img id="v1_0_0_image" src="flip1.gif" onClick="toggleBlock('v1_0_0', 'v1_0_0_image');"/> Detectors added in v1.0.0<br/> - <ul> - <li><b>Overly Concrete Parameters</b><br/> - Finds parameters to methods that are defined as concrete classes, - when they're usage pattern can be defined by an implemented interface. - By switching to the interface, you can reduce coupling, which helps making - the code more testable and changeable.</li> - <li><b>List Indexed Iterating</b><br/> - Looks for uses of using loop indexed variables to 'iterate' over a List by calling get(i) - each time thru the loop. Depending on the List implementation, using Iterators can be - significantly faster. Using Iterators also makes it easier to substitute other collection types.</li> - <li><b>Unrelated Collection Contents</b><br/> - Looks for collections that hold objects that are unrelated by class or interface inheritance, other - than java.lang.Object. Doing so, leads to brittle code, either by 'encoding' type knowledge in the position - of an element, or using instanceof tests. It is usually better to create a separate class, add the individual - types as members to it, and add an instance of this class to the collection.</li> - <li><b>Declared Runtime Exception</b><br/> - Finds methods that declare RuntimeExceptions in their throws clause. While not illegal, - this may indicate a misunderstanding as to how unchecked exceptions are handled. - If is felt that a RuntimeException is so prevalent that it should be declared, it - is probably a better idea to prevent the occurance in code.</li> - </ul> + <div id="v1_0_0" style="display:none;"> + <ul> + <li><b>Overly Concrete Parameters</b><br/> + Finds parameters to methods that are defined as concrete classes, + when they're usage pattern can be defined by an implemented interface. + By switching to the interface, you can reduce coupling, which helps making + the code more testable and changeable.</li> + <li><b>List Indexed Iterating</b><br/> + Looks for uses of using loop indexed variables to 'iterate' over a List by calling get(i) + each time thru the loop. Depending on the List implementation, using Iterators can be + significantly faster. Using Iterators also makes it easier to substitute other collection types.</li> + <li><b>Unrelated Collection Contents</b><br/> + Looks for collections that hold objects that are unrelated by class or interface inheritance, other + than java.lang.Object. Doing so, leads to brittle code, either by 'encoding' type knowledge in the position + of an element, or using instanceof tests. It is usually better to create a separate class, add the individual + types as members to it, and add an instance of this class to the collection.</li> + <li><b>Declared Runtime Exception</b><br/> + Finds methods that declare RuntimeExceptions in their throws clause. While not illegal, + this may indicate a misunderstanding as to how unchecked exceptions are handled. + If is felt that a RuntimeException is so prevalent that it should be declared, it + is probably a better idea to prevent the occurance in code.</li> + </ul> + </div> + <hr/> + <img id="v0_9_3_image" src="flip1.gif" onClick="toggleBlock('v0_9_3', 'v0_9_3_image');"/> Detectors added in v0.9.3 - <ul> - <li><b>Inefficient String Buffering</b><br/> - Finds Concatenation inside of a StringBuffer.append call, which - creates temporary StringBuffers.</li> - <li><b>Synchronized Collection Iterators</b><br/> - Finds use of iterators on collections built from Collection.synchronizedXXX() - calls. As iterators are by default not multithread safe, there use with a - synchronized collections seems dubious.</li> - <li><b>Cyclomatic Complexity</b><br/> - Finds methods that are overly complex based on the McCabe algorithm for - counting number of unique branches in the method</li> - </ul> + <div id="v0_9_3" style="display:none;"> + <ul> + <li><b>Inefficient String Buffering</b><br/> + Finds Concatenation inside of a StringBuffer.append call, which + creates temporary StringBuffers.</li> + <li><b>Synchronized Collection Iterators</b><br/> + Finds use of iterators on collections built from Collection.synchronizedXXX() + calls. As iterators are by default not multithread safe, there use with a + synchronized collections seems dubious.</li> + <li><b>Cyclomatic Complexity</b><br/> + Finds methods that are overly complex based on the McCabe algorithm for + counting number of unique branches in the method</li> + </ul> + </div> </div> </body> </html> \ No newline at end of file --- NEW FILE: flip1.gif --- (This appears to be a binary file; contents omitted.) --- NEW FILE: flip2.gif --- (This appears to be a binary file; contents omitted.) |
