The following document contains the results of FindBugs Report
FindBugs Version is 1.3.8
Threshold is Low
Effort is Default
| Classes | Bugs | Errors | Missing Classes |
|---|---|---|---|
| 60 | 18 | 0 | 0 |
| Bug | Category | Details | Line | Priority |
|---|---|---|---|---|
| Class sk.barti.diplomovka.amt.domain.Agent defines non-transient non-serializable instance field behaviors | BAD_PRACTICE | SE_BAD_FIELD | Not available | Low |
| Class sk.barti.diplomovka.amt.domain.Agent defines non-transient non-serializable instance field scheduledChanges | BAD_PRACTICE | SE_BAD_FIELD | Not available | Low |
| sk.barti.diplomovka.amt.domain.Agent is Serializable; consider declaring a serialVersionUID | BAD_PRACTICE | SE_NO_SERIALVERSIONID | 20-107 | Low |
| Bug | Category | Details | Line | Priority |
|---|---|---|---|---|
| sk.barti.diplomovka.amt.domain.Behavior is Serializable; consider declaring a serialVersionUID | BAD_PRACTICE | SE_NO_SERIALVERSIONID | 23-117 | Low |
| Bug | Category | Details | Line | Priority |
|---|---|---|---|---|
| sk.barti.diplomovka.amt.domain.ScheduledDeployments.getInitialized() may expose internal representation by returning ScheduledDeployments.initialized | MALICIOUS_CODE | EI_EXPOSE_REP | 64 | Medium |
| new sk.barti.diplomovka.amt.domain.ScheduledDeployments(int, int, Date, String, String, User, Agent) may expose internal representation by storing an externally mutable object into ScheduledDeployments.initialized | MALICIOUS_CODE | EI_EXPOSE_REP2 | 35 | Medium |
| new sk.barti.diplomovka.amt.domain.ScheduledDeployments(int, Date, User, Agent) may expose internal representation by storing an externally mutable object into ScheduledDeployments.initialized | MALICIOUS_CODE | EI_EXPOSE_REP2 | 28 | Medium |
| sk.barti.diplomovka.amt.domain.ScheduledDeployments.setInitialized(Date) may expose internal representation by storing an externally mutable object into ScheduledDeployments.initialized | MALICIOUS_CODE | EI_EXPOSE_REP2 | 68 | Medium |
| sk.barti.diplomovka.amt.domain.ScheduledDeployments is Serializable; consider declaring a serialVersionUID | BAD_PRACTICE | SE_NO_SERIALVERSIONID | 22-97 | Low |
| Bug | Category | Details | Line | Priority |
|---|---|---|---|---|
| Class sk.barti.diplomovka.amt.domain.User defines non-transient non-serializable instance field createdAgents | BAD_PRACTICE | SE_BAD_FIELD | Not available | Low |
| Class sk.barti.diplomovka.amt.domain.User defines non-transient non-serializable instance field modifiedBehaviors | BAD_PRACTICE | SE_BAD_FIELD | Not available | Low |
| Class sk.barti.diplomovka.amt.domain.User defines non-transient non-serializable instance field scheduledChanges | BAD_PRACTICE | SE_BAD_FIELD | Not available | Low |
| sk.barti.diplomovka.amt.domain.User is Serializable; consider declaring a serialVersionUID | BAD_PRACTICE | SE_NO_SERIALVERSIONID | 17-78 | Low |
| Bug | Category | Details | Line | Priority |
|---|---|---|---|---|
| Primitive value is boxed then unboxed to perform primitive coercion in sk.barti.diplomovka.amt.vo.BehaviorVO.hashCode() | PERFORMANCE | BX_BOXING_IMMEDIATELY_UNBOXED_TO_PERFORM_COERCION | 194 | Medium |
| Method sk.barti.diplomovka.amt.vo.BehaviorVO.hashCode() invokes inefficient new Long(long) constructor; use Long.valueOf(long) instead | PERFORMANCE | DM_NUMBER_CTOR | 194 | Medium |
| Redundant nullcheck of obj, which is known to be non-null in sk.barti.diplomovka.amt.vo.BehaviorVO.equals(Object) | STYLE | RCN_REDUNDANT_NULLCHECK_OF_NONNULL_VALUE | 183 | Medium |
| Bug | Category | Details | Line | Priority |
|---|---|---|---|---|
| sk.barti.diplomovka.amt.vo.ScheduledDeploymentsVO.getInitialized() may expose internal representation by returning ScheduledDeploymentsVO.initialized | MALICIOUS_CODE | EI_EXPOSE_REP | 86 | Medium |
| sk.barti.diplomovka.amt.vo.ScheduledDeploymentsVO.setInitialized(Date) may expose internal representation by storing an externally mutable object into ScheduledDeploymentsVO.initialized | MALICIOUS_CODE | EI_EXPOSE_REP2 | 89 | Medium |