PMD Results

The following document contains the results of PMD

Files

org/paneris/jammyjoes/mvp/AffiliateTotals.java

ViolationLine
Each class should declare at least one constructor3 - 26

org/paneris/jammyjoes/mvp/CollectOverTheTopCommand.java

ViolationLine
Use explicit scoping instead of the default package private level10
Use explicit scoping instead of the default package private level11

org/paneris/jammyjoes/mvp/EmptyStringConverter.java

ViolationLine
Use explicit scoping instead of the default package private level6

org/paneris/jammyjoes/mvp/VisitableCollectionAdaptor.java

ViolationLine
Use explicit scoping instead of the default package private level9

org/paneris/jammyjoes/mvp/CollectionSorter.java

ViolationLine
Each class should declare at least one constructor8 - 15

org/paneris/jammyjoes/mvp/MonthAdaptor.java

ViolationLine
A method should have only one exit point, and that should be the last statement in the method31
A method should have only one exit point, and that should be the last statement in the method37
A method should have only one exit point, and that should be the last statement in the method51
A method should have only one exit point, and that should be the last statement in the method52
A method should have only one exit point, and that should be the last statement in the method54
A method should have only one exit point, and that should be the last statement in the method56
A method should have only one exit point, and that should be the last statement in the method59
A method should have only one exit point, and that should be the last statement in the method65
A method should have only one exit point, and that should be the last statement in the method66
A method should have only one exit point, and that should be the last statement in the method68
A method should have only one exit point, and that should be the last statement in the method70
A method should have only one exit point, and that should be the last statement in the method71
A method should have only one exit point, and that should be the last statement in the method72
A method should have only one exit point, and that should be the last statement in the method73
A method should have only one exit point, and that should be the last statement in the method74

org/paneris/jammyjoes/mvp/OrdersByMonthPresenter.java

ViolationLine
It is a good practice to call super() in a constructor17 - 26

org/paneris/jammyjoes/mvp/Command.java

ViolationLine
Avoid modifiers which are implied by the context5
Avoid modifiers which are implied by the context7

org/paneris/jammyjoes/mvp/TimesliceValue.java

ViolationLine
Use explicit scoping instead of the default package private level5
Use explicit scoping instead of the default package private level6

org/paneris/jammyjoes/mvp/UncheckedProduct.java

ViolationLine
Assigning an Object to null is a code smell. Consider refactoring.11
Assigning an Object to null is a code smell. Consider refactoring.12
Assigning an Object to null is a code smell. Consider refactoring.17

org/paneris/jammyjoes/mvp/ReverseCollectionSorter.java

ViolationLine
Each class should declare at least one constructor9 - 26
Each class should declare at least one constructor17 - 25

org/paneris/jammyjoes/mvp/AbstractPresenter.java

ViolationLine
Each class should declare at least one constructor4 - 11

org/paneris/jammyjoes/mvp/ProductSelectionFromContext.java

ViolationLine
Use explicit scoping instead of the default package private level11
Use explicit scoping instead of the default package private level12
Avoid empty catch blocks26 - 27

org/paneris/jammyjoes/mvp/NullConverter.java

ViolationLine
Use explicit scoping instead of the default package private level6
A method should have only one exit point, and that should be the last statement in the method13

org/paneris/jammyjoes/mvp/DayAdaptor.java

ViolationLine
A method should have only one exit point, and that should be the last statement in the method30
A method should have only one exit point, and that should be the last statement in the method36
A method should have only one exit point, and that should be the last statement in the method50
A method should have only one exit point, and that should be the last statement in the method51
A method should have only one exit point, and that should be the last statement in the method53
A method should have only one exit point, and that should be the last statement in the method55
A method should have only one exit point, and that should be the last statement in the method59
A method should have only one exit point, and that should be the last statement in the method65
A method should have only one exit point, and that should be the last statement in the method66
A method should have only one exit point, and that should be the last statement in the method68
A method should have only one exit point, and that should be the last statement in the method70
A method should have only one exit point, and that should be the last statement in the method71
A method should have only one exit point, and that should be the last statement in the method72
A method should have only one exit point, and that should be the last statement in the method73
A method should have only one exit point, and that should be the last statement in the method74
A method should have only one exit point, and that should be the last statement in the method75
A method should have only one exit point, and that should be the last statement in the method76

org/paneris/jammyjoes/mvp/AffiliateStatsPresenter.java

