Angularjs reports "Cannot set property of undefined"

Question

I'm getting an error that says "cannot set property of undefined".

vm.dtr1 = {}; // maybe I'm initializing this wrong ?
vm.dtr1.date1 = {}; // maybe I'm initializing this wrong ?

for(var x=0; x<5; x++){
  vm.dtr1[x].date1 = new Date(days[x]); //getting undefined value
}

score 3 · Answer 1 · answered Mar 23 '18 at 05:38

3

vm.dtr is an object, but you are trying to use it as an array by accessing the index. Which will obviously undefined.

You need to declare vm.dtr as an array of type dtx.

answered Mar 23 '18 at 05:38

Sajeetharan

216,225
63
350
396

I've tried to change `vm.dtr1 = []` but it still throwing the error. – code.cycling Mar 23 '18 at 05:42
just doing that won't work, you need to have the objects of type dtx – Sajeetharan Mar 23 '18 at 05:55

Suren Srapyan · Accepted Answer · 2018-03-23T05:51:27.193

2

You need to change it to be an array, then before assigning a property it must be initialized.

vm.dtr1 = []; // make it an array

for(var x=0; x<5; x++) {
   vm.dtr1[x] = {}; // initialize it first
   vm.dtr1[x].date1 = new Date(days[x]);
}

Or in better way

vm.dtr1 = [];

for(var x=0; x<5; x++) {
   vm.dtr1[x] = { date1:  new Date(days[x])};
}

edited Mar 23 '18 at 05:51

answered Mar 23 '18 at 05:48

Suren Srapyan

66,568
14
114
112

Thank you so much. its working now i've used your second answer – code.cycling Mar 23 '18 at 05:51
The second is better. In the first I just want to show the initialization – Suren Srapyan Mar 23 '18 at 05:51

score 1 · Answer 3 · answered Mar 23 '18 at 05:50

1

Try following changes will suely work

vm.dtr1 = []; // change here
//vm.dtr1.date1 = {}; // remove this

for(var x=0; x<5; x++){
  vm.dtr1[x] ={}
  vm.dtr1[x].date1 = new Date(days[x]); //
}

answered Mar 23 '18 at 05:50

Neha Tawar

687
7
23

score 0 · Answer 4 · answered Mar 23 '18 at 05:41

The problem is that the possible values of x are not correct keys for vm.dtr1.

You may want to use for...in loop instead:

for(var x in vm.dtr1) {
  if (vm.dtr1.hasOwnProperty(x)) {
    vm.dtr1[x] = new Date(days[x]);
  }
}

I still don't know what days is. So, you need to figure that out. For more details about iterating over an object, see this post.

score 0 · Answer 5 · answered Mar 23 '18 at 05:57

I think you might need to do it like this

var vm = {}; vm.dtr1 = []; 
var day = ['7/7/2012','7/7/2012','7/7/2012','7/7/2012','7/7/2012']
  vm.dtr1[x] = {};
  vm.dtr1[x].date1 = new Date(days[x]);
}

Still don't know what days is, i did is as expected input for Date constructor

Angularjs reports "Cannot set property of undefined"

5 Answers5