J'en perds mon latin (pas très élevé, c'est vrai)

Version imprimable

Bonsoir,

voici un code qui marche très bien (les variables h_xxx sont des variables système) (et il y a des fonctions que j'ai définies ailleurs), mais quand je l'encapsule dans une fonction et que j'appelle celle-ci, j'obtiens une erreur js (mais pas si je mets en commentaires la boucle do-while). Peut-on m'éclairer là-dessus ?

Code:

1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
17
18
19
20
21
22
23
24
25
var sesainfo = new Array(3);
sesainfo[0] = new Array(11);
sesainfo[1] = new Array(11);
sesainfo[2] = new Array(11);
var sesaArray = new Array();
if(h_CurrentSkinType=='h_Edit') {
document.write("Use this space to provide information about your experience, interests, favorite links, etc." );
document.write("<table border=0 width='100%'><tr><td class='h-fieldHeaderEdit-bgtext'>");
document.write("Name of this page<br><"+"input size='50' type='text' name='h_getName'>");
document.write("</td></tr><tr><td>&nbsp;</td></tr><tr>");
document.write("<td class='h-fieldHeaderEdit-bgtext'>");
document.write("</td></tr></table>");
} else {
sesaArray=loadsesa(h_LastEditorDistinguishedName);
var idNotes=extract_NotesID(h_LastEditorDistinguishedName);
var ind=0;
var comp;
do {
load_personal_data(sesaArray,ind);
comp=extract_idnotes_sansfin(sesainfo[ind][8]);
ind++;
} while (comp!=idNotes);
display_photo(sesainfo[ind-1][9],sesainfo[ind-1][1],sesainfo[ind-1][0],ind-1);
//display_info(sesainfo[ind-1]);
}

15/10/2009, 09h01
vermine

Bonjour,

Peut-être qu'il y a une erreur grosse comme une maison dans le code mais étant fatigué, je n'arrive pas à me concentrer. Pourriez-vous nous donner plus de code (load_personal_data() et extract_idnotes_sansfin()) ainsi que le message d'erreur ?
15/10/2009, 09h05
le_chomeur

problème de variables globale non définit ?

Message d'erreur : juste, "il y a des erreurs sur la page".
load_personal_data :

Code:

1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
17
18
19
20
21
22
23
24
25
26
27
28
29
30
31
32
33
34
35
36
37
38
39
40
41
42
43
function load_personal_data(sesa,ind){
var URL,codesesa,xmlsesa;
var topEntry,entry;
var nomemploye,prenomemploye,adresseemploye,villeemploye,siteemploye,telemploye,cellemploye,mailemploye,idnotesemploye,urlphotoemploye;
//for (i=0;i<(sesa.length);i++) { // parcourir tous les codes SESA
// construire l'URL avec les infos donn&eacute;es par le code SESA(du type :
// http://ids.schneider-electric.com/global/corp/ecd.nsf/xmlemployeesactivebyguid?readviewentries&amp;StartKey=SESA21805&amp;count=1)
URL='http://ids.schneider-electric.com/global/corp/ecd.nsf/xmlemployeesactivebyguid?readviewentries'+'&'+'StartKey='+sesa[ind]+'&amp;count=1';
// appeler l'URL
codesesa="h_sesaList";
xmlsesa = new ActiveXObject("Microsoft.XMLDOM");
xmlsesa.async="false";
xmlsesa.load(URL); // appel URL du code XML à partir du code SESA
//document.write("URL appel&eacutee.<br>");
//pour chq item int&eacute;ressant,
// r&eacute;cup&eacute;rer la valeur
topEntry=xmlsesa.selectSingleNode("viewentries");
entry=xmlsesa.selectSingleNode("viewentries/viewentry");
var sUnid2 = entry.getAttribute('unid');
nomemploye=entry.childNodes.item(2);
prenomemploye=entry.childNodes.item(4);
adresseemploye=entry.childNodes.item(26);
villeemploye=entry.childNodes.item(30);
siteemploye=entry.childNodes.item(25);
telemploye=entry.childNodes.item(14);
cellemploye=entry.childNodes.item(16);
mailemploye=entry.childNodes.item(10);
idnotesemploye=entry.childNodes.item(9);
urlphotoemploye=entry.childNodes.item(39);
buemploye=entry.childNodes.item(19);
sesainfo[ind][0] = nomemploye.text;
sesainfo[ind][1] = prenfieldNames.REMOTE_USER omemploye.text;
sesainfo[ind][2] = adresseemploye.text;
sesainfo[ind][3] = villeemploye.text;
sesainfo[ind][4] = siteemploye.text;
sesainfo[ind][5] = telemploye.text;
sesainfo[ind][6] = cellemploye.text;
sesainfo[ind][7] = mailemploye.text;
sesainfo[ind][8] = idnotesemploye.text;
sesainfo[ind][9] = urlphotoemploye.text;
sesainfo[ind][10] = buemploye.text;
//return sesainfo;
}

extract_idnotes_sansfin :

Code:

1
2
3
4
5
6
function extract_idnotes_sansfin(ch)
{
indarobas=ch.indexOf("@");
ch=ch.substring(0,indarobas);
return(ch);
}

15/10/2009, 09h33
vermine
En cliquant sur le triangle jaune de l'erreur, il affiche une fenêtre avec un peu plus de détails sur l'erreur Javascript.

Je me pose des questions sur ceci:
Code:

1 2 3 4 5 6 7 function extract_idnotes_sansfin(ch) { indarobas=ch.indexOf("@"); ch=ch.substring(0,indarobas); return(ch); }
Imaginons qu'il n'y ai pas d'arobase, le substring va chercher de la position -1 à 0 et va renvoyer une chaine "vide". Est-ce bien ce que vous désiriez ?
15/10/2009, 09h59
laurentSc

Merci pour l'info du triangle jaune : je ne connaissais pas.
sinon, pas de lézard pour l'arobase : il y en a forcément un.
Enfin, j'ai vu qu'il bloquait sur des fonctions qui font appel à des variables système que je ne maîtrise pas (tout ça est sur un site intranet), mais ce qui me surprend, c'est que quand le code n'est pas dans une fonction, ça marche bien...
15/10/2009, 10h05
vermine

Je rejoins l'idée du Chômeur, peut-être que en mettant le code dans une fonction, vous rendez locales des variables qui aurait du être globales car elles sont utilisées par d'autres fonctions ?!

Détectez les variables en question et laissez-les globales.
15/10/2009, 10h53
le_chomeur

c'est certain ...
mais sans la page complète dur dur de savoir laquelle pose problème
15/10/2009, 10h57
laurentSc

Merci pour la piste ; j'ai renommé toutes mes variables locales, ce qui m'a permis d'aller un peu plus loin, mais maintenant il bute sur la ligne

Code:

var sUnid22 = entry2.getAttribute('unid');

avec une erreur du type "null a la valeur Null ou n'est pas un objet". Moi, ça ne m'aide pas : avez-vous une idée ?
15/10/2009, 11h19
vermine

Renommer les variables locales ? J'aurais plutôt sorti les variables globales. :?

Sinon, il faudrait voir (par un alert par exemple) si entry2 est bien un objet, et vérifier si il y a un "unid" définit dans le "viewentries/viewentry".
15/10/2009, 12h05
laurentSc

Qu'est-ce que vous appelez "sortir les variables globales" ? Sinon, quand, j'affiche entry2, il me donne entry2=null ; mais ça marche si je n'incorpore pas ce code dans une fonction : bizarre...

Faudrait voir la nouvelle version de load_personal_data(sesa,ind), c'est-à-dire avec vos nouveaux noms de variables.
Mais bon, je reviendrais en arrière, avec les anciens noms de variables et je sortirai de load_personal_data(sesa,ind) les variables qui sont normalement globales. Lesquelles exactement, ça je ne le sais pas. Soit vous avez tout le code et vous pouvez voir qui utilise quelles variables, soit vous testez une par une. :aie:

Exemple:
Code:

1 2 3 4 5 6 7 8 9 10 11 //Je sors quelques variables de la fonction, elles perdent le statut de local et deviennent globales car qqun d'autre les utilisent. var topEntry,entry; function load_personal_data(sesa,ind){ //Je garde quelques variables locales car elles n'ont lieu d'exister que dans cette fonction. var URL,codesesa,xmlsesa; var nomemploye,prenomemploye,adresseemploye,villeemploye,siteemploye,telemploye,cellemploye,mailemploye,idnotesemploye,urlphotoemploye; [...] //return sesainfo; }