ViolationLine
It is a good practice to call super() in a constructor19 - 33
A method should have only one exit point, and that should be the last statement in the method41

org/paneris/jammyjoes/mvp/CatalogueEntryPresenter.java

ViolationLine
Use explicit scoping instead of the default package private level9
Use explicit scoping instead of the default package private level10
Use explicit scoping instead of the default package private level11
It is a good practice to call super() in a constructor13 - 18

org/paneris/jammyjoes/mvp/IntegerConverter.java

ViolationLine
Each class should declare at least one constructor4 - 13

org/paneris/jammyjoes/mvp/Presenter.java

ViolationLine
Avoid modifiers which are implied by the context5
Avoid modifiers which are implied by the context7
Avoid modifiers which are implied by the context9

org/paneris/jammyjoes/mvp/ContextWrapper.java

ViolationLine
Use explicit scoping instead of the default package private level7

org/paneris/jammyjoes/mail/TransportAbapter.java

ViolationLine
Perhaps 'transport' could be replaced by a local variable.9

org/paneris/jammyjoes/mail/NewsletterInterface.java

ViolationLine
Avoid modifiers which are implied by the context9
Avoid modifiers which are implied by the context11

org/paneris/jammyjoes/mail/ImageParser.java

ViolationLine
Avoid modifiers which are implied by the context5

org/paneris/jammyjoes/mail/MailFascade.java

ViolationLine
A method should have only one exit point, and that should be the last statement in the method77
A method should have only one exit point, and that should be the last statement in the method80

org/paneris/jammyjoes/mail/WriterFascade.java

ViolationLine
Use explicit scoping instead of the default package private level8
Use explicit scoping instead of the default package private level9
It is a good practice to call super() in a constructor11 - 14

org/paneris/jammyjoes/mail/HtmlImageParser.java

ViolationLine
Each class should declare at least one constructor27 - 34

org/paneris/jammyjoes/mail/Sender.java

ViolationLine
Avoid modifiers which are implied by the context10

org/paneris/jammyjoes/mail/ImageTranslator.java

ViolationLine
Perhaps 'count' could be replaced by a local variable.17
A method should have only one exit point, and that should be the last statement in the method56

org/paneris/jammyjoes/util/JammyJoesContextUtil.java

ViolationLine
Each class should declare at least one constructor15 - 114
Use explicit scoping instead of the default package private level17
A method should have only one exit point, and that should be the last statement in the method24
A method should have only one exit point, and that should be the last statement in the method32
A method should have only one exit point, and that should be the last statement in the method37
A method should have only one exit point, and that should be the last statement in the method38
A method should have only one exit point, and that should be the last statement in the method43
A method should have only one exit point, and that should be the last statement in the method44
A method should have only one exit point, and that should be the last statement in the method49
A method should have only one exit point, and that should be the last statement in the method52
A method should have only one exit point, and that should be the last statement in the method59
A method should have only one exit point, and that should be the last statement in the method66

org/paneris/jammyjoes/util/JammyJoesUtil.java

ViolationLine
Each class should declare at least one constructor18 - 122
A method should have only one exit point, and that should be the last statement in the method66
A method should have only one exit point, and that should be the last statement in the method71

org/paneris/jammyjoes/shopping/ShoppingTrolley.java

ViolationLine
Each class should declare at least one constructor78 - 549
Use explicit scoping instead of the default package private level92
Use explicit scoping instead of the default package private level93
A method should have only one exit point, and that should be the last statement in the method353
A method should have only one exit point, and that should be the last statement in the method355
A method should have only one exit point, and that should be the last statement in the method381

org/paneris/jammyjoes/shopping/ShoppingConfigException.java

ViolationLine
Use explicit scoping instead of the default package private level60
It is a good practice to call super() in a constructor67 - 69

org/paneris/jammyjoes/shopping/Trolley.java

ViolationLine
Each class should declare at least one constructor50 - 552
A method should have only one exit point, and that should be the last statement in the method105
A method should have only one exit point, and that should be the last statement in the method106
A method should have only one exit point, and that should be the last statement in the method107
A method should have only one exit point, and that should be the last statement in the method109
A method should have only one exit point, and that should be the last statement in the method111
A method should have only one exit point, and that should be the last statement in the method113
A method should have only one exit point, and that should be the last statement in the method115
A method should have only one exit point, and that should be the last statement in the method116
A method should have only one exit point, and that should be the last statement in the method117
A method should have only one exit point, and that should be the last statement in the method118
A method should have only one exit point, and that should be the last statement in the method119

