Discussion :

[Gregory.M]

Bonjour,
J'ai fait une fonction recursive qui doit me retourner l'id d'un element parent.
Par contre j'ai un problème elle me retourne un objet undefined alors que quand je test en faisant un alert dans ma fonction, l'ID est bien trouvé.

Qu est ce qui ne va pas dans ma fonction?

Code :

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

1
2
3
4
5
6
7
8
9
10
 
function findElementId(element,word){
	if (element.id.toString().strip().startsWith(word))
	{ 
		return element.id.toString().strip().split('_').last();
	}
	else{ 
		findElementId(element.up(),word);
	}
}

Merci,
Grégory

[Auteur]

bonjour,

dans ta fonction où places-tu ton alert ?
Ensuite, que sont les méthodes strip() et up()

[Gregory.M]

bonjour,

dans ta fonction où places-tu ton alert ?
Ensuite, que sont les méthodes strip() et up()

Je le place à la place du return pour voir si la valeur est bonne, est c'est le cas.

J'ai aussi fait ca pour voir ma valeur de retour et c'est la que c'est undefined.

Code :

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

1
2
 
alert(findElementId(element,"mot");

[Auteur]

Sur quel type de noeud pointes-tu lorsque tu quittes ta fonction ? Tu peux, sans le savoir, pointer sur un noeud text.

Si j'ai bien compris ta fonction tu pars d'un élément "element" et tu remontes jusqu'au parent ayant un id commençant par "word".

Peut-on voir le code des fonctions up() et strip() ?

[Gregory.M]

Sur quel type de noeud pointes-tu lorsque tu quittes ta fonction ? Tu peux, sans le savoir, pointer sur un noeud text.

Si j'ai bien compris ta fonction tu pars d'un élément "element" et tu remontes jusqu'au parent ayant un id commençant par "word".

Peut-on voir le code des fonctions up() et strip() ?

Up et Strip sont des fonctions de Prototype. La fonction fonctionne correctement, c'est juste que ca me retourne un élément Undefined alors que ca ne devrait pas...

[Eric2a]

Salut,

Peut-être avec "return"

Code :

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

1
2
3
else{ 
		return findElementId(element.up(),word);
	}

[Bovino]

Tout d'abord :

Code :

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

element.id.toString()

Tu cherches à transformer une chaine en chaine ?

Sinon, return stoppe l'exécution d'une fonction et renvoie le résultat à la fonction appelante, donc dans le cas d'une récursivité, un des appels de ta fonction renvoie bien un résultat (celui que tu vois dans le alert), mais pas le premier appel de la fonction

PS :

La fonction fonctionne correctement, c'est juste que ca me retourne un élément Undefined alors que ca ne devrait pas...

Hum... donc elle fonctionne pas correctement

[le_chomeur]

ta fonction ne retourne pas toujours quelques chose
il te faut un return , et pas juste un simple appel