map per tijd
Ik heb een scriptje waarmee ik vier verschillende mappen aanspreek 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:
Mijn vraag is, kan ik dit korter maken?
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?
Tja... de vraag van de toppic was of het script korter kon en dan natuurlijk wel dezelfde functionaliteit behouden en altijd dezelfde uitvoer.
Hieraan is mijnsinziens voldaan.
Hieraan is mijnsinziens voldaan.
Pholeron:
Korter? Het gaat niet om de grootte, maar om wat je d'r mee doet toch? :P
Ik zou het overzichtelijker noemen.
De code is korter, dus wat dat betreft is het antwoord goed voor deze vraag (:
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 ;)
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 ;)