Discussion :

[sbill]

Bonsoir,

Je travaille actuellement avec nodeJS, je rencontre un problème pour faire une requête synchrone.
Avant tout mon but est de vérifier en base de donnée que l'email n'est pas déjà utilisé par un membre (j'utilise le module pg-promise) :
models/user.js

Code :

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

1
2
3
4
5
6
7
8
9
10
11
12
13
14
 
function existEmail(value) {
       			db.oneOrNone({
       				text : "SELECT * FROM users email = $1",
       				values : [value]
       			})
       			.then(function(data) {
       				return (data) ? true : false;
       			})
       			.catch(function(error) {
       				console.log('exist', error);
       				return false;
       			});
       		}

Puis dans mon controller, je demande au model (code du dessus) de me renvoyez vrai si l'adresse est déjà utiliser, cependant la requête étant asynchrone j'ai la valeur undefined :
controller/user.js

Code :

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

1
2
3
4
 
/* User représente mon models */
var test = User.existEmail('email@gmail.com');
console.log(test); //undefined

J'ai lu la doc du module pg-promise et je n'ai rien trouver de synchrone, ce que je comprend car ces contre la philosophie de node, du coup l'organisation de mon code et mal pensée ? Suis-je obligé de mélanger le model et le controller ?

Merci pour vos pistes.

[Gnuum]

Je pense que tu n'as pas encore tout à fait saisi comment fonctionne les promises.
La doc MDN pour les promises JavaScript natives: https://developer.mozilla.org/fr/doc...lobaux/Promise
Ta requête n'est pas synchrone, tu ne peux donc utiliser du code synchrone. Tu dois utiliser le mécanisme des promises.

Ca donnerai ça pour ton code:

Code JavaScript :

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

1
2
3
4
5
6
7
8
9
10
11
12
13
 function existEmail(value) {
    return db.oneOrNone({
        text : "SELECT * FROM users email = $1",
        values : [value]
    })
    .then(function(data) {
        return (data) ? true : false;
    })
    .catch(function(error) {
        console.log('exist', error);
        return false;
    });
}

Code JavaScript :

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

1
2
3
4
User.existEmail('email@gmail.com')
.then(function(isUserExisting) {
    console.log(isUserExisting);
});

[sbill]

Yes, merci j'avais trouvé la solution entre temps et c'est exactement ce que j'ai fais

[Gnuum]

Une fois qu'on a compris le mécanisme c'est facile.