Discussion :

[Valadone]

Bonsoir,

Je ne sais pas du tout comment empêcher qu'un objet JS ne se crée pas plus d'une fois dans la page car j'ai un problème avec un script qui me crée mon objet 2 fois dans la plage et ça fausse pas mal de mes données récupérée pour analytics.

J'ai une fonction de ce type là :

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
17
18
19
20
21
22
23
24
25
26
27
28
29
30
31
32
33
34
<script type="text/javascript">
 
var Object = Object|| {};
 
Object.config = {
  hideBanniere : '[%hideBanniere%]',
  containerID : '[%containerID%]',
  bgColor : '[%bgColor%]',
  backupImage : '[%backupImage%]',
  customClass : '[%customClass%]',
  mediaType : '[%mediaType%]',
  forcedDomain : '[%forcedDomain%]'
  }
}
</script>
 
<script type="text/javascript">
 
 
var creativeRevision = creativeRevision || 'creative.1.1.66-32';
var sc = document.createElement('script');
sc.type = "text/javascript";
sc.src = "../js/wallpaper/ctwallpaper.js";
document.body.appendChild(sc);
sc.onload = function(){
  try{
  var begin = new Object.wallpaper(Object.config);
  begin._init();
  }
  catch(e){
  console.error(e);
  }
};
</script>

Inutile de me demander de "googler" cela, si je demande ici c'est que j'ai cherché en amont et ce que j'ai trouvé ne m'a pas aidé.

Merci par avance

[danielhagnoul]

var Object = ... !

Vous ne pouvez pas nommer une variable "Object", vu qu'en JS Object est l'objet à la base de tous les autres.

Object.config = { ... } !

Modifier le contenu de Object est la pire chose que l'on puisse faire.

[Valadone]

A la base l'objet a un autre nom, c'est juste que l'objet a le nom de mon entreprise et je n'ai pas envie de la divulguer de partout sur le net voilà tout

Une astuce pour ma demande sinon ?

EDIT :

Il semblerait que quelque chose comme cela fonctionne :

Code :

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

1
2
3
4
5
6
if(typeof(window['Purch']) !== "undefined"){
alert("Déjà créé");
}
else{
alert("Undefined");
}

[topolino]

Code :

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

if(!window.tonObject){ window.tonObject = tonObject; }

[Valadone]

Les réponses que vous m'avez donné, ça doit être bon à coup sûr mais mon soucis maintenant c'est que mon objet est créé avant que la vérification ne se fasse et donc il ne rentre jamais dans le if...

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
17
18
19
20
21
22
23
24
25
26
27
28
29
30
31
32
33
34
35
<script type="text/javascript">
 
var monObjet = monObjet|| {};
 
monObjet.config = {
  hideBanniere : '[%hideBanniere%]',
  containerID : '[%containerID%]',
  bgColor : '[%bgColor%]',
  backupImage : '[%backupImage%]',
  customClass : '[%customClass%]',
  mediaType : '[%mediaType%]',
  forcedDomain : '[%forcedDomain%]'
  }
}
</script>
 
<script type="text/javascript">
 
 
var creativeRevision = creativeRevision || 'creative.1.1.66-32';
var sc = document.createElement('script');
sc.type = "text/javascript";
sc.src = "../js/wallpaper/ctwallpaper.js";
document.body.appendChild(sc);
sc.onload = function(){
  try{
  var begin = new monObjet.wallpaper(monObjet.config);
  /********** C'est ici que je voudrais que monObjet.wallpaper(monObjet.config)._init(); ne se reproduise pas une seconde fois  ********/
  begin._init();
  }
  catch(e){
  console.error(e);
  }
};
</script>

Dans mon code c'est dans le try que je voudrais qu'il ne se reproduise pas une seconde fois : begin._init()

[thejocker9]

Sort le du onload

Code :

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

1
2
3
4
5
6
7
8
9
10
11
12
13
 
sc.onload = function(){
  try{
  var begin = new monObjet.wallpaper(monObjet.config);
 
 
var begin; 
sc.onload = function(){
  try{
    if(begin==undefined){
      begin = new monObjet.wallpaper(monObjet.config);
      begin._init();
    }