Checkstyle Results

The following document contains the results of Checkstyle .

Summary

Files

io/IOUtilities.java

Error		Line
	Missing a header - not enough lines in file.	1
	Using the '.*' form of import should be avoided - java.io.*.	10
	Class IOUtilities should be declared as final.	16
	'{' should be on the previous line.	17
	Missing a Javadoc comment.	18
	'{' should be on the previous line.	19
	Missing a Javadoc comment.	22
	'(' is followed by whitespace.	22
	Parameter in should be final.	22
	')' is preceded with whitespace.	22
	'{' should be on the previous line.	23
	'(' is followed by whitespace.	25
	')' is preceded with whitespace.	25
	'256' is a magic number.	26
	'(' is followed by whitespace.	28
	'(' is followed by whitespace.	28
	'(' is followed by whitespace.	28
	')' is preceded with whitespace.	28
	')' is preceded with whitespace.	28
	')' is preceded with whitespace.	28
	'{' should be on the previous line.	29
	'(' is followed by whitespace.	32
	')' is preceded with whitespace.	32
	'(' is followed by whitespace.	33
	')' is preceded with whitespace.	33

io/RethrownException.java

Error		Line
	Missing a header - not enough lines in file.	1
	Line is longer than 80 characters.	10
	'{' should be on the previous line.	15
	Missing a Javadoc comment.	16
	Missing a Javadoc comment.	18
	'(' is followed by whitespace.	18
	Parameter e should be final.	18
	')' is preceded with whitespace.	18
	'{' should be on the previous line.	19
	Missing a Javadoc comment.	23
	'(' is followed by whitespace.	23
	Parameter s should be final.	23
	')' is preceded with whitespace.	23
	'{' should be on the previous line.	24
	'(' is followed by whitespace.	25
	')' is preceded with whitespace.	25
	Line has trailing spaces.	33
	Line has trailing spaces.	35
	Method 'getMessage' is not designed for extension - needs to be abstract, final or empty.	35
	'{' should be on the previous line.	36
	'(' is followed by whitespace.	38
	')' is preceded with whitespace.	38
	'{' should be on the previous line.	39
	'}' should be on the same line.	41
	'{' should be on the previous line.	43
	Line has trailing spaces.	53
	Method 'printStackTrace' is not designed for extension - needs to be abstract, final or empty.	53
	Line has trailing spaces.	54
	'{' should be on the previous line.	54
	'(' is followed by whitespace.	55
	')' is preceded with whitespace.	55
	'{' should be on the previous line.	56
	'}' should be on the same line.	58
	Line has trailing spaces.	59
	'{' should be on the previous line.	60
	Line has trailing spaces.	68
	Method 'printStackTrace' is not designed for extension - needs to be abstract, final or empty.	68
	Parameter s should be final.	68
	Expected @param tag for 's'.	68
	Line has trailing spaces.	69
	'{' should be on the previous line.	69
	'(' is followed by whitespace.	70
	')' is preceded with whitespace.	70
	'{' should be on the previous line.	71
	'(' is followed by whitespace.	72
	')' is preceded with whitespace.	72
	'}' should be on the same line.	73
	Line has trailing spaces.	74
	'{' should be on the previous line.	75
	'(' is followed by whitespace.	76
	')' is preceded with whitespace.	76
	Method 'printStackTrace' is not designed for extension - needs to be abstract, final or empty.	83
	Parameter s should be final.	83
	Expected @param tag for 's'.	83
	Line has trailing spaces.	84
	'{' should be on the previous line.	84
	'(' is followed by whitespace.	85
	')' is preceded with whitespace.	85
	'{' should be on the previous line.	86
	'(' is followed by whitespace.	87
	')' is preceded with whitespace.	87
	'}' should be on the same line.	88
	Line has trailing spaces.	89
	'{' should be on the previous line.	90
	'(' is followed by whitespace.	91
	')' is preceded with whitespace.	91
	Method 'toString' is not designed for extension - needs to be abstract, final or empty.	95
	Missing a Javadoc comment.	95
	Line has trailing spaces.	96
	'{' should be on the previous line.	96
	Line has trailing spaces.	98
	'(' is followed by whitespace.	99
	')' is preceded with whitespace.	99
	'{' should be on the previous line.	100
	'}' should be on the same line.	102
	Line has trailing spaces.	103
	'{' should be on the previous line.	104

io/package.html

Error		Line
	Missing package documentation file.	0

spellcast/beings/Being.java

Error		Line
	Line does not match expected header line of 'Spellcast has a restricted license.'.	1
	Comment matches to-do format 'TODO:'.	29
	Missing a Javadoc comment.	85
	Parameter isNullable should be final.	435

spellcast/beings/Female.java

Error		Line
	Missing a header - not enough lines in file.	1
	Using the '.*' form of import should be avoided - java.io.*.	7
	'{' should be on the previous line.	16
	Missing a Javadoc comment.	17
	Missing a Javadoc comment.	19
	'{' should be on the previous line.	20
	'(' is followed by whitespace.	21
	')' is preceded with whitespace.	21
	Expected an @return tag.	27
	Method 'pro_himself' is not designed for extension - needs to be abstract, final or empty.	27
	Name 'pro_himself' must match pattern '^[a-z][a-zA-Z0-9]*$'.	27
	'{' should be on the previous line.	28
	Expected an @return tag.	35
	Method 'pro_him' is not designed for extension - needs to be abstract, final or empty.	35
	Name 'pro_him' must match pattern '^[a-z][a-zA-Z0-9]*$'.	35
	'{' should be on the previous line.	36
	Expected an @return tag.	43
	Method 'pro_he' is not designed for extension - needs to be abstract, final or empty.	43
	Name 'pro_he' must match pattern '^[a-z][a-zA-Z0-9]*$'.	43
	'{' should be on the previous line.	44
	Line has trailing spaces.	50
	Expected an @return tag.	51
	Method 'pro_his' is not designed for extension - needs to be abstract, final or empty.	51
	Name 'pro_his' must match pattern '^[a-z][a-zA-Z0-9]*$'.	51
	'{' should be on the previous line.	52

