JavaScript sostituire / regex

Data questa funzione:

function Repeater(template) {

    var repeater = {

        markup: template,

        replace: function(pattern, value) {
            this.markup = this.markup.replace(pattern, value);
        }

    };

    return repeater;

};

Come faccio a this.markup.replace()sostituire globalmente? Ecco il problema. Se lo uso in questo modo:

alert(new Repeater("$TEST_ONE $TEST_ONE").replace("$TEST_ONE", "foobar").markup);

Il valore dell'avviso è "foobar $ TEST_ONE".

Se cambio Repeaterin quanto segue, non viene sostituito nulla in Chrome:

function Repeater(template) {

    var repeater = {

        markup: template,

        replace: function(pattern, value) {
            this.markup = this.markup.replace(new RegExp(pattern, "gm"), value);
        }

    };

    return repeater;

};

... e l'avviso è $TEST_ONE $TEST_ONE.

È necessario eseguire l'escape doppia di qualsiasi carattere RegExp (una volta per la barra nella stringa e una volta per la regexp):

  "$TESTONE $TESTONE".replace( new RegExp("\\$TESTONE","gm"),"foo")

Altrimenti, cerca la fine della riga e "TESTONE" (che non trova mai).

Personalmente, non sono un grande fan della creazione di regexp usando le stringhe per questo motivo. Il livello di fuga necessario potrebbe portarti a bere. Sono sicuro che gli altri si sentono in modo diverso però e amano bere quando scrivono regex.

In termini di interpretazione del modello, non c'è differenza tra le seguenti forme:

• /pattern/
• new RegExp("pattern")

Se vuoi sostituire una stringa letterale usando il replacemetodo, penso che puoi semplicemente passare una stringa invece di un'espressione regolare a replace.

Altrimenti, dovresti prima eseguire l'escape di tutti i caratteri speciali regexp nel pattern, forse in questo modo:

function reEscape(s) {
    return s.replace(/([.*+?^$|(){}\[\]])/mg, "\\$1");
}

// ...

var re = new RegExp(reEscape(pattern), "mg");
this.markup = this.markup.replace(re, value);

Il tuo pattern regex dovrebbe avere il modificatore g:

var pattern = /[somepattern]+/g;

notare la g alla fine. dice al sostituto di fare una sostituzione globale.

Inoltre non è necessario utilizzare l'oggetto RegExp, è possibile costruire il modello come sopra. Pattern di esempio:

var pattern = /[0-9a-zA-Z]+/g;

un modello è sempre circondato da / su entrambi i lati - con i modificatori dopo la / finale, il modificatore g è il globale.

EDIT: Perché è importante se il pattern è una variabile? Nel tuo caso funzionerebbe in questo modo (nota che il pattern è ancora una variabile):

var pattern = /[0-9a-zA-Z]+/g;
repeater.replace(pattern, "1234abc");

Ma dovresti cambiare la tua funzione di sostituzione in questo:

this.markup = this.markup.replace(pattern, value);