[PHP] Tips over code compacter te maken

[Verwijderd]

Ik ben voor mijn website bezig met een statistiekentooltje. Hier is onder andere een overzicht opgenomen van het aantal hits en sessions van vandaag, gisteren, deze -en vorige week, deze -en vorige maand. Ik gebruik hiervoor de volgende functie:

PHP:

function stats_period($timeframe) {
  $sql = 'SELECT date, SUM(hits) AS hits, SUM(sessions) AS visits FROM stats_dt ';
  switch ($timeframe) {
    case 'today':
      $today = date("Y-m-d", mktime(0, 0, 0, date('n'), date('j'), date('Y')));
      $sql.= "WHERE date = '$today'";
    break;
    
    case 'yesterday':
      $yesterday = date("Y-m-d", mktime(0, 0, 0, date('n'), date('j') - 1, date('Y')));
      $sql.= "WHERE date = '$yesterday'";
    break;
  
    case 'this_week':
      $today = date("Y-m-d", mktime(0, 0, 0, date('n'), date('j'), date('Y')));
      $week_start = date("Y-m-d", mktime(0, 0, 0, date('n'), date('j') - date('w'), date('Y')));
      $sql.= "WHERE date BETWEEN '$week_start' AND '$today'";
    break;
    
    case 'prev_week':
      $week_start = date("Y-m-d", mktime(0, 0, 0, date('n'), date('j') - (date('w') + 7), date('Y')));
      $week_end = date("Y-m-d", mktime(0, 0, 0, date('n'), date('j') - (date('w') + 1), date('Y')));
      $sql.= "WHERE date BETWEEN '$week_start' AND '$week_end'";
    break;
    
    case 'this_month':
      $today = date("Y-m-d", mktime(0, 0, 0, date('n'), date('j'), date('Y')));
      $month_start = date("Y-m-d", mktime(0, 0, 0, date('n'), 1, date('Y')));
      $sql.= "WHERE date BETWEEN '$month_start' AND '$today'";
    break;
    
    case 'prev_month':
      $month_start = date("Y-m-d", mktime(0, 0, 0, date('n') - 1, 1, date('Y')));
      $month_end = date("Y-m-d", mktime(0, 0, 0, date('n'), 0, date('Y')));
      $sql.= "WHERE date BETWEEN '$month_start' AND '$month_end'";
    break;
  }
  $sql.= ' GROUP BY date DESC';
  return db_fetch_object(db_query($sql));
}

Ik heb alleen erg het gevoel dat dit veel korter / efficienter kan, al zie ik het niet. Wat vinden jullie ervan? Wie heeft tips hoe dit korter op te schrijven?

[Cartman!]

http://nl2.php.net/strtotime misschien een idee?

Overigens is korter schrijven van code niet per definitie beter imo, het komt de leesbaarheid niet altijd ten goede. Als je er na een jaar weer naar kijkt snap je er misschien geen hout meer van en probeer dan die bug maar eens te vinden

[? ?]

code:

stats_period($date_from, $date_till)

een functie moet al die opties niet hebben, gewoon van een datum tot een datum

[ Voor 59% gewijzigd door ? ? op 25-10-2007 14:09 ]

[Kalentum]

je zou je database kunnen gebruiken om de intervallen te bepalen. Dus geen PHP meer maar de database-specifieke functies. BV van Mysql

[Verwijderd]

Wat ik meestal doe is als iets meer dan 2 keer voorkomt in een stuk code, kun je er beter een functie van maken. In jouw geval kun je dus voor het opbouwen van die where clause een functie maken waar je wat parameters aan mee geeft.

[Japius]

Als je Date veld van het type datetime is, kan je ook werken met de date_sub:

code:

WHERE date > date_sub(current_date(), interval 1 day)

[? ?]

delete

[ Voor 95% gewijzigd door ? ? op 25-10-2007 14:51 ]

[Erkens]

era.zer schreef op donderdag 25 oktober 2007 @ 14:49:
Zo 'ingewikkeld' allemaal?
Gewoon als INT in mysql. time() geeft je de huidige int waarde en met date kun je de int omzetten naar een menselijke datum

en een INT is zo makkelijk te vergelijken? Gewoon startdatum & stopdatum >= en <=