org/paneris/jammyjoes/shopping/JammyJoesShoppingTrolley.java

ViolationLine
Each class should declare at least one constructor47 - 554
A method should have only one exit point, and that should be the last statement in the method79
A method should have only one exit point, and that should be the last statement in the method82
Assigning an Object to null is a code smell. Consider refactoring.126
A method should have only one exit point, and that should be the last statement in the method230
A method should have only one exit point, and that should be the last statement in the method246
A method should have only one exit point, and that should be the last statement in the method249
Assigning an Object to null is a code smell. Consider refactoring.311
A method should have only one exit point, and that should be the last statement in the method313
A method should have only one exit point, and that should be the last statement in the method325
Assigning an Object to null is a code smell. Consider refactoring.330
A method should have only one exit point, and that should be the last statement in the method332
A method should have only one exit point, and that should be the last statement in the method335
Assigning an Object to null is a code smell. Consider refactoring.341
A method should have only one exit point, and that should be the last statement in the method343
A method should have only one exit point, and that should be the last statement in the method346
A method should have only one exit point, and that should be the last statement in the method355
A method should have only one exit point, and that should be the last statement in the method543
Perhaps 'jjutil' could be replaced by a local variable.547

org/paneris/jammyjoes/shopping/ShoppingContext.java

ViolationLine
Each class should declare at least one constructor9 - 13

org/paneris/jammyjoes/shopping/ShoppingTrolleyItem.java

ViolationLine
Each class should declare at least one constructor9 - 137

org/paneris/jammyjoes/shopping/JammyJoesShoppingTrolleyItem.java

ViolationLine
Each class should declare at least one constructor9 - 88
A method should have only one exit point, and that should be the last statement in the method54
A method should have only one exit point, and that should be the last statement in the method76
A method should have only one exit point, and that should be the last statement in the method84

org/paneris/jammyjoes/model/AgeTable.java

ViolationLine
A method should have only one exit point, and that should be the last statement in the method32
A method should have only one exit point, and that should be the last statement in the method34
A method should have only one exit point, and that should be the last statement in the method38
A method should have only one exit point, and that should be the last statement in the method42

org/paneris/jammyjoes/model/SexTable.java

ViolationLine
Use explicit scoping instead of the default package private level32
Use explicit scoping instead of the default package private level33
A method should have only one exit point, and that should be the last statement in the method51

org/paneris/jammyjoes/model/Graphable.java

ViolationLine
Avoid modifiers which are implied by the context9
Avoid modifiers which are implied by the context10
Avoid modifiers which are implied by the context11

org/paneris/jammyjoes/model/Supplier.java

ViolationLine
Avoid unnecessary constructors - the compiler will generate these for you6
It is a good practice to call super() in a constructor6

org/paneris/jammyjoes/model/DeliveryCharge.java

ViolationLine
Avoid unnecessary constructors - the compiler will generate these for you6
It is a good practice to call super() in a constructor6

org/paneris/jammyjoes/model/ShopCurrencyTable.java

ViolationLine
Perhaps 'UK' could be replaced by a local variable.21
Perhaps 'cache' could be replaced by a local variable.43
Perhaps 'cacheSerial' could be replaced by a local variable.44
A method should have only one exit point, and that should be the last statement in the method52
A method should have only one exit point, and that should be the last statement in the method54
A method should have only one exit point, and that should be the last statement in the method60

org/paneris/jammyjoes/model/ShopCurrency.java

ViolationLine
Avoid unnecessary constructors - the compiler will generate these for you10
It is a good practice to call super() in a constructor10
A method should have only one exit point, and that should be the last statement in the method14
A method should have only one exit point, and that should be the last statement in the method15

org/paneris/jammyjoes/model/StockTransaction.java

ViolationLine
Avoid unnecessary constructors - the compiler will generate these for you13
It is a good practice to call super() in a constructor13

org/paneris/jammyjoes/model/StockTransactionType.java

ViolationLine
Avoid unnecessary constructors - the compiler will generate these for you6
It is a good practice to call super() in a constructor6

org/paneris/jammyjoes/model/DeliveryZone.java

ViolationLine
Avoid unnecessary constructors - the compiler will generate these for you9 - 10
It is a good practice to call super() in a constructor9 - 10
A method should have only one exit point, and that should be the last statement in the method23
A method should have only one exit point, and that should be the last statement in the method34
A method should have only one exit point, and that should be the last statement in the method46

