The following document contains the results of FindBugs Report
FindBugs Version is 1.2.0
Threshold is Low
Effort is Default
Classes | Bugs | Errors | Missing Classes |
---|---|---|---|
398 | 21 | 0 | 0 |
Bug | Category | Details | Line |
---|---|---|---|
de.mud.jmd.BSXPolygon.clone() does not call super.clone() | BAD_PRACTICE | CN_IDIOM_NO_SUPER_CALL | 39-44 |
Bug | Category | Details | Line |
---|---|---|---|
Class de.mud.jmd.JMD defines non-transient non-serializable instance field ah | BAD_PRACTICE | SE_BAD_FIELD | Not available |
Bug | Category | Details | Line |
---|---|---|---|
de.mud.jmd.JMD$ActionHandler.actionPerformed(ActionEvent) might ignore java.lang.Exception | BAD_PRACTICE | DE_MIGHT_IGNORE | 395 |
de.mud.jmd.JMD$ActionHandler.actionPerformed(ActionEvent) invokes System.exit(...), which shuts down the entire virtual machine | BAD_PRACTICE | DM_EXIT | 304 |
Exception is caught when Exception is not thrown in de.mud.jmd.JMD$ActionHandler.actionPerformed(ActionEvent) | STYLE | REC_CATCH_EXCEPTION | 395 |
Bug | Category | Details | Line |
---|---|---|---|
Should de.mud.jmd.JMD$CloseHandler be a _static_ inner class? | PERFORMANCE | SIC_INNER_SHOULD_BE_STATIC | Not available |
Unwritten field: de.mud.jmd.JMD$CloseHandler.jmd | CORRECTNESS | UWF_UNWRITTEN_FIELD | 236 |
Bug | Category | Details | Line |
---|---|---|---|
Class de.mud.jmd.JMDBackground defines non-transient non-serializable instance field _scaled | BAD_PRACTICE | SE_BAD_FIELD | Not available |
Class de.mud.jmd.JMDBackground defines non-transient non-serializable instance field image | BAD_PRACTICE | SE_BAD_FIELD | Not available |
Usage of GetResource in de.mud.jmd.JMDBackground.initComponents() may be unsafe if class is extended | BAD_PRACTICE | UI_INHERITANCE_UNSAFE_GETRESOURCE | 198 |
Bug | Category | Details | Line |
---|---|---|---|
Use of non-localized String.toUpperCase() or String.toLowerCase | I18N | DM_CONVERT_CASE | 447 |
Bug | Category | Details | Line |
---|---|---|---|
The class de.mud.jmd.JMDDrawControls$Colorbox could be refactored into a _static_ inner class | PERFORMANCE | SIC_INNER_SHOULD_BE_STATIC_NEEDS_THIS | Not available |
Bug | Category | Details | Line |
---|---|---|---|
Method de.mud.jmd.JMDMainCanvas.clickedOn(boolean, int, int) uses the same code for two switch clauses | STYLE | DB_DUPLICATE_SWITCH_CLAUSES | 713 |
Method de.mud.jmd.JMDMainCanvas.movedTo(int, int) uses the same code for two switch clauses | STYLE | DB_DUPLICATE_SWITCH_CLAUSES | 767 |
Dead store to color in de.mud.jmd.JMDMainCanvas.loadFile(File, JFrame) | STYLE | DLS_DEAD_LOCAL_STORE | 409 |
Use of non-localized String.toUpperCase() or String.toLowerCase | I18N | DM_CONVERT_CASE | 398 |
Use of non-localized String.toUpperCase() or String.toLowerCase | I18N | DM_CONVERT_CASE | 465 |
Null pointer dereference of out in de.mud.jmd.JMDMainCanvas.saveFile(JFrame) on exception path | CORRECTNESS | NP_ALWAYS_NULL_EXCEPTION | 497 |
Switch statement found in de.mud.jmd.JMDMainCanvas.paintGuide(Graphics) where one case falls through to the next case | STYLE | SF_SWITCH_FALLTHROUGH | 1626-1628 |
Bug | Category | Details | Line |
---|---|---|---|
Use of non-localized String.toUpperCase() or String.toLowerCase | I18N | DM_CONVERT_CASE | 1763 |
Bug | Category | Details | Line |
---|---|---|---|
Class de.mud.jmd.JMDMainCanvas$MouseHandler implements same interface as superclass | STYLE | RI_REDUNDANT_INTERFACES | Not available |