Home/ Questions/Q 6244897
In Process

The Archive Base Latest Questions

Editorial Team
  • 0
Asked: 2026-05-24T12:24:38+00:00

I am learning jquery and started my first attempt at building a form validation

  • 0

I am learning jquery and started my first attempt at building a form validation script based on what I know so far (which isnt much).

This is really only the Radio button portion of the validation script, but I thought I get on the right track -coding wise- before I went too far. There are some fundamental issues that I know need addressing.

The Script (jsFiddle): http://jsfiddle.net/pkdsleeper/xNt5n/

The Questions:

a. How best to remove the global variables using 
b. jsLint recommends "use strict", so I added it, but im not sure what it does. 
c. any good refs?
d. Generally, feel free to rip this code apart (cuz I AM trying to learn) but 
   please explain my errors in noob-speak :)

Thanks In advance
sleeper

Share

Leave an answer

You must login to add an answer.

Need An Account,

1 Answer

  1. Editorial Team

    a. How best to remove the global variables using

    Wrap it in an anonymous function and assign it to the form as a submit listener.

    b. jsLint recommends “use strict”, so I added it, but im not sure what it does. any good ref’s?

    Don’t bother. It’s just a buzz-word for those trying to be hip. You can’t use strict mode features on the general web because way too many browsers don’t support them. You can use it with otherwise compliant ES 3 code, but it’s only useful as a debugging tool for errors that should have been found during testing anyway (e.g. calling constructors without new).

    No c?

    d. Generally, feel free to rip this code apart (cuz I AM trying to learn) but please explain my errors in noob-speak 🙂

    >   $rdoGroup = [], // an empty array which will be used to hold

    You seem to be using $ to indicate a variable that references a jQuery object, but $rdoGroup is just an array. That may be confusing later.

    >     $rdoGroup.push($(this).attr("name"));

    The $ function is very expensive, don’t use it if you don’t need to. Standard HTML attributes are available in all browsers as DOM properties, so:

          $rdoGroup.push(this.name);

    Is up to 100 times more efficient, depending on the browser.

    >     for (i = 0; i < $rdoGroup.length; i++) {               
>         if ($rdoGroup[c].toString() !== $(this).attr("name").toString()) {

    The values assigned to $rdoGroup are strings, calling their toString method is redundant.

    As above, use this.name. The name property is a string, so no need for toString.

    I think the exercise would be easier without jQuery, which seems to be getting in the way far more than helping. If you are trying to learn javascript, I’d suggest that you learn javascript without using any library.

    Once you are reasonably confident with using javascript, then you are far better equipped to use a library for the things the library is good with, and not using it for the things it isn’t good at.

    • 0