Discussion :

[Katian]

Salut !

Dans ce code :

Code :

Sélectionner tout - Visualiser dans une fenêtre à part

1
2
3
4
5
6
7
8
9
10
11
function connect()
{
  let con = mysql.createConnection(options);
 
  con.connect(function(error_)
  {
    if(error_) throw error_;
 
    console.log("connected!");
  });
}

est-ce une bonne pratique de gerer l'erreur par un throw ? car je ne trouve pas comment placer mon bloc try + catch...

un

Code :

Sélectionner tout - Visualiser dans une fenêtre à part

1
2
3
4
5
6
7
8
try
{
  connect();
}
catch(error_)
{
  console.error("exception ", error_);
}

ne semble pas fonctionner...

[danielhagnoul]

C'est une question pour le forum Node.js

Voir le chapitre Establishing connections dans https://github.com/mysqljs/mysql

est-ce une bonne pratique de gerer l'erreur par un throw ? [...]

La bonne méthode c'est toujours celle qui est recommandée par l'extension de code, ici :

Code :

Sélectionner tout - Visualiser dans une fenêtre à part

1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
17
// The recommended way to establish a connection is this:
 
var mysql      = require('mysql');
var connection = mysql.createConnection({
  host     : 'example.org',
  user     : 'bob',
  password : 'secret'
});
 
connection.connect(function(err) {
  if (err) {
    console.error('error connecting: ' + err.stack);
    return;
  }
 
  console.log('connected as id ' + connection.threadId);
});

[Katian]

La bonne méthode c'est toujours celle qui est recommandée par l'extension de code, ici :

Pourtant ici https://www.w3schools.com/nodejs/nodejs_mysql.asp je vois des throw ...

[psychadelic]

C'est presque une question de philosophie (en restant dans le domaine de la programmation) : qu'est-ce qu'une erreur, sur quels critère établir la valeur "erreur" pour un dysfonctionnement ?

Il faut aussi prendre en comme l'évolution des langages informatique, la gestion d'erreur et la commande throw sont récentes, et dans ce cas précis elle est même transverse, puisqu'une "erreur" d'acces sur un serveur SQL se répercute un autre système.

Pour moi, l'utilisation du throw est ici valable.
En revanche, si on entre dans une gestion d'erreurs, alors il faut faire les choses au carré, c'est a dire codifier toutes les erreurs possibles avec leur libellé et leur traçabilité et documenter.

[Katian]

En revanche, si on entre dans une gestion d'erreurs, alors il faut faire les choses au carré, c'est a dire codifier toutes les erreurs possibles avec leur libellé et leur traçabilité et documenter.

Complètement d'accord, c'est ce que je fait merci !