[ruby-list:50675] Re: [質問] キャプチャ付き正規表現の後方参照

From: dogatana <dogatana@...>
Date: 2018-06-10 10:19:56 UTC
List: ruby-list #50675
こんにちは、市田です。

2018年6月5日 21:42 Takahiro Yamaguchi <yamataka@u08.itscom.net>:

> もっと、プロっぽい記述とかになればいいんですが...

メソッド内部の処理の組み立てレベルは、「これはあきらかにだめ」なものも
あるでしょうが、そうでなければ個人の好みも大きく影響すると考えます。

そう言う意味でプロっぽいかどうかは別として、私なら変えようと思うのは

* regexp には正規表現リテラルにする
* 先頭と最後の空白をはぎ取るのは事前に strip を使う
* テーブルから探すのは each ではなく find を使う
* help も map と join で文字列生成した文字列を puts にする
* cmd にシンボルを入れているが、quue に積むときは文字列になるのが気持ち悪い

あたりです。

なお、書かれている例だと

* /q(uit)*/ だと uit が 0回以上の繰り返しにマッチする
* /q(uit)/ の uit が $1 に入るが考慮されてないのでは
* eval は $1 を使うなら不要(直接 $1 を使う)

のは確認されるのが良いかと。

In This Thread