PMD 4.2.5 Report
2012-05-05 - 00:01:23

Summary

FilesTotalPriority 1Priority 2Priority 3Priority 4Priority 5
6593
16
5
61
11
0

org.crosswire.jsword.versification.BibleInfo

PrioLineMethodDescription
3
getBookName Avoid catching NullPointerException; consider removing the cause of the NPE.
3
getPreferredBookName Avoid catching NullPointerException; consider removing the cause of the NPE.
3
getLongBookName Avoid catching NullPointerException; consider removing the cause of the NPE.
3
getShortBookName Avoid catching NullPointerException; consider removing the cause of the NPE.
3
getOSISName Avoid catching NullPointerException; consider removing the cause of the NPE.
Total number of violations for this class: 5

org.crosswire.jsword.bridge.BookInstaller

PrioLineMethodDescription
3
main Avoid printStackTrace(); use a logger call instead.
3
main Avoid printStackTrace(); use a logger call instead.
3
main Avoid printStackTrace(); use a logger call instead.
3
main Avoid printStackTrace(); use a logger call instead.
Total number of violations for this class: 4

org.crosswire.bibledesktop.display.basic.ActiveURITip

PrioLineMethodDescription
3
updateText Avoid printStackTrace(); use a logger call instead.
3
updateText Avoid printStackTrace(); use a logger call instead.
3
updateText Avoid printStackTrace(); use a logger call instead.
Total number of violations for this class: 3

org.crosswire.bibledesktop.display.basic.URITipHelper

PrioLineMethodDescription
3
retrieve Avoid printStackTrace(); use a logger call instead.
3
retrieve Avoid printStackTrace(); use a logger call instead.
3
retrieve Avoid printStackTrace(); use a logger call instead.
Total number of violations for this class: 3

org.crosswire.bibledesktop.display.basic.URITipMgr

PrioLineMethodDescription
3
updateText Avoid printStackTrace(); use a logger call instead.
3
updateText Avoid printStackTrace(); use a logger call instead.
3
updateText Avoid printStackTrace(); use a logger call instead.
Total number of violations for this class: 3

org.crosswire.common.util.Logger

PrioLineMethodDescription
3
doLogging This statement may have some unnecessary parentheses
2
 Use of modifier volatile is not recommended.
2
 Use of modifier volatile is not recommended.
Total number of violations for this class: 3

org.crosswire.jsword.examples.APIExamples

PrioLineMethodDescription
3
installBook Avoid printStackTrace(); use a logger call instead.
3
installBook Avoid printStackTrace(); use a logger call instead.
3
installBook Avoid printStackTrace(); use a logger call instead.
Total number of violations for this class: 3

org.crosswire.bibledesktop.desktop.XSLTProperty

PrioLineMethodDescription
4
 A getX() method which returns a boolean should be named isX()
4
 A getX() method which returns a boolean should be named isX()
Total number of violations for this class: 2

org.crosswire.common.swing.NudgeButton

PrioLineMethodDescription
4
 A getX() method which returns a boolean should be named isX()
4
 A getX() method which returns a boolean should be named isX()
Total number of violations for this class: 2

org.crosswire.common.diff.Commonality

PrioLineMethodDescription
1
 A class which only has private constructors should be final
3
halfMatch Avoid assignments in operands
Total number of violations for this class: 2

org.crosswire.common.diff.DiffCleanup

PrioLineMethodDescription
3
cleanupSemantic Avoid empty while statements
3
cleanupEfficiency Avoid empty while statements
Total number of violations for this class: 2

org.crosswire.common.icu.DateFormatter

PrioLineMethodDescription
1
 A class which only has private constructors should be final
3
parse An instanceof check is being performed on the caught exception. Create a separate catch clause for this exception type.
Total number of violations for this class: 2

org.crosswire.common.util.NetUtil

PrioLineMethodDescription
3
makeDirectory These nested if statements could be combined
3
getAsFile Do not add empty strings
Total number of violations for this class: 2

org.crosswire.jsword.book.sword.RawLDBackend

PrioLineMethodDescription
3
getRawText Consider simply returning the value vs storing it in local variable 'result'
3
get Avoid empty catch blocks
Total number of violations for this class: 2

org.crosswire.jsword.bridge.BookIndexer

PrioLineMethodDescription
3
main Avoid printStackTrace(); use a logger call instead.
3
main Avoid printStackTrace(); use a logger call instead.
Total number of violations for this class: 2

org.crosswire.jsword.bridge.BookLookup

PrioLineMethodDescription
3
main Avoid printStackTrace(); use a logger call instead.
3
main Avoid printStackTrace(); use a logger call instead.
Total number of violations for this class: 2

