[Groonga-commit] droonga/express-droonga at ab2834c [master] Split cache and middlewares that use cache

Back to archive index

Kouhei Sutou kou****@clear*****
Mon Apr 7 13:30:58 JST 2014


In <53422735.40209 �� clear-code.com>
  "Re: [Groonga-commit] droonga/express-droonga �� ab2834c [master] Split cache and middlewares that use cache" on Mon, 07 Apr 2014 13:19:01 +0900,
  YUKI Hiroshi <yuki �� clear-code.com> wrote:

> cacheのインスタンスを作ってそれを明示的にミドルウェアに渡す、というつく 
> りは、cache関係のモジュールの内部の都合が外に染み出ている感じがして、よ 
> くないのではないかと思いました。
> 
>           var cache = new droonga.cache({
>             size: cache
>           });
>           application.use('/cache/statistics',
>                           cache.statistics);
>           application.use(cache.middleware({
>             rules: [
>               {
>                 regex: /^\//,
>                 ttlInMilliSeconds: 1000
>               }
>             ]
>           }));
> 
> という感じで初期化するスタイルの方がいいと自分は思うのですが、どうでしょ 
> うか。

うーん、これだと染みだしていないように感じるってことなんです
よね。何がポイントなのかしら。エントリーポイントのオブジェク
トが1つだけってところなんですかねぇ。

個人的には、キャッシュオブジェクトがミドルウェアを作ることに
びっくりするので、キャッシュオブジェクトじゃないなにか
(cacheという名前じゃないなにかのオブジェクト)がエントリーポ
イントになるなら、私はエントリーポイントを1つ案でよいです。そ
のエントリーポイントが誰なのかはパッと浮かびませんが。。。




More information about the Groonga-commit mailing list
Back to archive index