Basic JavaScript - Record Collection

JavaScript
1

Hi all, could someone explain why dot notation isn’t working here, please?

EDIT: Found the error in the code where “value” wasn’t sqare bracketed to create initially absent array.
Still not sure why dot notation won’t work.

Thanks!

// Setup
const recordCollection = {
  2548: {
    albumTitle: 'Slippery When Wet',
    artist: 'Bon Jovi',
    tracks: ['Let It Rock', 'You Give Love a Bad Name']
  },
  2468: {
    albumTitle: '1999',
    artist: 'Prince',
    tracks: ['1999', 'Little Red Corvette']
  },
  1245: {
    artist: 'Robert Palmer',
    tracks: []
  },
  5439: {
    albumTitle: 'ABBA Gold'
  }
};

// Only change code below this line
function updateRecords(records, id, prop, value) {

if (prop !== "tracks" && value !== "") {

  records.id.prop = value;

} else if (prop === "tracks" && records.id.hasOwnProperty("tracks") === false) {

  records.id.prop = [value];

} else if (prop === "tracks" && value !== "") {

  records.id.prop.push(value);

} else if (value === "") {

  delete records.id.prop;

}



  return records;
}

updateRecords(recordCollection, 5439, 'artist', 'ABBA');

Your browser information:

User Agent is: Mozilla/5.0 (Windows NT 10.0; Win64; x64) AppleWebKit/537.36 (KHTML, like Gecko) Chrome/106.0.0.0 Safari/537.36 Edg/106.0.1370.52

Challenge: Basic JavaScript - Record Collection

Link to the challenge:

2

When using dot notation to access a property, the property name or identifier must not start a with a number. All the record collection keys are numbers, so that’s why the dot notation is not working.
(to test this, change all the numbers to letters like a b c d and then try to use the dot notation then and it will work)

1 Like
3

Thanks, this makes sense.
However it doesn’t seem to work if I bracket ids only, like this:
records[id].prop = [value]

4

I don’t think you can mix notations like this. (you should pick one method)
But also I just realized that dot notation doesn’t work with variables. It works with identifiers (so the name of the property but not the variable that contains the name of the property)

1 Like
5

Yes, you’re right. I totally forgot about that part, thinking about spaces in the names.
It’s a bummer about notation mixing, though.

6

to be honest, I’m not 100% sure about that. You can try and see if you can get it to work…
Edit: i just tried it and it worked. So i guess it is allowed after all…

7

This topic was automatically closed 182 days after the last reply. New replies are no longer allowed.