org/paneris/jammyjoes/model/AffiliateTransaction.java

ViolationLine
Avoid unnecessary constructors - the compiler will generate these for you10
It is a good practice to call super() in a constructor10

org/paneris/jammyjoes/model/Award.java

ViolationLine
Avoid unnecessary constructors - the compiler will generate these for you6
It is a good practice to call super() in a constructor6

org/paneris/jammyjoes/model/OrderStatusTable.java

ViolationLine
Use explicit scoping instead of the default package private level14
Use explicit scoping instead of the default package private level15
Use explicit scoping instead of the default package private level16
Use explicit scoping instead of the default package private level17
A method should have only one exit point, and that should be the last statement in the method37

org/paneris/jammyjoes/model/Wrapping.java

ViolationLine
Avoid unnecessary constructors - the compiler will generate these for you6
It is a good practice to call super() in a constructor6

org/paneris/jammyjoes/model/OrderStatus.java

ViolationLine
Avoid unnecessary constructors - the compiler will generate these for you6
It is a good practice to call super() in a constructor6

org/paneris/jammyjoes/model/TypeTable.java

ViolationLine
A method should have only one exit point, and that should be the last statement in the method23
A method should have only one exit point, and that should be the last statement in the method27

org/paneris/jammyjoes/model/Affiliate.java

ViolationLine
Avoid unnecessary constructors - the compiler will generate these for you10
It is a good practice to call super() in a constructor10

org/paneris/jammyjoes/model/Sex.java

ViolationLine
Avoid unnecessary constructors - the compiler will generate these for you37
It is a good practice to call super() in a constructor37

org/paneris/jammyjoes/model/User.java

ViolationLine
Avoid unnecessary constructors - the compiler will generate these for you13
It is a good practice to call super() in a constructor13
A method should have only one exit point, and that should be the last statement in the method41

org/paneris/jammyjoes/model/NotLoggedInException.java

ViolationLine
Each class should declare at least one constructor5 - 11

org/paneris/jammyjoes/model/DeliveryCarrier.java

ViolationLine
Avoid unnecessary constructors - the compiler will generate these for you6
It is a good practice to call super() in a constructor6

org/paneris/jammyjoes/model/Product.java

ViolationLine
Avoid unnecessary constructors - the compiler will generate these for you30 - 31
It is a good practice to call super() in a constructor30 - 31
A method should have only one exit point, and that should be the last statement in the method35
A method should have only one exit point, and that should be the last statement in the method42
A method should have only one exit point, and that should be the last statement in the method49
A method should have only one exit point, and that should be the last statement in the method53
A method should have only one exit point, and that should be the last statement in the method60
A method should have only one exit point, and that should be the last statement in the method72
Assigning an Object to null is a code smell. Consider refactoring.114
Assigning an Object to null is a code smell. Consider refactoring.115
Assigning an Object to null is a code smell. Consider refactoring.116
Assigning an Object to null is a code smell. Consider refactoring.117
Assigning an Object to null is a code smell. Consider refactoring.118
Assigning an Object to null is a code smell. Consider refactoring.119
Assigning an Object to null is a code smell. Consider refactoring.120
Assigning an Object to null is a code smell. Consider refactoring.121
Assigning an Object to null is a code smell. Consider refactoring.122
It is a good practice to call super() in a constructor303 - 306
A method should have only one exit point, and that should be the last statement in the method310
A method should have only one exit point, and that should be the last statement in the method337

org/paneris/jammyjoes/model/ProductStatus.java

ViolationLine
Avoid unnecessary constructors - the compiler will generate these for you6
It is a good practice to call super() in a constructor6

org/paneris/jammyjoes/model/ShopOrderItem.java

ViolationLine
Avoid unnecessary constructors - the compiler will generate these for you9 - 10
It is a good practice to call super() in a constructor9 - 10

org/paneris/jammyjoes/model/EcentiveCategory.java

ViolationLine
Avoid unnecessary constructors - the compiler will generate these for you6
It is a good practice to call super() in a constructor6

org/paneris/jammyjoes/model/Age.java

ViolationLine
Avoid unnecessary constructors - the compiler will generate these for you6
It is a good practice to call super() in a constructor6

org/paneris/jammyjoes/model/ShopOrder.java

