Javascript-如何使此正则表达式代码全局化？

Question

I think I am missing one line somewhere, or need to change one line. I tried various /g methods but I think the exec is killing me here, maybe I am missing some small easy addition, or maybe the way I have written it is flawed. Saw a lot of threads where one mentioned while looping the exec but that seemed to infinite loop because I am including the match in the replacement.

Here is the code I got so far

var system_url="http://somesite.com/";
$('.words').mouseover(function(){
  var re=new RegExp("[A-Z][A-Z][A-Z]-[0-9]{5}");
  var m=re.exec($(this).html());
  if (m == null) {
    re=new RegExp("[0-9]{5}");
    m=re.exec($(this).html());
    if (m == null) {}else{
      if ($(this).html().match("</a>")) {}else{
        tx=$(this).html().replace(m,"<a href='"+system_url+m+"' target='_blank' title='Ticket: "+m+"'>"+m+"</a>");
        $(this).html(tx);
      }
    }
  }else{
    if ($(this).html().match("</a>")) {}else{
      tx=$(this).html().replace(m,"<a href='"+system_url+m+"' target='_blank' title='Ticket: "+m+"'>"+m+"</a>");
      $(this).html(tx);
    }
  }
});

Some HTML Class:

<span class='words'>The quick brown ZAD-14034 jumped over the EAD-14534</span>
<span class='words'>The 13034 brown fox jumped over the ZEN-12274</span>

Answer 1

You define a regex literal as /expr/flags . The RegExp constructor is used as new RegExp("expr", "flags") . Thus, in your case the usage would be:

new RegExp("[0-9]{5}", "g");

At the same time, you gain no advantage of using the constructor over a literal, so try using this syntax:

re = /[0-9]{5}/g;

Reference:

• https://developer.mozilla.org/en-US/docs/JavaScript/Reference/Global_Objects/RegExp

UPDATE:

An attempt to fix your code:

var system_url = "http://somesite.com/";

$(".words").on("mouseover", function () {
    var $this = $(this);
    if (!$this.hasClass("replaced")) {
        var currentContent = $this.text();
        var re = /([A-Z]{3}-[0-9]{5})/g;
        var newContent = currentContent.replace(re, "<a href='" + system_url + "$1' target='_blank' title='Ticket $1'>$1</a>");
        $this.html(newContent);
        $this.addClass("replaced");
    }
});

DEMO: http://jsfiddle.net/5bJ5N/1/