Javascript - Making Array Index toLowerCase() not working

Question

I'm trying to make all array indexes lowercase strings, but it's not working. I looked at other answers on here and tried their solutions like using toString() before adding toLowerCase but it doesn't work, which is weird.

I created a jsfiddle of the problem here.

JS:

$(colorArr).each(function(i, item) // loop thru each of elements in colorArr and make lowercase + trim
{
    if(colorArr[i] !== undefined) // check if colorArr index undefined
      {
      colorArr[i].toString().toLowerCase().trim(); // FIX HERE
      /* TRIED - DIDN'T WORK!
      colorArr[i].toLowerCase().trim();
      */
       }
});

Possible duplicate of why does the trim doesn't really trim? trim not working in my case — JJJ
– JJJ, Commented Mar 23, 2016 at 14:03
@juhana This is has nothing to do with trim() even though it's in the code, I'm asking only about toLowerCase(). — TheAmazingKnight
– TheAmazingKnight, Commented Mar 23, 2016 at 14:04
The solution is the same: you're applying toLowerCase and trim but not doing anything with the result. — JJJ
– JJJ, Commented Mar 23, 2016 at 14:05

messerbill · Accepted Answer · 2016-03-23 14:03:18Z

4

i updated your fiddle

https://jsfiddle.net/af91r2cq/6/

colorArr[i] = colorArr[i].toString().toLowerCase().trim(); // FIX HERE

your way was really close ;)

answered Mar 23, 2016 at 14:03

messerbill

5,6391 gold badge31 silver badges39 bronze badges

Sign up to request clarification or add additional context in comments.

1 Comment

TheAmazingKnight Over a year ago

Comparing the other variable checkedAttr, why did it work for checkedAttr instead of colorArr when both are array of strings?

gurvinder372 · Accepted Answer · 2016-03-23 14:01:57Z

2

You need to set the value back

It should be

colorArr[i] = colorArr[i].toString().toLowerCase().trim(); // FIX HERE

Or simply

colorArr = colorArr.map(function(value){ return value ? value.toLowerCase().trim() : ""; });

answered Mar 23, 2016 at 14:01

gurvinder372

68.6k11 gold badges78 silver badges98 bronze badges

Comments

Gary S. · Accepted Answer · 2016-03-23 14:32:36Z

1

Another way change all defined values in the array to lowercase is to use the jQuery.map function like so:

colorArr = $.map(colorArr, function(item, i) {
  if(item !== undefined) {
    return item.toString().toLowerCase().trim();
  }
});

answered Mar 23, 2016 at 14:32

Gary S.

1465 bronze badges

Collectives™ on Stack Overflow

Javascript - Making Array Index toLowerCase() not working

3 Answers 3

1 Comment

Comments

Comments

Your Answer

Linked

Hot Network Questions

Collectives™ on Stack Overflow

3 Answers 3

1 Comment

Comments

Comments

Your Answer

Sign up or log in

Post as a guest

Linked

Related