Fix code style and remove unnecessary abbreviations

casperboone commented 7 years ago

This PR contains some small style fixes. For some of the fixes, a relatively large amount of lines is touched (making this PR a bit larger than ideal), but the changes itself are very minor.

Remove unnecessary abbreviations to improve readability and consistency throughout the code base (in some places there are many, in others none). They make the code harder to read and do not provide any benefit. I hope I caught all of them. (2d15bdc and 9336290)
Remove (some) unnecessary line breaks for lines short than 100 characters (2d15bdc)
Remove an unnecessary else case in the launcher. This is done to express the intent of the method better and make it consistent with other code. (9d0a4d4)
Improve readability of directions in PacManSprites by moving them to separate lines (a9f5f65)
Remove unnecessary empty lines from PlayerCollisions (3a05155)
Fix indentation of the switch statement in the createGhost method in LevelFactory (9336290)

avandeursen commented 7 years ago

Better indeed! Thanks!

mauricioaniche commented 7 years ago

Beautiful PR, @casperboone! Clean code for the win!!

SERG-Delft / jpacman-framework

Fix code style and remove unnecessary abbreviations #114