Discussion :

[Aooka]

Bonjour à tous,

Je déclare deux variables en global dans ma fenêtre principale (FEN_Main) :

Code :

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

1
2
gFichierXLS						est un xlsDocument
gsCheminFichierXLS				est une chaîne	= INILit("config", "sCheminFichierXLS", "", gsCheminConfig)

Ensuite, j'appel une procédure local avec ces 2 variables :

Code :

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

VérificationValiditeFichierImputationIT(gsCheminFichierXLS, gFichierXLS)

Enfin, voici le contenu de ma procédure. Mon problème est que je ne comprends pas l'avertissement sur les paramètres/arguments de ma procédure. Surtout que tout marche.

Code :

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

1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
PROCÉDURE VérificationValiditeFichierImputationIT(gsCheminFichierXLS est une chaîne, gFichierXLS est un xlsDocument)
	// Vérification fichier imputation - informatique.xlsx
	SI fFichierExiste(gsCheminFichierXLS) ALORS
		gFichierXLS = xlsOpen(gsCheminFichierXLS, xlsEcriture) 
		SI gFichierXLS <> Null ALORS
			// Le fichier excel existe et est bien valide donc on le ferme.
			xlsFerme(gFichierXLS)
			xlsFerme(gFichierXLS)
		SINON
			Error("Attention : Impossible d'initialiser le fichier Excel. Vérifier son emplacement et son nom.", ErreurInfo())
			FinProgramme()
		FIN
	SINON
		Error("Attention : Impossible d'initialiser le fichier Excel. Vérifier son emplacement et son nom.", ErreurInfo())
		FinProgramme()
	FIN

L'avertissement :


Quelqu'un peut-il m'expliquer cet avertissement ?

Merci pour votre aide,
Bonne journée,

[tatayo]

Bonjour,
Comme le paramètre et la variable globale ont le même nom, Windev t'explique que le paramètre masque la variable globale dans le code de la procédure.

Donc si dans ta procédure tu accèdes à gsCheminFichierXLS, se sera le paramètre et non la variable globale qui sera utilisé.

Tatayo.

[Aooka]

Bonjour Tatayo,

Du coup j'ai modifié le nom de mes deux arguments dans ma procédure et je n'ai plus l'avertissement. j'ai simplement enlever les préfix g de global.

Code :

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

1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
PROCÉDURE VérificationValiditeFichierImputationIT(sCheminFichierXLS est une chaîne, FichierXLS est un xlsDocument)
	// Vérification fichier imputation - informatique.xlsx
	SI fFichierExiste(sCheminFichierXLS) ALORS
		FichierXLS = xlsOpen(sCheminFichierXLS, xlsEcriture) 
		SI FichierXLS <> Null ALORS
			// Le fichier excel existe et est bien valide donc on le ferme.
			xlsFerme(FichierXLS)
			xlsFerme(FichierXLS)
		SINON
			Error("Attention : Impossible d'initialiser le fichier Excel. Vérifier son emplacement et son nom.", ErreurInfo())
			FinProgramme()
		FIN
	SINON
		Error("Attention : Impossible d'initialiser le fichier Excel. Vérifier son emplacement et son nom.", ErreurInfo())
		FinProgramme()
	FIN

En gros si je passe des variables global dans la définition de mes procédures tout fonctionne car on a bien les bonnes valeurs mais les variables global ne seront pas modifié une fois sortie de la procédure ?
Donc si notre procédure est uniquement une vérification de variables : c'est OK de mettre les mêmes noms dans la définition de la procédure ?
Et dans le cas où on souhaite modifier la valeur de la variable global alors il faut absolument que dans la définition de la procédure on passe des variables avec des noms inconnu puis on joue à l'intérieur de la procédure avec la variable global ?

Merci

[frenchsting]

Bonjour,

Il faudrait que tu déclares ton entête de procédure ainsi :

Code :

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

PROCÉDURE VérificationValiditeFichierImputationIT(chnCheminFichierXLS est une chaîne, xlsFichierXLS est un xlsDocument)

Et pour les appels, tu ne changes rien.

Mais come tu as des variables globales, tu pourrais faire un appel comme ça :

Code :

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

PROCÉDURE VérificationValiditeFichierImputationIT()

Si tu t'aperçois que tes variables globales n'ont pas (plus) lieu d'être, le cas 1 est ce qu'il te faut.

