Bonjour,

Je souhaite optimiser du Jquery. Après plusieurs test et visite de forum je n'arrive pas optimiser mon code.

// Partie Papa
$(".papa").click(function() {
	$(".tab_content").hide(500)
	var activeTab = $(this).find("a").attr("href");
	$(activeTab).toggle(1000);
	return false;
});

// Partie Maman
$(".maman").click(function() {
	$(".tab_content").hide(500);
	var activeTab = $(this).find("a").attr("href");
	$(activeTab).toggle(1000);
	return false;
});

J'ai mis une partie du code pour une meilleur visibilité.
Se qui fait appel à la même fonction j'aimerais le réécrire en une ligne.
Je suis passé par des variables, création d'une fonction pour faire un rappel dans celle-ci, je reste bloqué



Si vous aurez sous la main comment faire, lien, explication et autre, je suis preneur.

Merci d'avance
Kitzumé

// Partie Papa et Maman
$(".papa,.maman").click(function() {
	$(".tab_content").hide(500)
	var activeTab = $(this).find("a").attr("href");
	$(activeTab).toggle(1000);
	return false;
});

Méthode que je connaissait et je n'ai pas pensé à appliquer.

Existe t-il d'autre méthode ?
Comme par exemple dans l'as : création d'une fonction que l'ont peut rappeler dans d'autre fonction.

function fonction($el) {
	$(".tab_content").hide(500)
	var activeTab = $el.find("a").attr("href");
	$(activeTab).toggle(1000);
}

// Partie Papa
$(".papa").click(function() {
	fonction($(this));
	return false;
});

// Partie Maman
$(".maman").click(function() {
	fonction($(this));
	return false;
});

Citation : Golmote

function fonction() {
	$(".tab_content").hide(500)
	var activeTab = $(this).find("a").attr("href");
	$(activeTab).toggle(1000);
	
	return false;
}

// Partie Papa
$(".papa").click(fonction);

// Partie Maman
$(".maman").click(fonction);

Non?

Salut,

EIDT: Grillé



Ce qui me gêne avec jQuery, c'est qu'à chaque appel de $(), un nouvel objet est crée. Donc ici à chaque clic, on a rien que 3 nouveaux objets en mémoire (et des objets jQuery ça fait un max de propriétés et de méthodes, donc c'est "gros") !!

Citation : Tiller

Citation : Golmote

function fonction() {
	$(".tab_content").hide(500)
	var activeTab = $(this).find("a").attr("href");
	$(activeTab).toggle(1000);
	
	return false;
}

// Partie Papa
$(".papa").click(fonction);

// Partie Maman
$(".maman").click(fonction);

Non?

Aussi, oui. Mais il dit que certaines parties du code diffèrent entre Papa et Maman... donc j'avais préféré ne pas prendre de "risques"

@LCaba : C'est en effet un des "problèmes" de jQuery.

Merci bien pour cette méthode, elle est toute simple a comprendre.
A la fin de

$(".tab_content").hide(500);

manque juste un point ";"

Cdt
Kitzumé

*Honte*

Bien vu.

En effet, ce ";" manque depuis ton premier post... J'ai copié/collé partout sans rectifier.

Content que ton problème soit résolu.

Je dirait que ces moi qui a fait la faute de l'effacer