Discussion :

[zaboug]

Bonjour,

j'ai un bug très bizarre. je recherche une chaine de caractère dans une autre grâce à une fonction récursive. En testant cette fonction récursive avec firbug, on voit que cette fonction marche : quand on arrive au return, la valeur qui va être retourné est la bonne. Mais la variable qui est initialisée avec cette valeur reste à undefined !! Je comprends rien.

voici mon code :
la fonction récursive :

Code :

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

1
2
3
4
5
6
7
8
9
10
11
 
var Find_ref = function(ID, fin){
		var char_fin = ID.substring(fin, fin+1); 
		if(char_fin == '|'){
			var reference = ID.substring(13,fin);
			return reference;
		}
		else{
			Find_ref(ID, fin+1);
		}
	}

et l'endroit où elle est appelée :

Code :

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

1
2
3
4
5
6
7
 
this.Valider = function(val){
		var ID = val.getId();
		var ref_fiche = null;
		ref_fiche = Find_ref(ID, 13);
		alert(ref_fiche);
	}

"ref_fiche" reste toujours à undefined alors que dans firebug, on voit que la variable "ref" de la fonction fonction Find_ref à bien la valeur qu'elle doit avoir (une chaine de caractère).

[Bovino]

Pourquoi tu fais pas plutôt :

Code :

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

1
2
3
var Find_ref = function(ID){
		var reference = ID.substring(13,indexOf('|'));
}

[le_chomeur]

Pourquoi tu fais pas plutôt :

Code :

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

1
2
3
var Find_ref = function(ID){
		var reference = ID.substring(13,indexOf('|'));
}

et ??? ici tu ne récupère rien tu es dans un objet , il te faut sois un return , sois utiliser une variable de l'objet ...

sinon zaboug , je crois tout simplement, que ta méthode perd l'instance que tu appel explication :

var a = retour de B()

B() s'exécute , mais se réappel sans rien renvoyer une première fois
B() s'exécute donc nouvelle instance ! trouve le résultat, le renvois mais a quoi ?

il te fau utiliser une variable propre a ton objet pour stocker le résultat je pense

[DoubleU]

je dirais plutot un return sur le FindRef du else...

[zaboug]

Merci pour toutes vos réponses.

Finalement, pas besoin de récursion ici (à croire que je marchais en mode "pourquoi faire simple quand on peut faire compliqué?"), j'ai utilisé la suggestion de Bovino.

Le code devient :

Code :

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

1
2
3
4
5
 
var Find_ref = function(ID, debut){
		var reference = ID.substring(debut,ID.indexOf('|'));
		return reference;
	}

y'a limite même pas besoin d'une fonction pour faire ça !

encore merci à vous.