2 if in jquery script to valid form

Question

I have a form with two radio input. For one of these two radio input (type1) I display 2 input text. I would like before submiting the form to check if minimum one of these 2 input text boxes is not empty. But my script doesn't work.

HTML

<input id="type1" value="1" type="radio">
<input id="num1" value="" type="text">
<textarea id="numlist"></textarea>
<input id="type2" value="1" type="radio">

JQUERY

$('#form2').on('submit', function (e) {
    if ($("input:radio[id=type1]").is(":checked")) {
        var comment2 = $('#numliste').val();
        var comment1 = $("#num1").val();
        if ((comment1.length === 0) || (comment2.length === 0)) {
            e.preventDefault();
            alert('You have to write on minimum one input text');
            return false;
        }
    }
});

Maybe you have an other problem related to your type="radio". You have to add the same name attribute for both type1 and type2 to make them act like "radios" — Asenar
– Asenar, Commented Dec 5, 2013 at 15:39

Asenar · Accepted Answer · 2013-12-05 15:54:36Z

3

$('#form2').on('submit', function (e) {
  var type1_checked = $("#type1").is(":checked");
  if (type1_checked) {
    var comment2 = $('#numlist').val();
    var comment1 = $("#num1").val();
    if ((comment1 == "") && (comment2 == "")) {
        e.preventDefault();
        alert('You have to write on minimum one input text');
        return false;
    }
  }
  return true;
});

EDIT: Also I assume you want the radio buttons to act like options, so you need the property name. See below for an example (I changed the radio values too):

<form id="form2">
  <input name="options" id="type1" value="type1" type="radio">
  <input id="num1" value="" type="text">
  <textarea id="numlist"></textarea>
  <input name="options" id="type2" value="type2" type="radio">
</form>

Which allows to improve the jquery code

$('#form2').on('submit', function (e) {
  var radio_value = $("input[name=options]:checked").val();
  if (!radio_value)
  {
    alert("Choose an option !");
    return false;
  }
  if (radio_value == "type1")
  {
    var comment2 = $('#numlist').val();
    var comment1 = $("#num1").val();
    if ((comment1 == "") && (comment2 == "")) {
        e.preventDefault();
        alert('You have to write on minimum one input text');
        return false;
    }
  }
  return true;
});

edited Dec 5, 2013 at 15:54

answered Dec 5, 2013 at 15:30

Asenar

7,0763 gold badges44 silver badges53 bronze badges

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

1 Comment

Raphaël Althaus Over a year ago

you could simplify the test to if (!comment1 && !comment2)

Tomasz Buczeń · Accepted Answer · 2013-12-05 15:32:39Z

0

First of all you have a typo in var comment2 = $('#numliste').val(); change it to "numlist" And if you want this work you have to change the || to && in if statement.

answered Dec 5, 2013 at 15:32

Tomasz Buczeń

18813 bronze badges

Comments

Shaik Rilwan · Accepted Answer · 2013-12-05 15:40:32Z

-1

i think this will work

$('#form2').on('submit',function (e) {
    if($("input:radio[id=type1]").is(":checked"))
              {
                var comment2 = $('#numliste').text();
                var comment1 = $("#num1").text();
                if((comment1.length === 0) || (comment2.length === 0))
                {
                    e.preventDefault();
                    alert('You have to write on minimum one input text');
                    return false;
                }
              } });

edited Dec 5, 2013 at 15:40

answered Dec 5, 2013 at 15:26

Shaik Rilwan

3552 silver badges8 bronze badges

3 Comments

Raphaël Althaus Over a year ago

minimum one of these 2 input text boxes is not empty. With your code, the form won't be submitted if one of the text box is empty. You need an and not an or

Raphaël Althaus Over a year ago

No, you still need an and.

Raphaël Althaus Over a year ago

Yes, and that's what we want. With your code, if one condition is true (one input is empty, not the other), the form won't be submitted. This is in contradiction with "minimum one is not empty".

Collectives™ on Stack Overflow

2 if in jquery script to valid form

3 Answers 3

1 Comment

Comments

3 Comments

Your Answer

Hot Network Questions

Collectives™ on Stack Overflow

3 Answers 3

1 Comment

Comments

3 Comments

Your Answer

Sign up or log in

Post as a guest

Related