LunaTTvBot / LunaBot

0 stars 1 forks source link

Event Names #13

Closed CapCalamity closed 8 years ago

CapCalamity commented 8 years ago

Most if not all the names of Registered Event-Handlers are named like this:

ChannelEventManager.UserListEvent += ChannelEventManagerOnUserListEvent;
ChannelEventManager.UserJoinEvent += ChannelEventManagerOnUserJoinEvent;
ChannelEventManager.UserPartEvent += ChannelEventManagerOnUserPartEvent;
UserEventManager.UserPublicMessageEvent += UserEventManagerOnUserPublicMessageEvent;

I think it would be clearer to use function-names that describe their use, e.g.

UserEventManager.UserPublicMessageEvent += AddChatterToUserList;

In case one Event Handler does more than one thing we can always register more than one Handler.

iPaat commented 8 years ago

Good Point. I'll do that asap.