spellcast/beings/FireElemental.java

Error		Line
	Missing a header - not enough lines in file.	1

spellcast/beings/Gender.java

Error		Line
	Missing a header - not enough lines in file.	1
	Using the '.*' form of import should be avoided - java.io.*.	7
	'{' should be on the previous line.	15
	Missing a Javadoc comment.	16
	Missing a Javadoc comment.	17
	Missing a Javadoc comment.	18
	Missing a Javadoc comment.	19
	Line has trailing spaces.	25
	'{' is followed by whitespace.	25
	Line has trailing spaces.	26
	Missing a Javadoc comment.	30
	'{' is followed by whitespace.	30
	Missing a Javadoc comment.	32
	Missing a Javadoc comment.	34
	'(' is followed by whitespace.	34
	Parameter genderString should be final.	34
	'genderString' hides a field.	34
	')' is preceded with whitespace.	34
	'{' should be on the previous line.	35
	Line has trailing spaces.	37
	Line has trailing spaces.	38
	Line has trailing spaces.	44
	Method 'getGenderString' is not designed for extension - needs to be abstract, final or empty.	44
	'{' should be on the previous line.	45
	Line has trailing spaces.	48
	Line has trailing spaces.	54
	Method 'setGenderString' is not designed for extension - needs to be abstract, final or empty.	54
	'(' is followed by whitespace.	54
	Parameter v should be final.	54
	')' is preceded with whitespace.	54
	'{' should be on the previous line.	55
	Expected an @return tag.	62
	Line has trailing spaces.	62
	Name 'pro_himself' must match pattern '^[a-z][a-zA-Z0-9]*$'.	62
	Expected an @return tag.	67
	Name 'pro_him' must match pattern '^[a-z][a-zA-Z0-9]*$'.	67
	Expected an @return tag.	72
	Name 'pro_he' must match pattern '^[a-z][a-zA-Z0-9]*$'.	72
	Line has trailing spaces.	76
	Expected an @return tag.	77
	Name 'pro_his' must match pattern '^[a-z][a-zA-Z0-9]*$'.	77
	Definition of 'equals()' without corresponding definition of 'hashCode()'.	79
	Method 'equals' is not designed for extension - needs to be abstract, final or empty.	79
	Missing a Javadoc comment.	79
	'(' is followed by whitespace.	79
	Parameter o should be final.	79
	')' is preceded with whitespace.	79
	'{' should be on the previous line.	80
	Line has trailing spaces.	82
	'(' is followed by whitespace.	83
	')' is preceded with whitespace.	83
	'{' should be on the previous line.	84
	'cast' is not followed by whitespace.	85
	'(' is followed by whitespace.	86
	'(' is followed by whitespace.	86
	')' is preceded with whitespace.	86
	')' is preceded with whitespace.	86
	'{' should be on the previous line.	87
	Line has trailing spaces.	91
	Line has trailing spaces.	96
	Line is longer than 80 characters.	99
	'(' is followed by whitespace.	102
	Parameter genderString should be final.	102
	')' is preceded with whitespace.	102
	'{' should be on the previous line.	103
	'(' is followed by whitespace.	106
	')' is preceded with whitespace.	106
	'{' should be on the previous line.	107
	'(' is followed by whitespace.	108
	'(' is followed by whitespace.	108
	')' is preceded with whitespace.	108
	')' is preceded with whitespace.	108
	'{' should be on the previous line.	109

spellcast/beings/GenderNone.java

Error		Line
	Missing a header - not enough lines in file.	1
	Using the '.*' form of import should be avoided - java.io.*.	7
	Line has trailing spaces.	13
	'{' should be on the previous line.	16
	Missing a Javadoc comment.	17
	Missing a Javadoc comment.	19
	'{' should be on the previous line.	20
	'(' is followed by whitespace.	21
	')' is preceded with whitespace.	21
	Expected an @return tag.	27
	Method 'pro_himself' is not designed for extension - needs to be abstract, final or empty.	27
	Name 'pro_himself' must match pattern '^[a-z][a-zA-Z0-9]*$'.	27
	'{' should be on the previous line.	28
	Expected an @return tag.	35
	Method 'pro_him' is not designed for extension - needs to be abstract, final or empty.	35
	Name 'pro_him' must match pattern '^[a-z][a-zA-Z0-9]*$'.	35
	'{' should be on the previous line.	36
	Expected an @return tag.	43
	Method 'pro_he' is not designed for extension - needs to be abstract, final or empty.	43
	Name 'pro_he' must match pattern '^[a-z][a-zA-Z0-9]*$'.	43
	'{' should be on the previous line.	44
	Line has trailing spaces.	50
	Expected an @return tag.	51
	Method 'pro_his' is not designed for extension - needs to be abstract, final or empty.	51
	Name 'pro_his' must match pattern '^[a-z][a-zA-Z0-9]*$'.	51
	'{' should be on the previous line.	52

spellcast/beings/Giant.java

Error		Line
	Missing a header - not enough lines in file.	1

spellcast/beings/Goblin.java

Error		Line
	Missing a header - not enough lines in file.	1
	Missing a Javadoc comment.	11
	'(' is followed by whitespace.	12
	')' is preceded with whitespace.	12

spellcast/beings/IBeing.java

Error		Line
	File does not end with a newline.	0

spellcast/beings/IBeing.java

Error		Line
	Missing a header - not enough lines in file.	1

spellcast/beings/IMonster.java

Error		Line
	File does not end with a newline.	0

spellcast/beings/IMonster.java

Error		Line
	Missing a header - not enough lines in file.	1
	Missing a Javadoc comment.	6

spellcast/beings/IWizard.java

Error		Line
	File does not end with a newline.	0

spellcast/beings/IWizard.java

Error		Line
	Missing a header - not enough lines in file.	1
	Line is longer than 80 characters.	18
	Line is longer than 80 characters.	25
	Line is longer than 80 characters.	62

spellcast/beings/IceElemental.java

Error		Line
	Missing a header - not enough lines in file.	1

spellcast/beings/Male.java

