Express error handler (перенаправить в логи)

[Gozar]

глобал ж в ноде не принято использовать

Потихоньку выпиливаю, когда начинал 3 месяца назад, было не до тонкостей.

Если маленький проект подключаем так?

let app = express();

app.use(require("./get/info"));
app.use(require("./get/props"));

app.listen(3000);

Где:

./get/info

let app = express();

app.get('/get/info/', funcHandler);

А если большой?

let app = express();

app.use(require("./get/init"));

app.listen(3000);

Где:

./get/init

require('./info');
require('./props');
...

Где ./get/info
app.get('/get/info/', funcHandler);

Так?

[Gozar]

А в ноде не без разницы let или var? Вроде же и так по модулям распихано.

[kobezzza]

А в ноде не без разницы let или var? Вроде же и так по модулям распихано.

var депрекейтед в ES6 А так без разницы.

[Safort]

А в ноде не без разницы let или var? Вроде же и так по модулям распихано.

Так Babel же)

[Gozar]

Так Babel же)

Я не использую Babel на сервере. Хватает стандартного api+npm.

[Erolast]

А в ноде не без разницы let или var? Вроде же и так по модулям распихано.

Во внешнем-то скопе разницы нету, но не объявлять же в разных стилях.

А если большой?

Я сейчас выделяю роутеры в отдельную папочку по сущностям (у меня, правда, ES6-модули, но не суть):

let app = express();

app.use("/", require("./routers/main"));
app.use("/api", require("./routers/api"));

app.listen(3000);

//routers/main.js
let express = require("express");
let router = express.Router();

router.use("/*", (req, res) => {
  res.render("main");
});

module.exports = router;

//routers/api.js
let express = require("express");
let httpStatuses = require("statuses");

let Article = require("../models/Article");

let router = express.Router();

router.route("/articles")
    .get((req, res, err) => {
        Article.find().then(
            (articles) => res.json(articles),
            (err) => next(err)
        );
    })
    .post((req, res, err) => {
        let article = new Article();
        
        article.save().then((article) => {
            res.status(httpStatuses.Created);
            res.location(req.protocol +
                "://" +
                req.hostname +
                (config.get("port") != 80 ? ":" + config.get("port") : "") +
                req.originalUrl +
                article.id
            );
            
            res.json(article);
        }, (err) => next(err));
    });

module.exports = router;

Когда роутер разрастается - он просто заменяется на папочку и разбивается на несколько мелких (именно поэтому я не указываю расширение при импорте).

[Gozar]

Я немного по другому делю:
/get //-> читаем с сервера (в папке есть файл init в который подключаем файлы)
/set //-> пишем на сервер (в папке есть файл init в который подключаем файлы)

Т.к. я в основном использую тип JSON, то деление на get и post становиться немного неудобным, а то ещё присрется put ввести, хотя похоже мне он будет без надобности.

Мне было неясно зачем нужен let router = express.Router();. Теперь вижу, внутри ты делишь на get и post, если такого деление нет, то он без надобности и можно сразу писать app.get или app.post

[Erolast]

Т.к. я в основном использую тип JSON, то деление на get и post становиться немного неудобным, а то ещё присрется put ввести, хотя похоже мне он будет без надобности.

Нихрена не понял. При чем тут JSON и в чем проблема ввести put?

внутри ты делишь на get и post

Это необязательно деление по методам, это может быть и деление по путям.

[Gozar]

Нихрена не понял

Я тебя ни в чем не убеждаю. Просто поделился архитектурным решением.

Ты написал роутер, внутри get, post. А я делю на get и set. Абстрагируюсь от типа(set: post, put, delete).

чем проблема ввести put?

Да ни в чем. Просто он не нужен.

[Erolast]

Я тебя ни в чем не убеждаю. Просто поделился архитектурным решением.

А. Ну я тоже ни в чем не убеждаю, просто поделился своим)