is not a number in angular js

Question

I created this code to check if value is number or not but it doesn't seem to work.

app.controller('ForCtrl', function($scope) {
    $scope.calcul_factorielle = function() {

        var mm = 1;
        if ((typeof ($scope.test) !== 'number')) $scope.factorielle = " invalide saisie";

        for(i=2;i<=$scope.test;i++) {
            mm = mm * i;
        }
        $scope.factorielle = mm;
    }
});

Why, when I'm entering string stackoverflow am I not receiving the 'invalide saisie' message in the output?

Suren Srapyan · Accepted Answer · 2017-06-17 21:09:48Z

2

After this if statement, if your text is not a number, you need to break the execution of the function with the keyword return. In your case the code continues after the if statement, so you get a wrong result.

For first you need to parse it into number, because every input is a string and after that then try to use isNaN() function. If string can not be parsed to the number it will return an NaN.

var number = Number.parseInt($scope.test);
if (isNaN(number)) { // or i
       $scope.factorielle = " invalide saisie";
       return;
}

edited Jun 17, 2017 at 21:09

answered Jun 17, 2017 at 21:01

Suren Srapyan

68.9k14 gold badges128 silver badges119 bronze badges

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

2 Comments

paulo linko Over a year ago

I modifier my code with your code but I haven't any result , and when i entering a number value i haven't any result also

paulo linko Over a year ago

okey now when i entering a number value i have message invalid saisie :(

Andriy Ivaneyko · Accepted Answer · 2017-06-19 09:41:11Z

0

The problem is in the if statement, apply parseFloat before typeof operator, so replace:

typeof ($scope.test) !== 'number'

To:

(typeof (parseFloat($scope.test)) !== 'number')

edited Jun 19, 2017 at 9:41

answered Jun 17, 2017 at 21:13

Andriy Ivaneyko

22.3k7 gold badges70 silver badges89 bronze badges

1 Comment

paulo linko Over a year ago

Thank you to all to be of help me

Collectives™ on Stack Overflow

is not a number in angular js

2 Answers 2

2 Comments

1 Comment

Your Answer

Hot Network Questions

Collectives™ on Stack Overflow

2 Answers 2

2 Comments

1 Comment

Your Answer

Sign up or log in

Post as a guest

Related