ViolationLine
Avoid unnecessary constructors - the compiler will generate these for you16 - 17
It is a good practice to call super() in a constructor16 - 17
A method should have only one exit point, and that should be the last statement in the method31
A method should have only one exit point, and that should be the last statement in the method38
A method should have only one exit point, and that should be the last statement in the method46
A method should have only one exit point, and that should be the last statement in the method49
A method should have only one exit point, and that should be the last statement in the method67

org/paneris/jammyjoes/model/Manufacturer.java

ViolationLine
Avoid unnecessary constructors - the compiler will generate these for you6
It is a good practice to call super() in a constructor6

org/paneris/jammyjoes/model/JammyjoesDatabase.java

ViolationLine
Each class should declare at least one constructor9 - 91

org/paneris/jammyjoes/model/Type.java

ViolationLine
Avoid unnecessary constructors - the compiler will generate these for you6
It is a good practice to call super() in a constructor6

org/paneris/jammyjoes/model/OrderType.java

ViolationLine
Avoid unnecessary constructors - the compiler will generate these for you37
It is a good practice to call super() in a constructor37

org/paneris/jammyjoes/model/ManufacturerTable.java

ViolationLine
A method should have only one exit point, and that should be the last statement in the method24
A method should have only one exit point, and that should be the last statement in the method28

org/paneris/jammyjoes/servlet/JammyjoesFileDataAdaptorFactory.java

ViolationLine
Each class should declare at least one constructor59 - 73

org/paneris/jammyjoes/servlet/JammyJoesMelatiServlet.java

ViolationLine
Each class should declare at least one constructor19 - 98
A method should have only one exit point, and that should be the last statement in the method60
A method should have only one exit point, and that should be the last statement in the method87
A method should have only one exit point, and that should be the last statement in the method90
A method should have only one exit point, and that should be the last statement in the method93

org/paneris/jammyjoes/servlet/JammyjoesFileDataAdaptor.java

ViolationLine
It is a good practice to call super() in a constructor81 - 85
It is a good practice to call super() in a constructor97 - 103

org/paneris/jammyjoes/controller/NullSearchCriteria.java

ViolationLine
Avoid unnecessary constructors - the compiler will generate these for you7 - 8

org/paneris/jammyjoes/controller/SearchMaufacturer.java

ViolationLine
Assigning an Object to null is a code smell. Consider refactoring.14
Assigning an Object to null is a code smell. Consider refactoring.20
Assigning an Object to null is a code smell. Consider refactoring.25
Assigning an Object to null is a code smell. Consider refactoring.26
A method should have only one exit point, and that should be the last statement in the method30
A method should have only one exit point, and that should be the last statement in the method40

org/paneris/jammyjoes/controller/SalesAnalysis.java

ViolationLine
Each class should declare at least one constructor32 - 264
It is a good practice to call super() in a constructor88 - 90

org/paneris/jammyjoes/controller/AffiliateHome.java

ViolationLine
Each class should declare at least one constructor25 - 88
A method should have only one exit point, and that should be the last statement in the method40
A method should have only one exit point, and that should be the last statement in the method83

org/paneris/jammyjoes/controller/Logger.java

ViolationLine
Perhaps 'whereClause' could be replaced by a local variable.11

org/paneris/jammyjoes/controller/NewsletterSend.java

ViolationLine
Each class should declare at least one constructor18 - 45

org/paneris/jammyjoes/controller/ProductReport.java

ViolationLine
Each class should declare at least one constructor27 - 202

org/paneris/jammyjoes/controller/ParameterProvider.java

ViolationLine
Use explicit scoping instead of the default package private level8
Use explicit scoping instead of the default package private level9
A method should have only one exit point, and that should be the last statement in the method19
A method should have only one exit point, and that should be the last statement in the method25
Assigning an Object to null is a code smell. Consider refactoring.32
Assigning an Object to null is a code smell. Consider refactoring.33

org/paneris/jammyjoes/controller/TemplateContextSearchCriteria.java

ViolationLine
Use explicit scoping instead of the default package private level18
A method should have only one exit point, and that should be the last statement in the method71
A method should have only one exit point, and that should be the last statement in the method171

org/paneris/jammyjoes/controller/SimilarSearchCriteria.java

ViolationLine
Use explicit scoping instead of the default package private level10

org/paneris/jammyjoes/controller/SearchCriteria.java

ViolationLine
Avoid modifiers which are implied by the context6
Avoid modifiers which are implied by the context7

