3 svar · 186 visningar · startad av nordan
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 
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>
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).
Fast kanske Math.round(uträkning) vore bättre än parseInt(uträkning)...
Ja, det är klart. Man vill ju inte riskera att lagret sitter 1 pixel fel 