Je kan beter een timestamp type gebruiken in mysql voor tijden (tenzij je een groter bereik nodig hebt, dan moet je datetime gebruiken). Vergelijken etc kan je dan gewoon doen met de mysql functies wat gewoon veel handiger is.

[Verwijderd]

De statistieken van de website worden elke dag (en elk uur van de dag) samengevat in stats_dt:

code:

+------------+------+----------+------+
| date       | hour | sessions | hits |
+------------+------+----------+------+
| 2007-10-24 |   12 | 4        | 53   |
+------------+------+----------+------+
| 2007-10-24 |   13 | 8        | 65   |
+------------+------+----------+------+
| 2007-10-25 |    7 | 2        | 17   |
+------------+------+----------+------+

Het was een goed idee om de MySQL functies te gebruiken voor deze datum / tijd calculaties, ipv eerst te gaan rekenen in PHP Ik probeer eea nu om te zetten:

PHP:

// vandaag
// date("Y-m-d", mktime(0, 0, 0, date('n'), date('j'), date('Y')))
$sql.= "WHERE date = CURDATE()";

// gisteren
//date("Y-m-d", mktime(0, 0, 0, date('n'), date('j') - 1, date('Y')))
$sql.= "WHERE date = DATE_SUB(CURDATE(), INTERVAL 1 DAY)";

// deze week
// $today = date("Y-m-d", mktime(0, 0, 0, date('n'), date('j'), date('Y')));
// $week_start = date("D Y-m-d", mktime(0, 0, 0, date('n'), date('j') - date('w'), date('Y'))); 
$sql.= "WHERE YEARWEEK(date) = YEARWEEK(CURDATE())";

Maar ik loop vast bij gegevens ophalen van de afgelopen week. Ik gebruikte:

PHP:

$week_start = date("Y-m-d", mktime(0, 0, 0, date('n'), date('j') - (date('w') + 7), date('Y')));
$week_end = date("Y-m-d", mktime(0, 0, 0, date('n'), date('j') - (date('w') + 1), date('Y')));
$sql.= "WHERE date BETWEEN '$week_start' AND '$week_end'";

Nu zou ik voor de afgelopen week YEARWEEK(CURDATE()) - 1 kunnen pakken, maar dan kom ik in problemen in de eerste week van het jaar. Stel dat YEARWEEK(CURDATE()), 200801 oplevert. Wat gebeurt er dan als ik er daar 1 vanaf haal? Zet MySQL dat dan om in 200752?

ik weet trouwens dat "date" een gereserveerd woord in MySQL is, en daarom niet in je tabel als kolomnaam gebruikt mag worden...moet dit nog aanpassen, maar voorlopig werkt het wel

[Erkens]

Verwijderd schreef op donderdag 25 oktober 2007 @ 15:21:
ik weet trouwens dat "date" een gereserveerd woord in MySQL is, en daarom niet in je tabel als kolomnaam gebruikt mag worden...moet dit nog aanpassen, maar voorlopig werkt het wel

Dan zet je die naam tussen `backticks` wat je eigenlijk altijd om je kolomnamen moet zetten...

[Grijze Vos]

Erkens schreef op donderdag 25 oktober 2007 @ 14:52:
[...]

Je kan beter een timestamp type gebruiken in mysql voor tijden (tenzij je een groter bereik nodig hebt, dan moet je datetime gebruiken).

Euhm, volgens mij niet. Timestamps hebben side effects in MySQL, die wil je niet altijd.

[ Voor 11% gewijzigd door Grijze Vos op 25-10-2007 15:31 ]

[Erkens]

Grijze Vos schreef op donderdag 25 oktober 2007 @ 15:30:
[...]

Euhm, volgens mij niet. Timestamps hebben side effects in MySQL, die wil je niet altijd.

over welke side effects hebben we het dan?

[Grijze Vos]

Een timestamp (precies een, overigens), wordt geupdate naar now() als je row wordt geupdate.

Last time I checked iig.

[update]
Hmz, zo te zien is dat sinds versie 5.x anders.

[ Voor 20% gewijzigd door Grijze Vos op 25-10-2007 15:41 ]

[_Gekkie_]

Ik zelf hanteer vaak een goede structuur voor het inkorten van grotere switches zodat je cases samen kunt nemen of variabelen maar 1 keer hoeft te definieren...

