Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

リファクタリング #49

Open
tsutaj opened this issue Apr 15, 2021 · 5 comments
Open

リファクタリング #49

tsutaj opened this issue Apr 15, 2021 · 5 comments
Labels
Type: Maintenance Repository Maintenance

Comments

@tsutaj
Copy link
Owner

tsutaj commented Apr 15, 2021

リーダブルコードに影響された

@tsutaj
Copy link
Owner Author

tsutaj commented Aug 14, 2021

エラーテキストは出るが、どの設定ファイルで出たエラーなのか分からない

これはなおすべき

@tsutaj
Copy link
Owner Author

tsutaj commented Aug 17, 2021

logger 周りを wrap してもっといい感じに出来ないだろうか。logger.error()raise XXXError() をどっちも書くのは面倒だしミスのもとになる。

best practice を漁ったり他のリポジトリを見たりすると良いかな。

@tsutaj
Copy link
Owner Author

tsutaj commented Aug 25, 2021

ソースコード中のコメントは英語にそろえたい (なんか混在しててよくわからなくなってきた)

@tsutaj
Copy link
Owner Author

tsutaj commented Aug 27, 2021

入出力例のところ、各言語における設定をベタ書きしているけど、言語が仮に増えたときにコードが汚くなる一方なのでよろしくない。

本来なら、言語ごとに設定ファイル的なのがあって、そこにある内容を適宜引っ張ってくるべき。

追記: これは #103 にて対応済み

@tsutaj tsutaj added this to the v1.5.1 milestone Aug 28, 2021
@tsutaj tsutaj added the Type: Maintenance Repository Maintenance label Aug 28, 2021
@tsutaj
Copy link
Owner Author

tsutaj commented Jun 21, 2023

dict の key が生の string で書かれているけれど、これは変数かなにかで置き換えるべきじゃないか?

それか dict をクラスに変換してクラスをやりとりするという手もある

いずれにしてもテストを整備するのが先っぽい

@tsutaj tsutaj removed this from the v1.7.3 or later milestone May 14, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Type: Maintenance Repository Maintenance
Projects
None yet
Development

No branches or pull requests

1 participant