1

MongoDB サーバーへの接続を開始するこのコードを作成し、その接続内で Web サービス コードを実行します。

これは最適または安全な方法ですか? これを実装するためのより良い方法は何でしょうか?

var express = require("express");
var mongodb = require("mongodb");
var servidor = new express();

var BSON = mongodb.BSONPure;
servidor.use(express.static(__dirname+"/publico"));
servidor.use(express.bodyParser());
var cliente_mongodb = mongodb.MongoClient;

cliente_mongodb.connect("mongodb://localhost/db_escuela", function(err, db_escuela){
    if(err){
        console.log("Error de conexion"+err);
    }else{
        console.log("Se ha conectado a la base de datos");
        var cl_alumnos = db_escuela.collection("cl_alumnos");

        servidor.get("/alumnos",function(peticion, respuesta){
            cl_alumnos.find().toArray(function(err,respuesta_db){
                respuesta.send(respuesta_db);
            });
        });

        servidor.get("/alumnos/:id",function(peticion,respuesta){
            var id = new BSON.ObjectID(peticion.params.id);
            cl_alumnos.findOne({'_id':id},function(err,respuesta_db){
                respuesta.send(respuesta_db);           
            });
        });

        servidor.post("/alumnos",function(peticion,respuesta){
            var nuevo_alumno={
                nombre:peticion.body.nombre,
                edad:peticion.body.edad,
                promedio:peticion.body.promedio
            };
            cl_alumnos.insert(nuevo_alumno,function(err,respuesta_db){
                console.log("Se ha insertado: "+JSON.stringify(respuesta_db));
                respuesta.send(respuesta_db);           
            });
        });

        servidor.delete("/alumnos/:id",function(peticion,respuesta){
            var alumno_eliminar = {
                _id : new BSON.ObjectID(peticion.params.id)
            }

            cl_alumnos.remove(alumno_eliminar,function(err,respuesta_db){
                if(err)
                    console.log("No se ha podido eliminar el registro");
                else
                    respuesta.send("Registro Eliminado");               
            });         
        });

        servidor.put("/alumnos/:id",function(peticion,respuesta){
            var id_editado={
                _id : new BSON.ObjectID(peticion.params.id)         
            };
            var alumno_editado={
                nombre:peticion.body.nombre,
                edad:peticion.body.edad,
                promedio:peticion.body.promedio
            };      
            cl_alumnos.update(id_editado, alumno_editado, function(err, respuesta_db){
                if(err)
                {
                    console.log("Error:"+err);
                }
                else
                {
                    console.log("Resultado "+respuesta_db);
                    respuesta.send("Actualizacion Exitosa");
                }
            });
        });


        servidor.listen(8080,function(){
            console.log("El servidor esta escuchando en el puerto 8080");
        });             
    }


});
4

2 に答える 2

3

これで大丈夫、安全です。MongoDB への接続は 1 回だけです。サーバーのミドルウェアへのこの接続への参照を持つ匿名関数を追加しています。

コード編成に関する限り、これを少し分割することを提案する人もいるかもしれません。上記のサイズのアプリケーションでは、問題はありません。拡張を計画している場合は、リファクタリングが必要になる場合があります。

于 2013-08-12T23:02:41.687 に答える
2

Mongoose のような ODM を使用することを恐れないでください。思ったよりも簡単で、どんな問題も解決します。

于 2013-08-13T02:23:56.750 に答える