parseInt not converting multiple string to int

Question

it was a very simple but don't no why i can't solve it.

I am trying to convert string to integer from json, it converts 1st object but from second object it doesn't:

HTML:

<body ng-controller="myApp">
  <ul ng-repeat="vals in json_arr" ng-init="parseId(vals)">
    <li>
      <input type="number" ng-model="vals.id" />
    </li>
    <li>
      <input type="number" ng-model="vals.id1" />
    </li>
    <li>
      <input type="number" ng-model="vals.id2" />
    </li>
  </ul>
</body>

Controller:

app.controller('myApp', function($scope) {
  $scope.json_arr = [{
    'id': '1',
    'id1': '2',
    'id2': '3'
  }];

  $scope.parseId = function(val) {
    val.id = parseInt(val.id);
  }
});

DEMO PLUNKER

user3030089 · Accepted Answer · 2014-06-25 04:59:28Z

1

Aahhhhh that's my mistake, In parseId function i only parse val.id.

The correct 1 is:

$scope.parseId = function(val) {
  val.id = parseInt(val.id);
  val.id1 = parseInt(val.id1);
  val.id2 = parseInt(val.id2);
}

answered Jun 25, 2014 at 4:59

user3030089

2721 silver badge14 bronze badges

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

Comments

Jeetendra Chauhan · Accepted Answer · 2014-06-25 05:05:06Z

1

change your code with this one

$scope.parseId = function(val) {
    val.id = parseInt(val.id);
    val.id1 = parseInt(val.id1);
    val.id2 = parseInt(val.id2);
}

or you can change your json_arr variable than you don't need to change string to int

$scope.json_arr = [{
    'id': 1,
    'id1': 2,
    'id2': 3
}];

edited Jun 25, 2014 at 5:05

answered Jun 25, 2014 at 4:59

Jeetendra Chauhan

1,9772 gold badges15 silver badges21 bronze badges

Comments

Michael Kang · Accepted Answer · 2014-06-25 05:03:42Z

Directives are executed in order from highest priority to lowest priority when defined on the same element. ng-repeat has a directive priority level of 1000, but ng-init has a directive priority of 450. ng-repeat also has a terminal property - meaning it is the last directive to execute, and no other directives can execute after it. There-in lies the problem. The ng-repeat executes first, and because it is terminal, it will not allow ng-init to execute after.

To solve this, move the ng-init to the first li below:

<body ng-controller="myApp">
  <ul ng-repeat="vals in json_arr">
    <li  ng-init="parseId(vals)">
      <input type="number" ng-model="vals.id" />
    </li>
    <li>
      <input type="number" ng-model="vals.id1" />
    </li>
    <li>
      <input type="number" ng-model="vals.id2" />
    </li>
  </ul>
</body>

This works because ng-model has a priority of 0, so it will execute after ng-init.

[EDIT]

I realized there is a logic error in the code you posted. The other two answers have already addressed it, but I'll post it again here for completeness:

$scope.parseId = function(val) {
    val.id = parseInt(val.id);
    val.id1 = parseInt(val.id1);
    val.id2 = parseInt(val.id2);
}

Collectives™ on Stack Overflow

parseInt not converting multiple string to int

3 Answers 3

Comments

Comments

Comments

Your Answer

Hot Network Questions

Collectives™ on Stack Overflow

3 Answers 3

Comments

Comments

Comments

Your Answer

Sign up or log in

Post as a guest

Related