org.crosswire.jsword.index.lucene.analysis.AnalyzerFactory

PrioLineMethodDescription
4
 A getX() method which returns a boolean should be named isX()
4
 A getX() method which returns a boolean should be named isX()
Total number of violations for this class: 2

org.crosswire.jsword.versification.Versification

PrioLineMethodDescription
3
getLastChapter Avoid catching NullPointerException; consider removing the cause of the NPE.
3
getLastVerse Avoid catching NullPointerException; consider removing the cause of the NPE.
Total number of violations for this class: 2

org.crosswire.bibledesktop.BDMsg

PrioLineMethodDescription
3
 All methods are static. Consider using Singleton instead. Alternatively, you could add a private constructor or make the class abstract to silence this warning.
Total number of violations for this class: 1

org.crosswire.bibledesktop.book.BibleComboBoxModel

PrioLineMethodDescription
1
getElementAt A switch with less than 3 branches is inefficient, use a if statement instead.
Total number of violations for this class: 1

org.crosswire.bibledesktop.book.install.BookFont

PrioLineMethodDescription
1
 A class which only has private constructors should be final
Total number of violations for this class: 1

org.crosswire.bibledesktop.book.install.BookIcon

PrioLineMethodDescription
1
 A class which only has private constructors should be final
Total number of violations for this class: 1

org.crosswire.bibledesktop.book.install.IndexResolver

PrioLineMethodDescription
3
scheduleIndex Avoid appending characters as strings in StringBuffer.append.
Total number of violations for this class: 1

org.crosswire.bibledesktop.desktop.Desktop

PrioLineMethodDescription
2
 Use of modifier volatile is not recommended.
Total number of violations for this class: 1

org.crosswire.bibledesktop.display.basic.TabbedBookDataDisplay

PrioLineMethodDescription
3
getBooks Returning 'books' may expose an internal array.
Total number of violations for this class: 1

org.crosswire.bibledesktop.display.basic.TextPaneBookDataDisplay

PrioLineMethodDescription
3
setBookData The user-supplied array 'books' is stored directly.
Total number of violations for this class: 1

org.crosswire.bibledesktop.passage.KeyTreeNode

PrioLineMethodDescription
4
 A getX() method which returns a boolean should be named isX()
Total number of violations for this class: 1

org.crosswire.bibledesktop.passage.WholeBibleTreeNode

PrioLineMethodDescription
4
 A getX() method which returns a boolean should be named isX()
Total number of violations for this class: 1

org.crosswire.common.swing.CWLabel

PrioLineMethodDescription
1
 A class which only has private constructors should be final
Total number of violations for this class: 1

org.crosswire.common.swing.CWMsg

PrioLineMethodDescription
3
 All methods are static. Consider using Singleton instead. Alternatively, you could add a private constructor or make the class abstract to silence this warning.
Total number of violations for this class: 1

org.crosswire.common.swing.CWOptionPane

PrioLineMethodDescription
3
styleFromMessageType A switch statement does not contain a break
Total number of violations for this class: 1

org.crosswire.common.swing.CWOtherMsg

PrioLineMethodDescription
3
 All methods are static. Consider using Singleton instead. Alternatively, you could add a private constructor or make the class abstract to silence this warning.
Total number of violations for this class: 1

org.crosswire.common.swing.desktop.LayoutPersistence

PrioLineMethodDescription
1
 A class which only has private constructors should be final
Total number of violations for this class: 1

org.crosswire.jsword.view.web.DownloadSet

PrioLineMethodDescription
1
 A class which only has private constructors should be final
Total number of violations for this class: 1

org.crosswire.jsword.view.web.Msg

PrioLineMethodDescription
3
 All methods are static. Consider using Singleton instead. Alternatively, you could add a private constructor or make the class abstract to silence this warning.
Total number of violations for this class: 1

gnu.gpl.GPLMsg

PrioLineMethodDescription
3
 All methods are static. Consider using Singleton instead. Alternatively, you could add a private constructor or make the class abstract to silence this warning.
Total number of violations for this class: 1

gnu.lgpl.LGPLMsg

PrioLineMethodDescription
3
 All methods are static. Consider using Singleton instead. Alternatively, you could add a private constructor or make the class abstract to silence this warning.
Total number of violations for this class: 1

org.crosswire.common.config.AbstractReflectedChoice

PrioLineMethodDescription
3
setString Avoid catching NullPointerException; consider removing the cause of the NPE.
Total number of violations for this class: 1

org.crosswire.common.diff.Distance

PrioLineMethodDescription
1
 A class which only has private constructors should be final
