Problème avec document.write

André31
Javascript

Bonjour,

Voila je rencontre un petit problème avec mon code.

Ce que je fais

Décrivez ici votre code ou ce que vous cherchez à faire

<!--DEBUT 7 PARTENAIRES 1--><br />
<script type="text/javascript">
function nb_aleatoire(nb){nombre=Math.floor(Math.random()*nb)+0;return nombre;}var nbaleatoire=nb_aleatoire(7);
var tableau_image=new Array("http://petanquelaunaguet.wifeo.com/images/b/bor/Borderouge.jpg","http://petanquelaunaguet.wifeo.com/images/m/mar/Martignon3.jpg","http://petanquelaunaguet.wifeo.com/images/s/ste/Stef-er.jpg","http://petanquelaunaguet.wifeo.com/images/p/piz/Pizza1.jpg","http://petanquelaunaguet.wifeo.com/images/f/fol/Foltran1.jpg","http://petanquelaunaguet.wifeo.com/images/b/bri/Brillas.jpg","http://petanquelaunaguet.wifeo.com/images/c/coi/coiffeur1.jpg");
var tableau_liens=new Array("#","http://pneu-4x4-prix-pas-cher-achat-vente.martignonpneus.com","#","http://www.tutti-pizza.com/","http://garage-auto-entretien-voiture-vente-occasion-voitures-neuves.garage-foltran.com/","#","http://www.linstant-pour-soi.fr/");
document.write('<a target="_blank" href="'+tableau_liens[nbaleatoire]+'" ><img border="0" src=" '+tableau_image[nbaleatoire]+' " alt="'+'" /></a>'+"<br />");
<!-- target="_blank" POUR OUVRIR LA PAGE DANS UN NOUVEL ONGLET-->
</script><br />
<!--FIN-->

Ce que je veux

Sur le site http://petanquelaunaguet.wifeo.com/ créé il y a 4 ans, afficher de manière aléatoire un annonceur avec lien éventuel sur son site.

Ce que j'obtiens

Ce script a fonctionné pendant plusieurs mois mais est maintenant refusé par wifeo à cause de document.write.
Ce script fonctione parfaitement en testant sur https://jsfiddle.net/.
J'ajoute que je suis un bricoleur...

Quelqu'un a-t-il une solution ?

Merci

11 réponses

betaWeb

Salut,

C'est quoi cette horreur http://petanquelaunaguet.wifeo.com/ Ôo
Ce site n'a pas 4 ans il a 15 ans ! :o

Bref, wifeo a surement bloqué la fonction document.write() (qui n'est d'ailleurs plus vraiment utilisé).
Il faut ajouter le contenu souhaité à un élément HTML conteneur.

André31
Auteur

Cette horreur satisfait beaucoup de monde !
Pourquoi ce site aurait-il 15 ans ???

Huggy

@MacFly31 pour revenir dans le présent il te faut 1.21 GIGAWATT !!!!!

André31
Auteur

Je suis inculte en la matière...
je ne comprends pas !

Defy

Pourquoi ce site aurait-il 15 ans ???

ben simplement parceque ca ressemble a du web d'il y as 15 ans...

voila un exemple un peut plus moderne de site de boulliste et qui n'agresse pas les yeux et peut provoquer des crises d'epilepsie...

je ne sais pas si le design est de toi, mais faudrait revoir un peut ca quand meme, moderniser l'interfaces peut etre pas mal pour le référencement. avec du responsive

betaWeb

C'est surtout que les techniques utilisées sur ce site sont obsolètes depuis une bonne 10aine d'années maintenant, du coup le code est difficilement maintenable.
Et puis excusez mon franc parlé, mais le site est moche, point; On dirait qu'il a été fait sous MS Publisher 1997 :/

Huggy

Ah nostalgie !!! , c'est vrai qu'on s'attend à voir la pub du concessionnaire Simca
je vais aspirer le site pour montrer plus tard à mes petits enfants

Maintenant pour répondre à la question, le meilleur moyen de remplacer document.write c'est d'utiliser innerHTML
mais comme ce site est moderne, et qu'il utilise jquery, c'est $('#madiv').html

André31
Auteur

J'espère toucher les royalties...
On se moque mais on n'a toujours pas de solution précise à mon problème !

betaWeb

@André31 Nous ne sommes pas à ton service, nous sommes ici pour aider et non pour te pondre du code.
Nous t'avons donné plusieurs pistes, à toi de les creuser.
En ce qui concerne les critiques, elles sont fondées. Certes ce n'est que du visuel, mais ça compte quand même, et en 2016 il y a énromément de technos qui permettent de sortir un site ayant un design au minimum 'moderne'.
Enfin bref.

Huggy

Comme je l'ai dit plus haut, le mieux serait de remplacer le document.write par un innerHTML
mais comme ton site utilise jQuery (ça c'est moderne),
il suffirait de créer une div vide et d'y injecter ce qu'il y a dans le document.write

$('#blabla').html('<a target="_blank" href="'+tableau_liens[nbaleatoire]+'" ><img border="0" src=" '+tableau_image[nbaleatoire]+' " alt="'+'" /></a>'+"<br />");

<div id='blabla'></div>

On se moque mais c'est pas méchant :-)
Si ton site a du succès, c'est le principal
bon courage

André31
Auteur

Merci tu es sympa... Le site est très suivi mais Je vais modifier le design dans l'année !
Pour revenir à la correction proposée : où dois je positionner les instructions proposées ? dans le script ?
Merci