Map explications sur une ligne de code

Version imprimable

Bonjour,

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

Code:

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:

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

23/02/2018, 17h53
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:

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.

Bonjour,

Merci pour la réponse.

Voici le code pour le "cb"
Code:

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:

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

Merci,

Yves

23/02/2018, 20h04
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:

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:

(param) => expression

est équivalente à celle-ci :

Code:

(param) => { return expression; }

Ainsi,

Code:

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

est équivalent à :

Code:

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:

1 2 var messages = rows.map((row) => { return new Message(row); }); cb(messages)
23/02/2018, 20h20
Marco46

Citation:

Envoyé par Watilin

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é !
23/02/2018, 21h11
vandeyy

Merci à vous pour vos réponses.

Yves
24/02/2018, 11h26
vandeyy
Bonjour,

Juste un petit point qui me parrait étrange !

Code:

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

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

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
24/02/2018, 15h09
Marco46

Citation:

Envoyé par vandeyy

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.
26/02/2018, 09h56
blbird

Citation:

Envoyé par Marco46

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! 8-)