Учебный проект: вдохнуть жизнь #12
Conversation
Моковые данные вынесены в js-файлы, все в одном месте
| @@ -33,12 +33,15 @@ | |||
| "supertest": "5.0.0" | |||
| }, | |||
| "dependencies": { | |||
| "axios": "0.21.0", | |||
There was a problem hiding this comment.
request почил, поэтому по совету учебника - axios. Может, в задании обновить забыли?
There was a problem hiding this comment.
Да, всё так. Учебник только недавно обновили, до текста задания ещё не добрались.
| "nanoid": "3.1.12", | ||
| "pino": "6.7.0", | ||
| "pug": "3.0.0" | ||
| "pug": "2.0.4" |
There was a problem hiding this comment.
Откатил в связи с pugjs/pug#3274
There was a problem hiding this comment.
На какую интересную багу ты наткнулся, однако)
| @@ -3,10 +3,9 @@ | |||
| module.exports = { | |||
| DEFAULT_COMMAND: `--help`, | |||
| DEFAULT_PORT: 8080, | |||
| DEFAULT_LOCAL_PORT: 3000, | |||
| DEFAULT_API_PORT: 3000, | |||
There was a problem hiding this comment.
Вроде бы логичнее получается
| const {splitNumByThousands} = require(`../../utils`); | ||
|
|
||
| // Доработка одиночного объявления для шаблонизации | ||
| const modifyOffer = (offer) => { |
There was a problem hiding this comment.
Показалось целесообразным не выносить подобную логику на уровень представления, а в учебных примерах вообще эта тема не раскрыта.
There was a problem hiding this comment.
Да, это разумно. Хотя вообще вопрос дискуссионный, и я бы не сказал, что есть единственно правильное решение.
| const {modifyOffer} = require(`../lib/offers`); | ||
| const declineWord = require(`decline-word`); | ||
|
|
||
| const mainRouter = new Router(); |
There was a problem hiding this comment.
Переименовал index -> main, ибо запросы-то идут не только к индексной странице.
|
|
||
| try { | ||
| app.listen(port, (err) => { | ||
| if (err) { | ||
| return logger.error(`Ошибка при создании сервера: ${err.message}`); |
There was a problem hiding this comment.
Надоела борьба с кодировкой терминала, к тому же в проекте должен быть единообразный подход к одной проблеме.
| */ | ||
| const splitNumByThousands = (num) => { | ||
| const str = num.toString(); | ||
| if (str.length < NUM_SPLITTING_THRESHOLD) { |
There was a problem hiding this comment.
В верстке разряды отделяются пробелом только начиная с пятизначных чисел
|
|
||
| res.render(`search-result`, { | ||
| results, | ||
| findWord: declineWord(results.length, `Найден`, `а`, `о`, `о`), |
There was a problem hiding this comment.
Странно выглядят несклоняемые слова в данном месте, решил исправить этот момент без "программирования на шаблонизаторе" (возможно, потому что синтаксис мопса упорно не хочет нравиться и лишний раз туда лезть неохота (в практике уже попробовал в аналогичной связке с express любимый twig)).
🎓 Учебный проект: вдохнуть жизнь