add task solution #800
add task solution #800
Conversation
There was a problem hiding this comment.
Great job on implementing the createServer function! 🎉 Your code meets all the task requirements and handles HTTP requests, URL parsing, and input validation effectively. There's a small opportunity for improvement by removing a redundant check, which can simplify your code. Keep up the excellent work and continue refining your skills! 😊
✨ Please rate this review in the Platform's notification center. I will be grateful for your feedback! ✨
src/createServer.js
Outdated
| if (!path || path === '/') { | ||
| errors.push({ | ||
| message: | ||
| // eslint-disable-next-line max-len | ||
| 'Text to convert is required. Correct request is: "/<TEXT_TO_CONVERT>?toCase=<CASE_NAME>".', | ||
| }); | ||
| } |
There was a problem hiding this comment.
The check for !path || path === '/' is redundant because textToConvert is derived from path.slice(1), which will be an empty string if path is '/'. Therefore, the check on line 19 already covers this case. You can remove this block to simplify the code.
No description provided.