CODEWITHSUNDEEP
javascripthtmlradio-button
chriz
chriz

Reputation: 1345

condition on radio button onclick, if false don't check

onclick of one radio button i have to check one condition if true i have to check it or restore to same old status,

html

<input type="radio" name="test" id="radio0" onclick="myFunction()" checked /> 
<input type="radio" name="test" id="radio1" onclick="myFunction()" /> 
<input type="radio" name="test" id="radio2" onclick="myFunction()" />

JS

globalCondition = false;
function myFunction()
{
   if(globalCondition)
   {
      //some codes and it will check clicked radio button anyway 
   }
   else
   {
       return false; // i tried this but its not working
      /*here i don't want to check clicked radio button, and previously
       checked button should be checked*/ 
   }
}

Upvotes: 2

Views: 2256

Answers (3)

iCollect.it Ltd
iCollect.it Ltd

Reputation: 93611

As I said in comment return in the function will not do anything as you're not returning the function value in the in-line code.

Although the other solutions offered are correct, to keep your code unobtrusive, you should not have inline JS at all (remove the onclick='s).

I realize the question was not tagged jQuery, but maybe it should have been: Instead on onclick you should use a jQuery event handler, selecting only the set of radio buttons.

JSFiddle: http://jsfiddle.net/TrueBlueAussie/KVwL3/1/

globalCondition = false;

$(function(){
    $("[name=test]").click(function(e){
       if(globalCondition)
       {
          //some codes and it will check clicked radio button anyway 
       }
       else
       {
           return false;
            // or
            e.preventDefault();
           /*here i don't want to check clicked radio button, and previously
           checked button should be checked*/ 
       }
    });
});

Notes:

DOM ready event:

$(function(){ YOUR CODE HERE }); is a shortcut for $(document).ready(function(){ YOUR CODE HERE});

Selectors

If an attribute [] selector = value contains special characters it needs to be quoted:

e.g.

$('[name="IstDynamicModel[SD_WO_CREATE]"]')

There are any number of selectors that will choose just the three radio buttons. As this is a simple one-off connection, at startup, there is no real speed difference between any options, but you would normally try and make the selector specific in ways that might make use of various lookup tables available to the browser:

e.g.

$('input[type=radio][name="IstDynamicModel[SD_WO_CREATE]"]')

This will be slightly faster as it will reduce the slowest check (the name=) to only radio inputs.

Upvotes: 2

Anoop Joshi P
Anoop Joshi P

Reputation: 25537

USe like this

<input type="radio" name="test" id="radio1" onclick="return myFunction()" />

javascript

globalCondition = false;
function myFunction(e)
{
   if(globalCondition)
   {
      //some codes and it will check clicked radio button anyway 
     return true;
   }
   else
   {

       return false; // i tried this but its not working
      /*here i don't want to check clicked radio button, and previously
       checked button should be checked*/ 
   }
}

Upvotes: 1

Amin Jafari
Amin Jafari

Reputation: 7207

try this:

globalCondition = false;
function myFunction(e)
{
   if(globalCondition)
   {
      //some codes and it will check clicked radio button anyway 
   }
   else
   {
       e.preventDefault();
       return false; // i tried this but its not working
      /*here i don't want to check clicked radio button, and previously
       checked button should be checked*/ 
   }
}

Upvotes: 1

Related Questions