The following document contains the results of SpotBugs
SpotBugs Version is 3.1.12
Threshold is medium
Effort is default
| Bug | Category | Details | Line | Priority |
|---|---|---|---|---|
| ejava.examples.orm.map.annotated.ManyManyInverseEntity.toString() makes inefficient use of keySet iterator instead of entrySet iterator | PERFORMANCE | WMI_WRONG_MAP_ITERATOR | 41 | Medium |
| Bug | Category | Details | Line | Priority |
|---|---|---|---|---|
| ejava.examples.orm.map.annotated.OneManyInverseParent.toString() makes inefficient use of keySet iterator instead of entrySet iterator | PERFORMANCE | WMI_WRONG_MAP_ITERATOR | 42 | Medium |
| Bug | Category | Details | Line | Priority |
|---|---|---|---|---|
| ejava.examples.orm.rel.MediaCopyPK.equals(Object) does not check for null argument | BAD_PRACTICE | NP_EQUALS_SHOULD_HANDLE_NULL_ARGUMENT | 34-39 | Medium |
| Bug | Category | Details | Line | Priority |
|---|---|---|---|---|
| ejava.examples.orm.rel.MediaCopyPK2.equals(Object) does not check for null argument | BAD_PRACTICE | NP_EQUALS_SHOULD_HANDLE_NULL_ARGUMENT | 30-35 | Medium |
| Bug | Category | Details | Line | Priority |
|---|---|---|---|---|
| ejava.examples.orm.rel.annotated.Borrower.getEndDate() may expose internal representation by returning Borrower.endDate | MALICIOUS_CODE | EI_EXPOSE_REP | 59 | Medium |
| ejava.examples.orm.rel.annotated.Borrower.getStartDate() may expose internal representation by returning Borrower.startDate | MALICIOUS_CODE | EI_EXPOSE_REP | 64 | Medium |
| ejava.examples.orm.rel.annotated.Borrower.setEndDate(Date) may expose internal representation by storing an externally mutable object into Borrower.endDate | MALICIOUS_CODE | EI_EXPOSE_REP2 | 61 | Medium |
| ejava.examples.orm.rel.annotated.Borrower.setStartDate(Date) may expose internal representation by storing an externally mutable object into Borrower.startDate | MALICIOUS_CODE | EI_EXPOSE_REP2 | 66 | Medium |
| Bug | Category | Details | Line | Priority |
|---|---|---|---|---|
| ejava.examples.orm.rel.annotated.Checkout.getReturnDate() may expose internal representation by returning Checkout.returnDate | MALICIOUS_CODE | EI_EXPOSE_REP | 47 | Medium |
| new ejava.examples.orm.rel.annotated.Checkout(Date) may expose internal representation by storing an externally mutable object into Checkout.outDate | MALICIOUS_CODE | EI_EXPOSE_REP2 | 32 | Medium |
| ejava.examples.orm.rel.annotated.Checkout.setReturnDate(Date) may expose internal representation by storing an externally mutable object into Checkout.returnDate | MALICIOUS_CODE | EI_EXPOSE_REP2 | 49 | Medium |
| Bug | Category | Details | Line | Priority |
|---|---|---|---|---|
| ejava.examples.orm.rel.annotated.Library.toString() makes inefficient use of keySet iterator instead of entrySet iterator | PERFORMANCE | WMI_WRONG_MAP_ITERATOR | 42 | Medium |
| Bug | Category | Details | Line | Priority |
|---|---|---|---|---|
| Dead store to s in ejava.examples.orm.rel.annotated.Photo.toString() | STYLE | DLS_DEAD_LOCAL_STORE | 51 | Medium |
| Primitive boxed just to call toString in ejava.examples.orm.rel.annotated.Photo.toString() | PERFORMANCE | DM_BOXED_PRIMITIVE_TOSTRING | 49 | Medium |
| ejava.examples.orm.rel.annotated.Photo.toString() invokes inefficient new Long(long) constructor; use Long.valueOf(long) instead | PERFORMANCE | DM_NUMBER_CTOR | 49 | Medium |
| ejava.examples.orm.rel.annotated.Photo.getImage() may expose internal representation by returning Photo.image | MALICIOUS_CODE | EI_EXPOSE_REP | 35 | Medium |
| new ejava.examples.orm.rel.annotated.Photo(byte[]) may expose internal representation by storing an externally mutable object into Photo.image | MALICIOUS_CODE | EI_EXPOSE_REP2 | 27 | Medium |
| ejava.examples.orm.rel.annotated.Photo.setImage(byte[]) may expose internal representation by storing an externally mutable object into Photo.image | MALICIOUS_CODE | EI_EXPOSE_REP2 | 37 | Medium |