Codecov Report

Merging #228 (0a5f72c) into stable (8af0cbd) will decrease coverage by 0.43%.
The diff coverage is 96.08%.

@@            Coverage Diff             @@
##           stable     #228      +/-   ##
==========================================
- Coverage   89.64%   89.20%   -0.44%     
==========================================
  Files          55       56       +1     
  Lines        2877     2363     -514     
==========================================
- Hits         2579     2108     -471     
+ Misses        298      255      -43     

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update 8af0cbd...0a5f72c. Read the comment docs.

        if type == Type.float:
            return self.info["input_func"]["float"]
        elif type == Type.int:
            return self.info["input_func"]["int"]
        elif type == Type.str:
            return self.info["input_func"]["string"]

といった感じのif分岐は全部enumで置き換えたほうがいいですかね？つまり、return self.info["input_func"][type.value]とシンプルに書くことができると思います。

        if type == Type.float:
            return self.info["input_func"]["float"]
        elif type == Type.int:
            return self.info["input_func"]["int"]
        elif type == Type.str:
            return self.info["input_func"]["string"]

といった感じのif分岐は全部enumで置き換えたほうがいいですかね？つまり、return self.info["input_func"][type.value]とシンプルに書くことができると思います。

可読性の観点からこれは十分シンプルなのでこれでいいと思います。

メンテ性の観点からはあんまり今後頻繁に型が増えるとは思えないのでこのままで十分です

C++のbits/stdc++.hを使わないようにしました。また、formatのキーワードの指定を一々指定するのではなくて、専用の関数から展開するようにしました。これによって、割とコードがシンプルになったと思います。

上記のコメントを読む前にif分岐の部分もenumから配列アクセスするようにしてしまいましたが、不評でしたら戻します。

ありがとうございます。
正直のところ不必要にコードを間接的にしたくないという気持ちはあって、以前のほうが好きなんですよね。

あとinput_funcに関してなんですが、こっちもより間接的になっている気がします。この記法のユーザー的なメリットを教えて頂けたら納得できるかもしれません。教えて頂けますか?

では、if分岐のところはもとに戻します。キーワードをまとめるのも便利だと思ったんですが、間接化してることになっちゃいますかね？

input_funcを使うメリットはpythonで確保と入力をfor文を使わないでA = [int(next(tokens)) for _ in range({length})]といった形に書けるようなものもサポートしていて、これをやるときにint, float, strごとに違う関数を指定しなければならないところが、input_funcを定義しておくと下記のように共通して書けるということですかね。あと、もう一つは入力で使うint(next(tokens))というのの繰り返しを避けたり、変更があったときにinput_funcを変えるだけでできるようになるので少し便利ということもありますね。もともと、tomlではなくpythonで書いていた頃には文字列の継ぎ接ぎで対応していて、universal_code_generatorは感知しない状態だったんですが、tomlにするとuniversal_code_generatorの変数として扱わなければならなくなりました。

# 確保と入力
[allocate_and_input]
seq = "{name} = [{input_func} for _ in range({length})]"
2d_seq = "{name} = [[{input_func} for _ in range({length_j})] for _ in range({length_i})]"

# 宣言と確保と入力
[declare_and_allocate_and_input]
seq = "{name} = [{input_func} for _ in range({length})]  # type: \"List[{type}]\""
2d_seq = "{name} = [[{input_func} for _ in range({length_j})] for _ in range({length_i})]  # type: \"List[List[{type}]]\""

@chaemon なるほどなるほど、よくわかりました。ではinput_funcに関してはそれでいきましょう。

では、if分岐のところはもとに戻します。

これごめんなさい。戻してくれたところを改めて見て、やっぱり戻さないほうがキレイに見えました。これは完全に僕の勘違い&間違った指示です。スミマセン...

LGTM, @chaemon マージしてよかったら教えてください

では、if分岐ではなくenumのvalueを使う方にしました！

これでマージしていただいて構いませんよ〜