[SOLVED] What have I done wrong in this code?

acrikey · March 23rd, 2008, 12:40 PM

I am trying to make this script change my banner jpg dependent on the month:

Code:

<script>

var Digital=new Date()
var month=Digital.getMonth()

if (month <= 2){
    // month is between jan and march
    
    document.write[Winter]("<IMG SRC='fieldworks/images/img5.jpg'>");
} else if (month <= 5){
    // month is between april and june
    document.write[Spring]("<IMG SRC='fieldworks/images/img2.jpg'>");
} else if (month <= 8){    
    // july and september
    document.write[Summer]("<IMG SRC='fieldworks/images/img2.jpg'>");
} else {
    // otherwise its between oct and dec
    document.write[Autumn]("<IMG SRC='fieldworks/images/img3.jpg'>");
}
</script>

any ideas?

Jack Franklin · March 23rd, 2008, 01:28 PM

Are you getting an error or nothing at all? What is going wrong?

acrikey · March 23rd, 2008, 01:48 PM

blank screen

http://www.lightex.co.uk/fieldworks/season.html

Uername: Test
Pass: Webforumz

aihazm · March 23rd, 2008, 03:07 PM

write (in document.write) is a function not an array or an object, so you can't use [], you have tu use () :
document.write[], should be document.write() , and inside the paranthese yaou can put the string that you want to parse into the html document:

Code:

<script>
var Digital=new Date()
var month=Digital.getMonth()
if (month <= 2){    // month is between jan and march    
    document.write("<IMG SRC='fieldworks/images/img5.jpg'>");
} else if (month <= 5){
    // month is between april and june
    document.write("<IMG SRC='fieldworks/images/img2.jpg'>");
} else if (month <= 8){    // july and september
    document.write("<IMG SRC='fieldworks/images/img2.jpg'>");
} else {    // otherwise its between oct and dec
    document.write("<IMG SRC='fieldworks/images/img3.jpg'>");
}
</script>

But maybe you have a purpose in putting [Winter]...[Summer]-> can you explain?

acrikey · March 23rd, 2008, 03:16 PM

still nothing

acrikey · March 23rd, 2008, 03:18 PM

it was a hint for me

acrikey · March 23rd, 2008, 07:39 PM

Thank you guys, I have sorted it now, I had to put the whole url in in the end