Mijn persoonlijke stelregel is eigenlijk: als je code ergens dubbel ziet moet het intelligenter kunnen
(DRY technieken)

offtopic:
Even een korte side-track vraag: Hoe zit het het met queries die op een datetime-kolom zitten in een nogal grote database? Kun je deze indexen om zodanig snelheidswinst te halen uit je queries?

Overigens: sideeffect van timestamps in MySQL: het is niet hetzelfde als de UNIX-timestamp notatie dus dan zit je weer met FROM_UNIXTIME-functies voordat je ermee in PHP kunt werken...

Het automatisch updaten kun je in v5 van MySQL wel aan/uit zetten, dus dat is dan wel weer handig... (IMHO)

[ Voor 20% gewijzigd door _Gekkie_ op 25-10-2007 16:47 ]

[Japius]

_Gekkie_ schreef op donderdag 25 oktober 2007 @ 16:43:

offtopic:
Even een korte side-track vraag: Hoe zit het het met queries die op een datetime-kolom zitten in een nogal grote database? Kun je deze indexen om zodanig snelheidswinst te halen uit je queries?

Ja, je kan ook gewoon een index op een datetime zetten. Kost natuurlijk wel performance, bij elke insert. Dat moet je afwegen tegen het voordeel dat de index oplevert: als er slecht eenmaal per dag een cron draait die stats genereert, kan het zijn dat je die index achterwege wilt laten.

[Verwijderd]

Mooie discussie, maar kan iemand ook even ingaan op mijn laatste vraag ? Ik zit nog steeds met hetzelfde probleem namelijk...

[Japius]

code:

mysql> select yearweek(curdate());
+---------------------+
| yearweek(curdate()) |
+---------------------+
|              200742 | 
+---------------------+
1 row in set (0.00 sec)

mysql> select yearweek(curdate())-43;
+------------------------+
| yearweek(curdate())-43 |
+------------------------+
|                 200699 | 
+------------------------+
1 row in set (0.00 sec)

Gaat niet helemaal lekker dus..


Edit: daarmee was je vraag beantwoord. Je kan natuurlijk wel

code:

date_sub(current_date(), interval 1 week)

doen.

[ Voor 13% gewijzigd door Japius op 25-10-2007 18:00 ]

[FragFrog]

Grijze Vos schreef op donderdag 25 oktober 2007 @ 15:39:
Een timestamp (precies een, overigens), wordt geupdate naar now() als je row wordt geupdate.

Last time I checked iig.

[update]
Hmz, zo te zien is dat sinds versie 5.x anders.

Onzin, een timestamp kun je een 'ON UPDATE CUR_TIME' flag meegeven, dan wordt'ie elke keer geupdate naar de huidige tijd als je de rij aanpast. Als je een fatsoenlijke SQL admin tool (zoals Navicat of MySQL's Query Browser) hebt kun je dat gewoon zien en uitzetten Niet dat ik het gebruik van timestamp fields wil promoten trouwens, een date of datetime field is veelal beter.

[Erkens]

FragFrog schreef op donderdag 25 oktober 2007 @ 19:08:
[...]

Onzin, een timestamp kun je een 'ON UPDATE CUR_TIME' flag meegeven, dan wordt'ie elke keer geupdate naar de huidige tijd als je de rij aanpast. Als je een fatsoenlijke SQL admin tool (zoals Navicat of MySQL's Query Browser) hebt kun je dat gewoon zien en uitzetten Niet dat ik het gebruik van timestamp fields wil promoten trouwens, een date of datetime field is veelal beter.

In de oudere MySQL versies had je die optie niet en werd de eerste timestamp kolom automatisch ge-update zodra je een update op die row deed en niet expliciet een value in die kolom zette.

[Verwijderd]

Japius schreef op donderdag 25 oktober 2007 @ 17:55:
Edit: daarmee was je vraag beantwoord. Je kan natuurlijk wel

code:

date_sub(current_date(), interval 1 week)

doen.

@Japius: bedankt voor je reaktie, maar met jouw suggestie krijg ik de resultaten van de afgelopen 7 dagen, niet van de vorige week.

Een voorbeeld: vandaag is het 25 oktober. Als ik de resultaten van de vorige week wil, zoek ik naar de resultaten tussen maandag 15 en zondag 21 oktober (aangenomen dat een week op maandag begint). De code hierboven zal me de resultaten geven vanaf 18 oktober (7 dagen teruggeteld van 25 oktober).

