solution #3690
solution #3690
Conversation
There was a problem hiding this comment.
Great job on implementing the convertToObject function! 🎉 It meets all the task requirements and constraints, and your use of iteration methods instead of loops is spot on. Just make sure to define or pass stylesString as an argument to avoid any runtime errors. Keep up the good work! 😊
✨ Please rate this review in the Platform's notification center. I will be grateful for your feedback! ✨
| // write your code here | ||
| const obj = {}; | ||
|
|
||
| const stylesArray = stylesString |
There was a problem hiding this comment.
The variable stylesString is used here but is not defined anywhere in the function. This will result in a ReferenceError. You need to ensure that stylesString is defined or passed as an argument to the function.
| const obj = {}; | ||
|
|
||
| const stylesArray = stylesString |
There was a problem hiding this comment.
res/result/el/elem/arr/array/obj/object and etc are bad names, variable name should be descriptive
| .map(style => style.trim()) | ||
| .filter(style => style.length > 0); | ||
|
|
||
| stylesArray.forEach(style => { |
There was a problem hiding this comment.
You can use reduce to generate an object.
| .filter(style => style.length > 0); | ||
|
|
||
| stylesArray.forEach(style => { | ||
| const [property, value] = style.split(':').map(item => item.trim()); |
There was a problem hiding this comment.
item is not informative name as well
No description provided.