Total number of violations for this class: 1

org.crosswire.common.util.CWClassLoader

PrioLineMethodDescription
3
adjustPackageSearch Prefer StringBuffer over += for concatenating strings
Total number of violations for this class: 1

org.crosswire.common.util.CallContext

PrioLineMethodDescription
2
 Use of modifier volatile is not recommended.
Total number of violations for this class: 1

org.crosswire.common.util.Countries

PrioLineMethodDescription
1
 A class which only has private constructors should be final
Total number of violations for this class: 1

org.crosswire.common.util.IOUtil

PrioLineMethodDescription
3
unpackZip These nested if statements could be combined
Total number of violations for this class: 1

org.crosswire.common.util.Languages

PrioLineMethodDescription
1
 A class which only has private constructors should be final
Total number of violations for this class: 1

org.crosswire.common.util.Translations

PrioLineMethodDescription
1
 A class which only has private constructors should be final
Total number of violations for this class: 1

org.crosswire.common.xml.TransformingSAXEventProvider

PrioLineMethodDescription
3
getTemplateInfo These nested if statements could be combined
Total number of violations for this class: 1

org.crosswire.common.xml.XMLFeature

PrioLineMethodDescription
4
 A getX() method which returns a boolean should be named isX()
Total number of violations for this class: 1

org.crosswire.common.xml.XalanProcess

PrioLineMethodDescription
1
 A class which only has private constructors should be final
Total number of violations for this class: 1

org.crosswire.jsword.JSMsg

PrioLineMethodDescription
3
 All methods are static. Consider using Singleton instead. Alternatively, you could add a private constructor or make the class abstract to silence this warning.
Total number of violations for this class: 1

org.crosswire.jsword.JSOtherMsg

PrioLineMethodDescription
3
 All methods are static. Consider using Singleton instead. Alternatively, you could add a private constructor or make the class abstract to silence this warning.
Total number of violations for this class: 1

org.crosswire.jsword.book.BookData

PrioLineMethodDescription
3
getBooks Returning 'books' may expose an internal array.
Total number of violations for this class: 1

org.crosswire.jsword.book.OSISUtil

PrioLineMethodDescription
3
rtfToOsis This call to String.startsWith can be rewritten using String.charAt(0)
Total number of violations for this class: 1

org.crosswire.jsword.book.filter.FilterFactory

PrioLineMethodDescription
2
 Use of modifier volatile is not recommended.
Total number of violations for this class: 1

org.crosswire.jsword.book.readings.ReadingsBookDriver

PrioLineMethodDescription
3
getBooks Returning 'books' may expose an internal array.
Total number of violations for this class: 1

org.crosswire.jsword.book.sword.ConfigEntryTable

PrioLineMethodDescription
3
loadContents Prefer StringBuffer over += for concatenating strings
Total number of violations for this class: 1

org.crosswire.jsword.book.sword.RawFileBackend

PrioLineMethodDescription
3
createDataTextFile Prefer StringBuffer over += for concatenating strings
Total number of violations for this class: 1

org.crosswire.jsword.bridge.BookExporter

PrioLineMethodDescription
3
main Avoid printStackTrace(); use a logger call instead.
Total number of violations for this class: 1

org.crosswire.jsword.bridge.DwrBridge

PrioLineMethodDescription
3
getOSISString Avoid empty catch blocks
Total number of violations for this class: 1

org.crosswire.jsword.index.lucene.IndexMetadata

PrioLineMethodDescription
1
 A class which only has private constructors should be final
Total number of violations for this class: 1

org.crosswire.jsword.index.lucene.LuceneIndex

PrioLineMethodDescription
3
 These nested if statements could be combined
Total number of violations for this class: 1

org.crosswire.jsword.index.lucene.analysis.AbstractBookAnalyzer

PrioLineMethodDescription
4
 A getX() method which returns a boolean should be named isX()
Total number of violations for this class: 1

org.crosswire.jsword.passage.PassageUtil

PrioLineMethodDescription
4
 A getX() method which returns a boolean should be named isX()
Total number of violations for this class: 1

org.crosswire.jsword.util.WebWarning

PrioLineMethodDescription
1
 A class which only has private constructors should be final
Total number of violations for this class: 1

org.crosswire.jsword.versification.BibleBook

PrioLineMethodDescription
3
getBooks Returning 'books' may expose an internal array.
Total number of violations for this class: 1

org.crosswire.jsword.versification.system.Versifications

PrioLineMethodDescription
1
 A class which only has private constructors should be final
Total number of violations for this class: 1

Generated by PMD 4.2.5 on 2012-05-05 - 00:01:23.