Begynder søger hjælp til rettelser og optimering (Del 1)

<ole>

Det kommer anpå til, hvad det skal bruges ... og hvordan  :)

/mvh
</bole>

Hej Ole.

Ja scriptet skal bruges til at åbne et nyt vindue. Ja jeg ved der findes massere af scripts derude der gør det, men jeg arbejder med det af 2 grunde.
1. Jeg lære en del af at forsøge mig med det
2. jeg vil se om ikke jeg kan lave det til en action

Det jeg gerne ville vide er om denne del kan skrives lidt smartere (Fylde mindre/fære linier) ??

Den ser nu sådanne ud (og virker efter hensigt)

if (maxHoej == true ){
    vinTop = 0;
    vinHoej = winHight;
    }
if (maxBred == true){
    vinVens = 0;
    vinBred = winWidth;
    }
if (vinHoej == ""){
    vinHoej = 0;
    }
if (vinBred == ""){
    vinBred = 0;
    }
if (centsk == true){
    vinTop = (winHight-vinHoej)/2;
    vinVens = (winWidth-vinBred)/2;
    vinTop = Math.round(vinTop);
    vinVens = Math.round(vinVens);
    if (vinHoej < 1){
        vinTop = 0;
        vinHoej = winHight;
        }
    if (vinBred < 1){
        vinVens = 0;
        vinBred = winWidth;
        }
    }
if (fuldsk == true){
    vinTop = 0;
    vinVens = 0;
    vinHoej = winHight;
    vinBred = winWidth;
    }
var vinSet = "";
    vinSet += ",left=" + vinVens;
    vinSet += ",top=" + vinTop;

Mvh
Jesper

Bortset fra, at der er en hel række overflødige sammenligninger kan det zq nok ik' laves anderledes, nå man vil tage højde for andres dårlige kodestil ...

-- men du laver det vgel til dig selv, så du ved at tingene er initialiseret ordentligt ?-)

-- men disse er antageligt ens uanset kodningen udenfor:

if (centsk == true){
if (centsk){

nå... tingene virker efter hensigt nu .... så må hellere få lukket ....

Smid venligt et svar