Error		Line
	Missing a header - not enough lines in file.	1
	Using the '.*' form of import should be avoided - java.io.*.	7
	'{' should be on the previous line.	17
	Missing a Javadoc comment.	18
	Missing a Javadoc comment.	20
	'{' should be on the previous line.	21
	'(' is followed by whitespace.	22
	')' is preceded with whitespace.	22
	Expected an @return tag.	28
	Method 'pro_himself' is not designed for extension - needs to be abstract, final or empty.	28
	Name 'pro_himself' must match pattern '^[a-z][a-zA-Z0-9]*$'.	28
	'{' should be on the previous line.	29
	Expected an @return tag.	36
	Method 'pro_him' is not designed for extension - needs to be abstract, final or empty.	36
	Name 'pro_him' must match pattern '^[a-z][a-zA-Z0-9]*$'.	36
	'{' should be on the previous line.	37
	Expected an @return tag.	44
	Method 'pro_he' is not designed for extension - needs to be abstract, final or empty.	44
	Name 'pro_he' must match pattern '^[a-z][a-zA-Z0-9]*$'.	44
	'{' should be on the previous line.	45
	Line has trailing spaces.	51
	Expected an @return tag.	52
	Method 'pro_his' is not designed for extension - needs to be abstract, final or empty.	52
	Name 'pro_his' must match pattern '^[a-z][a-zA-Z0-9]*$'.	52
	'{' should be on the previous line.	53

spellcast/beings/Monster.java

Error		Line
	Missing a header - not enough lines in file.	1
	Missing a Javadoc comment.	14
	Line is longer than 80 characters.	16
	Missing a Javadoc comment.	16
	Missing a Javadoc comment.	20
	Missing a Javadoc comment.	21
	Missing a Javadoc comment.	22
	Missing a Javadoc comment.	23
	Line is longer than 80 characters.	24
	Missing a Javadoc comment.	24
	Missing a Javadoc comment.	25
	Line is longer than 80 characters.	27
	Missing a Javadoc comment.	27
	Missing a Javadoc comment.	31
	Missing a Javadoc comment.	32
	Missing a Javadoc comment.	33
	Missing a Javadoc comment.	34
	Line is longer than 80 characters.	35
	Missing a Javadoc comment.	35
	Line is longer than 80 characters.	36
	Missing a Javadoc comment.	36
	Comment matches to-do format 'TODO:'.	42
	'(' is followed by whitespace.	45
	')' is preceded with whitespace.	45
	Missing a Javadoc comment.	49
	Parameter name should be final.	49
	Parameter gender should be final.	49
	Parameter maxHitPoints should be final.	49
	'(' is followed by whitespace.	50
	')' is preceded with whitespace.	50
	Method 'createKurukName' is not designed for extension - needs to be abstract, final or empty.	53
	Missing a Javadoc comment.	53
	'cast' is not followed by whitespace.	58
	'*' is not preceded with whitespace.	58
	'*' is not followed by whitespace.	58
	'10' is a magic number.	58
	'%' is not preceded with whitespace.	58
	'%' is not followed by whitespace.	58
	'(' is followed by whitespace.	61
	')' is preceded with whitespace.	61
	'(' is followed by whitespace.	62
	')' is preceded with whitespace.	62
	Line is longer than 80 characters.	65
	Line is longer than 80 characters.	67
	'(' is followed by whitespace.	67
	'&&' should be on a new line.	67
	'cast' is not followed by whitespace.	68
	'*' is not preceded with whitespace.	68
	'*' is not followed by whitespace.	68
	'10' is a magic number.	68
	'%' is not preceded with whitespace.	68
	'%' is not followed by whitespace.	68
	')' is preceded with whitespace.	68
	Line is longer than 80 characters.	70
	'cast' is not followed by whitespace.	70
	'*' is not preceded with whitespace.	70
	'*' is not followed by whitespace.	70
	'10' is a magic number.	70
	'%' is not preceded with whitespace.	70
	'%' is not followed by whitespace.	70
	'(' is followed by whitespace.	72
	')' is preceded with whitespace.	72
	'(' is followed by whitespace.	73
	')' is preceded with whitespace.	73
	Line is longer than 80 characters.	78
	'(' is followed by whitespace.	78
	'(' is followed by whitespace.	78
	'(' is followed by whitespace.	78
	')' is preceded with whitespace.	78
	')' is preceded with whitespace.	78
	')' is preceded with whitespace.	78
	'(' is followed by whitespace.	79
	')' is preceded with whitespace.	79
	Line is longer than 80 characters.	80
	'(' is followed by whitespace.	80
	'(' is followed by whitespace.	80
	'(' is followed by whitespace.	80
	')' is preceded with whitespace.	80
	')' is preceded with whitespace.	80
	')' is preceded with whitespace.	80
	Line is longer than 80 characters.	81
	'(' is followed by whitespace.	82
	')' is preceded with whitespace.	82
	Missing a Javadoc comment.	88
	Line is longer than 80 characters.	89
	'cast' is not followed by whitespace.	89
	'*' is not preceded with whitespace.	89
	'*' is not followed by whitespace.	89
	'100' is a magic number.	89
	'%' is not preceded with whitespace.	89
	'%' is not followed by whitespace.	89
	Method 'createSnaffiName' is not designed for extension - needs to be abstract, final or empty.	92
	Missing a Javadoc comment.	92
	'cast' is not followed by whitespace.	97
	'*' is not preceded with whitespace.	97
	'*' is not followed by whitespace.	97
	'10' is a magic number.	97
	'%' is not preceded with whitespace.	97
	'%' is not followed by whitespace.	97
	'(' is followed by whitespace.	100
	')' is preceded with whitespace.	100
	'(' is followed by whitespace.	101
	')' is preceded with whitespace.	101
	Line is longer than 80 characters.	104
	'(' is followed by whitespace.	106
	'||' should be on a new line.	106
	Line is longer than 80 characters.	107
	'(' is followed by whitespace.	107
	'&&' should be on a new line.	107
	'cast' is not followed by whitespace.	108
	'*' is not preceded with whitespace.	108
	'*' is not followed by whitespace.	108
	'10' is a magic number.	108
	'%' is not preceded with whitespace.	108
	'%' is not followed by whitespace.	108
	')' is preceded with whitespace.	108
	')' is preceded with whitespace.	108
	Line is longer than 80 characters.	110
	'cast' is not followed by whitespace.	110
	'*' is not preceded with whitespace.	110
	'*' is not followed by whitespace.	110
	'10' is a magic number.	110
	'%' is not preceded with whitespace.	110
	'%' is not followed by whitespace.	110
	'(' is followed by whitespace.	112
	')' is preceded with whitespace.	112
	'(' is followed by whitespace.	113
	')' is preceded with whitespace.	113
	Line is longer than 80 characters.	118
	'(' is followed by whitespace.	118
	'(' is followed by whitespace.	118
	'(' is followed by whitespace.	118
	')' is preceded with whitespace.	118
	')' is preceded with whitespace.	118
	')' is preceded with whitespace.	118
	'(' is followed by whitespace.	119
	')' is preceded with whitespace.	119
	Line is longer than 80 characters.	120
	'(' is followed by whitespace.	120
	'(' is followed by whitespace.	120
	'(' is followed by whitespace.	120
	')' is preceded with whitespace.	120
	')' is preceded with whitespace.	120
	')' is preceded with whitespace.	120
	Line is longer than 80 characters.	121
	'(' is followed by whitespace.	122
	')' is preceded with whitespace.	122
	Missing a Javadoc comment.	128
	Line is longer than 80 characters.	129
	'cast' is not followed by whitespace.	129
	'*' is not preceded with whitespace.	129
	'*' is not followed by whitespace.	129
	'100' is a magic number.	129
	'%' is not preceded with whitespace.	129
	'%' is not followed by whitespace.	129
	Method 'getOwner' is not designed for extension - needs to be abstract, final or empty.	135

