JQuery IF statement not working when using .append()

Question

Im using .append() to print the value of an input field to a span. The form is AJAX so multiple values are inputted into the form therefore I want to print these multiple inputs. This works well, apart from the below IF statement isn't switching the string value of 'names', any ideas?

$("input").bind("keydown", function(event) {
    var keycode = (event.keyCode ? event.keyCode : (event.which ? event.which : event.charCode));
    if (keycode == 13) {
         var value = $(this).val();
        var names = '';
        if (names == '') {
            names = value;
        }
        else {
            names = value + ', ' + names '.';
        }
        $("span#name1").append(names);
        return false;
    } else  {
        return true;
    }
}

is names html or just a plain string? You probably want to use html() and not append(). — epascarello
– epascarello, Commented Mar 5, 2012 at 23:48

Soufiane Hassou · Accepted Answer · 2012-03-05 23:40:45Z

1

name == '' is always true in your code, and that's because it's the initialization value of your variable.

answered Mar 5, 2012 at 23:40

Soufiane Hassou

17.8k2 gold badges42 silver badges76 bronze badges

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

Comments

Malk · Accepted Answer · 2012-03-05 23:39:53Z

0

if (names == '')

Should be:

if (value == '')

That is my guess simply because you are setting names = '' in the preceding line so it will always be true.

answered Mar 5, 2012 at 23:39

Malk

12k4 gold badges35 silver badges34 bronze badges

2 Comments

Ryan Brodie Over a year ago

But aren't I changing it's value to that of 'value' when I do 'names = value;' in the IF? Or does var have a fixed value?

Nico Burns Over a year ago

You are, but you are then changing it back again just before the if.

Collectives™ on Stack Overflow

JQuery IF statement not working when using .append()

2 Answers 2

Comments

2 Comments

Hot Network Questions

Collectives™ on Stack Overflow

2 Answers 2

Comments

2 Comments

Related