Kleiner maken van mijn if/elseif constructie?

Door Richard Hansma op 11-03-2014 15:24

1.684 views

Hallo PHP-ers,

Inmiddels bezig met een PHP dart script en nu heb ik keuze menu waar je het aantal sets en legs kunt kiezen, nu is het zo dat wanneer je bijvoorbeeld een best of 1 leg kiest. Nu is dat niet zo moeilijk te maken. Nu is het probleem meer, moet ik nu een elseif aanmaken voor elke 'best of leg(s)'?

Op dit moment heb ik dit (voor speler 1):

<?php
if(isset($_SESSION['player1'], $_SESSION['scoreleft1']) && $_SESSION['scoreleft1'] == 0) {
$_SESSION['throws1'] = 0;
$_SESSION['throws2'] = 0;
unset($_SESSION['lastthrows1']);
unset($_SESSION['lastthrows2']);
if($_SESSION['bestofsets'] == 1) {
if($_SESSION['legs1'] == 0 && $_SESSION['bestoflegs'] == 1) {
$win = $_SESSION['player1'].' heeft gewonnen!';
$_SESSION['legs1'] = 0;
$_SESSION['legs2'] = 0;
$_SESSION['sets1']++;
}
elseif($_SESSION['legs1'] < 1 && $_SESSION['bestoflegs'] == 3) {
$_SESSION['legs1']++;
$_SESSION['scoreleft1'] = $_SESSION['game'];
$_SESSION['scoreleft2'] = $_SESSION['game'];
}
elseif($_SESSION['legs1'] == 1 && $_SESSION['bestoflegs'] == 3) {
$win = $_SESSION['player1'].' heeft gewonnen!';
$_SESSION['legs1'] = 0;
$_SESSION['legs2'] = 0;
$_SESSION['sets1']++;
}
elseif($_SESSION['legs1'] < 2 && $_SESSION['bestoflegs'] == 5) {
$_SESSION['legs1']++;
$_SESSION['scoreleft1'] = $_SESSION['game'];
$_SESSION['scoreleft2'] = $_SESSION['game'];
}
elseif($_SESSION['legs1'] == 2 && $_SESSION['bestoflegs'] == 5) {
$win = $_SESSION['player1'].' heeft gewonnen!';
$_SESSION['legs1'] = 0;
$_SESSION['legs2'] = 0;
$_SESSION['sets1']++;
}
elseif($_SESSION['legs1'] < 3 && $_SESSION['bestoflegs'] == 7) {
$_SESSION['legs1']++;
$_SESSION['scoreleft1'] = $_SESSION['game'];
$_SESSION['scoreleft2'] = $_SESSION['game'];
}
elseif($_SESSION['legs1'] == 3 && $_SESSION['bestoflegs'] == 7) {
$win = $_SESSION['player1'].' heeft gewonnen!';
$_SESSION['legs1'] = 0;
$_SESSION['legs2'] = 0;
$_SESSION['sets1']++;
}
elseif($_SESSION['legs1'] < 4 && $_SESSION['bestoflegs'] == 9) {
$_SESSION['legs1']++;
$_SESSION['scoreleft1'] = $_SESSION['game'];
$_SESSION['scoreleft2'] = $_SESSION['game'];
}
elseif($_SESSION['legs1'] == 4 && $_SESSION['bestoflegs'] == 9) {
$win = $_SESSION['player1'].' heeft gewonnen!';
$_SESSION['legs1'] = 0;
$_SESSION['legs2'] = 0;
$_SESSION['sets1']++;
}
}
}
?>

Mijn vraag is dus, kan dit korter en zo ja, hoe?

Ivo P

11-03-2014 15:44

misschien scheelt het als je de spelregels uitlegt?

zoals het nu lijkt, zou het verschil meer dan 2 "punten" moeten zijn, tenzij best gelijk is aan 9?

naast < en == bestaat er trouwens ook <= wat je in elk geval al de helft van je regels zou schelen.

Erwin H

11-03-2014 15:45 gewijzigd op 11-03-2014 15:46

