Basic JavaScript - Record Collection

JavaScript
1

hello fellow coders. your help would be much appreciated . this is my code. the part that troubles me is those few lines, that should delete given object property if the value is an empty string. in one example ( when the value of a “track” is empty) it does what it’s supposed to, but in the same example (when the value of an “artist” is empty) it does not. i just can’t figure out why one would work and the other would not…

Your code so far

// 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 6.1) AppleWebKit/537.36 (KHTML, like Gecko) Chrome/106.0.0.0 Safari/537.36

Challenge: Basic JavaScript - Record Collection

Link to the challenge:

2

the deleting feature is completely correct , copy your code reset the editor,
and paste it after refreshing the page , it will work ,

also just make it first condition with only if, and then check other with else,

I just ran your code with the swapped conditions and it worked fine.

3 
if (prop !== 'tracks' && 'value' !=='')

Are you sure you want to use the string 'value' for this condition?

4

thank you, I missed that…

5

:astonished: :astonished: :astonished:
I was losing my mind over this! thank you so much!

1 Like
6

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