Question

See the bottom of this post;

function isVowel(aCharacter)
    {
        return ((aCharacter == 'a') || (aCharacter == 'A')||
                (aCharacter == 'e') || (aCharacter == 'E')||
                (aCharacter == 'i') || (aCharacter == 'I')||
                (aCharacter == 'o') || (aCharacter == 'O')||
                (aCharacter == 'u') || (aCharacter == 'U')||
                (aCharacter == 'y') || (aCharacter == 'Y'));
    }


function myF(aString)
{
    // variable to hold resultString
    var resultString = '';

    // variable to hold the current and previous characters
    var currentCharacter = '';
    var precedingCharacter = '';

    // in the case of the first character in the string there is no
    // previous character, so we assign an empty string '' to the variable at first
    //precedingCharacter = '';

    // TODO  part (ii)
    // add code as directed in the question

     var i = 0;
    for( i; i < sString.length; ++i)
    {

        currentCharacter = sString.charAt(i);
        if (isVowel(currentCharacter) && (!isVowel(precedingCharacter)))
        {
        resultString = resultString + 'ub';
        }
        resultString = resultString + currentCharacter;
        precedingCharacter = currentCharacter;
    } 

    return resultString;
}

var string1 = "the cat sat on the mat"; 
var result1 = myF(string1);
document.write(string1);//THIS ISN'T GOING TO BE DISPLAYED, BUT WHY?

document.write('<BR>');
document.write(result1);

Answer 1

You iterate on sString wich doesn’t exist and not on your parameter aString.