If/else is not working in angularjs

Question

In this code if/else is not working. Am I making any mistake? data.success contains true/false. If I code like this if (data.success === true) then else block is working and if block is not working and vise versa.

$scope.verifyMobile = function () {
   var otp = {
      "otp": $scope.mobile.otp
   };
   $http({
      method: 'POST',
      url: 'verify_mobile',
      data: otp,
      headers: {
         'Content-Type': 'application/x-www-form-urlencoded'
      }
   }).success(function (data, status, headers, config) {
      if (data.success) {
          $scope.verified = true;
          $scope.sms_sent = false;
      } else {
          alert(data.message);
      }
   }).error(function (data, status, headers, config) {
   });
};

try to print the data type of the value data.success, alert(typeof data.success) before the if condition, becuase undefined and null can also be checked with the same if condition, in case your data.success is undefined or null then else part will trigger — Dickens A S
– Dickens A S, Commented Jul 13, 2016 at 4:49
@Scary he already told angularjs probably he is a beginner in stackoverflow and coding that is why he tagged as java — Dickens A S
– Dickens A S, Commented Jul 13, 2016 at 4:52
if (data.success) will return true as long as data.success is not undefined nor null. Use if (data.success == true) instead. — jegtugado
– jegtugado, Commented Jul 13, 2016 at 4:53
Change it to data.success[0], the success object is an array basing from your screenshot. — Adrian
– Adrian, Commented Jul 13, 2016 at 5:21

Ravi kumar Soni · Accepted Answer · 2016-07-13 05:56:17Z

You should change the data.success and data.message to data.success[0] and data.message[0], because that are not boolean values you returning array in response that's why you have to take it in a array format. Try below code.

$scope.verifyMobile = function () {
        var otp = {
            "otp": $scope.mobile.otp
        };
        $http({
            method: 'POST',
            url: 'verify_mobile',
            data: otp,
            headers: {
                'Content-Type': 'application/x-www-form-urlencoded'
            }
        }).success(function (data, status, headers, config) {
            if (data.success[0]) {
                $scope.verified = true;
                $scope.sms_sent = false;
            } else {
                alert(data.message[0]);
            }
        }).error(function (data, status, headers, config) {
        });
};

Surjeet Bhadauriya · Accepted Answer · 2016-07-13 05:01:01Z

0

This is because your data.success is not contains boolean value. So before your if-else block try to print type of data.success

console.log(typeof data.success);

And see is it boolean if not then just resolve it.

answered Jul 13, 2016 at 5:01

Surjeet Bhadauriya

7,2223 gold badges39 silver badges54 bronze badges

1 Comment

Mohammad Amir Over a year ago

when I do alert(typeof data.success), it returns object

developer033 · Accepted Answer · 2016-07-13 05:26:16Z

0

Instead of .success() use .then().

Response will return an object you should check the response as below

$scope.httpRequest = function() {
  $http({
    method: 'GET',
    url: 'http://jsonplaceholder.typicode.com/posts/1',
  }).then(function(success) {
    if (success.data.userId === 1) {
      $scope.name = 'Jason Statham'
    } else {
      $scope.name = 'Cristiano Ronaldo'
    }
  }, function(error) {
    console.log(error)
  })
}

DEMO

edited Jul 13, 2016 at 5:26

developer033

25k8 gold badges85 silver badges113 bronze badges

answered Jul 13, 2016 at 5:03

Arif

1,6419 silver badges18 bronze badges

Collectives™ on Stack Overflow

If/else is not working in angularjs

3 Answers 3

Comments

1 Comment

Comments

Your Answer

Hot Network Questions

Collectives™ on Stack Overflow

3 Answers 3

Comments

1 Comment

Comments

Your Answer

Sign up or log in

Post as a guest

Related