How to do if statment in AngularJS controller

Question

I have an angular controller, in which i have a function that should return an image depends on the json that is received from the server. The problem is that it does not matter which country is received it load only the first one (usa.png in my case).

$scope.getUser = function(){
		url = "/get";
		$scope.img = "";
		$http.post(url, {
			"Id":$scope.Id,
			"name":$scope.name
		}).then(
				function (response){
				$scope.returnedUser = response.data;
				if ($scope.returnedUser.country = "USA"){
					$scope.img = "/base_icons/usa.png";
				} else if ($scope.returnedUser.country = "Canada"){
					$scope.img = "/base_icons/canada.png";
				} else if ($scope.returnedUser.country = "Mexico"){
                    $scope.img = "/base_icons/mexico.png";
                }
				}, $scope.negativeMessage);};

<img ng-src="{{img}}"/>

Callum Linington · Accepted Answer · 2016-08-04 08:38:16Z

6

You aren't doing equality:

if ($scope.returnedUser.country = "USA") // this is an assignment operator in if

Should be:

if ($scope.returnedUser.country == "USA")

Or you can have strict equality (recommended):

if ($scope.returnedUser.country === "USA")

Strict equality is good (in most cases), because it means, things like '1' === 1 don't return true, where as '1' == 1 would return true.

answered Aug 4, 2016 at 8:38

Callum Linington

14.4k14 gold badges80 silver badges156 bronze badges

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

2 Comments

Fima Taf Over a year ago

Cant believe that it was my mistake, I thought that it is somethng more complicated, Thank you!

Callum Linington Over a year ago

It happens to the best of us mate, don't worry!

Antipod · Accepted Answer · 2016-08-04 08:39:09Z

1

You assign value, but you have to check it:

$scope.returnedUser.country == "USA"

answered Aug 4, 2016 at 8:39

Antipod

1,6234 gold badges25 silver badges43 bronze badges

Comments

Hardik Vaghani · Accepted Answer · 2016-08-04 08:46:23Z

0

You can use as this way if countries are already fixed from particular group of countries and comparing is not compulsory.

$scope.image_base = "/base_icons/";
$scope.returnedUser = response.data;
$scope.img = $scope.image_base + ($scope.returnedUser.country).toLowerCase() + ".png";

Regards.

answered Aug 4, 2016 at 8:46

Hardik Vaghani

2,18330 silver badges48 bronze badges

Collectives™ on Stack Overflow

How to do if statment in AngularJS controller

3 Answers 3

2 Comments

Comments

Comments

Your Answer

Hot Network Questions

Collectives™ on Stack Overflow

3 Answers 3

2 Comments

Comments

Comments

Your Answer

Sign up or log in

Post as a guest

Related