Need mini-code review

[Phil Powell]

$sql = 'DELETE FROM nnet_produkt_be stilling ' .
'WHERE nnet_produkt_pl acement > ' .
'nnet_produkt_p lacement + \'' . date("Y-m-d H:i:s", time() +
$maxCartTime) . '\'';

This is supposed to delete all rows in nnet_produkt_be stilling whose value
in the field nnet_produkt_pl acement is more than $maxCartTime seconds old
($maxCartTime, in this case = 86400 seconds, or 24 hours). I am unsure if
this is the correct syntax using mySQL to do such an action upon a DATETIME
field column datatype, so if someone could look that over and see how far
out of the ballpark I really am I'd appreciate it.

Thanx
Phil

[Andy Jeffries]

Re: Need mini-code review

On Tue, 30 Sep 2003 00:32:31 -0400, Phil Powell wrote:[color=blue]
> $sql = 'DELETE FROM nnet_produkt_be stilling ' .
> 'WHERE nnet_produkt_pl acement > ' .
> 'nnet_produkt_p lacement + \'' . date("Y-m-d H:i:s", time() +
> $maxCartTime) . '\'';[/color]

The easiest way is to use MySQL's built in functions:

$sql = 'DELETE FROM nnet_produkt_be stilling ' .
'WHERE nnet_produkt_pl acement < DATE_SUB(NOW(), INTERVAL 24 HOURS)';


Cheers,


Andy

[Phil Powell]

Re: Need mini-code review

Thanx but don't you mean > DATE_SUB(NOW(), INTERVAL 24 HOURS)?

Phil

"Andy Jeffries" <news@andyjeffr ies.remove.co.u k> wrote in message
news:pan.2003.0 9.30.08.53.39.9 40884@andyjeffr ies.remove.co.u k...[color=blue]
> On Tue, 30 Sep 2003 00:32:31 -0400, Phil Powell wrote:[color=green]
> > $sql = 'DELETE FROM nnet_produkt_be stilling ' .
> > 'WHERE nnet_produkt_pl acement > ' .
> > 'nnet_produkt_p lacement + \'' . date("Y-m-d H:i:s", time() +
> > $maxCartTime) . '\'';[/color]
>
> The easiest way is to use MySQL's built in functions:
>
> $sql = 'DELETE FROM nnet_produkt_be stilling ' .
> 'WHERE nnet_produkt_pl acement < DATE_SUB(NOW(), INTERVAL 24[/color]
HOURS)';[color=blue]
>
>
> Cheers,
>
>
> Andy[/color]

[Andy Jeffries]

Re: Need mini-code review

On Tue, 30 Sep 2003 07:19:36 -0400, Phil Powell wrote:[color=blue]
> Thanx but don't you mean > DATE_SUB(NOW(), INTERVAL 24 HOURS)?[/color]

No, you want to delete all baskets older than a certain time, e.g. 24
hours old.

So, say you have the following table:

+--+-------------------+
|ID|DateCreated |
+--+-------------------+
| 1|2003-09-28 00:00:00|
| 2|2003-09-30 12:31:00|
+--+-------------------+

DATE_SUB(NOW(), INTERVAL 24 HOURS) will equate to:

NOW(): 2003-09-30 12:41:56
SUB : 2003-09-29 12:41:56

As you can see from the table, you want to delete basket ID 1, but not ID
2 (as it's was only created 10 minutes ago).

So the question you want to ask is 2003-09-28 (ID1:DateCreate d) less than or greater than
2003-09-29 (SUB)?

Of course it's less than, therefore your WHILE criteria should be less
than.

If you use greater than you will be constantly deleting baskets that are
less than 24 hours old and keeping all your old ones.

Does that make more sense now or have I rambled on and confused the issue?

Cheers,


Andy

[Phil Powell]

Re: Need mini-code review

My date-math skills are absolutely nonexistent since it's pure logic and I
don't have that, so everything you said made no sense until you used an
example. I still don't get it between "<" and ">" but in this case I'll
just take your word for it.

Unfortunately, it doesn't work :( Produces a mysql_error near 'HOURS)'

I honestly have found nothing on www.mysql.com or anywhere else that can
address this problem. All I simply want to do is to delete rows from a
table where nnet_produkt_re cord_entered is more than 24 hours old. I
thought this would do it but it produces a mysql_error near 'HOURS)':

$sql = 'DELETE FROM nnet_produkt_be stilling ' .
'WHERE nnet_produkt_pl acement < DATE_SUB(NOW(), INTERVAL 24
HOURS)';

Any ideas, sorry I'm fresh out as usual.

Thanx
Phil

"Andy Jeffries" <news@andyjeffr ies.remove.co.u k> wrote in message
news:pan.2003.0 9.30.11.44.51.1 30321@andyjeffr ies.remove.co.u k...[color=blue]
> On Tue, 30 Sep 2003 07:19:36 -0400, Phil Powell wrote:[color=green]
> > Thanx but don't you mean > DATE_SUB(NOW(), INTERVAL 24 HOURS)?[/color]
>
> No, you want to delete all baskets older than a certain time, e.g. 24
> hours old.
>
> So, say you have the following table:
>
> +--+-------------------+
> |ID|DateCreated |
> +--+-------------------+
> | 1|2003-09-28 00:00:00|
> | 2|2003-09-30 12:31:00|
> +--+-------------------+
>
> DATE_SUB(NOW(), INTERVAL 24 HOURS) will equate to:
>
> NOW(): 2003-09-30 12:41:56
> SUB : 2003-09-29 12:41:56
>
> As you can see from the table, you want to delete basket ID 1, but not ID
> 2 (as it's was only created 10 minutes ago).
>
> So the question you want to ask is 2003-09-28 (ID1:DateCreate d) less than[/color]
or greater than[color=blue]
> 2003-09-29 (SUB)?
>
> Of course it's less than, therefore your WHILE criteria should be less
> than.
>
> If you use greater than you will be constantly deleting baskets that are
> less than 24 hours old and keeping all your old ones.
>
> Does that make more sense now or have I rambled on and confused the issue?
>
> Cheers,
>
>
> Andy
>[/color]

[Adam i Agnieszka Gasiorowski FNORD]

Re: Need mini-code review

Phil Powell wrote:
[color=blue]
> My date-math skills are absolutely nonexistent since it's pure logic and I
> don't have that, so everything you said made no sense until you used an
> example. I still don't get it between "<" and ">" but in this case I'll
> just take your word for it.
>
> Unfortunately, it doesn't work :( Produces a mysql_error near 'HOURS)'
>
> I honestly have found nothing on www.mysql.com or anywhere else that can
> address this problem. All I simply want to do is to delete rows from a
> table where nnet_produkt_re cord_entered is more than 24 hours old. I
> thought this would do it but it produces a mysql_error near 'HOURS)':
>
> $sql = 'DELETE FROM nnet_produkt_be stilling ' .
> 'WHERE nnet_produkt_pl acement < DATE_SUB(NOW(), INTERVAL 24
> HOURS)';
>
> Any ideas, sorry I'm fresh out as usual.[/color]

Use 'HOUR' instead of 'HOURS'.

--
Seks, seksiæ, seksolatki... news:pl.soc.sek s.moderowana
http://hyperreal.info { iWanToDie } WiNoNa ) (
http://szatanowskie-ladacznice.0-700.pl foReVeR( * )
Poznaj jej zwiewne kszta³ty... http://www.opera.com 007