This means all the variables and methods and arraylists of the animation frames and logic are all in one class.
I just want to know if there are any issues doing it this way or can it be improved for the sake of better maintenance and readability or any convenience for the other programmers looking at the code in the future.
This is far too convoluted—the class does way too much.
You should have a CSprite class for anything related to sprites.
A CAnimation class uses sprites and draws them in order to play animations.
A CEntity is a thing in the game world and has a COrientation.
A CCharacter is any character-type object in the game and inherits from CEntity.
A CDrawableCharacter inherits from CCharacter and has a CAnimation for rendering.
A CLink inherits from CDrawableCharacter and adds logic specifically about Link.
It is amazing how often people try to be unique, and yet they are always trying to make others be like them. - L. Spiro 2011
I spent most of my life learning the courage it takes to go out and get what I want. Now that I have it, I am not sure exactly what it is that I want. - L. Spiro 2013
I went to my local Subway once to find some guy yelling at the staff. When someone finally came to take my order and asked, “May I help you?”, I replied, “Yeah, I’ll have one asshole to go.”
L. Spiro Engine: http://lspiroengine.com
L. Spiro Engine Forums: http://lspiroengine.com/forums