shoftee / JavaStory

Solving the big issue.
6 stars 1 forks source link

The ChannelCharacter class is huge. #5

Open shoftee opened 12 years ago

shoftee commented 12 years ago

It has too much things in it - database loading, packet sending, general bad code.

Database loading should be separated into data access classes and ORM if necessary. Cohesive classes should be created to encapsulate the packet sending for each operation.