node.jsの例外処理について

Question

ご覧いただきありがとうございます。
node.js×Expressでローカルでサーバーを構築しています。
エラー処理についてお伺いしたいのですが、
node.jsでは例外発生時に、例外処理を記述しないとサーバーが落ちてしまうと色々なサイトで知りました。

//app.js
var IndexPage       = require('./routes/index.js');
var dbSettings = require(__dirname+'/settings/mongo_setting.js');


process.on('uncaughtException', function (err) {
    console.log('uncaughtException => ' + err);
});

app.get('/',IndexPage.index);

//index.js
var MongoClient = require('mongodb').MongoClient;
var dbSettings = require('../settings/mongo_setting.js');

exports.index = function(req,res){
 MongoCliant.connect(dbSettings.dbUrl,(err,database)=>{
  try{
   const myDB = database.db(dbSettings.dbName);
   //以下I/O処理
  }catch(e){
  　console.log(e);
    res.render('../views/Erro.ejs');
  };
 });
};

//mongo_settings.js
exports.dbUrl = "mongodb://localhost";
exports.dbName = "test";

I/O処理に負荷をかけた際に起きるエラーが
TypeError: Cannot read property 'db' of nullと表示されます。
trycatchでログを残して、エラーページに飛ばせてはいるのですが、その後アクセスしてもエラーページしか表示されず、ログには同じエラーが表示されています。
エラーを放置せずにtrycatchで受けてあげればサーバーは落ちないというのを教本で読み試してみました。
確かに、サーバー自体は落ちていないのですが、ページをリロードしてもエラーページが表示されたままです。
エラー処理の方法を大きく勘違いしているような気がしてなりません。
もしよろしければ教えていただければ幸いです。

Answer 1

connectのみをしclose処理をしていないためmongodbの最大同時接続数に到達し、それ以上は接続できないためエラーが発生しているのではないでしょうか？

close処理を入れるか、connectしたobjectを使い回す(singleton)ようにすればエラーを回避することができると思いますよ。

追記

基本的な使い方まマニュアルに載ってますね。

https://docs.mongodb.com/manual/tutorial/atlas-free-tier-setup/

var MongoClient = require('mongodb').MongoClient;

var uri = "mongodb://user123:[e-mail]:27017,gettingstarted-shard-00-01-hyjsm.mongodb.net:27017,gettingstarted-shard-00-02-hyjsm.mongodb.net:27017/test?ssl=true&replicaSet=GettingStarted-shard-0&authSource=admin";
MongoClient.connect(uri, function(err, db) {
  // Paste the following examples here

  db.close();
});

この場合最後のコールバック関数時(res.render)に、myDB.close()とすれば一括で閉じることはできるのでしょうか？

catchでエラーページに飛ぶときのみcloseしても正常時にはcloseしないことになるのでfinallyが妥当です。

  }catch(e){
  　console.log(e);
    res.render('../views/Erro.ejs');
  } finally {
    database.close();  // ←こっち
  };

Comments

[退会済みユーザー]

ご回答いただきありがとうございます。
重ねて質問になってしまい大変申し訳ないのですが、
I/O処理の部分で、複数のコレクションを参照(してtoArrayで引き出しています)してデータを取得しています。
この場合最後のコールバック関数時(res.render)に、myDB.close()とすれば一括で閉じることはできるのでしょうか？それともそれぞれのコレクション参照時にclose処理が必要なのでしょうか？
無知で申し訳ありません。

[mosapride]

回答に追記しました。

[退会済みユーザー]

マニュアルの参照ありがとうございます。もう一度見直してみます。
また詳しくご回答頂き本当にありがとうございました。
すごくわかりやすく、本当に助かりました。

Answer 2

connect() 何度も実行しているのに close() を行っていないために、何かの内部リソース（MongoClient内部のコネクションプールやNode.jsの既定のファイルディスクリプタの最大数に到達など）が枯渇しているのではないかというtenpaさんの意見に同意します。

その上での話ですが MongoClient.connect() のコールバック関数の冒頭（database.db()をコールする直前）で err が null であることを確認したらどうでしょうか？

接続エラー発生時には err にエラーの内容を示すメッセージ文字列が入りますので、それを見ると問題が具体的になると思います。

あと、connect() のコールバック関数の第二引数（変数名 database）はそれ自体がDbオブジェクトなので、複数のカーソルを使うのでなければ、database.db() の呼び出しは不要です。

ご参考になれば。

Comments

[退会済みユーザー]

ご回答いただきありがとうございます。
connect()のご指摘いただき、リファレンス等読み返したところ、
dbUrl = "mongodb://localhost:27017/<データベース名>としていてtkanda様のご指摘どおりでした。
自分も早速、"mongodb://localhost:27017/test"と書き換えてみたのですが、collection of nullのエラーが発生してしまい、データベースに正しくアクセスできないようです。
何分サーバーサイドに不慣れなもので、dotinstallをみてローカルサーバーに言われるがままインストールしたのでその際に何か設定漏れ？のようなものがあったのかもしれません。
もう一度見直してみます。

Answer 3

まず第一に、同じ処理でもIOに負荷がかかる時とかからない時があるのでしょうか？
失敗するような動作を反復しているのであれば、例外処理をしていても、再度実行してもやはり失敗します。

次に、ブラウザ側のキャッシュは確認しましたか？
ただ、リクエストするたびに、サーバ側でエラー文が出るのであれば、このキャッシュのせいである可能性は低いです。

Comments

[退会済みユーザー]

I/O処理を連続(1000回ぐらいです)させた際に定期的に発生しています。
I/O処理の記述自体には問題がない(リファレンスのペーストです)と思われます。
ブラウザのキャッシュ確認いたしましたが、問題ないようなので、おそらくサーバー側でのエラーによるものだと思われます。
ご回答ありがとうございます。