Using a for loop to display an array

Question

I am new to arrays, and I have been trouble with making a for loop work with an array. What do I need to do?

function start() {
    var arrPeople = ['Adele', 'Ted', 'Jo', 'Jim', 'Emma', 'Kate', 'Christopher'];
    var text = '';

    for (i = 0; 1 < arrPeople.length; i++) {
        text += (arrPeople[i] + '<br />');
    }
}

if you want to avoid the for loop and only want to join the array element with <br/> tag you can also use this text= arrPeople.toString().replace(new RegExp(',', 'g'), '<br/>'). — vikscool
– vikscool, Commented Apr 14, 2017 at 5:27

miken32 · Accepted Answer · 2017-09-13 16:07:20Z

3

There were a couple of minor errors in your function. The for loop statement is three parts: variable initialization, a condition that's checked after each loop, and an action that's performed after each loop. You had mistyped your condition as 1 < arrPeople.length which would always be true. Also you weren't returning a value from the function to be used by the calling code.

function start() {
    var arrPeople = ['Adele', 'Ted', 'Jo', 'Jim', 'Emma', 'Kate', 'Christopher'];
    var text = '';

    for (var i = 0; i < arrPeople.length; i++) {
        text += (arrPeople[i] + '<br />');
    }
    return text;
}

edited Sep 13, 2017 at 16:07

answered Apr 14, 2017 at 5:19

miken32

42.5k16 gold badges127 silver badges177 bronze badges

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

Comments

rbtLong · Accepted Answer · 2017-04-14 06:30:08Z

1

pick your poison ...

you can also use array.foreach ...

function start() {
    var arrPeople = ['Adele', 'Ted', 'Jo', 'Jim', 'Emma', 'Kate', 'Christopher'];
    var text = '';

    arrPeople.forEach(function(person) { 
        text += person + '<br />';
    });
}

see https://developer.mozilla.org/en-US/docs/Web/JavaScript/Reference/Global_Objects/Array/forEach

here's an possibly an even simpler solution using array.join ...

function start() {
    var arrPeople = ['Adele', 'Ted', 'Jo', 'Jim', 'Emma', 'Kate', 'Christopher'];
    var text = arrPeople.join("<br />");

    text += "<br />";
}

see https://developer.mozilla.org/en-US/docs/Web/JavaScript/Reference/Global_Objects/Array/join

or if you prefer, you can also use the functional style using array.reduce ...

function start() {
    var arrPeople = ['Adele', 'Ted', 'Jo', 'Jim', 'Emma', 'Kate', 'Christopher'];

    var text = arrPeople.reduce(function(txt, itm) { 
        return txt + '<br />' + itm;
    })

    text += "<br />";
}

see https://developer.mozilla.org/en-US/docs/Web/JavaScript/Reference/Global_Objects/Array/join

edited Apr 14, 2017 at 6:30

answered Apr 14, 2017 at 6:03

rbtLong

1,5823 gold badges14 silver badges33 bronze badges

1 Comment

miken32 Over a year ago

Some good options there, but you repeated the OP's mistake of not returning anything from your functions!

LINQ Newbee · Accepted Answer · 2017-04-14 05:50:50Z

0

Looks like you just want to display the list of names in the array vertically. You can do it without for-loop.

function start() {
    var text = [ 'Adele', 'Ted', 'Jo', 'Jim', 'Emma', 'Kate', 'Christopher' ].join( '<br />' );
}

You can use join to concatenate each array item and you can specify a separator for each. This way there is no extra <br/>. see array.join

answered Apr 14, 2017 at 5:50

LINQ Newbee

3331 gold badge4 silver badges13 bronze badges

1 Comment

miken32 Over a year ago

Good thought, but you repeated the OP's mistake of not returning anything from the function.

Collectives™ on Stack Overflow

Using a for loop to display an array

3 Answers 3

Comments

1 Comment

1 Comment

Your Answer

Hot Network Questions

Collectives™ on Stack Overflow

3 Answers 3

Comments

1 Comment

1 Comment

Your Answer

Sign up or log in

Post as a guest

Related