spellcast/beings/Neuter.java

Error		Line
	Missing a header - not enough lines in file.	1
	Using the '.*' form of import should be avoided - java.io.*.	7
	Line has trailing spaces.	13
	'{' should be on the previous line.	16
	Missing a Javadoc comment.	17
	Missing a Javadoc comment.	19
	'{' should be on the previous line.	20
	'(' is followed by whitespace.	21
	')' is preceded with whitespace.	21
	Expected an @return tag.	27
	Method 'pro_himself' is not designed for extension - needs to be abstract, final or empty.	27
	Name 'pro_himself' must match pattern '^[a-z][a-zA-Z0-9]*$'.	27
	'{' should be on the previous line.	28
	Expected an @return tag.	35
	Method 'pro_him' is not designed for extension - needs to be abstract, final or empty.	35
	Name 'pro_him' must match pattern '^[a-z][a-zA-Z0-9]*$'.	35
	'{' should be on the previous line.	36
	Expected an @return tag.	43
	Method 'pro_he' is not designed for extension - needs to be abstract, final or empty.	43
	Name 'pro_he' must match pattern '^[a-z][a-zA-Z0-9]*$'.	43
	'{' should be on the previous line.	44
	Line has trailing spaces.	50
	Expected an @return tag.	51
	Method 'pro_his' is not designed for extension - needs to be abstract, final or empty.	51
	Name 'pro_his' must match pattern '^[a-z][a-zA-Z0-9]*$'.	51
	'{' should be on the previous line.	52

spellcast/beings/NullBeing.java

Error		Line
	Missing a header - not enough lines in file.	1

spellcast/beings/NullMonster.java

Error		Line
	File does not end with a newline.	0

spellcast/beings/NullMonster.java

Error		Line
	Missing a header - not enough lines in file.	1

spellcast/beings/NullWizard.java

Error		Line
	Missing a header - not enough lines in file.	1

spellcast/beings/Ogre.java

Error		Line
	Missing a header - not enough lines in file.	1

spellcast/beings/Troll.java

Error		Line
	Missing a header - not enough lines in file.	1
	Missing a Javadoc comment.	11
	'(' is followed by whitespace.	12
	')' is preceded with whitespace.	12

spellcast/beings/Wizard.java

Error		Line
	Got an exception - expecting EOF, found '}'	0

spellcast/beings/package.html

Error		Line
	Missing package documentation file.	0

spellcast/client/Client.java

Error		Line
	Missing a header - not enough lines in file.	1
	Line is longer than 80 characters.	34
	Missing a Javadoc comment.	34
	Name 'programVersionDetails' must match pattern '^[A-Z][A-Z0-9]*(_[A-Z0-9]+)*$'.	34
	Missing a Javadoc comment.	37
	Missing a Javadoc comment.	39
	Missing a Javadoc comment.	41
	Missing a Javadoc comment.	43
	Method 'createGUI' is not designed for extension - needs to be abstract, final or empty.	52
	Missing a Javadoc comment.	52
	Method 'setVisible' is not designed for extension - needs to be abstract, final or empty.	57
	Missing a Javadoc comment.	57
	Parameter visible should be final.	57
	First sentence should end with a period.	63
	Method 'propertyChange' is not designed for extension - needs to be abstract, final or empty.	66
	Parameter evt should be final.	66
	Expected @param tag for 'evt'.	66
	Method 'addMessage' is not designed for extension - needs to be abstract, final or empty.	88
	Parameter message should be final.	88
	Expected @param tag for 'message'.	88
	Method 'addQuery' is not designed for extension - needs to be abstract, final or empty.	97
	Parameter query should be final.	97
	Expected @param tag for 'query'.	97
	Method 'addWizard' is not designed for extension - needs to be abstract, final or empty.	106
	Parameter w should be final.	106
	Expected @param tag for 'w'.	106
	Method 'removeWizard' is not designed for extension - needs to be abstract, final or empty.	115
	Parameter w should be final.	115
	Expected @param tag for 'w'.	115
	Method 'connected' is not designed for extension - needs to be abstract, final or empty.	129
	Parameter success should be final.	129
	Method 'disconnected' is not designed for extension - needs to be abstract, final or empty.	138
	Method 'setID' is not designed for extension - needs to be abstract, final or empty.	150
	Parameter v should be final.	150
	Method 'setGame' is not designed for extension - needs to be abstract, final or empty.	163
	Parameter v should be final.	163
	Line is longer than 80 characters.	174
	Line is longer than 80 characters.	180

