web-dev-qa-db-fra.com

node.js La connexion globale existe déjà. Appelle d'abord sql.close ()

J'essaie de créer des services Web en utilisant node.js à partir d'une base de données de serveur SQL. Dans l'interface frontale, lorsque j'appelle ces 2 services Web simultanément, une erreur est générée. La connexion globale existe déjà. Appelez d'abord sql.close ().

Toute solution ?

var express = require('express');
var router = express.Router();
var sql = require("mssql");

router.get('/Plant/:server/:user/:password/:database', function(req, res, next) {

    user = req.params.user;
    password = req.params.password;
    server = req.params.server;
    database = req.params.database;


    // config for your database
    var config = {
        user: user,
        password: password,
        server: server,
        database:database

    };

    sql.connect(config, function (err) {



        // create Request object
        var request = new sql.Request();

        // query to the database and get the records
        request.query("SELECT  distinct PlantName FROM MachineryStateTable"
            , function (err, recordset) {

                if (err) console.log(err)
                else {

                    for(i=0;i<recordset.recordsets.length;i++) {
                        res.send(recordset.recordsets[i])
                    }

                }
               sql.close();

            });

    });

});





router.get('/Dep/:server/:user/:password/:database/:plantname', function(req, res, next) {

    user = req.params.user;
    password = req.params.password;
    server = req.params.server;
    database = req.params.database;
    plantname = req.params.plantname;

    // config for your database
    var config = {
        user: user,
        password: password,
        server: server,
        database:database

    };

    sql.connect(config, function (err) {



        // create Request object
        var request = new sql.Request();

        // query to the database and get the records
        request.query("SELECT  distinct DepName FROM MachineryStateTable where PlantName= '"+plantname+"'"
            , function (err, recordset) {

                if (err) console.log(err)
                else {

                    for(i=0;i<recordset.recordsets.length;i++) {
                        res.send(recordset.recordsets[i])

                    }
                    sql.close();
                }

            });
    });
});



module.exports = router;
sql-servernode.jsangularweb-services
18
Laamiri Oussema

Vous devez créer un poolConnection essayez ceci:

new sql.ConnectionPool(config).connect().then(pool => {
  return pool.request().query("")
  }).then(result => {
    let rows = result.recordset
    res.setHeader('Access-Control-Allow-Origin', '*')
    res.status(200).json(rows);
    sql.close();
  }).catch(err => {
    res.status(500).send({ message: "${err}"})
    sql.close();
  });
40
bams

A partir de la documentation , la méthode close doit être utilisée sur la connexion et non sur le module requis,

Donc, devrait être utilisé comme

var connection = new sql.Connection({
user: '...',
password: '...',
server: 'localhost',
database: '...'
});
connection.close().

Aussi quelques suggestions,
1. mettre res.send dans une boucle n'est pas une bonne idée. Vous pouvez répondre à l'intégralité des jeux d'enregistrements ou effectuer des opérations dessus, stocker le résultat dans une variable et le renvoyer.
2. Essayez d'utiliser promesses , au lieu de rappels, cela rendrait le flux plus ordonné

3
Shivam

Vous devez utiliser ConnectionPool.

La fonction suivante retourne un jeu d'enregistrements avec les résultats de ma requête.

async function execute2(query) {

    return new Promise((resolve, reject) => {

        new sql.ConnectionPool(dbConfig).connect().then(pool => {
            return pool.request().query(query)
        }).then(result => {

            resolve(result.recordset);

            sql.close();
        }).catch(err => {

            reject(err)
            sql.close();
        });
    });


}

Fonctionne bien dans mon code!

2
ehitel rodriguez castro

si ce problème vous dérange toujours, changez l’API principale. allez à node_modules\mssql\lib\base.js à la ligne 1723, ajoutez le code ci-dessous avant si la condition globalConnection = null

0
Mohammed Sabbir

Au cas où quelqu'un viendrait ici essayer de savoir comment utiliser la connexion de pool SQL Server avec des paramètres:

var executeQuery = function(res,query,parameters){

   new sql.ConnectionPool(sqlConfig).connect().then(pool =>{

    // create request object
    var request = new sql.Request(pool);

   // Add parameters
    parameters.forEach(function(p) {
    request.input(p.name, p.sqltype, p.value);
    });

    // query to the database
    request.query(query,function(err,result){

            res.send(result);
            sql.close();

    });
 })       
}
0
Jared