View by Classes

Findings - Overview

info Method uses instanceof on multiple types to arbitrate logic (Findbugs)

Further info on the wiki.

This method uses the instanceof operator in a series of if/else statements to differentiate blocks of code based on type. If these types are related by inheritance, it is cleaner to just define a method in the base class, and use overridden methods in these classes.

1org.jcoderz.commons.logging.LoggerUtil
Method org.jcoderz.commons.logging.LoggerUtil.checkNestedItem(LogItem, Throwable) uses instanceof on multiple types to arbitrate logic (test code) [107]
1org.jcoderz.phoenix.dbview.DbView
Method org.jcoderz.phoenix.dbview.DbView.objectFormater(Object, String, String) uses instanceof on multiple types to arbitrate logic [669]