spellcast/client/ClientConnectionHandler.java

Error		Line
	File does not end with a newline.	0

spellcast/client/ClientConnectionHandler.java

Error		Line
	Line does not match expected header line of 'Spellcast has a restricted license.'.	1
	Missing a Javadoc comment.	27
	Missing a Javadoc comment.	28
	Missing a Javadoc comment.	29
	Missing a Javadoc comment.	30
	Missing a Javadoc comment.	31
	Missing a Javadoc comment.	32
	Missing a Javadoc comment.	33
	Missing a Javadoc comment.	34
	Missing a Javadoc comment.	35
	Missing a Javadoc comment.	36
	Missing a Javadoc comment.	37
	Line is longer than 80 characters.	40
	'10' is a magic number.	43
	Missing a Javadoc comment.	44
	Missing a Javadoc comment.	51
	Name 'logger' must match pattern '^[A-Z][A-Z0-9]*(_[A-Z0-9]+)*$'.	51
	Missing a Javadoc comment.	53
	Parameter view should be final.	54
	'view' hides a field.	54
	Parameter programVersionDetails should be final.	55
	'programVersionDetails' hides a field.	55
	Method 'run' is not designed for extension - needs to be abstract, final or empty.	60
	Missing a Javadoc comment.	60
	'}' should be on the same line.	72
	Line has trailing spaces.	83
	Parameter e should be final.	86
	Expected @param tag for 'e'.	86
	Line has trailing spaces.	96
	Line is longer than 80 characters.	96
	Line has trailing spaces.	98
	Line has trailing spaces.	99
	Expected an @return tag.	102
	Method 'receive' is not designed for extension - needs to be abstract, final or empty.	102
	'}' should be on the same line.	109
	Method 'send' is not designed for extension - needs to be abstract, final or empty.	132
	Parameter event should be final.	132
	Expected @param tag for 'event'.	132
	'}' should be on the same line.	137
	Line is longer than 80 characters.	155
	'}' should be on the same line.	164
	Method 'connect' is not designed for extension - needs to be abstract, final or empty.	180
	Parameter serverAddressString should be final.	181
	'serverAddressString' hides a field.	181
	Parameter wizardName should be final.	182
	'wizardName' hides a field.	182
	Parameter gender should be final.	183
	'gender' hides a field.	183
	Method 'disconnect' is not designed for extension - needs to be abstract, final or empty.	196
	Method 'join' is not designed for extension - needs to be abstract, final or empty.	203
	'}' should be on the same line.	207
	Line has trailing spaces.	215
	Line is longer than 80 characters.	216
	Line is longer than 80 characters.	217
	Line is longer than 80 characters.	218
	Line has trailing spaces.	219
	Line has trailing spaces.	220

spellcast/client/Main.java

Error		Line
	File does not end with a newline.	0

spellcast/client/Main.java

Error		Line
	Missing a header - not enough lines in file.	1
	Utility classes should not have a public or default constructor.	21
	Missing a Javadoc comment.	22
	Variable 'logger' must be private and have accessor methods.	22
	Missing a Javadoc comment.	24
	Parameter args should be final.	24
	Line is longer than 80 characters.	27
	Missing a Javadoc comment.	48
	Parameter args should be final.	48

spellcast/client/cengine.java

Error		Line
	File does not end with a newline.	0

spellcast/client/cengine.java

Error		Line
	Line does not match expected header line of 'Spellcast has a restricted license.'.	1
	Missing a Javadoc comment.	29
	Name 'cengine' must match pattern '^[A-Z][a-zA-Z0-9]*$'.	29
	Missing a Javadoc comment.	30
	Missing a Javadoc comment.	31
	Missing a Javadoc comment.	32
	Missing a Javadoc comment.	33
	Missing a Javadoc comment.	34
	Missing a Javadoc comment.	36
	'static' modifier out of order with the JLS suggestions.	36
	Name 'logger' must match pattern '^[A-Z][A-Z0-9]*(_[A-Z0-9]+)*$'.	36
	Line is longer than 80 characters.	38
	Missing a Javadoc comment.	40
	Parameter view should be final.	41
	'view' hides a field.	41
	Parameter clientConnectionHandler should be final.	42
	'clientConnectionHandler' hides a field.	42
	Line is longer than 80 characters.	49
	Line is longer than 80 characters.	50
	Method 'run' is not designed for extension - needs to be abstract, final or empty.	52
	Line is longer than 80 characters.	56
	Method 'cengine_execute' is not designed for extension - needs to be abstract, final or empty.	58
	Missing a Javadoc comment.	58
	Name 'cengine_execute' must match pattern '^[a-z][a-zA-Z0-9]*$'.	58
	Parameter args should be final.	58
	Array brackets at illegal position.	58
	Line is longer than 80 characters.	65
	Method 'initialise_the_program' is not designed for extension - needs to be abstract, final or empty.	67
	Missing a Javadoc comment.	67
	Name 'initialise_the_program' must match pattern '^[a-z][a-zA-Z0-9]*$'.	67
	Line is longer than 80 characters.	71
	Method 'get_external_event' is not designed for extension - needs to be abstract, final or empty.	73
	Missing a Javadoc comment.	73
	Name 'get_external_event' must match pattern '^[a-z][a-zA-Z0-9]*$'.	73
	Method 'mapGameEventToStateMachineEvent' is not designed for extension - needs to be abstract, final or empty.	78
	Missing a Javadoc comment.	78
	'}' should be on the same line.	83
	'}' should be on the same line.	87
	'}' should be on the same line.	91
	'}' should be on the same line.	95
	'}' should be on the same line.	99
	'}' should be on the same line.	103
	Line is longer than 80 characters.	114
	Line is longer than 80 characters.	117
	Method 'add_connection' is not designed for extension - needs to be abstract, final or empty.	120
	Name 'add_connection' must match pattern '^[a-z][a-zA-Z0-9]*$'.	120
	Line is longer than 80 characters.	126
	Method 'connect_this_client' is not designed for extension - needs to be abstract, final or empty.	132
	Name 'connect_this_client' must match pattern '^[a-z][a-zA-Z0-9]*$'.	132
	Line is longer than 80 characters.	138
	Method 'remove_connection' is not designed for extension - needs to be abstract, final or empty.	144
	Name 'remove_connection' must match pattern '^[a-z][a-zA-Z0-9]*$'.	144
	Line is longer than 80 characters.	160
	Method 'display_message' is not designed for extension - needs to be abstract, final or empty.	165
	Name 'display_message' must match pattern '^[a-z][a-zA-Z0-9]*$'.	165
	Line is longer than 80 characters.	170
	Line is longer than 80 characters.	173
	Line is longer than 80 characters.	174
	Method 'update_game' is not designed for extension - needs to be abstract, final or empty.	177
	Name 'update_game' must match pattern '^[a-z][a-zA-Z0-9]*$'.	177
	Line is longer than 80 characters.	189
	Method 'handle_unknown_event' is not designed for extension - needs to be abstract, final or empty.	191
	Missing a Javadoc comment.	191
	Name 'handle_unknown_event' must match pattern '^[a-z][a-zA-Z0-9]*$'.	191
	Line is longer than 80 characters.	193
	Line is longer than 80 characters.	196
	Method 'sync_game' is not designed for extension - needs to be abstract, final or empty.	202
	Name 'sync_game' must match pattern '^[a-z][a-zA-Z0-9]*$'.	202
	'}' should be on the same line.	207
	Line is longer than 80 characters.	210
	Line is longer than 80 characters.	214
	Missing a Javadoc comment.	216
	Name 'handle_io_error' must match pattern '^[a-z][a-zA-Z0-9]*$'.	216
	Line is longer than 80 characters.	217

