Error script?

Hej!
Om man har detta script för att placera ett lager på rätt ställe beroende av upplösning.

<script type="text/javascript" language="JavaScript">
<!--
function pos(){
 if(document.layers){document.layers["lager1"].left=(window.innerWidth/2)-252;}
else if(document.all){document.all("lager1").style.left=(document.body.offsetWidth/2)-252;}
else
if(document.getElementById){document.getElementById("lager1").style.left=(window.innerWidth/2)-252;}
</script>

Kan detta script leda till ett javascript error som ibland och ibland inte dyker upp?
Kan man byta ut paranteseran till klamrar på lagernamnen :q utan att det blir fel? eller finns det någon risk med detta?
Nuvarande kod:
else if(document.all){document.all("lager1")
byter ut
else if(document.all){document.all["lager1"]
Jag har bytt ut detta på en sida och det verka funka utan att det blir error
Finns det någon risk med detta script?

<script type="text/javascript" language="JavaScript">
<!--
function pos(){
 if(document.layers){document.layers["lager1"].left=(window.innerWidth/2)-252;}
else if(document.all){document.all["lager1"].style.left=(document.body.offsetWidth/2)-252;}
else
if(document.getElementById){document.getElementById("lager1").style.left=(window.innerWidth/2)-252;}
</script>

------------------
/nordan

[Redigerat av nordan den 09 sep 2001]

Problemet beror på att du delar fönstrets bredd med 2 och därmed riskerar en 50% chans att få ett decimaltal som svar. Det är svårt för webbläsaren att placera ett lager på 112,5 pixlar :)

Använd alltså parseInt(...uträkning...) för att få ut heltal.

Sedan föreslår jag den senare varianten (hakparenteser).

------------------
/ J

Fast kanske Math.round(uträkning) vore bättre än parseInt(uträkning)...

------------------
God, root, what is difference? /Pitr

Ja, det är klart. Man vill ju inte riskera att lagret sitter 1 pixel fel ;)

------------------
/ J