備忘録
- pep8に対応していないので対応する
- とくに理由なく関数内でimportしている。importは原則トップレベルに書くべき
あと、なんだろうな。実行と定義が分離できていない気がする。 __name__ == '__main__' の部分は実行のみにしたい。
定義の部分は別途他のでも使えるようにしたい。
- たぶんコマンドの登録はobjectが良い。
- テスト用の入力値(実行コード)と関数定義がまとまっていない感じなので辛い。
とりあえず、ひとつのオブジェクトにしてみる。
何だか1つのオブジェクトだと再利用性が悪い気がする。
以下の3つくらいに分ける
- 登録していたアクションを実行するexecutor
- 渡された入力値を解析して、実行しやすい形式に変更するparser
- デフォルトの動作を決めたりするbot
botがexecutor,parserを利用するという感じにすると良い気がする。
あと、細かな関数と全体のinterfaceは別なのでたぶん細かな関数はそのまま残しておいて、 各オブジェクトのメソッドなどでは直接実装を書かないという方が良い。