spellcast/client/cenginei.java

Error		Line
	File does not end with a newline.	0

spellcast/client/cenginei.java

Error		Line
	Missing a header - not enough lines in file.	1
	Missing a Javadoc comment.	7
	'public' modifier out of order with the JLS suggestions.	7
	Name 'cenginei' must match pattern '^[A-Z][a-zA-Z0-9]*$'.	7
	Missing a Javadoc comment.	10
	Name '_LR_event' must match pattern '^[a-z][a-zA-Z0-9]*$'.	10
	Name '_LR_state' must match pattern '^[a-z][a-zA-Z0-9]*$'.	11
	Name '_LR_savest' must match pattern '^[a-z][a-zA-Z0-9]*$'.	12
	Name '_LR_index' must match pattern '^[a-z][a-zA-Z0-9]*$'.	13
	Missing a Javadoc comment.	15
	Name 'the_next_event' must match pattern '^[a-z][a-zA-Z0-9]*$'.	15
	Variable 'the_next_event' must be private and have accessor methods.	15
	Name 'the_exception_event' must match pattern '^[a-z][a-zA-Z0-9]*$'.	16
	Variable 'the_exception_event' must be private and have accessor methods.	16
	Missing a Javadoc comment.	18
	Name 'exception_raised' must match pattern '^[a-z][a-zA-Z0-9]*$'.	18
	Missing a Javadoc comment.	22
	Name '_LR_STOP' must match pattern '^[a-z][a-zA-Z0-9]*$'.	22
	'0xFFFF' is a magic number.	22
	Name '_LR_NULL_EVENT' must match pattern '^[a-z][a-zA-Z0-9]*$'.	22
	'-2' is a magic number.	22
	Missing a Javadoc comment.	24
	Name '_LR_STATE_after_init' must match pattern '^[a-z][a-zA-Z0-9]*$'.	24
	Variable '_LR_STATE_after_init' must be private and have accessor methods.	24
	Name '_LR_STATE_not_connected' must match pattern '^[a-z][a-zA-Z0-9]*$'.	24
	Variable '_LR_STATE_not_connected' must be private and have accessor methods.	24
	Name '_LR_STATE_connected' must match pattern '^[a-z][a-zA-Z0-9]*$'.	25
	Variable '_LR_STATE_connected' must be private and have accessor methods.	25
	Name '_LR_STATE_defaults' must match pattern '^[a-z][a-zA-Z0-9]*$'.	25
	Variable '_LR_STATE_defaults' must be private and have accessor methods.	25
	'3' is a magic number.	25
	Name '_LR_defaults_state' must match pattern '^[a-z][a-zA-Z0-9]*$'.	26
	Variable '_LR_defaults_state' must be private and have accessor methods.	26
	'3' is a magic number.	26
	Name 'connect_event' must match pattern '^[a-z][a-zA-Z0-9]*$'.	26
	Variable 'connect_event' must be private and have accessor methods.	26
	Name 'disconnect_event' must match pattern '^[a-z][a-zA-Z0-9]*$'.	26
	Variable 'disconnect_event' must be private and have accessor methods.	26
	Name 'game_sync_event' must match pattern '^[a-z][a-zA-Z0-9]*$'.	27
	Variable 'game_sync_event' must be private and have accessor methods.	27
	Name 'game_update_event' must match pattern '^[a-z][a-zA-Z0-9]*$'.	27
	Variable 'game_update_event' must be private and have accessor methods.	27
	'3' is a magic number.	27
	Name 'io_error_event' must match pattern '^[a-z][a-zA-Z0-9]*$'.	27
	Variable 'io_error_event' must be private and have accessor methods.	27
	'4' is a magic number.	27
	Name 'message_event' must match pattern '^[a-z][a-zA-Z0-9]*$'.	28
	Variable 'message_event' must be private and have accessor methods.	28
	'5' is a magic number.	28
	Name 'ok_event' must match pattern '^[a-z][a-zA-Z0-9]*$'.	28
	Variable 'ok_event' must be private and have accessor methods.	28
	'6' is a magic number.	28
	Name 'stop_event' must match pattern '^[a-z][a-zA-Z0-9]*$'.	28
	Variable 'stop_event' must be private and have accessor methods.	28
	'7' is a magic number.	28
	Name 'unknown_event' must match pattern '^[a-z][a-zA-Z0-9]*$'.	28
	Variable 'unknown_event' must be private and have accessor methods.	28
	'8' is a magic number.	28
	Name 'terminate_event' must match pattern '^[a-z][a-zA-Z0-9]*$'.	29
	Variable 'terminate_event' must be private and have accessor methods.	29
	Missing a Javadoc comment.	33
	Name '_LR_nextst' must match pattern '^[a-z][a-zA-Z0-9]*$'.	33
	Array brackets at illegal position.	33
	'{' is followed by whitespace.	33
	'{' is followed by whitespace.	33
	'{' is followed by whitespace.	34
	'{' is followed by whitespace.	34
	'{' is followed by whitespace.	35
	'3' is a magic number.	35
	'3' is a magic number.	35
	Missing a Javadoc comment.	37
	Name '_LR_action' must match pattern '^[a-z][a-zA-Z0-9]*$'.	37
	Array brackets at illegal position.	37
	'{' is followed by whitespace.	37
	'{' is followed by whitespace.	37
	'{' is followed by whitespace.	38
	'{' is followed by whitespace.	38
	'4' is a magic number.	38
	'5' is a magic number.	38
	'6' is a magic number.	38
	'3' is a magic number.	38
	'7' is a magic number.	38
	'{' is followed by whitespace.	39
	'8' is a magic number.	39
	'9' is a magic number.	39
	Missing a Javadoc comment.	41
	Name '_LR_vector' must match pattern '^[a-z][a-zA-Z0-9]*$'.	41
	Array brackets at illegal position.	41
	'{' is followed by whitespace.	41
	'{' is followed by whitespace.	41
	'{' is followed by whitespace.	41
	'{' is followed by whitespace.	41
	'{' is followed by whitespace.	42
	'7' is a magic number.	42
	'{' is followed by whitespace.	42
	'{' is followed by whitespace.	42
	'5' is a magic number.	42
	'{' is followed by whitespace.	42
	'6' is a magic number.	42
	'{' is followed by whitespace.	43
	'3' is a magic number.	43
	'{' is followed by whitespace.	43
	'{' is followed by whitespace.	43
	'4' is a magic number.	43
	Missing a Javadoc comment.	45
	Name '_LR_mname' must match pattern '^[a-z][a-zA-Z0-9]*$'.	45
	Array brackets at illegal position.	45
	'{' is followed by whitespace.	45
	Missing a Javadoc comment.	50
	Name '_LR_sname' must match pattern '^[a-z][a-zA-Z0-9]*$'.	50
	Array brackets at illegal position.	50
	'{' is followed by whitespace.	50
	Missing a Javadoc comment.	53
	Name '_LR_ename' must match pattern '^[a-z][a-zA-Z0-9]*$'.	53
	Array brackets at illegal position.	53
	'{' is followed by whitespace.	53
	Missing a Javadoc comment.	57
	'public' modifier out of order with the JLS suggestions.	57
	Name 'initialise_the_program' must match pattern '^[a-z][a-zA-Z0-9]*$'.	57
	Missing a Javadoc comment.	59
	'public' modifier out of order with the JLS suggestions.	59
	Name 'get_external_event' must match pattern '^[a-z][a-zA-Z0-9]*$'.	59
	Missing a Javadoc comment.	61
	'public' modifier out of order with the JLS suggestions.	61
	Name 'add_connection' must match pattern '^[a-z][a-zA-Z0-9]*$'.	61
	Missing a Javadoc comment.	63
	'public' modifier out of order with the JLS suggestions.	63
	Name 'connect_this_client' must match pattern '^[a-z][a-zA-Z0-9]*$'.	63
	Missing a Javadoc comment.	65
	'public' modifier out of order with the JLS suggestions.	65
	Name 'display_message' must match pattern '^[a-z][a-zA-Z0-9]*$'.	65
	Missing a Javadoc comment.	67
	'public' modifier out of order with the JLS suggestions.	67
	Name 'handle_io_error' must match pattern '^[a-z][a-zA-Z0-9]*$'.	67
	Missing a Javadoc comment.	69
	'public' modifier out of order with the JLS suggestions.	69
	Name 'handle_unknown_event' must match pattern '^[a-z][a-zA-Z0-9]*$'.	69
	Missing a Javadoc comment.	71
	'public' modifier out of order with the JLS suggestions.	71
	Name 'remove_connection' must match pattern '^[a-z][a-zA-Z0-9]*$'.	71
	Missing a Javadoc comment.	73
	'public' modifier out of order with the JLS suggestions.	73
	Name 'sync_game' must match pattern '^[a-z][a-zA-Z0-9]*$'.	73
	Missing a Javadoc comment.	75
	'public' modifier out of order with the JLS suggestions.	75
	Name 'update_game' must match pattern '^[a-z][a-zA-Z0-9]*$'.	75
	Method 'execute' is not designed for extension - needs to be abstract, final or empty.	79
	Missing a Javadoc comment.	79
	Name 'next_module' must match pattern '^[a-z][a-zA-Z0-9]*$'.	80
	'9' is a magic number.	86
	'if' construct must use '{}'s.	116
	switch without "default" clause.	119
	'3' is a magic number.	129
	'4' is a magic number.	132
	'5' is a magic number.	135
	'6' is a magic number.	138
	'7' is a magic number.	141
	'if' construct must use '{}'s.	148
	'else' construct must use '{}'s.	152
	'if' construct must use '{}'s.	155
	Method 'raise_exception' is not designed for extension - needs to be abstract, final or empty.	172
	Missing a Javadoc comment.	172
	Name 'raise_exception' must match pattern '^[a-z][a-zA-Z0-9]*$'.	172
	Parameter event should be final.	172
	'if' construct must use '{}'s.	174

