Méthode inconnue avec un Array

Version imprimable

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:

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:

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:

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:

1 2 datatest[1].node.expandToArray() ["1", "2", "3"]
l'expandToArray fonctionne
Code:

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:

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:

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

Citation:

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 ???????:calim2:

Merci cordialement.

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
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 );
    });
  });
  
});

08/04/2016, 13h55
SylvainPV

Salut,

Avec cette ligne là :

Code:

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.

Citation:

Envoyé par 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...

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

Citation:

Envoyé par renaudfrancois_orl

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

et

Citation:

Envoyé par renaudfrancois_orl

Code:

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:

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:

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:

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:

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