How can I remove the last ma from this function.

   for(var i = 0; i <= 100; i++) { 
          if(i % 2 === 0) { 
            div.innerHTML += l + ","; 
          } 
         else { 
            div.innerHTML += " ";
         }
   } 

How can I remove the last ma from this function.

   for(var i = 0; i <= 100; i++) { 
          if(i % 2 === 0) { 
            div.innerHTML += l + ","; 
          } 
         else { 
            div.innerHTML += " ";
         }
   } 

• javascript

Share Improve this question Follow edited Oct 12, 2015 at 11:07 Kevbear asked Oct 12, 2015 at 10:15 KevbearKevbear 36111 gold badge44 silver badges1010 bronze badges 4

• Add this after the for loop, dividedThree.innerHTML = dividedThree.innerHTML.substring(0, dividedThree.innerHTML.length - 1); – Tushar Commented Oct 12, 2015 at 10:18
• if (i % 2 === 0 && i !== 100) { ... ? – adeneo Commented Oct 12, 2015 at 10:18
• in practice, you should avoid updating innerHTML in a loop like that – Jaromanda X Commented Oct 12, 2015 at 10:20
• do you really add l? or is it i? with i you get 0, 2, 4, 6, ..., 98, 100,. the function is missing some (). – Nina Scholz Commented Oct 12, 2015 at 10:26

Add a ment  | 

4 Answers 4

Sorted by: Reset to default 6

First, use arrays instead strings for sum strings, its faster. Second:

var arr = [];
for (var i = 0 ;i <= 100; i+=2) arr.push(l);
dividedThree.innerHTML = arr.join(', ');

function loop {
    for (var i = 0; i <= 100; i++) {
        if (i % 2 === 0) {
            dividedThree.innerHTML += l;
            if (i < 100)
            {
                dividedThree.innerHTML += ",";
            }
        } else {
            dividedThree.innerHTML += " ";
        }
    }
}

Try using and (&&) condition for 100 and show only i value for 100.

function loop() {
  var dividedThree = document.getElementById('MY_ID');
  for (var i = 0; i <= 100; i++) {
    if (i % 2 === 0 && i !== 100) {
      dividedThree.innerHTML += i + ", ";
    } else if (i === 100) {
      dividedThree.innerHTML += i;
    } else {
      dividedThree.innerHTML += " ";
    }
  }
}
loop();

<p id="MY_ID">
  <p>

does this version work for you?

function loop() {
    var data = '';
    for (var i = 0; i <= 49; i++) { 
        data  += l + ',' + ' ';
    }
    data  += l;
    dividedThree.innerHTML = data;
}

What are you trying to do? where does 'l' e from?