The following document contains the results of FindBugs Report
FindBugs Version is 1.0.0
Threshold is Low
Effort is Default
| Bug | Category | Details | Line |
|---|---|---|---|
| org.apache.shale.usecases.rolodex.WebPagerRenderer$PageInfoIterator.next() can't throw NoSuchElement exception | CORRECTNESS | IT_NO_SUCH_ELEMENT | 587-592 |
| Bug | Category | Details | Line |
|---|---|---|---|
| org.apache.shale.usecases.rolodex.GenericComparator implements Comparator but not Serializable | STYLE | SE_COMPARATOR_SHOULD_BE_SERIALIZABLE | Not available |
| Possible null pointer dereference in org.apache.shale.usecases.rolodex.GenericComparator.compare(Object,Object) | CORRECTNESS | NP_NULL_ON_SOME_PATH | 149 |
| Bug | Category | Details | Line |
|---|---|---|---|
| org.apache.shale.usecases.rolodex.HeaderSorterRenderer.wasSelected(javax.faces.context.FacesContext,javax.faces.component.UIData,HeaderSorter) might ignore java.lang.Exception | CORRECTNESS | DE_MIGHT_IGNORE | 122 |
| Unchecked/unconfirmed cast from javax.faces.component.UIComponent to class org.apache.shale.usecases.rolodex.HeaderSorter in org.apache.shale.usecases.rolodex.HeaderSorterRenderer.encodeBegin(javax.faces.context.FacesContext,javax.faces.component.UIComponent) | STYLE | BC_UNCONFIRMED_CAST | 157 |
| Unchecked/unconfirmed cast from javax.faces.component.UIComponent to class org.apache.shale.usecases.rolodex.HeaderSorter in org.apache.shale.usecases.rolodex.HeaderSorterRenderer.encodeEnd(javax.faces.context.FacesContext,javax.faces.component.UIComponent) | STYLE | BC_UNCONFIRMED_CAST | 210 |
| Bug | Category | Details | Line |
|---|---|---|---|
| org.apache.shale.usecases.rolodex.HeaderSorter.saveState(javax.faces.context.FacesContext) invokes dubious Boolean constructor; use Boolean.valueOf(...) instead | PERFORMANCE | DM_BOOLEAN_CTOR | 80 |
| Bug | Category | Details | Line |
|---|---|---|---|
| Method org.apache.shale.usecases.rolodex.Contact.getSortName() invokes dubious String.toUpperCase() or String.toLowerCase; use the Locale parameterized version instead | I18N | DM_CONVERT_CASE | 169 |
| Bug | Category | Details | Line |
|---|---|---|---|
| Uncallable method org.apache.shale.usecases.rolodex.RolodexDao$1.addState(State) defined in anonymous class | CORRECTNESS | UMAC_UNCALLABLE_METHOD_OF_ANONYMOUS_CLASS | 183-201 |
| Uncallable method org.apache.shale.usecases.rolodex.RolodexDao$1.addContact(Contact) defined in anonymous class | CORRECTNESS | UMAC_UNCALLABLE_METHOD_OF_ANONYMOUS_CLASS | 204-205 |
| Bug | Category | Details | Line |
|---|---|---|---|
| Questionable cast from Collection to abstract class java.util.SortedSet in org.apache.shale.usecases.rolodex.RolodexDao.findContactsForTab(int) | STYLE | BC_BAD_CAST_TO_ABSTRACT_COLLECTION | 153 |
| Questionable cast from java.util.Collection to concrete class java.util.TreeSet in org.apache.shale.usecases.rolodex.RolodexDao.findContact(String) | STYLE | BC_BAD_CAST_TO_CONCRETE_COLLECTION | 330 |
| org.apache.shale.usecases.rolodex.GenericComparator.log isn't final but should be | MALICIOUS_CODE | MS_SHOULD_BE_FINAL | 48 |
| org.apache.shale.usecases.rolodex.RolodexDao.TAB_INDEX should be package protected | MALICIOUS_CODE | MS_PKGPROTECT | 77 |
| Should org.apache.shale.usecases.rolodex.WebPagerRenderer$PageInfo be a _static_ inner class? | PERFORMANCE | SIC_INNER_SHOULD_BE_STATIC | Not available |