Discussion :

[rafoim]

Bonjour,

Une des utilisations possibles de Array.from, si j'ai bien compris, est de faire une copie d'un tableau en préservant l'original ( = deep copy).
J'ai fait un premier test:

Code :

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

1
2
3
4
5
6
 
const a1 = [2, 3, 5];
const a2 = Array.from(a1);
a2[0] += 1;
console.log(a1, a2);
// renvoie (3) [2, 3, 5] (3) [3, 3, 5]

Mon tableau d'origine est bien intact.
Par ailleurs, je suis en train de rédiger une classe me permettant de travailler avec des matrices. Le constructeur prend en argument un tableau de tableaux:

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
 
class Matrix {
  constructor(array) {
    this.array = array;
    this.height = array.length;
    this.width = array[0].length;
  }
  multiply(k) {
    const temp = Array.from(this.array);
    for (let i = 0; i < this.height; i += 1) {
      for (let j = 0; j < this.width; j += 1) {
        temp[i][j] *= k;
      }
    }    
    return new Matrix(temp);
  }
}
 
const M = new Matrix([[1, 2], [1, 0]]);
const N = M.multiply(2);
console.log(M);

Et là, cela ne fonctionne pas.
Ma matrice M initiale est multipliée par 2 alors que je voulais que seule ma matrice N soit modifiée.
j'ai fait quelques tests: si je me contente d'ajouter 1 à un des nombres de la matrice temp, la matrice désignée par this.array est aussi modifiée.
Quelqu'un peut-il m'éclairer?
Merci d'avance.

Raphaël

[mathieu]

Array.from ne fait une copie que de la 1re dimension. pour faire une copie de la 2e dimension, vous devrez faire une boucle qui parcourt toute la matrice.

[rafoim]

Merci à vous.

[rafoim]

Au cas où cela intéresse quelqu'un, je colle la solution:

Code :

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

1
2
3
4
5
6
7
8
9
10
11
12
13
14
 
 multiply(k) {
    // deep copy de this.array
    const temp = [];
    for (let i = 0; i < this.height; i += 1) {
      temp.push(Array.from(this.array[i]));
    }
    for (let i = 0; i < this.height; i += 1) {
      for (let j = 0; j < this.width; j += 1) {
        temp[i][j] *= k;
      }
    }
    return new Matrix(temp);
  }

[NoSmoking]

Bonjour,
indépendamment tu soucis bien réel sur la copie des Array, ce qui m'interpelle c'est la logique de ta Class.

La méthode multiply devrait agir sur l'instance en cours et non pas sur une nouvelle !
Si tu souhaites utiliser les données déjà initialisé alors il te faut créer une méthode clone pour au final écrire :

Code :

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

1
2
3
4
const M = new Matrix([[1, 2], [1, 0]]);
const N = M.clone().multiply(3);
console.log(M);
console.log(N);

ce qui reste somme toute bien plus compréhensible.

J'aurais donc plus vu quelque chose comme :

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
class Matrix {
  constructor(array) {
    this.array = array;
    this.height = array.length;
    this.width = array[0].length;
  }
  multiply(k) {
    for (let i = 0; i < this.height; i += 1) {
      for (let j = 0; j < this.width; j += 1) {
        this.array[i][j] *= k;
      }
    }
    return this;
  }
  clone() {
    return new Matrix(deepCopyArray(this.array));
  }
}

en utilisant une fonction générique deepCopyArray comme :

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
/**
 * Function récursive pour copier un Array en profondeur
 * @param   {Array} arrayIn  - Array devant être copier
 * @returns {Array} arrayOut - copie de l'Array d'entrée
 */
const deepCopyArray = (arrayIn) => {
  const arrayOut = [];
  if (!Array.isArray(arrayIn)) {
    return arrayIn;
  }
  arrayIn.forEach((val, ind) => {
    arrayOut[ind] = deepCopyArray(val);
  });
  return arrayOut;
}

[rafoim]

Merci pour ta réponse et pour ton analyse.

J'avais effectivement prévu de faire comme tu l'indiques au départ mais il me semble que j'avais es-lint qui me reprochait quelque chose, je ne sais plus quoi.
Peut-être avais-je mal géré l'affaire et oublié le "return this" en fin de méthode.
Pour ce que je veux faire, ma méthode marche tout de même mais c'est vrai que je génère une nouvelle instance de manière inutile.

Merci pour ta fonction récursive de copie de tableau: élégant.

Raphaël