array pushing on button click jquery

Question

I'm trying to improve on some javaScript I wrote to push elements into an array if they were not previously in the array. Then, I want that array displayed in a selection field. I had this all working in javascript but thought it would be an interesting exercise for myself to try doing the same thing in jQuery. I think I'm close but for some reason, the console in my browser gives me an error as follows:

"[Error] SyntaxError: Cannot use the keyword 'var' as a variable name. (anonymous function)"

Here is my code so far. I've included (in this order) my attempt at the jQuery and then my working javascript code (commented out), as well as my function updating the drop list. I'll eventually attempt to recreate that function in jQuery as well. Can anyone tell me why I'm getting this error and or what I'm doing wrong in my jQuery? Thank you.

     var clientArray = [];
        var clientInput = document.getElementById("newClt");
        var sel = document.getElementById("cltList");


            $("#addCltBtn").click(function(){

                var found = $.inArray(clientInput, clientArray) > -1;

                if(found >= 0) {
                    //Element was found update errorMessage div
                    $(errorMessage).html("This Client Already Exists");

                }   else{
                        clientArray.push(clientInput.value);

                }   
                });
    //  
    //      document.getElementById("addCltBtn").onclick=function(){
    //          
    //          console.log(clientArray.length);
    //          if (!contains(clientArray,clientInput.value)) {
    //              clientArray.push(clientInput.value);
    //              console.log("Objects: " + clientArray.join(", "));
    //              updateDropList(clientInput.value);
    //          }
    //          else alert("You already have a client by that name!");
    //          }


        function updateDropList(value){
                var opt = document.createElement('option');
                opt.innerHTML = value;
                opt.value = value;
                sel.appendChild(opt);

        }

function contains(a, obj){
            for(var i =0; i< a.length; i++){
                if(a[i] === obj){
                    return true;
                }

            }
            return false;
        }

When does this error occur? What line does it point to? The code sample you've included seems ok. — Pavlo
– Pavlo, Commented May 5, 2015 at 14:39
it points to line 65 which on my code is var found = $.inArray(clientInput, clientArray) > -1; — Panayiotis Spanos
– Panayiotis Spanos, Commented May 5, 2015 at 14:40
"doing the same thing in jQuery" - jQuery is not a language. You are still using JavaScript. — Tomalak
– Tomalak, Commented May 5, 2015 at 14:40
perhaps it would be more accurate to say "using jQuery" Tomalak, but I feel like your comment is unnecessary. — Panayiotis Spanos
– Panayiotis Spanos, Commented May 5, 2015 at 14:44

Joe Urc · Accepted Answer · 2015-05-05 14:40:19Z

4

var found = $.inArray(clientInput, clientArray) > -1;

Why are you placing a comparison in this statement? This is what is throwing your error. You check if found is greater than or equal to 0 elsewhere, so removing the > -1 should make everything work.

answered May 5, 2015 at 14:40

Joe Urc

4975 silver badges19 bronze badges

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

1 Comment

A. Wolff Over a year ago

Or as it returns a boolean, just use it

Collectives™ on Stack Overflow

array pushing on button click jquery

1 Answer 1

1 Comment

Your Answer

Linked

Hot Network Questions

Collectives™ on Stack Overflow

1 Answer 1

1 Comment

Your Answer

Sign up or log in

Post as a guest

Linked

Related