2013-01-08 30 views
1

我有一个代码在页面加载时随机化div元素中的背景图像,但它似乎没有工作。JavaScript无法正常工作?随机发生器代码

JS中的部分:

function Randomize() { 
    var images = new Array("<?php echo SITE_URL?>/lib/skins/flyeuro/images/banners/ban.ner1.jpg","<?php echo SITE_URL?>/lib/skins/flyeuro/images/banners/banner2.jpg"); 
    var imageNum = Math.floor(Math.random() * images.length); 
    document.getElementById("banner").style.backgroundImage = "url('" + images[imageNum] + "') repeat scroll center center rgb(0, 0, 0)"; 
} 

window.onload = Randomize; 

HTML:

<div id="banner"></div> 

CSS:

div#banner { 
    background: url("../images/banners/banner1.jpg"); 
    width: 100%; 
    height: 270px; 
    z-index: 1; 
    margin-bottom: 25px; 
    -webkit-box-shadow: 0px 2px 15px 2px ; 
    box-shadow: 0px 2px 15px 2px ; 
} 
+1

”但它似乎没有工作。“详细说明 – George

+2

定义“不工作”。当你在调试器中遍历代码时,观察到的行为在什么时候会偏离预期的行为?发生这种情况时,相关的运行时间值是什么? – David

+0

您如何足够自信地使用'Math.random()* images.length',您想将它用作索引到数组中? –

回答

2

你设置多个背景属性到backgroundImage财产。相反,只需设置一个url()

如果你想总是有其他的值,我会把它们放在CSS中。

jsFiddle。 “

+0

我认为这是问题,现在它在我用CSS设置时正常工作,谢谢。 – zzwyb89