expressのルーティングについて

Question

let h = Express.Router();
app.use('/', (req, res, next) => {
  h = hoge()
  next()
});

app.use('/root', h)

function hoge() {
  let r = Express.Router()
  r.get('/hoge', (req, res) => {
    return res.send('hoge')
  })
  return r
}

Expressにてこのようなコードを書いたのですが、/root/hogeにアクセスしてもhogeと出力されず、Cannot GET /root/hogeとなってしまい、適切にルーティングできてないことがわかります。
どなたかこの理由わかりますでしょうか。

Answer 1

もう一回手元で動かしましたが、質問のコードでconsole.logを埋め込んでみたところapp.useで/rootにhを設定する際にからのルーターになっています。
これはhogeが実行されていないためです。
app.useで設定しているrouter、登録時のものが設定されるため、先にRouterを設定しておくといいでしょう。

let h = express.Router();
app.use('/', (req, res, next) => {
  h = hoge()
  next()
});

console.log(h)

app.use('/root', h)

ですので/と/rootで同じRooterを利用したいのであれば、次のようなコードにしてみてはどうでしょうか？
便宜上appとの関係性がわかりやすいように1つのファイルとして書いています。

const express = require('express');
const app = express()
const router = express.Router();

router.get('/hoge', (req, res) => {
  return res.send('hoge')
})

app.use('/', router);
app.use('/root', router)


app.listen(3000, () => console.log('Example app listening on port 3000!'))

Comments

[sakas1231]

ご回答有り難うございます！
自分の勘違いなのかもしれないのですが、 `app.use` はミドルウェアとしてリクエストパスに合致するものを上から順に実行するものだと認識しています。
そのため、自分の書いたコードでは `/root/hoge` にアクセスした際、

1. まず `app.use('/' ~~ )` が実行され、hにhoge()の返り値のrouterがsetされる
2. 次に `app.use('/root', ~~ )` が実行され、先程setしたrouterが読み込まれ、 `router.get('/hoge')` が実行される

という流れになり、routerが空であることはないのではないかと思っていますがいかがでしょうか。

[mottox2]

useはミドルウェア的な動作はしているので1.までは正しいように見えますが（回答は修正しました）、2の`app.use('/root', ~~ )`で先程したrouterではなく定義時の空Routerが読み込まれているように見えます。

[sakas1231]

なるほど、app.useが呼ばれるタイミングでグローバル変数はロックされるんですかね、ちょっとまだしっくり来ていませんがご親切にありがとうございました！