Tijd verschil berekenen

Door Jeremy Evers op 24-11-2016 11:07 gewijzigd op 24-11-2016 11:08

5.917 views

Hallo,

Ik ben voor een website aan het berekenen of een restaurant open is. De restaurants zijn vaak tot na middernacht open, zoals 01:00, nu moet ik dus gaan kijken of de huidige tijd voor de tijd is van de sluitingstijd, enkel ondervind ik wat problemen mee. Tot 23:59 gaat het prima, maar daarna niet. Dit is mijn code op dit moment, het werkt tot 23:59, maar daarna geeft hij aan gesloten, terwijl hij nog tot 01:20 open is. In explode[1] en explode[2] zit de open tijd en gesloten tijd en daar maak ik time van.


$time = time();
$open_time = strtotime($explode[1]);
$close_time = strtotime($explode[2]);
                       
$diff_open = ($time - $open_time);
$diff_gesloten = ($time - $close_time);
                      
if ($diff_open > 0 && $diff_gesloten > 0) {
    echo "<div class='companypage__restaurantopen'>Open</div>";
}else {
    echo "<div class='companypage__restaurantclosed'>Gesloten</div>";
}

Jeremy Evers

24-11-2016 15:21

Toch gaat het nog niet helemaal lekker. Hij is namelijk van 16:00 tot 01:30 geopend. Nu is het bijvoorbeeld 01:10 en geeft hij aan dat het gesloten is, omdat die al wel de open tijd pakt van de volgende dag. En dat komt weer omdat ik er dit boven heb staan:

if ($dagNamen[date('D')] == $dagen[$explode[0]]) {

Maar dat heb ik erboven staan omdat hij natuurlijk moet checken welke dag het is voor de tijd..

- SanThe -

24-11-2016 19:05

Zit dat array wel logisch in elkaar?

Jeremy Evers

25-11-2016 09:24

Dit zijn de arrays die ik zelf heb gemaakt:


$dagNamen = array('Mon' => 'Maandag', 'Tue' => 'Dinsdag', 'Wed' => 'Woensdag', 'Thu' => 'Donderdag', 'Fri' => 'Vrijdag', 'Sat' => 'Zaterdag', 'Sun' => 'Zondag');
    
$dagen = array('Zondag', 'Maandag', 'Dinsdag', 'Woensdag', 'Donderdag', 'Vrijdag', 'Zaterdag');

En dit is de array die ik krijg vanuit de database (aangeleverd van de leverancier waardoor ik daar aan vast zit)


0,16:00,00:50;1,16:00,21:20;2,16:00,21:20;4,16:00,00:00;5,16:00,01:20;6,16:00,01:50;3,16:00,00:00

En bovenstaande array explode ik dan als volgt:


$explode = explode(';', $delivery_times);

foreach ($explode as $status) {
   $explode = explode(',', $status);

   if ($dagNamen[date('D')] == $dagen[$explode[0]]) {
       $time = time()."<br/>";
       $open_time = strtotime($explode[1])."<br/>";
       $close_time = strtotime($explode[2])."<br/>";
                       
       if ($close_time < $open_time) {
           $close_time += 86400;
       }
                        
       $diff_open = ($time - $open_time);
       $diff_gesloten = ($time - $close_time);
                       
       if ($diff_open > 0 && $diff_gesloten < 0) {
           echo "<div class='companypage__restaurantopen'>Open</div>";
       } else {
           echo "<div class='companypage__restaurantclosed'>Gesloten</div>";
       }
    }
}

- SanThe -

25-11-2016 10:58

<?php
foreach ($explode as $status) {
$explode = explode(',', $status);
?>
Dit vraagt om problemen.
Je doorloopt $explode en het eerste wat je doet is $explode compleet wijzigen.

Jeremy Evers

25-11-2016 11:09

Maar ik doe dat explode in de foreach omdat ik dan elke keer één dag heb, die heb ik namelijk erboven al ge-explode.

- SanThe -

25-11-2016 11:12

Maar gebruik dan altijd een andere naam.

Jeremy Evers

25-11-2016 11:16

Oke, ik begrijp wat je bedoelt inderdaad, nu gebruik ik de array die erboven in de foreach zit.
Wat kan daarnaast het probleem zijn van de tijd? Want daar zit ik nog wel mee, en ben ten einde raad.

- SanThe -

25-11-2016 11:39

Door deze regel kijk je na middernacht reeds bij de volgende dag.
En dat klopt niet altijd.
<?php
if ($dagNamen[date('D')] == $dagen[$explode[0]]) {
?>

Jeremy Evers

25-11-2016 11:49

Klopt inderdaad, dat is het probleem. Ik haal daar de dag op en de tijd van die dag. Maar soms zijn ze geopend tot 01:00 en dan zegt hij dus om 00:00 dat het een nieuwe dag is en pakt hij de volgende tijd weer van die dag, terwijl hij tot 01:00 de tijd van de vorige dag moet pakken, maar hoe

Ger van Steenderen

25-11-2016 20:19 gewijzigd op 25-11-2016 20:22


<?php
echo date('Y-m-d H:i', strtotime('16:00');
// geeft 1970-01-01 16:00
?>

Daarmee kan je dus nooit de tijd van nu vergelijken.
Zo wel


<?php
function TimeToSec($time)
{
	$parts = explode(':', $time);
	return $parts[0] * 3600 + $parts[2] * 60 + (count($parts) == 3 ? $parts[2] : 0);
}

$midnight = strtotime(date('Y-m-d 00:00:00'));
$open_time = $midnight + TimeToSec($times[1]);
$closed_time = $midnight + TimeToSec($times[2]);
if ($open_time >= $closed_time)
{
	$closed_time += 86400;
}
$now = time();
if ($open_time <= $now && $closed_time >= $now)
{
	echo 'Open';
}
else
{
	echo 'Gesloten';
}
?>

Maar beter kan je de manier waarop je de tijden in de db opslaat aanpassen door hiervoor een aparte tabel aan te maken:
- restaurant_id
- day_id (0[maandag] - 6 [zondag])
- time_open (TIME)
- time_closed (TIME)
Dan met de volgende SQL


SELECT
	r.restaurant_id,
	r.restaurant_name,
	CASE WHEN NOW() BETWEEN o.time_open HOUR_SECOND
		AND o.time_closed + INTERVAL IF(o.time_closed <= o.time_open, 24, 0) HOUR
	THEN 1 ELSE 0 END is_open
FROM
	restaurants r
JOIN
	(SELECT
		restaurant_id,
		CURRENT_DATE + INTERVAL time_open HOUR_SECOND time_open
		CURRENT_DATE + INTERVAL time_closed HOUR_SECOND time_closed
	FROM
		opening_times
	WHERE
		day_id = WEEKDAY(CURRENT_DATE)
	) o USING (restaurant_id)
ORDER BY is_open DESC

Reageren

Inloggen om te reageren