map per tijd
Dus:
Tussen 0:00 t/m 4:59 = nacht
Tussen 5:00 t/m 11:59 = ochtend
Tussen 12:00 t/m 17:59 = dag
Tussen 18:00 t/m 22:59 = avond
Tussen 23:00 t/m 0:00 = nacht
Mijn script is als volgt:
Code (php)
1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
17
18
19
20
21
22
23
24
25
26
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
17
18
19
20
21
22
23
24
25
26
<?php
if(date(G) >= 18){ // dus 18:00 of later
if(date(G) >= 23) { // dus 23 en later
$random_map = "nacht/";
}
else { //dus tussen 18:00 en 22:59
$random_map = "avond/";
}
}
elseif(date(G) <= 12){ // dus 12:59 of vroeger
if(date(G) <= 4) { //dus 4:59 of vroeger
$random_map = "nacht/";
}
else { //dus tussen 5:00 en 11:59
$random_map = "ochtend/";
}
}
else{
$random_map = "dag/";
}
?>
if(date(G) >= 18){ // dus 18:00 of later
if(date(G) >= 23) { // dus 23 en later
$random_map = "nacht/";
}
else { //dus tussen 18:00 en 22:59
$random_map = "avond/";
}
}
elseif(date(G) <= 12){ // dus 12:59 of vroeger
if(date(G) <= 4) { //dus 4:59 of vroeger
$random_map = "nacht/";
}
else { //dus tussen 5:00 en 11:59
$random_map = "ochtend/";
}
}
else{
$random_map = "dag/";
}
?>
Mijn vraag is, kan ik dit korter maken?
even helemaal alles op 1 regel gepropt :P
Korter? Het gaat niet om de grootte, maar om wat je d'r mee doet toch? :P
Hieraan is mijnsinziens voldaan.
Woopie100:
even helemaal alles op 1 regel gepropt :P
Dit snap ik niet.
Pholeron:
Korter? Het gaat niet om de grootte, maar om wat je d'r mee doet toch? :P
Ik zou het overzichtelijker noemen.
santhe, volgens mij mis je de boodschap :)
Als je $random_map vervangt door $r wordt 'ie nóg korter ;-)
Doorgaans vind ik leesbaarheid, hoe flexibel je zaken kan uitbreiden, en hoe eenvoudig je er gepast commentaar bij kan zetten, belangrijker dan het feit dat het op 3 regels kan in plaats van 10..
Het was dan ook meer een reactie naar de maker van het oorspronkelijke script, dan naar de geboden oplossing, waant daar is niets mis mee ;)