Images aléatoire avec popup paramétré

Bonjour,

Je suis pas du tous fort pour le javascript, j'ai tenté de résoudre mon problème sur plusieurs forum déja, et personne ne m'a encore donné de réponse qui fonctionne.

voici mon problème :

j'ai trouvé un code javascript pour affiché des images de manière aléatoire avec lien. Mais j'aimerais que les liens s'ouvre dans une fenêtre différente (façon popup) et en plus paramétré. (exigent le gars ).

Je vous présente donc 2 codes, celui que je viens d'énoncer et le code pour ouvrir un popup paramétré

Voici celui avec les images aléatoires et avec des liens classique :

PARTIE HEAD :

<SCRIPT LANGUAGE="JavaScript">

var nbimage= 20;
var width;
var height;
var url;
var alte;
function banniere()
{
numimage= Math.round(Math.random()*(nbimage-1)+1);
if (numimage <= 10)
{
gotosite = "le_bon_plan/plan_01.html";
url = "le_bon_plan/plan_01.gif";
alte = "";
}
if (numimage > 10)
{
gotosite = "le_bon_plan/plan_02.html";
url = "le_bon_plan/plan_02.gif";
alte = "";
}
if(gotosite != "")
{
document.write ('<A HREF="' + gotosite + '">'); 
}
document.write('<IMG SRC="' + url + '" ALT="' + alte + '" BORDER=0>')
if(gotosite != "")
{
document.write('</A>')
}
}
</SCRIPT>

PARTIE BODY :

<SCRIPT language="JavaScript">
banniere();
</SCRIPT>

Voici maintenant le code pour popup paramétré :

<a href="#" class="dplayer" onClick="window.open('le_bon_plan/plan_01.html','_blank','toolbar=0, location=0, directories=0, status=0, scrollbars=0, resizable=0, copyhistory=0, menuBar=0, width=753, height=556, left=0, top=0');return(false)">

Comment pui-je faire pour combiner les deux ?

Merci d'avance pour votre aide


Pour suivre l'évolution de ce script sur les autres forums, voici les liens :

- EditeurJavascript
- Developpez
Modifié par hoodvy (03 Oct 2008 - 13:01)

Bonjour, je vais peut-être paraître (être ?) méchant, mais c'est une horreur, ce code, autant le HTML que le javascript, c'est ce qu'on écrivait dans les années... 90.

Qqs conseils :

- écrire les balises HTML en minuscule, et les attributs aussi ;
- proscrire les scripts à l'intérieur du HTML, et a fortiori les document.write() ;
- l'attribut language de <script> n'a jamais été standardisé, et les navigateurs n'en tiennent pas compte (en tout cas pour le javascript) autrement dit, il ne sert à rien, le remplacer par type="text/javascript" ;
- et enfin, les popups... c'est mal, utilise plutôt des lightboxes (http://livepipe.net/control/window ou même http://ajax.phpmagazine.net/upload/2006/03/lightbox.html)

Si tu tiens vraiment à garder ce code, il suffit de "combiner" cette ligne :

document.write ('<A HREF="' + gotosite + '">'); 

avec cette ligne :

<a href="#" class="dplayer" onClick="window.open('le_bon_plan/plan_01.html','_blank','toolbar=0, location=0, directories=0, status=0, scrollbars=0, resizable=0, copyhistory=0, menuBar=0, width=753, height=556, left=0, top=0');return(false)">

A bientôt