list コマンドのイテレーション改善

[toyboot4e]

list コマンドに変更を加え、 30% 前後のスピードアップを確認しました。 劇的な変化でもないのですが、一応 PR を提出させていただきました。

仕様変更

ちょっと手抜きがあります。 narou list をパイプに通して id を渡す場合、最後の id の後に空白が 1 つ入ります。

$ narou list | cat
1 2 3 
     ^ ココに空白

その他の変化は無いと思います。

動作確認

pipe, grep, echo で期待通り動作することを確認しました。

備考

確認も一手間かかると思いますので、このまま閉じていただいても大丈夫です。

[ghost]

RuboCop detected 8 issues on the commit 2e17e02e5ae7a26cde77c0d8cc5ca9d3ef29add5. Visit the analysis result to review all issues.

[takotakot]

横からすみません。30% はすごいですね。寄与が大きいのは何の影響なのでしょう…。

output_list メソッドは limit を無視するようにしたのでしょうか。であれば、引数を消してしまっても良い気がします。

[toyboot4e]

@takotakot チェックありがとうございました。

limit 忘れてました……。 そして、ごめんなさい、スピードアップは勘違いでした。

gem で入れた narou.rb には、間にスクリプト (which narou) が入っているため、 余分に時間がかかっていたみたいです。ゴメンナサイ……