Even iets anders groeperen en je ziet meteen een patroon:


<?php
        if($_SESSION['legs1'] == 0 && $_SESSION['bestoflegs'] == 1) {
            $win =  $_SESSION['player1'].' heeft gewonnen!';
            $_SESSION['legs1'] = 0;
            $_SESSION['legs2'] = 0;
            $_SESSION['sets1']++;
        }
        elseif($_SESSION['legs1'] == 1 && $_SESSION['bestoflegs'] == 3) {
            $win =  $_SESSION['player1'].' heeft gewonnen!';
            $_SESSION['legs1'] = 0;
            $_SESSION['legs2'] = 0;
            $_SESSION['sets1']++;  
        }
        elseif($_SESSION['legs1'] == 2 && $_SESSION['bestoflegs'] == 5) {
            $win =  $_SESSION['player1'].' heeft gewonnen!';
            $_SESSION['legs1'] = 0;
            $_SESSION['legs2'] = 0;
            $_SESSION['sets1']++;
        }
        elseif($_SESSION['legs1'] == 3 && $_SESSION['bestoflegs'] == 7) {
            $win =  $_SESSION['player1'].' heeft gewonnen!';
            $_SESSION['legs1'] = 0;
            $_SESSION['legs2'] = 0;
            $_SESSION['sets1']++;
        }
        elseif($_SESSION['legs1'] == 4 && $_SESSION['bestoflegs'] == 9) {
            $win =  $_SESSION['player1'].' heeft gewonnen!';
            $_SESSION['legs1'] = 0;
            $_SESSION['legs2'] = 0;
            $_SESSION['sets1']++;
        }

        elseif($_SESSION['legs1'] < 1 && $_SESSION['bestoflegs'] == 3) {
            $_SESSION['legs1']++;
            $_SESSION['scoreleft1'] = $_SESSION['game'];
            $_SESSION['scoreleft2'] = $_SESSION['game'];
        }
        elseif($_SESSION['legs1'] < 2 && $_SESSION['bestoflegs'] == 5) {
            $_SESSION['legs1']++;
            $_SESSION['scoreleft1'] = $_SESSION['game'];
            $_SESSION['scoreleft2'] = $_SESSION['game'];
        }
        elseif($_SESSION['legs1'] < 3 && $_SESSION['bestoflegs'] == 7) {
            $_SESSION['legs1']++;
            $_SESSION['scoreleft1'] = $_SESSION['game'];
            $_SESSION['scoreleft2'] = $_SESSION['game'];
        }
        elseif($_SESSION['legs1'] < 4 && $_SESSION['bestoflegs'] == 9) {
            $_SESSION['legs1']++;
            $_SESSION['scoreleft1'] = $_SESSION['game'];
            $_SESSION['scoreleft2'] = $_SESSION['game'];
        }

?>

En dan lijkt het mij een koud kunstje om het in twee opties te schrijven (maar test het nog wel, want dat heb ik niet gedaan):


<?php
if ( $_SESSION['legs1'] == floor( $_SESSION['bestoflegs'] / 2 ) ){
  $win =  $_SESSION['player1'].' heeft gewonnen!';
  $_SESSION['legs1'] = 0;
  $_SESSION['legs2'] = 0;
  $_SESSION['sets1']++;
} else {
  $_SESSION['legs1']++;
  $_SESSION['scoreleft1'] = $_SESSION['game'];
  $_SESSION['scoreleft2'] = $_SESSION['game'];
}
?>

Michael -

11-03-2014 16:03

Hier was laatst ook al een vraag over een dart spel. Toen leek hij mij wel leuk om te kijken of het nog lastig zou gaan met die legs, sets, matches etc. En wanneer de één moet beginnen, etc. Uiteindelijk heb ik dit ervan gemaakt (source) misschien kan je er iets mee.

Richard Hansma

11-03-2014 17:21

Bedankt, door jullie input kan ik weer verder.

Leuk gedaan trouwens Michael!

Reageren

Inloggen om te reageren