SpotBugs Bug Detector Report
The following document contains the results of SpotBugs
SpotBugs Version is 4.2.1
Threshold is medium
Effort is default
Summary
| Classes |
Bugs |
Errors |
Missing Classes |
| 405 |
52 |
0 |
0 |
com.gh.mygreen.xlsmapper.XlsLoader
| Bug |
Category |
Details |
Line |
Priority |
| null とわかっている値をロードしています。com.gh.mygreen.xlsmapper.XlsLoader.loadDetail(InputStream, Class) |
STYLE |
NP_LOAD_OF_KNOWN_NULL_VALUE |
111 |
Medium |
| null とわかっている値をロードしています。com.gh.mygreen.xlsmapper.XlsLoader.loadMultipleDetail(InputStream, Class) |
STYLE |
NP_LOAD_OF_KNOWN_NULL_VALUE |
186 |
Medium |
| null とわかっている値をロードしています。com.gh.mygreen.xlsmapper.XlsLoader.loadMultipleDetail(InputStream, Class[]) |
STYLE |
NP_LOAD_OF_KNOWN_NULL_VALUE |
293 |
Medium |
com.gh.mygreen.xlsmapper.XlsSaver
| Bug |
Category |
Details |
Line |
Priority |
| null とわかっている値をロードしています。com.gh.mygreen.xlsmapper.XlsSaver.saveDetail(InputStream, OutputStream, Object) |
STYLE |
NP_LOAD_OF_KNOWN_NULL_VALUE |
113 |
Medium |
| null とわかっている値をロードしています。com.gh.mygreen.xlsmapper.XlsSaver.saveMultipleDetail(InputStream, OutputStream, Object[]) |
STYLE |
NP_LOAD_OF_KNOWN_NULL_VALUE |
192 |
Medium |
com.gh.mygreen.xlsmapper.annotation.XlsHorizontalRecordsForIterateTables
| Bug |
Category |
Details |
Line |
Priority |
| com.gh.mygreen.xlsmapper.annotation.XlsHorizontalRecordsForIterateTables.cases() は，XlsHorizontalRecordsForIterateTables._cases を返すことによって内部表現を暴露するかもしれません。 |
MALICIOUS_CODE |
EI_EXPOSE_REP |
138 |
Medium |
com.gh.mygreen.xlsmapper.annotation.XlsLabelledArrayCellsForIterateTable
| Bug |
Category |
Details |
Line |
Priority |
| com.gh.mygreen.xlsmapper.annotation.XlsLabelledArrayCellsForIterateTable.cases() は，XlsLabelledArrayCellsForIterateTable.cases を返すことによって内部表現を暴露するかもしれません。 |
MALICIOUS_CODE |
EI_EXPOSE_REP |
124 |
Medium |
com.gh.mygreen.xlsmapper.annotation.XlsLabelledCellForIterateTable
| Bug |
Category |
Details |
Line |
Priority |
| com.gh.mygreen.xlsmapper.annotation.XlsLabelledCellForIterateTable.cases() は，XlsLabelledCellForIterateTable._cases を返すことによって内部表現を暴露するかもしれません。 |
MALICIOUS_CODE |
EI_EXPOSE_REP |
93 |
Medium |
com.gh.mygreen.xlsmapper.annotation.XlsLabelledCommentForIterateTable
| Bug |
Category |
Details |
Line |
Priority |
| com.gh.mygreen.xlsmapper.annotation.XlsLabelledCommentForIterateTable.cases() は，XlsLabelledCommentForIterateTable.cases を返すことによって内部表現を暴露するかもしれません。 |
MALICIOUS_CODE |
EI_EXPOSE_REP |
68 |
Medium |
com.gh.mygreen.xlsmapper.annotation.XlsVerticalRecordsForIterateTables
| Bug |
Category |
Details |
Line |
Priority |
| com.gh.mygreen.xlsmapper.annotation.XlsVerticalRecordsForIterateTables.cases() は，XlsVerticalRecordsForIterateTables._cases を返すことによって内部表現を暴露するかもしれません。 |
MALICIOUS_CODE |
EI_EXPOSE_REP |
158 |
Medium |
com.gh.mygreen.xlsmapper.cellconverter.OptionalProcessCase
| Bug |
Category |
Details |
Line |
Priority |
| 潜在的な非短絡論理の危険な使用。com.gh.mygreen.xlsmapper.cellconverter.OptionalProcessCase.isPresent(ProcessCase) |
STYLE |
NS_DANGEROUS_NON_SHORT_CIRCUIT |
76 |
High |
com.gh.mygreen.xlsmapper.cellconverter.impl.ArrayCellConverterFactory$ArrayCellConverter
| Bug |
Category |
Details |
Line |
Priority |
| com.gh.mygreen.xlsmapper.cellconverter.impl.ArrayCellConverterFactory$ArrayCellConverter は，static 内部クラスにすべきです。 |
PERFORMANCE |
SIC_INNER_SHOULD_BE_STATIC |
69-93 |
Medium |
com.gh.mygreen.xlsmapper.cellconverter.impl.BigDecimalCellConverterFactory$BigDecimalCellConverter
| Bug |
Category |
Details |
Line |
Priority |
| com.gh.mygreen.xlsmapper.cellconverter.impl.BigDecimalCellConverterFactory$BigDecimalCellConverter は，static 内部クラスにすべきです。 |
PERFORMANCE |
SIC_INNER_SHOULD_BE_STATIC |
33-46 |
Medium |
com.gh.mygreen.xlsmapper.cellconverter.impl.BigIntegerCellConverterFactory$BigIntegerCellConverter
| Bug |
Category |
Details |
Line |
Priority |
| com.gh.mygreen.xlsmapper.cellconverter.impl.BigIntegerCellConverterFactory$BigIntegerCellConverter は，static 内部クラスにすべきです。 |
PERFORMANCE |
SIC_INNER_SHOULD_BE_STATIC |
36-49 |
Medium |
com.gh.mygreen.xlsmapper.cellconverter.impl.BooleanCellConverterFactory$BooleanCellConverter
| Bug |
Category |
Details |
Line |
Priority |
| com.gh.mygreen.xlsmapper.cellconverter.impl.BooleanCellConverterFactory$BooleanCellConverter.parseCell(Cell, String) は，戻り型が Boolean なのに明示的に null を返しています。 |
BAD_PRACTICE |
NP_BOOLEAN_RETURN_NULL |
85 |
Medium |
| com.gh.mygreen.xlsmapper.cellconverter.impl.BooleanCellConverterFactory$BooleanCellConverter は，static 内部クラスにすべきです。 |
PERFORMANCE |
SIC_INNER_SHOULD_BE_STATIC |
57-107 |
Medium |
com.gh.mygreen.xlsmapper.cellconverter.impl.ByteCellConverterFactory$ByteCellConverter
| Bug |
Category |
Details |
Line |
Priority |
| com.gh.mygreen.xlsmapper.cellconverter.impl.ByteCellConverterFactory$ByteCellConverter は，static 内部クラスにすべきです。 |
PERFORMANCE |
SIC_INNER_SHOULD_BE_STATIC |
36-48 |
Medium |
com.gh.mygreen.xlsmapper.cellconverter.impl.CalendarCellConverterFactory$CalendarCellConverter
| Bug |
Category |
Details |
Line |
Priority |
| com.gh.mygreen.xlsmapper.cellconverter.impl.CalendarCellConverterFactory$CalendarCellConverter は，static 内部クラスにすべきです。 |
PERFORMANCE |
SIC_INNER_SHOULD_BE_STATIC |
74-103 |
Medium |
com.gh.mygreen.xlsmapper.cellconverter.impl.CellLinkCellConverterFactory$CellLinkCellConverter
| Bug |
Category |
Details |
Line |
Priority |
| com.gh.mygreen.xlsmapper.cellconverter.impl.CellLinkCellConverterFactory$CellLinkCellConverter は，static 内部クラスにすべきです。 |
PERFORMANCE |
SIC_INNER_SHOULD_BE_STATIC |
66-123 |
Medium |
com.gh.mygreen.xlsmapper.cellconverter.impl.CharacterCellConverterFactory$CharacterCellConverter
| Bug |
Category |
Details |
Line |
Priority |
| com.gh.mygreen.xlsmapper.cellconverter.impl.CharacterCellConverterFactory$CharacterCellConverter は，static 内部クラスにすべきです。 |
PERFORMANCE |
SIC_INNER_SHOULD_BE_STATIC |
74-96 |
Medium |
com.gh.mygreen.xlsmapper.cellconverter.impl.DateCellConverterFactory$DateCellConverter
| Bug |
Category |
Details |
Line |
Priority |
| com.gh.mygreen.xlsmapper.cellconverter.impl.DateCellConverterFactory$DateCellConverter は，static 内部クラスにすべきです。 |
PERFORMANCE |
SIC_INNER_SHOULD_BE_STATIC |
50-62 |
Medium |
com.gh.mygreen.xlsmapper.cellconverter.impl.DoubleCellConverterFactory$DoubleCellConverter
| Bug |
Category |
Details |
Line |
Priority |
| com.gh.mygreen.xlsmapper.cellconverter.impl.DoubleCellConverterFactory$DoubleCellConverter は，static 内部クラスにすべきです。 |
PERFORMANCE |
SIC_INNER_SHOULD_BE_STATIC |
33-45 |
Medium |
com.gh.mygreen.xlsmapper.cellconverter.impl.EnumCellConverterFactory$EnumCellConverter
| Bug |
Category |
Details |
Line |
Priority |
| com.gh.mygreen.xlsmapper.cellconverter.impl.EnumCellConverterFactory$EnumCellConverter は，static 内部クラスにすべきです。 |
PERFORMANCE |
SIC_INNER_SHOULD_BE_STATIC |
58-88 |
Medium |
com.gh.mygreen.xlsmapper.cellconverter.impl.FloatCellConverterFactory$FloatCellConverter
| Bug |
Category |
Details |
Line |
Priority |
| com.gh.mygreen.xlsmapper.cellconverter.impl.FloatCellConverterFactory$FloatCellConverter は，static 内部クラスにすべきです。 |
PERFORMANCE |
SIC_INNER_SHOULD_BE_STATIC |
33-45 |
Medium |
com.gh.mygreen.xlsmapper.cellconverter.impl.IntegerCellConverterFactory$IntegerCellConverter
| Bug |
Category |
Details |
Line |
Priority |
| com.gh.mygreen.xlsmapper.cellconverter.impl.IntegerCellConverterFactory$IntegerCellConverter は，static 内部クラスにすべきです。 |
PERFORMANCE |
SIC_INNER_SHOULD_BE_STATIC |
36-48 |
Medium |
com.gh.mygreen.xlsmapper.cellconverter.impl.ListCellConverterFactory$ListCellConverter
| Bug |
Category |
Details |
Line |
Priority |
| com.gh.mygreen.xlsmapper.cellconverter.impl.ListCellConverterFactory$ListCellConverter は，static 内部クラスにすべきです。 |
PERFORMANCE |
SIC_INNER_SHOULD_BE_STATIC |
74-103 |
Medium |
com.gh.mygreen.xlsmapper.cellconverter.impl.LocalDateCellConverterFactory$LocalDateCellConverter
| Bug |
Category |
Details |
Line |
Priority |
| com.gh.mygreen.xlsmapper.cellconverter.impl.LocalDateCellConverterFactory$LocalDateCellConverter は，static 内部クラスにすべきです。 |
PERFORMANCE |
SIC_INNER_SHOULD_BE_STATIC |
49-65 |
Medium |
com.gh.mygreen.xlsmapper.cellconverter.impl.LocalDateTimeCellConverterFactory$LocalDateTimeCellConverter
| Bug |
Category |
Details |
Line |
Priority |
| com.gh.mygreen.xlsmapper.cellconverter.impl.LocalDateTimeCellConverterFactory$LocalDateTimeCellConverter は，static 内部クラスにすべきです。 |
PERFORMANCE |
SIC_INNER_SHOULD_BE_STATIC |
48-63 |
Medium |
com.gh.mygreen.xlsmapper.cellconverter.impl.LocalTimeCellConverterFactory$LocalTimeCellConverter
| Bug |
Category |
Details |
Line |
Priority |
| com.gh.mygreen.xlsmapper.cellconverter.impl.LocalTimeCellConverterFactory$LocalTimeCellConverter は，static 内部クラスにすべきです。 |
PERFORMANCE |
SIC_INNER_SHOULD_BE_STATIC |
50-74 |
Medium |
com.gh.mygreen.xlsmapper.cellconverter.impl.LongCellConverterFactory$LongCellConverter
| Bug |
Category |
Details |
Line |
Priority |
| com.gh.mygreen.xlsmapper.cellconverter.impl.LongCellConverterFactory$LongCellConverter は，static 内部クラスにすべきです。 |
PERFORMANCE |
SIC_INNER_SHOULD_BE_STATIC |
36-48 |
Medium |
com.gh.mygreen.xlsmapper.cellconverter.impl.SetCellConverterFactory$SetCellConverter
| Bug |
Category |
Details |
Line |
Priority |
| com.gh.mygreen.xlsmapper.cellconverter.impl.SetCellConverterFactory$SetCellConverter は，static 内部クラスにすべきです。 |
PERFORMANCE |
SIC_INNER_SHOULD_BE_STATIC |
74-102 |
Medium |
com.gh.mygreen.xlsmapper.cellconverter.impl.ShortCellConverterFactory$ShortCellConverter
| Bug |
Category |
Details |
Line |
Priority |
| com.gh.mygreen.xlsmapper.cellconverter.impl.ShortCellConverterFactory$ShortCellConverter は，static 内部クラスにすべきです。 |
PERFORMANCE |
SIC_INNER_SHOULD_BE_STATIC |
36-48 |
Medium |
com.gh.mygreen.xlsmapper.cellconverter.impl.SqlDateCellConverterFactory$SqlDateCellConverter
| Bug |
Category |
Details |
Line |
Priority |
| com.gh.mygreen.xlsmapper.cellconverter.impl.SqlDateCellConverterFactory$SqlDateCellConverter は，static 内部クラスにすべきです。 |
PERFORMANCE |
SIC_INNER_SHOULD_BE_STATIC |
51-63 |
Medium |
com.gh.mygreen.xlsmapper.cellconverter.impl.SqlTimeCellConverterFactory$SqlTimeCellConverter
| Bug |
Category |
Details |
Line |
Priority |
| com.gh.mygreen.xlsmapper.cellconverter.impl.SqlTimeCellConverterFactory$SqlTimeCellConverter は，static 内部クラスにすべきです。 |
PERFORMANCE |
SIC_INNER_SHOULD_BE_STATIC |
59-71 |
Medium |
com.gh.mygreen.xlsmapper.cellconverter.impl.SqlTimestampCellConverterFactory$SqlTimestampCellConverter
| Bug |
Category |
Details |
Line |
Priority |
| com.gh.mygreen.xlsmapper.cellconverter.impl.SqlTimestampCellConverterFactory$SqlTimestampCellConverter は，static 内部クラスにすべきです。 |
PERFORMANCE |
SIC_INNER_SHOULD_BE_STATIC |
50-62 |
Medium |
com.gh.mygreen.xlsmapper.cellconverter.impl.StringCellConverterFactory$StringCellConverter
| Bug |
Category |
Details |
Line |
Priority |
| com.gh.mygreen.xlsmapper.cellconverter.impl.StringCellConverterFactory$StringCellConverter は，static 内部クラスにすべきです。 |
PERFORMANCE |
SIC_INNER_SHOULD_BE_STATIC |
66-87 |
Medium |
com.gh.mygreen.xlsmapper.cellconverter.impl.URICellConverterFactory$URICellConverter
| Bug |
Category |
Details |
Line |
Priority |
| com.gh.mygreen.xlsmapper.cellconverter.impl.URICellConverterFactory$URICellConverter は，static 内部クラスにすべきです。 |
PERFORMANCE |
SIC_INNER_SHOULD_BE_STATIC |
70-133 |
Medium |
com.gh.mygreen.xlsmapper.fieldaccessor.FieldAccessor
| Bug |
Category |
Details |
Line |
Priority |
| com.gh.mygreen.xlsmapper.fieldaccessor.FieldAccessor は，equals メソッドを定義して，Object.hashCode() を使用しています。 |
BAD_PRACTICE |
HE_EQUALS_USE_HASHCODE |
148-163 |
High |
com.gh.mygreen.xlsmapper.fieldaccessor.FieldAccessorProxy
| Bug |
Category |
Details |
Line |
Priority |
| com.gh.mygreen.xlsmapper.fieldaccessor.FieldAccessorProxy は，equals メソッドを定義して，Object.hashCode() を使用しています。 |
BAD_PRACTICE |
HE_EQUALS_USE_HASHCODE |
53-73 |
High |
com.gh.mygreen.xlsmapper.fieldaccessor.FieldAccessorProxyComparator
| Bug |
Category |
Details |
Line |
Priority |
| com.gh.mygreen.xlsmapper.fieldaccessor.FieldAccessorProxyComparator は，Comparator を実装していますが，Serializable を実装していません。 |
BAD_PRACTICE |
SE_COMPARATOR_SHOULD_BE_SERIALIZABLE |
21-54 |
Medium |
com.gh.mygreen.xlsmapper.fieldprocessor.impl.HorizontalRecordsProcessor
| Bug |
Category |
Details |
Line |
Priority |
| newMergedRegion への無効な代入です。com.gh.mygreen.xlsmapper.fieldprocessor.impl.HorizontalRecordsProcessor.copyCellStyle(Cell, Cell) |
STYLE |
DLS_DEAD_LOCAL_STORE |
1679 |
Medium |
| null ではないことがわかっている値 result の冗長な null チェックがあります。com.gh.mygreen.xlsmapper.fieldprocessor.impl.HorizontalRecordsProcessor.loadArrayColumns(Sheet, List, List, CellPosition, Class, Object, Configuration, LoadingWorkObject) |
STYLE |
RCN_REDUNDANT_NULLCHECK_OF_NONNULL_VALUE |
670 |
Medium |
com.gh.mygreen.xlsmapper.fieldprocessor.impl.VerticalRecordsProcessor
| Bug |
Category |
Details |
Line |
Priority |
| newMergedRegion への無効な代入です。com.gh.mygreen.xlsmapper.fieldprocessor.impl.VerticalRecordsProcessor.copyCellStyle(Cell, Cell) |
STYLE |
DLS_DEAD_LOCAL_STORE |
1652 |
Medium |
| null ではないことがわかっている値 result の冗長な null チェックがあります。com.gh.mygreen.xlsmapper.fieldprocessor.impl.VerticalRecordsProcessor.loadArrayColumns(Sheet, List, List, CellPosition, Class, Object, Configuration, LoadingWorkObject) |
STYLE |
RCN_REDUNDANT_NULLCHECK_OF_NONNULL_VALUE |
678 |
Medium |
com.gh.mygreen.xlsmapper.util.CellPosition
| Bug |
Category |
Details |
Line |
Priority |
| com.gh.mygreen.xlsmapper.util.CellPosition.clone() は，super.clone() を呼び出していません。 |
BAD_PRACTICE |
CN_IDIOM_NO_SUPER_CALL |
188 |
Medium |
com.gh.mygreen.xlsmapper.util.ClassUtils
| Bug |
Category |
Details |
Line |
Priority |
| 例外がスローされないのに例外をキャッチしています。com.gh.mygreen.xlsmapper.util.ClassUtils.getAnnotationAttribute(Annotation, String, Class) |
STYLE |
REC_CATCH_EXCEPTION |
276 |
Medium |
com.gh.mygreen.xlsmapper.util.Utils
| Bug |
Category |
Details |
Line |
Priority |
| Collection から抽象クラス java.util.List への疑わしいキャストです。com.gh.mygreen.xlsmapper.util.Utils.convertCollectionToList(Collection) |
STYLE |
BC_BAD_CAST_TO_ABSTRACT_COLLECTION |
600 |
High |
com.gh.mygreen.xlsmapper.validation.FieldErrorBuilder
| Bug |
Category |
Details |
Line |
Priority |
| new com.gh.mygreen.xlsmapper.validation.FieldErrorBuilder(String, String, String[]) は，FieldErrorBuilder.codes の中に外部の可変オブジェクトを格納することによって内部表現を暴露するかもしれません。 |
MALICIOUS_CODE |
EI_EXPOSE_REP2 |
47 |
Medium |
com.gh.mygreen.xlsmapper.validation.ObjectError
| Bug |
Category |
Details |
Line |
Priority |
| com.gh.mygreen.xlsmapper.validation.ObjectError.getCodes() は，ObjectError.codes を返すことによって内部表現を暴露するかもしれません。 |
MALICIOUS_CODE |
EI_EXPOSE_REP |
78 |
Medium |
| new com.gh.mygreen.xlsmapper.validation.ObjectError(String, String[], Map) は，ObjectError.codes の中に外部の可変オブジェクトを格納することによって内部表現を暴露するかもしれません。 |
MALICIOUS_CODE |
EI_EXPOSE_REP2 |
61 |
Medium |
com.gh.mygreen.xlsmapper.validation.ObjectErrorBuilder
| Bug |
Category |
Details |
Line |
Priority |
| new com.gh.mygreen.xlsmapper.validation.ObjectErrorBuilder(String, String[]) は，ObjectErrorBuilder.codes の中に外部の可変オブジェクトを格納することによって内部表現を暴露するかもしれません。 |
MALICIOUS_CODE |
EI_EXPOSE_REP2 |
35 |
Medium |
com.gh.mygreen.xlsmapper.xml.XmlIO
| Bug |
Category |
Details |
Line |
Priority |
| java.io.File.mkdirs() の例外的戻り値を無視しています。com.gh.mygreen.xlsmapper.xml.XmlIO.save(AnnotationMappingInfo, File, String) |
BAD_PRACTICE |
RV_RETURN_VALUE_IGNORED_BAD_PRACTICE |
172 |
Medium |
