Find and remove text from input value string

Question

Our form has a hidden input that contains all the required values we send to the server for server-side validation. This is formed of a comma-separated string:

i.e. <input type="hidden" value="name,address,telephone,email">

We now have new inputs (i.e. website) that appear on a radio button check and I can successfully add these to the value="" string:

var nowReq = $('input[name="required"]').val();
if ($('#website').is(':checked')) {
    $('input[name="required"]').val(nowReq + ',website');
}

But when a different radio is checked, I can't get it to remove ,website from the string.

I believe I need to grab the string, split it by comma, find and remove website and then re-join the string, but I'm not sure how to implement this (or if this is even the best way):

if ($('#fax').is(':checked')) {
    var splitReq = nowReq.split(',');
    // Something goes here?
    $('input[name="required"]').val(nowReq2);
}

Create a working fiddle

Rahil Wazir
– Rahil Wazir

2014-04-30 09:41:34 +00:00
Commented Apr 30, 2014 at 9:41 — Rahil Wazir
– Rahil Wazir, Commented Apr 30, 2014 at 9:41

Software Engineer · Accepted Answer · 2014-04-30 09:56:21Z

1

You can use string.indexOf(searchValue); and string.replace(oldValue, newValue); to search and remove non-selected elements or better pass Regular expression in the first parameter of string.replace(regex, newValue):

Non-regex Example:

var nowReq = $('input[name="required"]').val();
if ($('#website').is(':checked')) {
    nowReq = nowReq + ',website';
}
else {
    if(string.indexOf(',website') > -1) // If website is the last element
        nowReq = nowReq.replace(',website', '');
    else if(string.indexOf('website,') > -1) // If website is not the last element
        nowReq = nowReq.replace('website,', '');
}

$('input[name="required"]').val(nowReq);

answered Apr 30, 2014 at 9:56

Software Engineer

3,9861 gold badge28 silver badges35 bronze badges

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

Comments

Camille Wintz · Accepted Answer · 2014-04-30 09:36:37Z

0

You simply need to remove the last element in your array and join it with the ',' character :

splitReq.pop();
$('input[name="required"]').val(splitReq.join(','));

answered Apr 30, 2014 at 9:36

Camille Wintz

9517 silver badges9 bronze badges

2 Comments

Banana Over a year ago

who said its gonna be the last elemet

Camille Wintz Over a year ago

well, since he just added the value at the end of the string, it's always going to be the last element...

Wilmer · Accepted Answer · 2014-04-30 09:38:12Z

0

Try:

var splitReq = nowReq.split(',website').join("");

answered Apr 30, 2014 at 9:38

Wilmer

2,5091 gold badge14 silver badges8 bronze badges

Comments

Tom · Accepted Answer · 2014-04-30 09:46:27Z

0

the following would work in most cases:

var s = "name,address,telephone,email";
s = s.replace(",address", "").replace("address,", "").replace("address", "");

answered Apr 30, 2014 at 9:46

Tom

4,1702 gold badges26 silver badges40 bronze badges

Comments

David Thomas · Accepted Answer · 2014-04-30 09:50:21Z

0

I'd suggest that you avoid parsing the value string, and simply recalculate the value on the change of the relevant input elements; the following is a demonstrative example because of the lack of information presented in the question (a more specific example could be provided if we can see your HTML, and current jQuery, rather than a description of it):

$('input[type="checkbox"]').on('change', function(){
    var requiredString = $('input[type="checkbox"]').map(function(){
        if (this.checked) {
            return this.value;
        }
    }).get().join(',');
    $('input[name="required"]').val(requiredString);
}).change();

JS Fiddle demo.

References:

edited Apr 30, 2014 at 9:50

answered Apr 30, 2014 at 9:42

David Thomas

254k53 gold badges382 silver badges421 bronze badges

Collectives™ on Stack Overflow

Find and remove text from input value string

5 Answers 5

Comments

2 Comments

Comments

Comments

Comments

Your Answer

Hot Network Questions

Collectives™ on Stack Overflow

5 Answers 5

Comments

2 Comments

Comments

Comments

Comments

Your Answer

Sign up or log in

Post as a guest

Related