The following document contains the results of FindBugs Report
FindBugs Version is 1.3.9
Threshold is low
Effort is max
| Bug | Category | Details | Line | Priority |
|---|---|---|---|---|
| Method net.sourceforge.jeuclid.app.mathviewer.InsertPolynomDialog.updateMathMLText() concatenates strings using + in a loop | PERFORMANCE | SBSC_USE_STRINGBUFFER_CONCATENATION | 191 | Medium |
| Bug | Category | Details | Line | Priority |
|---|---|---|---|---|
| Method net.sourceforge.jeuclid.app.mathviewer.InsertTableDialog.buildText(int, int, String) concatenates strings using + in a loop | PERFORMANCE | SBSC_USE_STRINGBUFFER_CONCATENATION | 264 | Medium |
| Bug | Category | Details | Line | Priority |
|---|---|---|---|---|
| Class net.sourceforge.jeuclid.app.mathviewer.MainFrame defines non-transient non-serializable instance field biTree | BAD_PRACTICE | SE_BAD_FIELD | Not available | Low |
| Bug | Category | Details | Line | Priority |
|---|---|---|---|---|
| net.sourceforge.jeuclid.app.mathviewer.MainFrame$1.actionPerformed(ActionEvent) invokes System.exit(...), which shuts down the entire virtual machine | BAD_PRACTICE | DM_EXIT | 315 | Medium |
| The class net.sourceforge.jeuclid.app.mathviewer.MainFrame$1 could be refactored into a named _static_ inner class | PERFORMANCE | SIC_INNER_SHOULD_BE_STATIC_ANON | 313-316 | Low |
| Bug | Category | Details | Line | Priority |
|---|---|---|---|---|
| net.sourceforge.jeuclid.app.mathviewer.MainFrameAppListener.handleQuit(ApplicationEvent) invokes System.exit(...), which shuts down the entire virtual machine | BAD_PRACTICE | DM_EXIT | 74 | Medium |
| Bug | Category | Details | Line | Priority |
|---|---|---|---|---|
| The class net.sourceforge.jeuclid.app.mathviewer.ParametersDialog$3 could be refactored into a named _static_ inner class | PERFORMANCE | SIC_INNER_SHOULD_BE_STATIC_ANON | 172-174 | Low |
| Bug | Category | Details | Line | Priority |
|---|---|---|---|---|
| The class net.sourceforge.jeuclid.app.mathviewer.ParametersDialog$4 could be refactored into a named _static_ inner class | PERFORMANCE | SIC_INNER_SHOULD_BE_STATIC_ANON | 191-193 | Low |
| Bug | Category | Details | Line | Priority |
|---|---|---|---|---|
| The class net.sourceforge.jeuclid.app.mathviewer.ParametersDialog$5 could be refactored into a named _static_ inner class | PERFORMANCE | SIC_INNER_SHOULD_BE_STATIC_ANON | 200-202 | Low |
| Bug | Category | Details | Line | Priority |
|---|---|---|---|---|
| The class net.sourceforge.jeuclid.app.mathviewer.ParametersDialog$6 could be refactored into a named _static_ inner class | PERFORMANCE | SIC_INNER_SHOULD_BE_STATIC_ANON | 232-237 | Low |
| Bug | Category | Details | Line | Priority |
|---|---|---|---|---|
| The class net.sourceforge.jeuclid.app.mathviewer.ParametersDialog$7 could be refactored into a named _static_ inner class | PERFORMANCE | SIC_INNER_SHOULD_BE_STATIC_ANON | 241-243 | Low |
| Bug | Category | Details | Line | Priority |
|---|---|---|---|---|
| The class net.sourceforge.jeuclid.app.mathviewer.ParametersDialog$9 could be refactored into a named _static_ inner class | PERFORMANCE | SIC_INNER_SHOULD_BE_STATIC_ANON | 276-278 | Low |
| Bug | Category | Details | Line | Priority |
|---|---|---|---|---|
| BiTree.doc not initialized in constructor | STYLE | UWF_FIELD_NOT_INITIALIZED_IN_CONSTRUCTOR | Not available | Low |
| BiTree.root not initialized in constructor | STYLE | UWF_FIELD_NOT_INITIALIZED_IN_CONSTRUCTOR | Not available | Low |
| Bug | Category | Details | Line | Priority |
|---|---|---|---|---|
| BiTreeCreationHelper.currentBiNode not initialized in constructor | STYLE | UWF_FIELD_NOT_INITIALIZED_IN_CONSTRUCTOR | Not available | Low |
| Bug | Category | Details | Line | Priority |
|---|---|---|---|---|
| Unread field: net.sourceforge.jeuclid.biparser.TextNode.ladder; should this field be static? | PERFORMANCE | SS_SHOULD_BE_STATIC | 40 | Medium |
