The following document contains the results of FindBugs Report
FindBugs Version is 2.0.3
Threshold is medium
Effort is min
Class | Bugs |
---|---|
gboat2.base.dao.Activator | 6 |
gboat2.base.dao.aspect.GboatSessionFactoryAspect | 1 |
gboat2.base.dao.impl.BaseDAO | 4 |
gboat2.base.dao.impl.GenerateQL | 4 |
Bug | Category | Details | Line | Priority |
---|---|---|---|---|
gboat2.base.dao.Activator.start(BundleContext) invokes System.exit(...), which shuts down the entire virtual machine | BAD_PRACTICE | DM_EXIT | 106 | Medium |
Inconsistent synchronization of gboat2.base.dao.Activator.baseDAO; locked 83% of time | MT_CORRECTNESS | IS2_INCONSISTENT_SYNC | 409 | Medium |
Inconsistent synchronization of gboat2.base.dao.Activator.sessionFactoryAspect; locked 66% of time | MT_CORRECTNESS | IS2_INCONSISTENT_SYNC | 88 | Medium |
Incorrect lazy initialization of static field gboat2.base.dao.Activator.sessionFactoryBeans in gboat2.base.dao.Activator.setApplicationContext(ApplicationContext) | MT_CORRECTNESS | LI_LAZY_INIT_STATIC | 77 | Medium |
Incorrect lazy initialization of static field gboat2.base.dao.Activator.gboatLoader in gboat2.base.dao.Activator.start(BundleContext) | MT_CORRECTNESS | LI_LAZY_INIT_STATIC | 101 | Medium |
Write to static field gboat2.base.dao.Activator.sessionFactoryBeans from instance method gboat2.base.dao.Activator.setApplicationContext(ApplicationContext) | STYLE | ST_WRITE_TO_STATIC_FROM_INSTANCE_METHOD | 81 | Medium |
Bug | Category | Details | Line | Priority |
---|---|---|---|---|
Dead store to clazz in gboat2.base.dao.aspect.GboatSessionFactoryAspect.dynamicSetSessionFactory(JoinPoint) | STYLE | DLS_DEAD_LOCAL_STORE | 79 | Medium |
Bug | Category | Details | Line | Priority |
---|---|---|---|---|
Boxed value is unboxed and then immediately reboxed in gboat2.base.dao.impl.BaseDAO.getValue(Field, Object) | PERFORMANCE | BX_UNBOXING_IMMEDIATELY_REBOXED | 654 | Medium |
Boxed value is unboxed and then immediately reboxed in gboat2.base.dao.impl.BaseDAO.getValue(Field, Object) | PERFORMANCE | BX_UNBOXING_IMMEDIATELY_REBOXED | 662 | Medium |
Boxed value is unboxed and then immediately reboxed in gboat2.base.dao.impl.BaseDAO.getValue(Field, Object) | PERFORMANCE | BX_UNBOXING_IMMEDIATELY_REBOXED | 666 | Medium |
gboat2.base.dao.impl.BaseDAO.convertParamsForVO(Map) makes inefficient use of keySet iterator instead of entrySet iterator | PERFORMANCE | WMI_WRONG_MAP_ITERATOR | 339 | Medium |
Bug | Category | Details | Line | Priority |
---|---|---|---|---|
Redundant nullcheck of field, which is known to be non-null in gboat2.base.dao.impl.GenerateQL.getBlockValue(String, Class) | STYLE | RCN_REDUNDANT_NULLCHECK_OF_NONNULL_VALUE | 577 | Medium |
Redundant nullcheck of classField, which is known to be non-null in gboat2.base.dao.impl.GenerateQL.getColumnName(Class, String) | STYLE | RCN_REDUNDANT_NULLCHECK_OF_NONNULL_VALUE | 684 | Medium |
Redundant nullcheck of getMethod, which is known to be non-null in gboat2.base.dao.impl.GenerateQL.getColumnName(Class, String) | STYLE | RCN_REDUNDANT_NULLCHECK_OF_NONNULL_VALUE | 671 | Medium |
Redundant nullcheck of setMethod, which is known to be non-null in gboat2.base.dao.impl.GenerateQL.getColumnName(Class, String) | STYLE | RCN_REDUNDANT_NULLCHECK_OF_NONNULL_VALUE | 705 | Medium |