Combining two simple scripts

Question

How can I modify the script below so that it alerts not only when the field is empty, but when the field is empty or contains less than 10 digits (it can contain anything, not only digits)?

  if (myform.mytextinput.value=="")
  alert ('Please enter something!");

A good script that checks for ten digits is this:

  var input_dg = document.getElementById("mytextinput");
  var text_dg = input_dg.value;
  var totalNrOfDigits = 0;
  for(var i = 0; i < text_dg.length; i++){
  if(/\d/.test(text_dg[i])){
  totalNrOfDigits++;
  }
  }
  alert ('Please enter at least 10 digits!');

I just need to "paste" the second script in the first, but I lack the know-how. Both scripts are inside a large form validation script...

Community · Accepted Answer · 2017-05-23 12:13:14Z

5

Just use ||, the OR operator:

if (myform.mytextinput.value=="" || myform.mytextinput.length < 10)

To count the number of digits in a string I recommend this code (From this SO answer):

var totalNrOfDigits = myform.mytextinput.replace(/[^0-9]/g,"").length;

// And now combine both checks:
if (myform.mytextinput.value=="" || totalNrOfDigits < 10){
    // Alert the user
}

If you want to use the same code you were using you only need to replace the first part:

var input_dg = document.getElementById("mytextinput");
var text_dg = input_dg.value;
var totalNrOfDigits = 0;
for(var i = 0; i < text_dg.length; i++){
    if(/\d/.test(text_dg[i])){
       totalNrOfDigits++;
    }
}

if (myform.mytextinput.value=="" || totalNrOfDigits < 10){
    // Alert the user
}

edited May 23, 2017 at 12:13

CommunityBot

11 silver badge

answered Apr 9, 2014 at 8:24

Alejandro Urbano Alvarez

3,3361 gold badge27 silver badges39 bronze badges

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

Comments

Rahul Tripathi · Accepted Answer · 2014-04-09 08:24:27Z

2

Try this using the OR(||):

if (myform.mytextinput.value=="" || myform.mytextinput.length < 10)

answered Apr 9, 2014 at 8:24

Rahul Tripathi

173k33 gold badges292 silver badges341 bronze badges

2 Comments

a-d Over a year ago

This does not count how many digits are contained in the input.

Rahul Tripathi Over a year ago

@aduch:- Yes thats true actually I focussed on the main part of the question which asks about combining the scripts. However the answer provided by aurbano is adding that detail(already upvoted that!)myform.mytextinput.replace(/[^0-9]/g,"").length

a-d · Accepted Answer · 2014-04-09 08:26:24Z

2

You can count them with a regex

var str ='1a2b3',
    digits = (str).match(/\d+/g); //  returns ["1", "2", "3"]

if ((! str.length) || (! digits) || (digits.length < 10)) {
    alert ('Please enter at least 10 digits!');
}

answered Apr 9, 2014 at 8:26

a-d

10.9k2 gold badges34 silver badges50 bronze badges

Comments

chris97ong · Accepted Answer · 2014-04-09 08:33:20Z

1

Did you want this?

var input_dg = document.getElementById("mytextinput");
var text_dg = input_dg.value;
if(text_dg == "") {
alert ('Please enter something!');
} else if(text_dg.length < 10) {
alert("Please enter at least 10 digits!");
}

Fiddle.

edited Apr 9, 2014 at 8:33

answered Apr 9, 2014 at 8:26

chris97ong

7,0809 gold badges37 silver badges54 bronze badges

Collectives™ on Stack Overflow

Combining two simple scripts

4 Answers 4

Comments

2 Comments

Comments

Comments

Your Answer

Linked

Hot Network Questions

Collectives™ on Stack Overflow

4 Answers 4

Comments

2 Comments

Comments

Comments

Your Answer

Sign up or log in

Post as a guest

Linked

Related