expressにてapp.use([別ファイルのexpressモジュール])という書き方でコードを整理するのにデメリットは存在するのか？

Question

質問

• app.ts

// import 諸々

var app = express()

// ■■■■■■■■■■■部分Ａ■■■■■■■■■■■
var RedisStore = connectRedis(session)
var redisClient = redis.createClient({
  host: 'session',
  port: 6379,
  prefix: 'sid:',
})

app.use(
  session({
    secret: 'hoge',
    resave: false,
    saveUninitialized: true,
    store: new RedisStore({
      client: redisClient,
    }),
    cookie: {
      path: '/',
    },
  })
)
// ■■■■■■■■■■■部分Ａ■■■■■■■■■■■■

app.set('view engine', 'ejs')
app.use('/static', express.static(__dirname + '/public'))

app.use(express.json())
app.use(express.urlencoded({extended: true}))

ー省略ー

のような app.ts があったときに部分Ａはセッションに関わるものがつらつらと書かれているので別ファイルに分けたいなと思っています。（実際はもっと長い部分を分けようと思っている）

その際

• sessionApp.ts

const app = express()

var RedisStore = connectRedis(session)
var redisClient = redis.createClient({
  host: 'session',
  port: 6379,
  prefix: 'sid:',
})
app.use(
  session({
    secret: 'hoge',
    resave: false,
    saveUninitialized: true,
    store: new RedisStore({
      client: redisClient,
    }),
    cookie: {
      path: '/',
    },
  })
)

export default app

と切り出して

• app.ts

import sessionApp from './path/to/sessionApp'
// import 諸々

var app = express()

// ■■■■■■■■■■■部分Ａ■■■■■■■■■■■
app.use(sessionApp)
// ■■■■■■■■■■■部分Ａ■■■■■■■■■■■

app.set('view engine', 'ejs')
app.use('/static', express.static(__dirname + '/public'))

app.use(express.json())
app.use(express.urlencoded({extended: true}))

ー省略ー

と記述する事に処理的なデメリットはありますでしょうか？（一応切り出す前と同じ動作は確認できます）
app.tsのappを引数に渡すような関数モジュールを作成したほうがいいでしょうか？

app.get('/api',apiRouter)のようにルーティング処理を切り出す事は様々なサイトに載っているのですが、app.useなどの塊を切り出したい時にどうすればいいのか探せなかったので質問させていただきます。

ちょっと思うこと

感覚的にはexpressのインスタンスがexpressのインスタンスのミドルウェアとして機能することになる？ので、expressのインスタンス1つ分メモリ消費のデメリットがあるのかな？とも思ったりもします・・・

Comments

[hoshi-takanori]

express のインスタンスを複数作るのはおかしいというか、sessionApp.ts が export するのは app.use の引数として渡してる関数のはず。

[miyabi-sun]

全体的には良いと思います。そしてまぁまぁ一般的な感覚だと思います。
`var app = express()`も丸コピーで移植はダメですね。
session関数の返り値は関数だと思うので、
`export default session({...})`とする感じでしょうか。

[domidomi]

@hoshi-takanori さん
コメントありがとうございます。
> sessionApp.ts が export するのは app.use の引数として渡してる関数のはず
というのは、自分が記載した上記の二パターンの書き方はまったく同じといって良いということでしょうか？

@miyabi-sun さん
コメントありがとうございます。
> `export default session({...})`とする感じでしょうか。
例が悪く申し訳ないです。上では部分Aに app.use が一度しか使われていないですが実際は切り出したいのは複数の app.use 等が含まれている箇所になります。