Discussion :

[renaudfrancois_orl]

Bonjour,

Je commence avec javascript et venant du monde C++, je galère un peu.
Le but de mon code est de faire un expand d'une chaine "5-12" en un array "5","6","7","8","9","10","11","12"
voici mon problème

Mes datas de test

Code :

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

datatest = [{id:0,node:"0"},{id:1,node:"1-3"},{id:2,node:"5"},{id:3,node:"6-10"}]

Mon code expandToArray protitype de String:

Code :

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

1
2
3
4
5
6
7
8
9
10
11
String.prototype.expandToArray = function () {
  var d = this.valueOf();
  var out = '';
  if (d.split('-').length == 1) return d;
  var p = new Number;
  for (var i = parseInt(d.split('-') [0]); i <= parseInt(d.split('-') [1]); i++) {
    out = out + i + ',';
  }
  out = out.slice(0, - 1);
  return out.split(',');
}

test_1:

Code :

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

1
2
3
4
5
6
datatest.forEach ( function (d) {console.log(d.node)})
0
1-3
5
6-10
undefined

ok pour les datas si ce n'est ce undefined que je n'explique pas

Code :

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

1
2
datatest[1].node.expandToArray()
["1", "2", "3"]

l'expandToArray fonctionne

Code :

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

1
2
3
4
datatest[1].node.expandToArray()[0].constructor
String() { [native code] }
datatest[1].node.expandToArray().constructor
Array() { [native code] }

expandToArray renvoie bien un Array

Code :

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

1
2
3
4
datatest[1].node.expandToArray().forEach (function (d) {console.log(d)})
1
2
3

encore ok

si je passe tous les enregistrement de data j'ai l"erreur suivante:

Code :

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

datatest.forEach ( function (d) { d.node.expandToArray().forEach (function (d) {console.log(d)})})

Uncaught TypeError: d.node.expandToArray(...).forEach is not a function(…)

pourtant d.node est bien un String , je peu donc utiliser expandToArray, expandToArray est bien un Array je devrait pouvoir faire un forEach comme au dessus ???????

Merci cordialement.

[danielhagnoul]

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
let datatest = [ { id : 0, node : "0"}, { id : 1, node : "1-3" }, { id : 2, node : "5" }, { id : 3, node : "6-10" } ];
    
String.prototype.expandToArray = function(){
  let
    d = this.valueOf(),
    out = '',
    p = new Number;
    
  if ( d.split('-').length == 1 ) return [ d ]; // doit être un Array
  
  for ( let i = parseInt( d.split( '-' )[0], 10 ); i <= parseInt( d.split( '-' )[1], 10 ); i++ ){
    out = out + i + ',';
  }
  
  out = out.slice( 0, - 1 );
  
  return out.split( ',' );
};

document.addEventListener( 'DOMContentLoaded', function( ev ){
 
  datatest.forEach ( function( d ){
    d.node.expandToArray().forEach( function( k ){
      console.log( k );
    });
  });
  
});

[SylvainPV]

Salut,

Avec cette ligne là :

Code :

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

if(d.split('-').length==1)return d;

, tu renvoies la String telle quelle s'il n'y a pas de tirets. Du coup, tu ne peux pas appeler forEach sur une String. Il faudrait retourner [d] dans ce cas pour toujours retourner une Array.

[sekaijin]

Bonjour,

Je commence avec javascript et venant du monde C++, je galère un peu.
Le but de mon code est de faire un expand d'une chaine "5-12" en un array...

Bonjour il en va en Javascript comme avec tout langage même le C++
un array est une structure de donnée et ce n'est pas une string
alors quand je lis

expand d'une chaine "5-12" en un array

et

Code :

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

 out = out + i + ',';

j'avoue que je me pose des questions.

j'imagine qu'en C++ tu aurais utilisé un int[] alors pourquoi utiliser une string et pas utiliser un tableau d'entier ?

Code :

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

1
2
3
4
5
6
7
8
9
10
String.prototype.expandToArray = function () {
  var d = this.valueOf();
  var out = [];
  if (d.split('-').length == 1) return d;
  var p = new Number;
  for (var i = parseInt(d.split('-') [0]); i <= parseInt(d.split('-') [1]); i++) {
    out.push(i);
  }
  return out;
}

Ensuite tout comme en C++ il est préférable qu'une fonction retourne un type de donnée et un seul
or dans ton code d est une String et out.split(',') est un tableau

c'est comme si en C++ tu essayais d'écrire

Code :

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

1
2
3
4
5
6
7
int[] String::expandToArray() {
 String d = this;
 int[] out 
 if (d.notContaint('-')) return d;
 ....
 return out;
}

Tu va te chopper une erreur de compilation car ta méthode retourne un string au return d et un array au return out

il suffit donc de retourner un tableau avec la valeur de la string

Code :

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

1
2
3
4
5
6
7
8
9
10
11
12
13
String.prototype.expandToArray = function () {
  var d = this.valueOf();
  var out = [];
  if (d.split('-').length == 1) {
     out.push(parseInt(d));
  } else { 
    var p = new Number;
    for (var i = parseInt(d.split('-') [0]); i <= parseInt(d.split('-') [1]); i++) {
      out.push(i);
    }
  }
  return out;
}

ainsi lorsque tu fais

Code :

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

1
2
3
4
  datatest.forEach ( function( d ){
    d.node.expandToArray().forEach( function( k ){
      console.log( k );
    });

si d.node contient une intervalle tu as un tableau avec plusieurs éléments
et sinon tu as un tableau qui contient un élément.

comme tu as toujours un tableau tu peut utiliser la méthode foreach
A+JYT