Checkstyle Results
The following document contains the results of
Checkstyle.
Summary
| Files | Infos | Warnings | Errors |
| 81 | 0 | 0 | 1810 |
Files
org/neuclear/ledger/Book.java
| Error | Line |
| Missing a header - not enough lines in file. |
1
|
| First sentence should end with a period. |
26
|
| Missing a Javadoc comment. |
32
|
| Line is longer than 80 characters. |
35
|
| Missing a Javadoc comment. |
35
|
| Parameter id should be final. |
35
|
| 'id' hides a field. |
35
|
| Parameter nickname should be final. |
35
|
| 'nickname' hides a field. |
35
|
| Parameter type should be final. |
35
|
| 'type' hides a field. |
35
|
| Parameter source should be final. |
35
|
| 'source' hides a field. |
35
|
| Parameter registered should be final. |
35
|
| 'registered' hides a field. |
35
|
| Parameter updated should be final. |
35
|
| 'updated' hides a field. |
35
|
| Parameter registrationid should be final. |
35
|
| 'registrationid' hides a field. |
35
|
| Missing a Javadoc comment. |
45
|
| Parameter id should be final. |
45
|
| 'id' hides a field. |
45
|
| Parameter registered should be final. |
45
|
| 'registered' hides a field. |
45
|
| '32' is a magic number. |
48
|
| Avoid inline conditionals. |
48
|
| '4' is a magic number. |
49
|
| '4' is a magic number. |
50
|
| '8' is a magic number. |
50
|
| Method 'getId' is not designed for extension - needs to be abstract, final or empty. |
59
|
| Missing a Javadoc comment. |
59
|
| Method 'getNickname' is not designed for extension - needs to be abstract, final or empty. |
63
|
| Missing a Javadoc comment. |
63
|
| Method 'getType' is not designed for extension - needs to be abstract, final or empty. |
67
|
| Missing a Javadoc comment. |
67
|
| Method 'getSource' is not designed for extension - needs to be abstract, final or empty. |
71
|
| Missing a Javadoc comment. |
71
|
| Method 'getRegistered' is not designed for extension - needs to be abstract, final or empty. |
75
|
| Missing a Javadoc comment. |
75
|
| Method 'getUpdated' is not designed for extension - needs to be abstract, final or empty. |
79
|
| Missing a Javadoc comment. |
79
|
| Method 'getRegistrationId' is not designed for extension - needs to be abstract, final or empty. |
83
|
| Missing a Javadoc comment. |
83
|
| Method 'equals' is not designed for extension - needs to be abstract, final or empty. |
87
|
| Missing a Javadoc comment. |
87
|
| Parameter o should be final. |
87
|
| Line is longer than 80 characters. |
88
|
| Method 'hashCode' is not designed for extension - needs to be abstract, final or empty. |
91
|
| Missing a Javadoc comment. |
91
|
| Missing a Javadoc comment. |
95
|
| Variable 'id' must be private and have accessor methods. |
95
|
| Missing a Javadoc comment. |
96
|
| Variable 'nickname' must be private and have accessor methods. |
96
|
| Missing a Javadoc comment. |
97
|
| Variable 'type' must be private and have accessor methods. |
97
|
| Missing a Javadoc comment. |
98
|
| Variable 'source' must be private and have accessor methods. |
98
|
| Missing a Javadoc comment. |
99
|
| Variable 'registered' must be private and have accessor methods. |
99
|
| Missing a Javadoc comment. |
100
|
| Variable 'updated' must be private and have accessor methods. |
100
|
| Missing a Javadoc comment. |
101
|
| Variable 'registrationid' must be private and have accessor methods. |
101
|
org/neuclear/ledger/ExceededHeldAmountException.java
| Error | Line |
| Missing a header - not enough lines in file. |
1
|
| Line is longer than 80 characters. |
26
|
| First sentence should end with a period. |
30
|
| Line is longer than 80 characters. |
36
|
| Missing a Javadoc comment. |
36
|
| Line is longer than 80 characters. |
37
|
org/neuclear/ledger/HeldTransaction.java
| Error | Line |
| Missing a header - not enough lines in file. |
1
|
| First sentence should end with a period. |
5
|
| Line has trailing spaces. |
6
|
| Missing a Javadoc comment. |
12
|
| Redundant 'public' modifier. |
12
|
org/neuclear/ledger/InsufficientFundsException.java
| Error | Line |
| Missing a header - not enough lines in file. |
1
|
| Line is longer than 80 characters. |
11
|
| Missing a Javadoc comment. |
11
|
| 'book' hides a field. |
11
|
| 'amount' hides a field. |
11
|
| Line is longer than 80 characters. |
12
|
| Line is longer than 80 characters. |
17
|
| Missing a Javadoc comment. |
17
|
| 'book' hides a field. |
17
|
| 'amount' hides a field. |
17
|
| Line is longer than 80 characters. |
18
|
| Method 'getAmount' is not designed for extension - needs to be abstract, final or empty. |
23
|
| Missing a Javadoc comment. |
23
|
| Method 'getBook' is not designed for extension - needs to be abstract, final or empty. |
27
|
| Missing a Javadoc comment. |
27
|
| Missing a Javadoc comment. |
31
|
| Missing a Javadoc comment. |
32
|
org/neuclear/ledger/InvalidTransactionException.java
| Error | Line |
| Missing a header - not enough lines in file. |
1
|
| Line is longer than 80 characters. |
11
|
| Missing a Javadoc comment. |
11
|
| 'message' hides a field. |
11
|
| Missing a Javadoc comment. |
16
|
| Missing a Javadoc comment. |
20
|
org/neuclear/ledger/Ledger.java
| Error | Line |
| Missing a header - not enough lines in file. |
1
|
| First sentence should end with a period. |
26
|
| Missing a Javadoc comment. |
32
|
| Line is longer than 80 characters. |
35
|
| Missing a Javadoc comment. |
35
|
| More than 7 parameters. |
35
|
| Parameter id should be final. |
35
|
| 'id' hides a field. |
35
|
| Parameter nickname should be final. |
35
|
| 'nickname' hides a field. |
35
|
| Parameter type should be final. |
35
|
| 'type' hides a field. |
35
|
| Parameter source should be final. |
35
|
| 'source' hides a field. |
35
|
| Parameter registered should be final. |
35
|
| 'registered' hides a field. |
35
|
| Parameter updated should be final. |
35
|
| 'updated' hides a field. |
35
|
| Parameter registrationid should be final. |
35
|
| 'registrationid' hides a field. |
35
|
| Parameter unit should be final. |
35
|
| 'unit' hides a field. |
35
|
| Parameter decimal should be final. |
35
|
| 'decimal' hides a field. |
35
|
| Missing a Javadoc comment. |
47
|
| Parameter id should be final. |
47
|
| 'id' hides a field. |
47
|
| Parameter registered should be final. |
47
|
| 'registered' hides a field. |
47
|
| '32' is a magic number. |
50
|
| Avoid inline conditionals. |
50
|
| '4' is a magic number. |
51
|
| '4' is a magic number. |
52
|
| '8' is a magic number. |
52
|
| Method 'getId' is not designed for extension - needs to be abstract, final or empty. |
63
|
| Missing a Javadoc comment. |
63
|
| Method 'getNickname' is not designed for extension - needs to be abstract, final or empty. |
67
|
| Missing a Javadoc comment. |
67
|
| Method 'getType' is not designed for extension - needs to be abstract, final or empty. |
71
|
| Missing a Javadoc comment. |
71
|
| Method 'getSource' is not designed for extension - needs to be abstract, final or empty. |
75
|
| Missing a Javadoc comment. |
75
|
| Method 'getRegistered' is not designed for extension - needs to be abstract, final or empty. |
79
|
| Missing a Javadoc comment. |
79
|
| Method 'getUpdated' is not designed for extension - needs to be abstract, final or empty. |
83
|
| Missing a Javadoc comment. |
83
|
| Method 'getRegistrationId' is not designed for extension - needs to be abstract, final or empty. |
87
|
| Missing a Javadoc comment. |
87
|
| Method 'getUnit' is not designed for extension - needs to be abstract, final or empty. |
91
|
| Missing a Javadoc comment. |
91
|
| Method 'getDecimalPoint' is not designed for extension - needs to be abstract, final or empty. |
95
|
| Missing a Javadoc comment. |
95
|
| Missing a Javadoc comment. |
99
|
| Variable 'id' must be private and have accessor methods. |
99
|
| Missing a Javadoc comment. |
100
|
| Variable 'nickname' must be private and have accessor methods. |
100
|
| Missing a Javadoc comment. |
101
|
| Variable 'type' must be private and have accessor methods. |
101
|
| Missing a Javadoc comment. |
102
|
| Variable 'source' must be private and have accessor methods. |
102
|
| Missing a Javadoc comment. |
103
|
| Variable 'registered' must be private and have accessor methods. |
103
|
| Missing a Javadoc comment. |
104
|
| Variable 'updated' must be private and have accessor methods. |
104
|
| Missing a Javadoc comment. |
105
|
| Variable 'registrationid' must be private and have accessor methods. |
105
|
| Missing a Javadoc comment. |
106
|
| Variable 'unit' must be private and have accessor methods. |
106
|
| Missing a Javadoc comment. |
107
|
| Variable 'decimal' must be private and have accessor methods. |
107
|
org/neuclear/ledger/LedgerController.java
| Error | Line |
| Missing a header - not enough lines in file. |
1
|
| Line is longer than 80 characters. |
7
|
| Line is longer than 80 characters. |
8
|
| Line is longer than 80 characters. |
11
|
| Line is longer than 80 characters. |
14
|
| Line is longer than 80 characters. |
17
|
| Line is longer than 80 characters. |
18
|
| Line is longer than 80 characters. |
31
|
| Line is longer than 80 characters. |
34
|
| Line is longer than 80 characters. |
42
|
| Line is longer than 80 characters. |
45
|
| Line is longer than 80 characters. |
56
|
| Line is longer than 80 characters. |
59
|
| Line is longer than 80 characters. |
60
|
| Line is longer than 80 characters. |
61
|
| Line is longer than 80 characters. |
78
|
| Line is longer than 80 characters. |
81
|
| Line is longer than 80 characters. |
82
|
| Line is longer than 80 characters. |
90
|
| Line is longer than 80 characters. |
95
|
| Line is longer than 80 characters. |
96
|
| Line is longer than 80 characters. |
100
|
| Line is longer than 80 characters. |
101
|
| Line is longer than 80 characters. |
102
|
| Line is longer than 80 characters. |
103
|
| Line is longer than 80 characters. |
107
|
| Line is longer than 80 characters. |
112
|
| Line is longer than 80 characters. |
113
|
| Line is longer than 80 characters. |
114
|
| Line is longer than 80 characters. |
127
|
| Line is longer than 80 characters. |
156
|
| Line is longer than 80 characters. |
163
|
| Line is longer than 80 characters. |
164
|
| Line is longer than 80 characters. |
168
|
| Line is longer than 80 characters. |
169
|
| Line is longer than 80 characters. |
174
|
| Line is longer than 80 characters. |
176
|
| Line is longer than 80 characters. |
188
|
| First sentence should end with a period. |
192
|
| 'id' hides a field. |
197
|
| Expected @param tag for 'id'. |
197
|
| Missing a Javadoc comment. |
201
|
| 'id' hides a field. |
201
|
| Line is longer than 80 characters. |
205
|
| Line is longer than 80 characters. |
210
|
| Parameter trans should be final. |
210
|
| Expected @throws tag for 'UnBalancedTransactionException'. |
210
|
| Redundant throws: 'UnBalancedTransactionException' is subclass of 'InvalidTransactionException'. |
210
|
| Expected @throws tag for 'LowlevelLedgerException'. |
210
|
| Expected @throws tag for 'InvalidTransactionException'. |
210
|
| Line is longer than 80 characters. |
213
|
| Line is longer than 80 characters. |
218
|
| Parameter trans should be final. |
218
|
| Expected @throws tag for 'UnBalancedTransactionException'. |
218
|
| Redundant throws: 'UnBalancedTransactionException' is subclass of 'InvalidTransactionException'. |
218
|
| Expected @throws tag for 'LowlevelLedgerException'. |
218
|
| Expected @throws tag for 'InvalidTransactionException'. |
218
|
| Line is longer than 80 characters. |
222
|
| Line is longer than 80 characters. |
223
|
| Expected an @return tag. |
227
|
| Line is longer than 80 characters. |
227
|
| Parameter trans should be final. |
227
|
| Expected @throws tag for 'UnBalancedTransactionException'. |
227
|
| Redundant throws: 'UnBalancedTransactionException' is subclass of 'InvalidTransactionException'. |
227
|
| Expected @throws tag for 'LowlevelLedgerException'. |
227
|
| Expected @throws tag for 'InvalidTransactionException'. |
227
|
| Expected an @return tag. |
238
|
| Line is longer than 80 characters. |
238
|
| Parameter hold should be final. |
238
|
| Expected @param tag for 'hold'. |
238
|
| Expected @throws tag for 'LowlevelLedgerException'. |
238
|
| Expected @throws tag for 'UnknownTransactionException'. |
238
|
| Line is longer than 80 characters. |
242
|
| Line is longer than 80 characters. |
245
|
| Expected an @return tag. |
252
|
| Line is longer than 80 characters. |
252
|
| Parameter hold should be final. |
252
|
| Parameter amount should be final. |
252
|
| Parameter comment should be final. |
252
|
| Expected @param tag for 'comment'. |
252
|
| Expected @throws tag for 'InvalidTransactionException'. |
252
|
| Expected @throws tag for 'LowlevelLedgerException'. |
252
|
| Expected @throws tag for 'TransactionExpiredException'. |
252
|
| Expected @throws tag for 'UnknownTransactionException'. |
252
|
| Line is longer than 80 characters. |
256
|
| Line is longer than 80 characters. |
261
|
| Expected an @return tag. |
268
|
| Line is longer than 80 characters. |
268
|
| Parameter hold should be final. |
268
|
| Parameter origbook should be final. |
268
|
| Parameter newbook should be final. |
268
|
| Parameter amount should be final. |
268
|
| Parameter comment should be final. |
268
|
| Expected @param tag for 'comment'. |
268
|
| Expected @throws tag for 'InvalidTransactionException'. |
268
|
| Expected @throws tag for 'LowlevelLedgerException'. |
268
|
| Expected @throws tag for 'TransactionExpiredException'. |
268
|
| Expected @throws tag for 'UnknownTransactionException'. |
268
|
| First sentence should end with a period. |
270
|
| Line is longer than 80 characters. |
276
|
| Parameter id should be final. |
276
|
| 'id' hides a field. |
276
|
| Expected @throws tag for 'LowlevelLedgerException'. |
276
|
| Expected @throws tag for 'UnknownTransactionException'. |
276
|
| Line is longer than 80 characters. |
279
|
| Line is longer than 80 characters. |
286
|
| Line is longer than 80 characters. |
291
|
| Line is longer than 80 characters. |
299
|
| Parameter ledger should be final. |
299
|
| Expected @param tag for 'ledger'. |
299
|
| Parameter book should be final. |
299
|
| Expected @param tag for 'book'. |
299
|
| Expected @throws tag for 'LowlevelLedgerException'. |
299
|
| Method 'getBalance' is not designed for extension - needs to be abstract, final or empty. |
301
|
| Missing a Javadoc comment. |
301
|
| Parameter book should be final. |
301
|
| Line is longer than 80 characters. |
307
|
| Line is longer than 80 characters. |
313
|
| Line is longer than 80 characters. |
318
|
| Line is longer than 80 characters. |
326
|
| Parameter ledger should be final. |
326
|
| Expected @param tag for 'ledger'. |
326
|
| Parameter book should be final. |
326
|
| Expected @param tag for 'book'. |
326
|
| Expected @throws tag for 'LowlevelLedgerException'. |
326
|
| Line is longer than 80 characters. |
328
|
| Method 'getAvailableBalance' is not designed for extension - needs to be abstract, final or empty. |
328
|
| Missing a Javadoc comment. |
328
|
| Parameter book should be final. |
328
|
| Line is longer than 80 characters. |
332
|
| Missing a Javadoc comment. |
332
|
| Parameter ledger should be final. |
332
|
| Method 'getBookCount' is not designed for extension - needs to be abstract, final or empty. |
334
|
| Missing a Javadoc comment. |
334
|
| Line is longer than 80 characters. |
340
|
| Missing a Javadoc comment. |
340
|
| Parameter ledger should be final. |
340
|
| Method 'getTransactionCount' is not designed for extension - needs to be abstract, final or empty. |
342
|
| Missing a Javadoc comment. |
342
|
| Line is longer than 80 characters. |
346
|
| Missing a Javadoc comment. |
346
|
| Parameter id should be final. |
346
|
| 'id' hides a field. |
346
|
| Line is longer than 80 characters. |
348
|
| Missing a Javadoc comment. |
348
|
| Parameter id should be final. |
348
|
| 'id' hides a field. |
348
|
| First sentence should end with a period. |
350
|
| Expected an @return tag. |
361
|
| Line is longer than 80 characters. |
361
|
| Parameter id should be final. |
361
|
| 'id' hides a field. |
361
|
| Expected @param tag for 'id'. |
361
|
| Parameter nickname should be final. |
361
|
| Expected @param tag for 'nickname'. |
361
|
| Parameter type should be final. |
361
|
| Expected @param tag for 'type'. |
361
|
| Parameter source should be final. |
361
|
| Expected @param tag for 'source'. |
361
|
| Parameter registrationid should be final. |
361
|
| Expected @param tag for 'registrationid'. |
361
|
| Expected @throws tag for 'LowlevelLedgerException'. |
361
|
| Line is longer than 80 characters. |
363
|
| Missing a Javadoc comment. |
363
|
| Parameter id should be final. |
363
|
|