javascript replace not working for me with global tag

[jonpfl]

All,

I cannot get my javascript code to work with the global flag for replace.

Here is the code I have (I am trying to find a certain character and replace it with a new line)

strTemp = splitString(strTemp, 80, '*');
 
function splitString(stringToSplit, sizeOfSplit, extraCharacterToSplit)
{
    var i =  stringToSplit.length /parseInt(sizeOfSplit);
    var sTemp;
    var sFinalString = "";
 
    if (extraCharacterToSplit != "")
    {
        stringToSplit = stringToSplit.replace('/' + extraCharacterToSplit + '/g', extraCharacterToSplit + '\n');
    }
 
    for (i = 0; i < Math.floor(stringToSplit.length / parseInt(sizeOfSplit)); i++)
    {
        sTemp = stringToSplit.substr(i * sizeOfSplit, sizeOfSplit);
        sFinalString = sFinalString + sTemp + "\n"; 
    }
    sTemp = stringToSplit.substr(i * sizeOfSplit, sizeOfSplit);
    sFinalString = sFinalString + sTemp; 
 
    return sFinalString;
}

Any ideas what I am doing wrong?

Thx
jonpfl

[pbmods]

Heya, jonpfl. Welcome (back) to TSDN!

Please use CODE tags when posting source code. See the REPLY GUIDELINES on the right side of the page next time you post.

[acoder]

Any ideas what I am doing wrong?

See link on how to use regular expressions in Javascript.

Don't use a string unless you're going to use the RegExp object.

[jonpfl]

See link on how to use regular expressions in Javascript.

Don't use a string unless you're going to use the RegExp object.

I changed my code to the following and it still does not work with a siimple string passed in :

function splitStringJon(stringToSplit, sizeOfSplit, extraCharacterToSplit)
{
    var nStringLength = stringToSplit.length;;
    var sTemp;
    var nCurrentPos, nOriginalPos, nSplitSize;
    var sFinalString = "";
 
    // This is used to replace the extraCharacterToSplit with a newline
    if (extraCharacterToSplit != "")
    {
        var re = new RegExp(stringToSplit, "g");
        stringToSplit = stringToSplit.replace(re, extraCharacterToSplit + '\n');
    }

Can you see a problem with this?

Thx
jonpfl

[pbmods]

Heya, jonpfl.
The first parameter to the RegExp() constructor is the regular expression that you want to use, not the string that you want to process.