update #390
update #390
Conversation
There was a problem hiding this comment.
Great job on the test cases! They are well-structured and comprehensive. Just a small note: consider translating comments to English for broader accessibility. Keep up the great work! 🌟
✨ Please rate this review in the Platform's notification center. I will be grateful for your feedback! ✨
| @@ -7,15 +7,15 @@ | |||
| */ | |||
| function getCoinCombination(cents) { | |||
| let currentAmount = cents; | |||
| const values = [1, 5, 10, 25]; | |||
| const values = [25, 10, 5, 1]; // Починаємо з кварталів | |||
There was a problem hiding this comment.
The comment is in Ukrainian. Consider translating it to English for consistency and better understanding by a wider audience.
| coins[i] = Math.floor(currentAmount / values[i]); | ||
| currentAmount -= coins[i] * values[i]; | ||
| } | ||
|
|
||
| return coins; | ||
| return [coins[3], coins[2], coins[1], coins[0]]; |
There was a problem hiding this comment.
The return statement reverses the order of the coins array. This means the function will return the number of pennies, nickels, dimes, and quarters, instead of quarters, dimes, nickels, and pennies. If the task requires returning the coins in the order of quarters, dimes, nickels, and pennies, you should return the 'coins' array as is.
No description provided.