appending values to textbox using for loop javascript

Question

I am trying to add values to a textbox when looping through an array when checking checkboxes but as it is at the moment getting undefined.

Advice perhaps as to why the values are 'undefined'

    var txtBoxValues = [];
    $(document).on("click", "input[name=chkRelatedTopics]", function () {

    var nameAdminUser = $(this).val();
    var txtBox = document.getElementById("txtTraningTopics");
    txtBox.value = '';

    txtBoxValues.push(nameAdminUser);

    for (var i in txtBoxValues) {

        var str = txtBoxValues[i].value;
        txtBox.value += str + '; ';

    }
});

Do you know how to use firebug or Chrome console? printing the value of txtBoxValues[i] would be enough to spot the issue — JohnJohnGa
– JohnJohnGa, Commented Sep 2, 2013 at 14:13

Denys Séguret · Accepted Answer · 2013-09-02 14:03:14Z

2

nameAdminUser is already a string, so don't take .value from it.

You could replace

 var str = txtBoxValues[i].value;

with

 var str = txtBoxValues[i];

But instead of using this loop, and assuming you don't want, as I suppose, the last ";", you could also do

txtBox.value = txtBoxValues.join(';');

answered Sep 2, 2013 at 14:03

Denys Séguret

384k90 gold badges813 silver badges780 bronze badges

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

Comments

JohnJohnGa · Accepted Answer · 2013-09-02 14:04:36Z

1

nameAdminUser seems to be a String and in your for loop you expect an object. What if you simply do:

for (var i in txtBoxValues) {
    var str = txtBoxValues[i];
    txtBox.value += str + '; ';
}

answered Sep 2, 2013 at 14:04

JohnJohnGa

15.7k20 gold badges65 silver badges87 bronze badges

Collectives™ on Stack Overflow

appending values to textbox using for loop javascript

2 Answers 2

Comments

Comments

Your Answer

Hot Network Questions

Collectives™ on Stack Overflow

2 Answers 2

Comments

Comments

Your Answer

Sign up or log in

Post as a guest

Related