org/paneris/jammyjoes/controller/SearchAge.java

ViolationLine
Assigning an Object to null is a code smell. Consider refactoring.16
Assigning an Object to null is a code smell. Consider refactoring.23
Assigning an Object to null is a code smell. Consider refactoring.24
Assigning an Object to null is a code smell. Consider refactoring.30
A method should have only one exit point, and that should be the last statement in the method38
A method should have only one exit point, and that should be the last statement in the method39
A method should have only one exit point, and that should be the last statement in the method41
A method should have only one exit point, and that should be the last statement in the method52

org/paneris/jammyjoes/controller/PurchaseOrder.java

ViolationLine
Each class should declare at least one constructor30 - 142

org/paneris/jammyjoes/controller/AffiliateSignup.java

ViolationLine
Each class should declare at least one constructor12 - 34

org/paneris/jammyjoes/controller/TitleGenerator.java

ViolationLine
A method should have only one exit point, and that should be the last statement in the method27
A method should have only one exit point, and that should be the last statement in the method30
An empty statement (semicolon) not part of a loop33

org/paneris/jammyjoes/controller/Search.java

ViolationLine
Each class should declare at least one constructor26 - 218
Perhaps 'db' could be replaced by a local variable.31
Avoid empty catch blocks83
Avoid empty catch blocks149 - 150
A method should have only one exit point, and that should be the last statement in the method184
Assigning an Object to null is a code smell. Consider refactoring.194
Avoid empty catch blocks198 - 199
A method should have only one exit point, and that should be the last statement in the method208
A method should have only one exit point, and that should be the last statement in the method214

org/paneris/jammyjoes/controller/FroogleSearchCriteria.java

ViolationLine
Perhaps 'db' could be replaced by a local variable.10

org/paneris/jammyjoes/controller/StockEntry.java

ViolationLine
Each class should declare at least one constructor22 - 120
Avoid instantiating Boolean objects; you can usually invoke Boolean.TRUE instead.78
It is a good practice to call super() in a constructor96 - 99
Assigning an Object to null is a code smell. Consider refactoring.97
Assigning an Object to null is a code smell. Consider refactoring.98
It is a good practice to call super() in a constructor101 - 104
Assigning an Object to null is a code smell. Consider refactoring.103
It is a good practice to call super() in a constructor106 - 109

org/paneris/jammyjoes/controller/Toy.java

ViolationLine
Each class should declare at least one constructor13 - 57

org/paneris/jammyjoes/controller/SupplierNameSort.java

ViolationLine
Each class should declare at least one constructor7 - 15

org/paneris/jammyjoes/controller/SearchType.java

ViolationLine
Assigning an Object to null is a code smell. Consider refactoring.14
Assigning an Object to null is a code smell. Consider refactoring.20
Assigning an Object to null is a code smell. Consider refactoring.25
Assigning an Object to null is a code smell. Consider refactoring.26
A method should have only one exit point, and that should be the last statement in the method30
A method should have only one exit point, and that should be the last statement in the method41

org/paneris/jammyjoes/controller/Login.java

ViolationLine
Each class should declare at least one constructor53 - 69
Use explicit scoping instead of the default package private level57

org/paneris/jammyjoes/controller/Page.java

ViolationLine
Each class should declare at least one constructor10 - 32
A method should have only one exit point, and that should be the last statement in the method22

org/paneris/jammyjoes/controller/OrdersByMonth.java

ViolationLine
Each class should declare at least one constructor11 - 28

org/paneris/jammyjoes/controller/Catalogue.java

ViolationLine
Each class should declare at least one constructor16 - 67

org/paneris/jammyjoes/controller/Competition.java

ViolationLine
Each class should declare at least one constructor11 - 43

org/paneris/jammyjoes/upload/Warden.java

ViolationLine
Use explicit scoping instead of the default package private level37
A method should have only one exit point, and that should be the last statement in the method149
A method should have only one exit point, and that should be the last statement in the method169
A method should have only one exit point, and that should be the last statement in the method177

org/paneris/jammyjoes/upload/Froogle.java

ViolationLine
Use explicit scoping instead of the default package private level17

org/paneris/jammyjoes/upload/AmazonExport.java

ViolationLine
Use explicit scoping instead of the default package private level17

org/paneris/jammyjoes/upload/WardenLauncher.java

ViolationLine
Each class should declare at least one constructor17 - 80
Use explicit scoping instead of the default package private level20