How can I delete from an array [duplicate]

Question

i know there are similar threads but i have tried what you suggested there but still couldn´t solve my problem.

for(var i = text.length-1; i>=0; i--)
    {
        if(text.charCodeAt(i) < 42 || text.charCodeAt(i) > 57)
        {
            text.splice(i,1);
        }
    }
    alert(text);

Why is alert(text); not executed? Something is wrong with the Splice, please help me.

@AbdennourToumi possible confusion of lanaguage.. Java != JavaScript — Sterling Archer
– Sterling Archer, Commented Jun 2, 2014 at 19:08
You're working with a string. There is no splice method for strings. — Jonathan M
– Jonathan M, Commented Jun 2, 2014 at 19:10

Stefano Ortisi · Accepted Answer · 2014-06-02 23:31:23Z

3

It's because the string in javascript doesn't have a splice method. You need to convert it in an array.

text = text.split('');
text.splice(i,1);
text = text.join('');

And, as an array, you can't use the method charAt but

if(text[i] < 42 || text[i] > 57)

Talking about what's the best way to achieve this task, I would rather suggest to have a look at different solutions way better than this, performance wise.

Here you can find a benchmark where you can see how using the string method slice is way more performant.

http://jsperf.com/split-or-slice-for-string-splice/2

edited Jun 2, 2014 at 23:31

answered Jun 2, 2014 at 19:09

Stefano Ortisi

5,3163 gold badges35 silver badges44 bronze badges

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

2 Comments

Ruan Mendes Over a year ago

Converting it to an array works, but sounds like overkill. How about function splice(string, index, howMany) { return string.substring(0, index) + string.substring(index + howMany);}

Stefano Ortisi Over a year ago

yes I could be another solution. I just answered his question wondering why the alert wasn't been executed. That was because of the javascript error in the for loop. Going forward, it's plenty of different solutions to solve this particular task. Thank you!

Dan C · Accepted Answer · 2014-06-02 19:11:20Z

0

You might be looking for the .slice() method.

text.slice(i,1);

answered Jun 2, 2014 at 19:11

Dan C

353 bronze badges

Comments

lfarroco · Accepted Answer · 2014-06-02 19:09:06Z

-4

The alert is not executed because you did not declare it in the beginning of your script.

var text = [];

for(var i = text.length-1; i>=0; i--)
    {
        if(text.charCodeAt(i) < 42 || text.charCodeAt(i) > 57)
        {
            text.splice(i,1);
        }
    }
    alert(text);

answered Jun 2, 2014 at 19:09

lfarroco

6247 silver badges15 bronze badges

1 Comment

mplungjan Over a year ago

text is obviously a string and not an empty array

Collectives™ on Stack Overflow

How can I delete from an array [duplicate]

3 Answers 3

2 Comments

Comments

1 Comment

Linked

Hot Network Questions

Collectives™ on Stack Overflow

3 Answers 3

2 Comments

Comments

1 Comment

Linked

Related