Anders gezegd, ik zoek het SQL equivalent van:

PHP:

$prev_week_start = date("Y-m-d", mktime(0, 0, 0, date('n'), date('j') - (date('w') + 7), date('Y')));
$prev_week_end = date("Y-m-d", mktime(0, 0, 0, date('n'), date('j') - (date('w') + 1), date('Y')));

Heeft iemand een suggestie?

[ Voor 16% gewijzigd door Verwijderd op 25-10-2007 19:27 ]

[Erkens]

Verwijderd schreef op donderdag 25 oktober 2007 @ 19:21:
Een voorbeeld: vandaag is het 25 oktober. Als ik de resultaten van de vorige week wil, zoek ik naar de resultaten tussen maandag 15 en zondag 21 oktober (aangenomen dat een week op maandag begint). De code hierboven zal me de resultaten geven vanaf 18 oktober (7 dagen teruggeteld van 25 oktober).

Heeft iemand een suggestie?

gebruik de mysql functie "WEEK( )"

SQL:

SELECT * FROM tabel WHERE WEEK(`datefield`)=(WEEK(NOW())-1)

[Verwijderd]

Erkens schreef op donderdag 25 oktober 2007 @ 19:32:
[...]
gebruik de mysql functie "WEEK( )"

SQL:

SELECT * FROM tabel WHERE WEEK(`datefield`)=(WEEK(NOW())-1)

Dit gaat fout als de datum bijvoorbeeld 1 januari 2007 is:

code:

mysql > SELECT WEEK( '2007-01-01' ) -1
mysql > -1

Bij date_sub gaat dat wel goed:

code:

mysql > SELECT DATE_SUB('2007-01-01', INTERVAL 1 DAY)
mysql > 2006-12-31

Suggesties?

[Raynman]

Wil je niet gewoon dit:

SQL:

... WHERE WEEK(`date`) = WEEK(DATE_SUB(CURDATE(), INTERVAL 1 WEEK))

edit: wel liever YEARWEEK() dan waarschijnlijk

[ Voor 18% gewijzigd door Raynman op 25-10-2007 21:51 ]

[Erkens]

Overigens bedenk ik me nu dat je niet alleen het weeknummer goed wilt hebben, maar ook het jaartal, maar ook dat gaat wat minder goed met een jaarwisseling. Kan je niet gewoon BETWEEN gebruiken?

[Megamind]

PHP:

<?php 
$prev_week_start = strtotime("7 days ago");
$prev_week_end = strtotime("Yesterday");
?> 

kan ook Maar ik zag pagina 2 niet

[ Voor 15% gewijzigd door Megamind op 25-10-2007 21:24 ]

[FragFrog]

Erkens schreef op donderdag 25 oktober 2007 @ 19:14:
[...]

In de oudere MySQL versies had je die optie niet en werd de eerste timestamp kolom automatisch ge-update zodra je een update op die row deed en niet expliciet een value in die kolom zette.

I stand corrected

The following items summarize TIMESTAMP initialization and updating properties prior to MySQL 4.1.2:

The first TIMESTAMP column in table row automatically is set to the current timestamp when the record is created if the column is set to NULL or is not specified at all.

The first TIMESTAMP column in table row automatically is updated to the current timestamp when the value of any other column in the row is changed, unless the TIMESTAMP column explicitly is assigned a value other than NULL.

If a DEFAULT value is specified for the first TIMESTAMP column when the table is created, it is silently ignored.

Other TIMESTAMP columns in the table can be set to the current TIMESTAMP by assigning NULL to them, but they do not update automatically.

Beginning with MySQL 4.1.2, you have more flexible control over when automatic TIMESTAMP initialization and updating occur and which column should have those behaviors:

For one TIMESTAMP column in a table, you can assign the current timestamp as the default value and the auto-update value. It is possible to have the current timestamp be the default value for initializing the column, for the auto-update value, or both. It is not possible to have the current timestamp be the default value for one column and the auto-update value for another column.

Any single TIMESTAMP column in a table can be used as the one that is initialized to the current date and time, or updated automatically. This need not be the first TIMESTAMP column.

Maargoed, dan hebben we het wel over MySQL voor 4.1.2, da's een beetje hetzelfde als nu nog PHP3 gebruiken