spellcast/client/package.html

Error		Line
	Missing package documentation file.	0

spellcast/damage/Damage.java

Error		Line
	Missing a header - not enough lines in file.	1
	Line has trailing spaces.	15
	Missing a Javadoc comment.	16
	Parameter type_ should be final.	16
	Name 'type_' must match pattern '^[a-z][a-zA-Z0-9]*$'.	16
	Parameter value_ should be final.	16
	Name 'value_' must match pattern '^[a-z][a-zA-Z0-9]*$'.	16
	Line has trailing spaces.	20
	Missing a Javadoc comment.	21
	Method 'getType' is not designed for extension - needs to be abstract, final or empty.	22
	Missing a Javadoc comment.	22
	Method 'setType' is not designed for extension - needs to be abstract, final or empty.	23
	Missing a Javadoc comment.	23
	Parameter type_ should be final.	23
	Name 'type_' must match pattern '^[a-z][a-zA-Z0-9]*$'.	23
	Line has trailing spaces.	24
	Missing a Javadoc comment.	25
	Method 'getValue' is not designed for extension - needs to be abstract, final or empty.	26
	Missing a Javadoc comment.	26
	Line has trailing spaces.	27
	Method 'setValue' is not designed for extension - needs to be abstract, final or empty.	27
	Missing a Javadoc comment.	27
	Parameter value_ should be final.	27
	Name 'value_' must match pattern '^[a-z][a-zA-Z0-9]*$'.	27

