Comments on: Remove an element in a JavaScript Array

By: Fabian

Fabian — Wed, 04 Feb 2009 12:56:11 +0000

great!
it was of great aid…

By: Nilesh

Nilesh — Wed, 03 Sep 2008 05:22:18 +0000

I want to remove repeated values in the string. suppose the string is
123:234:123:234
values are repeating I want the output
as 123:234

By: Hangman

Hangman — Sat, 30 Aug 2008 21:51:48 +0000

Works great!

By: Tom Van Schoor

Tom Van Schoor — Tue, 05 Aug 2008 11:28:03 +0000

To make it all complete and tidy:

Array.prototype.remove=function(s){
var i = this.indexOf(s);
if(i != -1) this.splice(i, 1);
}

Gigi has a point, but it would have been better if this was “his” contribution. No hard feelings though ^^

By: Tom Van Schoor

Tom Van Schoor — Tue, 05 Aug 2008 08:51:59 +0000

oops again…

Array.prototype.remove=function(s){
var i = this.indexOf(s);
if(this.indexOf(s) != -1)this.splice(i, 1);
}
}

By: Tom Van Schoor

Tom Van Schoor — Tue, 05 Aug 2008 08:50:12 +0000

Thanks Billis for defending, commenting and breaking down is easy.
Some ppl think they have to copy paste stuff without thinking about it.
All i put was a suggestion out of my head.
Instead of breaking it down like that, rather improve the code so others can learn from it.

var i = this.indexOf(s);
if(this.indexOf(s) != -1)this.splice(i, 1);
}

error removed from my comment

By: Billis

Billis — Fri, 01 Aug 2008 13:27:08 +0000

Gigi, before do some critics like u….
do it better okay? ;)

I like that script… i did just put a “return;” after splice

By: Gigi

Gigi — Wed, 30 Jul 2008 08:43:35 +0000

Tom Van Schoor, your function is full of errors. i, what’s i? index, of course. Improve performance? 2 calls to indexOf, when you have the result in index variable.

By: Jon

Jon — Thu, 03 Jul 2008 12:36:33 +0000

a.splice(1,1); // Removes 1 element from index 1

Both those ’1′s look the same to me…

splice(index, number of elements) in case anyone is wondering

By: Deo

Deo — Fri, 16 May 2008 09:25:42 +0000

This was very helpful !
Thanks.