test_31331回答ポイント 80ptウォッチ 1

JavascriptのRegExpオブジェクトをStringから生成したいです。

普通は
js> a = new RegExp("abc", "i")
js> a.source # => "abc"
とすればよいのですが、
"/abc/i"のような文字列から先のようなRegExpを作ろうと思うと
js> a = new RegExp("/abc/i")
js> a.source # => "\/abc\/i"
のようになってしまいます。そこでevalを使うと
js> a = eval("/abc/i")
js> a.source # => "abc"
のようにほしいRegExpが作れることまでわかっています。
しかし、"/abc/i"のような文字列が安全かどうかわからない場合に、evalを使いたくありません。つまり
js> a = eval("while(true){alert('abc')}")
のようなことができないようにしたいです。
RegExpで、"/abc/i"の"abc"と"i"を分離させて、そこからRegExpを新たに生成するという方法もありますが、
ちょっと泥臭いので、もう少しスマートに、"/abc/i"のような文字列から、任意のコードを実行されないように、適切なRegExpを生成することができますか?

• はてなアカウントでログイン
• 無料ユーザー登録

ベストアンサー

• Mug2007/09/10 01:56:00

  満足40pt

  > 任意のコードを実行されないようにしたいので、

  半分はGEN111さんのと同じですが、以下のような感じでは如何でしょうか？

  
  	
  var Pattern = (function() {
    var PatternOfPattern = /^\/(.+)\/([gim]*)$/i;
    return function(pattern) {
      if((typeof pattern) != "string") {
        throw new Error("引数patternは文字列でなければなりません。");
      } else if(!PatternOfPattern.test(pattern)) {
        throw new Error("引数patternの値は正規表現リテラルの形式を満たしていません。");
      }
      PatternOfPattern.exec(pattern);
      return new RegExp(RegExp.$1, RegExp.$2);
    };
  })();
  
  	
  //-------------------------------
  	
  var regObj1 =  Pattern ("/^hoge\\s+.*;$/i");
  var regObj2 = new RegExp("^hoge\\s+.*;$", "i");
  	
  alert(regObj1 == regObj2);                // -> false
  alert(regObj1.source == regObj2.source);  // -> true
  	
  

  引数patternの文字列自体を取り出せるように拡張することも可能です。

  また、new RegExpの部分はわざとtry～catchしていません。これは正規表現としての文法エラーはこのPattern関数内部で隠蔽しないほうが逆によいのでは？と考えたためです。

  クロージャを使ったのは、２箇所(test,exec)で使用する変数PatternOfPatternのためで、毎回実行される関数外に書きたいけれど、外から参照・変更されたくないため、使用しました。

  スター

  • test_31331 2007/09/10 18:01:40

    一番ベターな方法のようです。

    これでやってみようかと思います。ありがとうございました。

その他の回答

• GEN1112007/09/09 22:50:37

  満足27pt

  こういうのはどうでしょう。

  var res1 = "/abc/i" ;
  var res2 = 'alert("")' ;
  
  var reo1 = eval(String(res1.match(/^\/.*\/[gim]*$/))) ;
  alert(reo1 == undefined) ;
  var reo2 = eval(String(res2.match(/^\/.*\/[gim]*$/))) ;
  alert(reo2 == undefined) ; // 文字列が正規表現にならない場合は undefined
  

  スター

  • test_31331 2007/09/09 23:45:28

    "/abc/i"の"abc"と"i"を分離させずに正規表現となっているかを正規表現で調べる、という方法ですね。ありがとうございます。

    ただ、やっぱりevalで泥臭さが見えるのかも、というのは私の偏見ですかね(笑)

    "/abc/i"という表現が、すでに正規表現を表す「記号」としての役割を果たしているので、new RegExp("/abc/i") みたいなもので生成できるように実装されててもいいんじゃないかと思えます。/abc/iがRegExpオブジェクトと振る舞う中で、引数としてRegExpオブジェクトをとるように見えるnew RegExp(/abc/i)はちょっと異質にも見えます。