spellcast/damage/DamageType.java

Error		Line
	Missing a header - not enough lines in file.	1
	Line has trailing spaces.	18
	Parameter isNullable should be final.	28
	Line has trailing spaces.	38
	Line has trailing spaces.	39
	Parameter type_ should be final.	43
	Expected @param tag for 'type_'.	43
	Name 'type_' must match pattern '^[a-z][a-zA-Z0-9]*$'.	43
	Expected an @return tag.	50
	Definition of 'equals()' without corresponding definition of 'hashCode()'.	50
	Method 'equals' is not designed for extension - needs to be abstract, final or empty.	50
	'(' is followed by whitespace.	50
	Parameter o should be final.	50
	Expected @param tag for 'o'.	50
	')' is preceded with whitespace.	50
	'{' should be on the previous line.	51
	'(' is followed by whitespace.	54
	')' is preceded with whitespace.	54
	'{' should be on the previous line.	55
	'(' is followed by whitespace.	57
	')' is preceded with whitespace.	57
	Expected an @return tag.	65
	Method 'toString' is not designed for extension - needs to be abstract, final or empty.	65
	Line has trailing spaces.	68
	Missing a Javadoc comment.	69
	Method 'getType' is not designed for extension - needs to be abstract, final or empty.	70
	Missing a Javadoc comment.	70
	Method 'setType' is not designed for extension - needs to be abstract, final or empty.	71
	Missing a Javadoc comment.	71
	Parameter type_ should be final.	71
	Name 'type_' must match pattern '^[a-z][a-zA-Z0-9]*$'.	71
	Line has trailing spaces.	72
	Missing a Javadoc comment.	73
	Name 'physicalDamageType_' must match pattern '^[a-z][a-zA-Z0-9]*$'.	73
	Line has trailing spaces.	74
	Missing a Javadoc comment.	75
	Line has trailing spaces.	81
	Missing a Javadoc comment.	82
	Name 'fireDamageType_' must match pattern '^[a-z][a-zA-Z0-9]*$'.	82
	Line has trailing spaces.	83
	Missing a Javadoc comment.	84
	Line has trailing spaces.	90
	Missing a Javadoc comment.	91
	Name 'coldDamageType_' must match pattern '^[a-z][a-zA-Z0-9]*$'.	91
	Missing a Javadoc comment.	92
	Line has trailing spaces.	98
	Line has trailing spaces.	107

spellcast/damage/KnifeAttack.java

Error		Line
	Missing a header - not enough lines in file.	1
	Missing a Javadoc comment.	17
	Missing a Javadoc comment.	19
	Method 'setTarget' is not designed for extension - needs to be abstract, final or empty.	26
	Parameter theTarget should be final.	26

spellcast/damage/NullDamageType.java

Error		Line
	Missing a header - not enough lines in file.	1
	Line has trailing spaces.	13
	First sentence should end with a period.	14
	Line has trailing spaces.	19

spellcast/damage/package.html

Error		Line
	Missing package documentation file.	0

spellcast/enchantment/AmnesiaEnchantment.java

Error		Line
	Missing a header - not enough lines in file.	1
	Unused import - spellcast.beings.Being.	6

spellcast/enchantment/BlindEnchantment.java

Error		Line
	Missing a header - not enough lines in file.	1

spellcast/enchantment/CharmPersonEnchantment.java

Error		Line
	Missing a header - not enough lines in file.	1

spellcast/enchantment/ConfusionEnchantment.java

Error		Line
	Missing a header - not enough lines in file.	1
	Line is longer than 80 characters.	51

spellcast/enchantment/DelayedEffectEnchantment.java

Error		Line
	Missing a header - not enough lines in file.	1

spellcast/enchantment/DiseasedEnchantment.java

Error		Line
	Missing a header - not enough lines in file.	1

spellcast/enchantment/Enchantment.java

Error		Line
	Missing a header - not enough lines in file.	1

spellcast/enchantment/EnchantmentImpl.java

Error		Line
	Line does not match expected header line of 'Spellcast has a restricted license.'.	1

spellcast/enchantment/FearEnchantment.java

Error		Line
	Missing a header - not enough lines in file.	1

spellcast/enchantment/HasteEnchantment.java

Error		Line
	Missing a header - not enough lines in file.	1

spellcast/enchantment/InvisibleEnchantment.java

Error		Line
	Missing a header - not enough lines in file.	1
	Line is longer than 80 characters.	51

spellcast/enchantment/ParalysisEnchantment.java

Error		Line
	Missing a header - not enough lines in file.	1
	Line is longer than 80 characters.	60

spellcast/enchantment/PermanencyEnchantment.java

Error		Line
	Missing a header - not enough lines in file.	1

spellcast/enchantment/PoisonedEnchantment.java

Error		Line
	Missing a header - not enough lines in file.	1