Question

I am using a onclick function and adding a value to by paasing a value to it. If the value already exists den i remove that value from the list.

function send_value(str)
    {
    //alert(str);
    var sub_id = document.getElementById('selected_sub_id').value;
    //alert(sub_id.indexOf(str));
    if(sub_id.indexOf(str)==-1)
    {
        if(sub_id=="")
        {
            sub_id+=str;
        }
        else
        {
        sub_id+=',';
        sub_id+=str;
        }
    }
    else
    {
        sub_id = sub_id.replace(str+",", "");
        sub_id = sub_id.replace(","+str, "");
        sub_id = sub_id.replace(str, "");
    }
    //alert(sub_id);
    document.getElementById('selected_sub_id').value=sub_id;
}

This is the function. Suppose i have the values 1,2,3,4 in the selected_sub_id and i am passing 5 to it, it will be stored as 1,2,3,4,5
now i am passing 24 it will be stored as 1,2,3,4,5,24
No suppose i want to remove 2, so when i send 2 to the function it removes all the occurrences of 2 so i am left with only 1,3,4,54…
kindly help me with this thanks in advance..

Answer 1

You most probably want an array for this. It makes things much easier.

Create one first:

var arr = [];

Adding goes with .push:

arr.push(24);

Removing goes with .splice and .indexOf (the 1 means ‘removing 1 element’):

arr.splice(arr.indexOf(24), 1);

Converting to a string goes with .join:

arr.join(); // 1,2,3,24 or something similar depending on elements

---

For example: http://jsfiddle.net/ScBNQ/.

var arr = [];

// adding

arr.push(1);
arr.push(2);
arr.push(3);
arr.push(24);

// removing

arr.splice(arr.indexOf(2), 1);

// joining

arr.join(); // 1,3,24