Converting String to upper and lower case

Question

I'm developing a program that takes a string, splits it, returns the first string with only the first letter capitalized and returns the second string with all the letters capitalized. The code is below:

var name = "ThEoDORe RoOseVElT"; function nameChanger(oldName) { var finalName = oldName; var splitNames = finalName.split(" "); var secondName = splitNames.pop(); var firstName = splitNames; var secondName2 = secondName.toUpperCase(); var firstName2 = firstName.toLowerCase(); var finalName = firstName + " " + secondName; return finalName; };

The error given states 'Uncaught' and 'TypeError: undefined is not a function'. I know my problem is line 11 and 12 with the toUpperCase() and toLowerCase() methods but I don't know why.

bvaughn · Accepted Answer · 2015-03-21 15:28:57Z

The current error you're getting is because your firstName variable contains an Array and not a String. You can fix that by changing this

var firstName = splitNames;

...to this:

var firstName = splitNames.pop();

However you should add some checking in place instead of just assuming that incoming names will also be in a "word word" format.

JLRishe · Accepted Answer · 2015-03-21 15:29:26Z

Even after you call pop() on it, splitNames is still an array, and if you assign it to firstName, firstName will be that same array, which doesn't have a toLowerCase() method.

Try:

var secondName = splitNames.pop(); var firstName = splitNames.pop(); var secondName2 = secondName.toUpperCase(); var firstName2 = firstName.toLowerCase();

Millie Smith · Accepted Answer · 2015-03-21 16:06:42Z

When you assign splitNames to firstName, you are assigning an array to firstName, and arrays don't have a lowercase or uppercase method.

var firstName = splitNames.pop()

Also, you don't need to reuse var if you've already declared a variable, and you forgot to add the new names together on the second to last line:

finalName = firstName2 + secondName2;

felix91 · Accepted Answer · 2016-11-11 12:36:50Z

function titleCase(str) { return str.toLowerCase().replace(/( |^)[a-z]/g, (L) => L.toUpperCase()); }

Collectives™ on Stack Overflow

Converting String to upper and lower case

4 Answers 4

1 Comment

Comments

Comments

Comments

Hot Network Questions

Collectives™ on Stack Overflow

4 Answers 4

1 Comment

Comments

Comments

Comments

Related