[EDIT] Tu as répondu entretemps, ce qui fait que pour moi, tu devrais faire l'appel sans argument. Mais j'avoue que ce n'est pas propre du tout.

[Aooka]

Ok je crois que je saisie bien mieux la chose avec vos explications et je vous en remercie.

Ce n'est pas propre de faire des procédure sans arguments ?

Le but de la procédure VérificationValiditeFichierImputationIT est simplement de vérifier si le fichier Excel s'ouvre.
Si c'est le cas je le referme.

Le chemin et l'ID du fichier Excel sont en global car je les utilises hors de ma procédure VérificationValiditeFichierImputationIT() et dans des champs interne à la fenêtre.
Par exemple, sur la fenêtre où ont été déclaré ces variables global, il y a un bouton "Afficher Excel" où j'utilisent ces variables. C'est pour cela que je les ais mises en variables global.



Bonne journée,

[tatayo]

En gros si je passe des variables global dans la définition de mes procédures tout fonctionne car on a bien les bonnes valeurs mais les variables global ne seront pas modifié une fois sortie de la procédure ?

Non. Tu ne passes pas de variable globale dans la définition de ta procédure, tu ne fait que la définir, et nommer ses paramètres.
Là comme les paramètres ont le même nom que des variables globales, Windev t'avertit que si te tentes d'accéder aux variables globales dans le code de ta procédure, tu accèderas en fait aux paramètres de celle-ci. La variable globale ne sera pas accessible.

"Normalement" la procédure ne devrait pas accéder aux variables globales, mais uniquement à ses paramètres.
Et comme par défaut dans Windev les paramètres sont passés par adresse (dixit l'aide en ligne), si tu modifies la valeur d'une paramètre dans la procédure, tu vas modifier la valeur de la variable qui a été passée en paramètre lors de l'appel de la procédure.

Tatayo.

[Voroltinquo]

Bonjour,
Contrairement à de nombreux autres langages, le WL utilise la passage par adresse.
Dans un passage par adresse, si la variable passée en paramètre est modifiée à l'intérieur de la fonction/procédure, cette modification se répercutera dans le code appelant.
Dans un passage par valeur, la valeur ne sera pas modifiée. Le passage par valeur doit être explicitement défini dans la déclaration de la fonction/procédure grâce au mot clé LOCAL.
En ràgle générale, il est déconseillé d'utiliser les variables globales dans une fonction/procédure alors qu'un passage par adresse fait le même boulot.
Exemple :
Soient les fonctions

Code :

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

1
2
3
4
5
 
FONCTION ParAdresse(nParamètre est entier) : entier
nParamètre=2*nParamètre+1
 
RENVOYER nParamètre

Code :

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

1
2
3
4
5
 
FONCTION ParValeur(LOCAL nParamètre est entier) : entier
nParamètre=2*nParamètre+1
 
RENVOYER nParamètre

Ces deux fonctions semblent similaires, toutefois, à l'utilisation on voit la différence :

Code :

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

1
2
3
4
5
6
7
8
9
10
11
12
13
14
 
nVal1 est un entier
nVal2 est un entier
 
nRes1 est un entier
nRes2 est un entier
 
nVal1=2
nVal2=2
 
nRes1=ParAdresse(nVal1)
Trace("Paramètre avant appel :2","Résultat :5","Paramètre après appel :5")
nRes2=ParValeur(nVal2)
Trace("Paramètre avant appel :2","Résultat :5","Paramètre après appel :2")

On pourrait écrire :

Code :

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

1
2
3
 
PROCÉDURE ParAdresse_2(nParamètre est entier)
nParamètre=2*nParamètre+1

Code :

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

1
2
3
4
5
6
7
8
9
 
nVal1 est un entier
 
nRes1 est un entier
 
nVal1=2
 
ParAdresse_2(nVal1)
nRes1=nVal1

Ainsi une fonction déclarée en C

Code C :

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

1
2
 
int MaFonction(int , int* nNoErr, char* saLibErr)

devrait se déclarer

Code :

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

1
2
 
FONCTION MaFonction(LOCAL nParamètre est entier, nNoErr est entier, saLibErr est chaine) : entier

Dans les faits LOCAL est la plupart du temps omis et avec la permissivité de WL,on se retrouve le plus souvent avec:

Code :

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

PROCEDURE MaFonction(Paramètre,NoErr,LibErr)

[Aooka]

Merci pour tout ceux qui ont pris le temps de me répondre.
Résolu.