add server script #1
Conversation
scripts/server.js
Outdated
| * @param {http responce} response | ||
| */ | ||
| let getStream = response => filename => fs | ||
| .createReadStream(`./${filename}`) |
There was a problem hiding this comment.
Думаю, здесь можно указать просто .createReadStream(filename).
Так немного чище, а результат равнозначный. Плюс т.о. мы позволяем вызывать функцию с абсолютными путями
| let getStream = response => filename => fs | ||
| .createReadStream(`./${filename}`) | ||
| .on('error', () => { | ||
| console.log(color('red'), "Error! Can't read file 'index.html'. Check, that file is exists & correct."); |
There was a problem hiding this comment.
Опечатка:
Error! Can't read file 'index.html'. Check that file exists & correct.
scripts/server.js
Outdated
| let showHelp = () => void console.log(` | ||
| Welcome to server.js script! This script run the server. | ||
|
|
||
| You can use agruments: |
There was a problem hiding this comment.
Общепринятым названием, думаю, будет options.
Принято разделять аргументы (arguments) и опции (options). Аргументы - порядковы, опции - именованные. Например,
program argument1 argument2 --option1 value --option2=value
scripts/server.js
Outdated
| index => { | ||
| if (isNaN(index)){ | ||
| console.error(color('red'), 'Index must be a Number. For more information use --help'); |
There was a problem hiding this comment.
Я правильно понимаю, что index здесь это номер порта?
Если так, то, думаю, в сообщении стоит отразить это. Как пользователю, мне было бы не очень понятно такое сообщение об ошибке.
scripts/server.js
Outdated
| index => { | ||
| if (isNaN(index)){ | ||
| console.error(color('red'), 'Index must be a Number. For more information use --help'); |
There was a problem hiding this comment.
Лишний пробел в
Index must be a Number. For more information
^
😄
scripts/server.js
Outdated
| } | ||
| return index || 3000; | ||
| } | ||
| )(agrv); |
There was a problem hiding this comment.
Не очень понимаю, почему здесь указываются аргументы.
Разве так не получается, что getPort это значение, а не функция? Аналогично с getFile и getHelp
scripts/server.js
Outdated
| let agrv = process.argv.slice(2); | ||
| getHelp(agrv); | ||
| let port = getPort(agrv); | ||
| createServer(serverCallback(getFile(agrv)))(port).listen(port) |
There was a problem hiding this comment.
Мне довольно сложно читать это.
Думаю, стоит выделить дополнительные переменные или т.п.
Интересным вариантом записать это было бы:
let server = pipe(
getFile,
serverCallback,
createServer,
)(argv)
server(port).listen(port)
No description provided.