array split using jquery?

Question

i have some value stored in array and i wana split them and wana know length of its contains value but when i am running function it is not working

<head>

<script type="text/javascript" src="jquery.js"></script>

<script type="text/javascript">

$(function(){


    var valData= ['songs','video','movie','games','other'];

    var valNew=valData.split(',');

    for(i=0;i<valNew.length;i++);

    alert(valNew.length)

    })


</script>


</head>

<body>

<select id="me"></select>
</body>

@bažmegakapa he is using the shorthand for $(document).ready(), although it is not relevant to the question — Kevin Bowersox
– Kevin Bowersox, Commented Apr 21, 2012 at 9:14
@kmb385 Yeah, that's true, but even that is not needed, there is no DOM access in the code. — kapa
– kapa, Commented Apr 21, 2012 at 9:24

Kevin Bowersox · Accepted Answer · 2012-04-21 09:19:12Z

13

Split is used to separate a delimited string into an array based upon some delimiter passed into the split function. Your values are already split into an array. Also your for loop syntax is incorrect.

Split Documentation: https://developer.mozilla.org/en/JavaScript/Reference/Global_Objects/String/split

Corrected code:

$(function(){
    var valData= "songs,video,movie,games,other";

    var valNew=valData.split(',');

    for(var i=0;i<valNew.length;i++){
        alert(valNew.length)
    }
});

http://jsfiddle.net/tmHea/

edited Apr 21, 2012 at 9:19

answered Apr 21, 2012 at 9:06

Kevin Bowersox

94.6k19 gold badges164 silver badges198 bronze badges

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

3 Comments

kapa Over a year ago

+1 But if you are advocating array iteration best practices, it would be nice to cache the length AND declare i as local (it is GLOBAL now).

Kevin Bowersox Over a year ago

@bažmegakapa good catch, I didn't see that i was global, I have updated. I agree caching the length would be a good idea but I think the alert will help the original poster realize the code is working.

kapa Over a year ago

How? It would alert the same value all the time.

elclanrs · Accepted Answer · 2012-04-21 09:06:00Z

5

You don't need to split anything, it's already an array. And your for loop syntax is wrong...

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

    alert(valData[i].length);

}

answered Apr 21, 2012 at 9:06

elclanrs

94.2k21 gold badges137 silver badges171 bronze badges

2 Comments

Tats_innit Over a year ago

Bruv also if you notice lol code is using array of string in valData that needs to be corrected as well because without it split won't work :) just mentioning bruv, have a good one, cheerios!

kapa Over a year ago

for (var i = 0, len = valData.length; i < len; i++) to cache the length and make a really "right" iteration.

Tats_innit · Accepted Answer · 2017-08-21 01:42:28Z

3

Hiya demo http://jsfiddle.net/YCarA/8/

2 things are wrong

1) for loop.

2) your valData is array it should be string for split to format as array.

jquery code

$(function(){


    var valData = "songs,video,movie,games,other";


    var valNew = valData.split(',');

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

      alert(valNew.length);

   }
    });

edited Aug 21, 2017 at 1:42

answered Apr 21, 2012 at 9:06

Tats_innit

34.1k10 gold badges74 silver badges77 bronze badges

2 Comments

kapa Over a year ago

Please make that i local (and cache the length).

Tats_innit Over a year ago

@bažmegakapa sure bruv gimme 1 minute please I used the code from above, 1 min :) thanks! and done now! +1

Shaikh Farooque · Accepted Answer · 2012-04-21 09:23:57Z

1

Kmb385 is right. Your data is you can not split to array it is already separated.

Also your for loop is faulty the correct one is

for(var i=0;i<valNew.length;i++)
      alert(valNew[i].length);

edited Apr 21, 2012 at 9:23

answered Apr 21, 2012 at 9:10

Shaikh Farooque

2,6402 gold badges22 silver badges36 bronze badges

1 Comment

kapa Over a year ago

Please make that i local (and cache the length) - if talking about faulty/correct loops.

tusar · Accepted Answer · 2012-04-21 10:15:12Z

1

The jQuery.each() way :

var valData = ["songs","video","movie","games","other"];
$.each(valData, function(key, value) {
   alert("Index ---> " + key + ' & length of item ---> ' + value.length);
});

jsfiddle

edited Apr 21, 2012 at 10:15

answered Apr 21, 2012 at 9:59

tusar

3,4306 gold badges41 silver badges62 bronze badges

Collectives™ on Stack Overflow

array split using jquery?

5 Answers 5

3 Comments

2 Comments

2 Comments

1 Comment

Comments

Linked

Hot Network Questions

Collectives™ on Stack Overflow

5 Answers 5

3 Comments

2 Comments

2 Comments

1 Comment

Comments

Linked

Related