ofk2007/09/09 23:26:04

満足13pt

function getRegExp(exp) {
	try {
		return (new Function('', 'return ' + exp))();
	}
	catch(e) {
		return new RegExp;
	}
}
var a = getRegExp("/abc/i");
alert(a.source);

こんなのはどうでしょうか？

スター

• test_31331 2007/09/09 23:55:08

  任意のコードを実行されないようにしたいので、

  js> a = getRegExp("alert('abc')")

  のようなコードを実行すると、任意のコードが実行できるかたちになってしまいます。

  これはちょっと質問に沿わなくなってしまいます。

  また、よい方法を思いついたらご回答ください。

  ありがとうございます。

• その他の回答を読む

この質問へのコメント

• ofk2007/09/09 23:58:01
  >>
  "/abc/i"という表現が、すでに正規表現を表す「記号」としての役割を果たしているので、new RegExp("/abc/i") みたいなもので生成できるように実装されててもいいんじゃないかと思えます。
  <<
  そうしてしまうと、/\/abc\/i/で書き表せる正規表現をnew RegExpで生成する方法がなくなってしまいます。
• test_313312007/09/10 07:52:25
  >>
  そうしてしまうと、/\/abc\/i/で書き表せる正規表現をnew RegExpで生成する方法がなくなってしまいます。
  <<
  new RegExp("/\/abc\/i/")
  という感じでどうでしょうか?
• GEN1112007/09/10 19:42:34
  Number("2*3") も parseInt("2*3") も 6 にはならないし、JavaScript は引数の文字列にはあまり手を入れないのがスタンスなんでしょうね。
  
  >>
  引数としてRegExpオブジェクトをとるように見えるnew RegExp(/abc/i)はちょっと異質にも見えます。
  <<
  object の代入が参照渡しになってしまうので、それができないとコピー作るのが面倒ですからね。
  
  >>
  一番ベター
  <<
  それってベスト?
• test_313312007/09/11 19:57:44
  「これら回答の中ではベスト。ただ、求めていた回答の中では、2番手。」という感じですかね。
  求めていた回答では、デフォルトの関数ひとつで、具体的にいえば、new RegExp("/abc/i")のような感じで
  できるというのがベストでした。
  ただ、今回はそういった方法はやっぱり存在しないようなので、1番はボツ。だから一番ベターだった2番手が
  いるか賞というわけです。
  
  >>
  object の代入が参照渡しになってしまうので、それができないとコピー作るのが面倒ですからね。
  <<
  駆け出しなもので、いまいち掴めてないですが、
  a = new RegExp("abc")
  b = a
  b == a // => true
  b = new RegExp(a)
  b == a // => false
  あたりのことと想像します。まだそこら辺を意識したことがなかったので、
  そういうこともあったと片隅においておくことにします。ありがとうございます。
  

質問の情報

登録日時

2007-09-09 20:58:56

終了日時

2007-09-10 18:04:15

回答条件

1人2回まで

この質問のカテゴリ

• コンピュータ
• ウェブ制作

この質問に含まれるキーワード

人気の質問

• この英文どうなってますか？ The boy seemed to be waiting for his mother to come.（その少年は母親が来…匿名質問者112日前
• 不定詞の形容詞的用法らしいんですけど、副詞的用法と書いたら✖️ですか？ I want to buy a book to read …匿名質問者112日前
• 日向陽葵という○V女優が好きなのですが似ている○V女優はいますか匿名質問者16時間前
• 共通テストで使われるプログラミング言語は共通テスト特有の言語と聞いたのですが、受験勉強で身につけたプ…匿名質問者116時間前
• 内定してほしいポケモンはいますか？匿名質問者17時間前

メニュー

• 質問一覧
• カテゴリ一覧
• 無料ユーザー登録
• 人力検索はてなトップ

PC版

• お問い合わせ
• ヘルプ
• お知らせ
• はてなトップ