Discussion :

[vandeyy]

Bonjour,

Je n'arrive pas à comprendre complètement l'instruction suivante:

Code :

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

cb(rows.map((row) => new Message(row)))

Je sais qu'il va créer un tableau de clé et de valeur.
mais, j'ai beaucoup de mal à intégrer/digérer cette simple ligne de codage !


voici le code complet !

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
18
19
20
21
22
23
24
25
26
27
28
29
30
31
32
33
34
35
36
37
38
 
let connection = require('../config/db')
let moment = require('moment')
 
class Message
{
 
    constructor(row)
    {
        this.row = row
    }
 
    get content()   {  return this.row.content }
    get created_at() { return moment(this.row.created_at).startOf('hour').fromNow();  }
 
    static create (content, cb)
    {
        connection.query('INSERT INTO messages SET content = ?, created_at = ?',
                    [content, new Date()],
                    (err, result) => {
                                        if (err) throw err
                                        cb(result)
                                    } 
                    )
    }
    static all (cb) 
    {
        connection.query('SELECT * FROM messages', 
                    (err, rows) => {
                                        if (err) throw err
                                        cb(rows.map((row) => new Message(row)))
                                    }
                        )
    }
 
}
 
module.exports = Message

Pouvez-vous m'aider ?

Merci

Yves

[Marco46]

Je ne sais pas ce que fait ta fonction cb, elle est injectée dans create et all (par ?).

En revanche pour ce morceau si c'est ça qui pose problème :

Code :

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

rows.map((row) => new Message(row))

Ça va te transformer retourner une nouvelle nommée rows retournée par ta db en une array d'objets de classe Message.

[vandeyy]

Bonjour,

Merci pour la réponse.

Voici le code pour le "cb"

Code :

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

1
2
3
4
5
6
7
8
app.get('/', (request, response) => {
    let Message = require('./models/message')
    Message.all(
                function (messages) {
                    response.render('pages/index', {messages: messages})   
                    }
                )
 })

En fait, c'est la construction (la syntaxe) de la ligne qui est difficile à digérer !
comme je peux comprendre on instancie la classe dans la classe !!!

Code :

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

rows.map((row) => new Message(row))

Merci,

Yves

[Watilin]

J’ajoute quelques précisions : la méthode map retourne un nouveau tableau, elle ne modifie pas le tableau original.
Sa syntaxe est :

Code :

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

unTableau.map(uneFonction)

où uneFonction est appelée autant de fois qu’il y a d’items dans le tableau ; elle reçoit à chaque fois l’item en paramètre, et doit retourner (avec return) ce qui sera l’item correspondant dans le nouveau tableau.

L’article MDN de map montre différents exemples.

Dans le cas présent, la fonction passée a la forme d’une arrow function dont le corps est une expression (pas un bloc encadré par des accolades). Dans ce cas le return est implicite, c’est le résultat de l’expression qui est retournée par la fonction.

Pour parler en code, cette forme :

Code :

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

(param) => expression

est équivalente à celle-ci :

Code :

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

(param) => { return expression; }

Ainsi,

Code :

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

cb(rows.map((row) => new Message(row)))

est équivalent à :

Code :

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

cb(rows.map((row) => { return new Message(row); }))

Pour rendre le code plus lisible, on peut le réécrire avec une variable intermédiaire :

Code :

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

1
2
var messages = rows.map((row) => { return new Message(row); });
cb(messages)

[Marco46]

J’ajoute quelques précisions : la méthode map retourne un nouveau tableau, elle ne modifie pas le tableau original.

Watilin a raison, la précision est très importante j'ai très mal formulé ma réponse que j'ai modifié !

[vandeyy]

Merci à vous pour vos réponses.

Yves

[vandeyy]

Bonjour,

Juste un petit point qui me parrait étrange !

Code :

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

cb(rows.map(  (row) => new Message(row)  )  )

le "new Message(row)" fait appel au constructeur

Code :

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

1
2
3
4
constructor(row)
    {
        this.row = row
    }

J'ai l'impression que celui-ci écrase la valeur du row précédent et pourtant ce n'est pas le cas, il ajoute chaque row de l'objet rows.
Donc, dans mon raisonnement row devrait contenir le dernier row de l'objet rows (car c'est juste une assignation) !

Là, je suis perdu !!!

Quelqu'un peut-il m'expliquer pourquoi les valeurs ne sont pas écrasées ?

Merci d'avance,

Yves

[Marco46]

Quelqu'un peut-il m'expliquer pourquoi les valeurs ne sont pas écrasées ?

Parce qu'en JavaScript une assignation sur un type non-primitif est une copie par référence, pas par valeur. Tu obtiens donc une array d'objets Message contenant une référence row vers le litéral row de l'array rows.

Il faudrait que le constructeur de Message effectue un clonage de row ou décompose propriété par propriété l'objet row passé en paramètre.

Utiliser les classes en JavaScript je trouve perso que c'est une fausse bonne idée, on croit qu'on fait de l'orienté objet alors que JavaScript n'est pas du tout conçu pour. C'est un keyword qui a été ajouté au langage pour pas trop brusquer ces pauvres développeurs Java et .NET qui sont dégoutés de devoir utiliser un langage qu'ils détestent et n'ont jamais pris le temps d'apprendre mais il n'a rien à y faire.

Un très bon article sur le sujet.

[blbird]

Utiliser les classes en JavaScript je trouve perso que c'est une fausse bonne idée, on croit qu'on fait de l'orienté objet alors que JavaScript n'est pas du tout conçu pour. C'est un keyword qui a été ajouté au langage pour pas trop brusquer ces pauvres développeurs Java et .NET qui sont dégoutés de devoir utiliser un langage qu'ils détestent et n'ont jamais pris le temps d'apprendre mais il n'a rien à y faire.

Un très bon article sur le sujet.

/Agree 2000%. Que dire de plus!