Refactoring

[sornas]

The code as it stands is not the best. Current issues:

• Code duplication when sending messages
• Very deep indentation in some places

Other suggestions:

• Clearer abstractions

[sornas]

parse_message could take a closure that sends a message.

[FredTheDino]

parse_message should maybe use the "match" functionality? You can make it a lot more declarative. If we split the string upto the first space can simplify a lot of the code from what I've seen. Maybe a dict mapped to a function that takes the rest of the string? Closures might be overkill...

[sornas]

The closure is for sending response messages. I just pushed 4568425 that does this. It opens up the return type for parse_message which otherwise would have been an almost-too-complex enum containing some like

pub enum ParseResponse {
    SendMessage(String),
    EmojiOk,
    EmojiErr,
    // ...
}

Parsing commands in a better way is definitely on the todo however.