Oldsoftboss
AWF VIP
- Local time
- Today, 13:06
- Joined
- Oct 28, 2001
- Messages
- 2,499
I have spent today updating our home page (raining outside, nothing much else to do!)
http://members.dodo.com.au/predatorsoftware/index.html
I coded the background page to load a random image each time the page refreshes, but with my access experience, I feel the code is messy and could be better... Any suggestions on the following:
I feel I should be able to combine the "cur" with the "backgr" rather than using the long winded if statement.
Dave
http://members.dodo.com.au/predatorsoftware/index.html
I coded the background page to load a random image each time the page refreshes, but with my access experience, I feel the code is messy and could be better... Any suggestions on the following:
Code:
<script>
<!--
var backgr1="backgrounds/adelaide.jpg"
var backgr2="backgrounds/blackboys.jpg"
var backgr3="backgrounds/cathederal.jpg"
var backgr4="backgrounds/christmas.jpg"
var backgr5="backgrounds/creek.jpg"
var backgr6="backgrounds/dad.jpg"
var backgr7="backgrounds/emmylou.jpg"
var backgr8="backgrounds/fernery.jpg"
var backgr9="backgrounds/fire.jpg"
var backgr10="backgrounds/houseboats.jpg"
var backgr11="backgrounds/jetboat.jpg"
var backgr12="backgrounds/kookaburra.jpg"
var backgr13="backgrounds/muffin1.jpg"
var backgr14="backgrounds/muffin2.jpg"
var backgr15="backgrounds/river1.jpg"
var backgr16="backgrounds/tash1.jpg"
var backgr17="backgrounds/tash2.jpg"
var backgr18="backgrounds/wayne.jpg";
var cur=Math.round(18*Math.random())
if (cur<=1)
backgrnd=backgr1
else if (cur<=2)
backgrnd=backgr2
else if (cur<=3)
backgrnd=backgr3
else if (cur<=4)
backgrnd=backgr4
else if (cur<=5)
backgrnd=backgr5
else if (cur<=6)
backgrnd=backgr6
else if (cur<=7)
backgrnd=backgr7
else if (cur<=8)
backgrnd=backgr8
else if (cur<=9)
backgrnd=backgr9
else if (cur<=10)
backgrnd=backgr10
else if (cur<=11)
backgrnd=backgr11
else if (cur<=12)
backgrnd=backgr12
else if (cur<=13)
backgrnd=backgr13
else if (cur<=14)
backgrnd=backgr14
else if (cur<=15)
backgrnd=backgr15
else if (cur<=16)
backgrnd=backgr16
else if (cur<=17)
backgrnd=backgr17
else
backgrnd=backgr18
document.write('<body background="'+backgrnd+'" bgcolor="#FFFFFF">')
I feel I should be able to combine the "cur" with the "backgr" rather than using the long winded if statement.
Dave