Record Collection using switch

can i use switch statements for this too?

  **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) {
switch (true){
  case (prop!== "tracks" && value !==""):
  records[id][prop] = value;
  
  case (prop==="tracks" && records[id].hasOwnProperty("tracks")=== false):
  records[id][prop] =[value];
 case (prop==="tracks" && value !==""):
  records[id][prop].push(value);
  case (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; rv:101.0) Gecko/20100101 Firefox/101.0

Challenge: Record Collection

Link to the challenge:

switch(true) is an antipattern. Switches are for choosing behavior based on a variable or expression taking one one of a finite list of values. That isn’t really applicable here.

2 Likes

Switch is for matching values, not checking conditions. You want if statements.

(You can use switch, but it’s not idiomatic, it’s the wrong tool for this job)

2 Likes

you could, as much as it is an antipattern

the main reason your code will not work is that you are not using break statements

2 Likes