Discussion :

[Seta-san]

Bonjour !

J'ai besoin d'appeler une fonction (qui défini des variables) dans d'autre fonctions, pour éviter justement d'avoir à définir plusieurs fois les variables.

Par exemple :

Code :

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

1
2
3
4
5
6
7
8
9
10
11
12
13
14
 
/* Je génère mes variables */
function variables();
	{
           var nom = document.getElementById('nom').value;
	}
 
 
/* Je génere le récapitulatif */
function recapitulatif()
	{
		variables(); /* Je récupère les variables */
		document.getElementById('recapitulatif').innerHTML = nom;
	}

Bien entendu j'ai comme retour que la variable nom is not defined.
Est-ce au moins possible de récupérer les variables de cette façon ? Sinon tant pis je les définirais à chaque fois.

Merci !

[NoSmoking]

Bonjour,

Bien entendu j'ai comme retour que la variable nom is not defined.
Est-ce au moins possible de récupérer les variables de cette façon ? Sinon tant pis je les définirais à chaque fois.

- ta fonction ne retournant rien le retour est undefined
- ta variable nom est définie en locale dans la fonction variables, donc non reconnue dans la fonction recapitulatif

tu dois donc faire à minima

Code :

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

1
2
3
4
5
6
7
8
9
10
/* Je génère mes variables */
function variables(); {
  return document.getElementById('nom').value;
}
/* Je génere le récapitulatif */
function recapitulatif() {
  var nom = variables();
  /* Je récupère les variables */
  document.getElementById('recapitulatif').innerHTML = nom;
}

[Seta-san]

Dans l'exemple j'ai juste écrit "var nom = .." mais en faite j'en ai beaucoup d'autre à la suite.

Code :

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

1
2
3
4
5
6
7
function variables()
	{	
		var nom  = document.getElementById('nom').value ;
		var prenom = document.getElementById('prenom').value ;
		var id_client = document.getElementById('id_client').value;
                /* et plein d'autre... */
         }

Par contre ton bout de code m'a fait réfléchir autrement et du coup je viens de faire quelque chose encore plus simple :

Code :

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

1
2
3
4
5
6
7
8
9
10
11
12
13
 
/* On récupère la variable demandé */
function variable(id_champ)
		{
			return document.getElementById(id_champ).value ;
		}
 
/* On génere le récapitulatif du rdv */
function recapitulatif()
	{
		 /* On récupere les variables */
		document.getElementById('recapitulatif').innerHTML = 'Le nom : '+variable('nom')+'Le prénom'+variable('prenom')+' et l\'id : '+variable('id_client');
	}

Je n'ai désormais plus qu'a appeler mes élément directement avec variable('id_de_l'élément').

Merci !

[Bovino]

Dans ce cas, le plus cohérent et le plus efficace (selon moi ) serait de passer par un objet, ce qui évite l'appel multiple de ta fonction :

Code :

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

1
2
3
4
5
6
7
8
function variables(){
	var monobjet = {};
	monobjet.nom  = document.getElementById('nom').value ;
	monobjet.prenom = document.getElementById('prenom').value ;
	monobjet.id_client = document.getElementById('id_client').value;
        /* et plein d'autre... */
	return monobjet
}

[NoSmoking]

Dans ce cas, le plus cohérent et le plus efficace (selon moi ) serait de passer par un objet, ce qui évite l'appel multiple de ta fonction :...

certes...mais le plus efficace serait encore de ne pas appeller de fonction et de glisser le code dans la fonction appelante...

Je pense que l'objet de la fonction est d'avoir une fonction générique du style getValue() pour simplification d'écriture, dans ce cas le but est atteint.