jQuery syntax error

Question

I am trying to set saveData when there is at least one field filled in:

    $(document).ready(function () {
        $(":input").each(function() {
            if($(this).val() != "") 
                window.setInterval(saveData, 5000);
        }
    });

There is a syntax error Expected ')' in this code.

What am I missing here?

You should try to use some IDE that highlights matching/missing parenthesis — Oleg Mikheev
– Oleg Mikheev, Commented Feb 8, 2012 at 9:36
Yeah. Visual Studio 2008 does not seem to have that. I have no option for the 2010 version now. — Animesh D
– Animesh D, Commented Feb 8, 2012 at 9:40
if you're doing just front-end programming you could try Eclipse IDE for JavaScript Web Developers — Oleg Mikheev
– Oleg Mikheev, Commented Feb 8, 2012 at 9:45
No it is both server and client side development. So no other option as of now. Existing website is in ASP.NET. So not possible. Thanks for letting me know of the possibility. — Animesh D
– Animesh D, Commented Feb 8, 2012 at 9:47

alex · Accepted Answer · 2012-02-08 09:34:45Z

3

each() is a function, so you need to close its invocation parentheses. You are missing the trailing one.

$(document).ready(function () {
    $(":input").each(function() {
        if($(this).val() != "") 
            window.setInterval(saveData, 5000);
    }); // <-- You were missing the closing `)` here.
});

answered Feb 8, 2012 at 9:34

alex

492k205 gold badges890 silver badges992 bronze badges

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

1 Comment

Animesh D Over a year ago

Hey thanks your answer helped me. I will mark it in a few minutes. If you don't mind can you let me know how to modify the function to save data only when there is at least one field filled in. Currently it is doing the save when the form loads after the interval of 5 seconds. After that, it doesn't matter even if data is present in the form or not.

user971401 · Accepted Answer · 2012-02-08 09:34:51Z

1

You are missing a ')' :

$(document).ready(function () {
    $(":input").each(function() {
        if($(this).val() != "") 
            window.setInterval(saveData, 5000);
    });
});

answered Feb 8, 2012 at 9:34

user971401

Comments

Bruno Costa · Accepted Answer · 2012-02-08 09:35:29Z

1

 $(document).ready(function () {
        $(":input").each(function() {
            if($(this).val() != "") 
                window.setInterval(saveData, 5000);
        });  <-- here
    });

answered Feb 8, 2012 at 9:35

Bruno Costa

2,7202 gold badges17 silver badges25 bronze badges

Comments

rjovic · Accepted Answer · 2012-02-08 09:35:45Z

1

$(document).ready(function () {
        $(":input").each(function() {
            if($(this).val() != "") 
                window.setInterval(saveData, 5000);
        });
    });

If I'm correct you forgot to close each() method..

answered Feb 8, 2012 at 9:35

rjovic

1,2473 gold badges16 silver badges38 bronze badges

Comments

Anders Lien · Accepted Answer · 2012-02-08 09:36:31Z

1

You need to close your .each function:

$(document).ready(function () {
    $(":input").each(function() {
        if($(this).val() != "") 
            window.setInterval(saveData, 5000);
    });
});

answered Feb 8, 2012 at 9:36

Anders Lien

4553 silver badges6 bronze badges

Comments

viebel · Accepted Answer · 2012-02-09 06:39:41Z

1

In general, it is very helpful to use jslint to validate your javascript code

answered Feb 9, 2012 at 6:39

viebel

20.9k10 gold badges54 silver badges86 bronze badges

Comments

Reporter · Accepted Answer · 2012-02-08 09:35:52Z

0

It must be window.setInterval("saveData()", 5000);

answered Feb 8, 2012 at 9:35

Reporter

3,9345 gold badges36 silver badges50 bronze badges

2 Comments

Reporter Over a year ago

why? According some different guides, they use this syntax. Ok, he forgot the close braces, thoough I think he will run into next question.

alex Over a year ago

They would be bad guides. The only reason I could think of to use that way is to preserve the this when calling an object's method and you must for some reason have terse code. Placing the function invocation in a string means that string needs to be eval()'d internally.

Collectives™ on Stack Overflow

jQuery syntax error

7 Answers 7

1 Comment

Comments

Comments

Comments

Comments

Comments

2 Comments

Hot Network Questions

Collectives™ on Stack Overflow

7 Answers 7

1 Comment

Comments

Comments

Comments

Comments

Comments

2 Comments

Related