Solution #245
Solution #245
Conversation
There was a problem hiding this comment.
Your code meets the task requirements and successfully implements the file moving functionality similar to the Linux mv command. Great job! 🎉 While your solution works well, consider adding checks to ensure the source file exists and handling potential errors more gracefully. Additionally, verifying that both source and dest are provided as command-line arguments can help prevent runtime issues. Keep up the good work and continue refining your skills! 😊
✨ Please rate this review in the Platform's notification center. I will be grateful for your feedback! ✨
|
|
||
| (() => { | ||
| try { | ||
| const [source, dest] = process.argv.slice(2); |
There was a problem hiding this comment.
Consider adding a check to ensure that both source and dest are provided as command-line arguments. If they are not, you could log an error message and exit the process early to prevent runtime errors.
| finalPath = path.resolve(dest, path.basename(source)); | ||
| } | ||
|
|
||
| fs.renameSync(source, finalPath); |
There was a problem hiding this comment.
Using fs.renameSync can throw an error if the source file does not exist or if there are permission issues. You might want to add checks to ensure the source file exists before attempting to rename it, or handle specific error types in the catch block.
Solution