Wygaszanie rekordów Opcje

[kajzur]

Witam,
napisałem skrypt, który docelowo będzie wykonywany przez CRON'a. Ma on za zadanie wygaszanie rekordów w bazie (tj. zmiana statusu na deactive). Wygasznie ma zajść w 2 przypadkach:

• Gdy ilość będzie równa 0
• Gdy czas z bazy danych + 10 dni będzie dalej (przejdzie lub będzie równy) od aktualnej daty.

Skrypt wygląda tak, jednak że, nie wygasza aukcji których data przeszła:

[PHP] pobierz, plaintext 
include("pdo.php");
$kt = time();
$dane = $pdo -> query("select `date` from `goods` where `status` = 'activ'");
$zmiana = 0;
while($data = $dane->fetch(PDO::FETCH_NUM))
{
 
$zmiana = 0;
		if(strtotime($data[0]) + (86400*10)  >= $kt )
 
		{
		$d = $data[0];
		$zmiana++;
		$zmiana = $pdo->exec("Update `goods` SET `status`='deactive' WHERE `date`='$d'");
 
 
		}
 
}
[PHP] pobierz, plaintext 

Proszę o pomoc.

PS. Jest jakiś inny sensowny sposób poza CRON'em, a by tego dokonać?

[tjaden]

zamień exec na query

[kajzur]

Nic to nie zmieniło..

[vokiel]

Możesz ewentualnie przy pierwszym wejściu na stronę danego dnia. Jednak cron np o 01:15 będzie bardzo dobrym rozwiązaniem.

Pomijając to, ciekawi mnie po co pobierasz te dane, sprawdzasz i następnie dla każdego oddzielnie robisz update. Nie lepiej rozwiązać to w jednym zapytaniu?

btw: czy goods to ilość?

[SQL] pobierz, plaintext 
UPDATE `goods` SET `status`='deactive' WHERE DATE_ADD(`date` INTERVAL 10 DAY)>=CURDATE() OR 'goods'=0
[SQL] pobierz, plaintext 

Lub

[SQL] pobierz, plaintext 
UPDATE `goods` SET `status`='deactive' WHERE DATEDIFF(`date`,CURDATE())>=10 OR 'goods'=0
[SQL] pobierz, plaintext 

[kajzur]

Oby dwa zapytania nie działają. Data w bazie jest datą dodanie rekordu, ważność rekordu to 10 dni. Rekord na którym robimy testy, został dodany 2009-08-02 00:26:13, więc kończy się 2009-08-12 0:26:13, dziś mamy 13, więc skrypt powinien zmienić status na deactive, jednak tego nie robi ;>

[Fifi209]

Oby dwa zapytania nie działają. Data w bazie jest datą dodanie rekordu, ważność rekordu to 10 dni. Rekord na którym robimy testy, został dodany 2009-08-02 00:26:13, więc kończy się 2009-08-12 0:26:13, dziś mamy 13, więc skrypt powinien zmienić status na deactive, jednak tego nie robi ;>

Może powiesz w jakim formacie przetrzymujesz ów datę w bazie danych? Bo to jest dość istotne.

[wookieb]

Nie napiszesz człowiekowi jak krowie na miedzy TO NI W ZĄB sam nie pomyśli.

w Pierwszym zapytaniu powinno być <= a nie >= i ten goods=0 wywal narazie.

P.s. Nie powiedziałeś o co ci chodzi w twoim pierwszym punkcie w słowie "ilosc"

[kajzur]

(IMG:http://image.mmazur.info/files/151592.jpg)

[vokiel]

Najpierw sprawdź co zwracają Ci dane zapytania:

[SQL] pobierz, plaintext 
SELECT * FROM `goods` WHERE DATE_ADD(`date` INTERVAL 10 DAY)<=CURDATE() OR 'ilosc'=0;
SELECT * FROM `goods` WHERE DATEDIFF(CURDATE(),`date`)>=10 OR 'ilosc'=0;
 
SELECT * FROM `goods` WHERE DATE_ADD(`date` INTERVAL 10 DAY)<=CURDATE();
SELECT * FROM `goods` WHERE DATEDIFF(CURDATE(),`date`)>=10;
[SQL] pobierz, plaintext 

[kajzur]

vokiel,
1 zapytanie - zwraca wszytkie rekordy.
2 zapytanie - wszystkie rekordy.
3 zapytanie - wydaje się że zwrócił poprawne rekordy.
4 zapytanie - również wydaje się że poprawne rekordy zwrócił.

wookieb, ilosc to pole w bazie danych, które jeśli posiada wartość 0, równiez kwalifikuje się do wygaszenia.

[vokiel]

Możesz pójść na łatwiznę i zrobić 2 zapytania:

[SQL] pobierz, plaintext 
UPDATE `goods` SET `status`='deactive' WHERE DATEDIFF(CURDATE(),`date`)>=10;
UPDATE `goods` SET `status`='deactive' WHERE 'ilosc'=0;
[SQL] pobierz, plaintext 

Lub pokusić się o zrobienie joina tabeli z nią samą(IMG:style_emoticons/default/smile.gif)

[kajzur]

Witam, nadal nie działa to, popatrzmy, mam tak spreparowaną baze:
(IMG:http://image.mmazur.info/files/799442.jpg)

Po wykonaniu się tego skrypt, wyłączone powinny być rekordy:
1 - ze względu na ilość
6 - ze względu na datę
natomiast 4 powinien zostać aktywny, lecz skrypt wyłącza wszystkie.


(IMG:style_emoticons/default/smile.gif)

Ten post edytował kajzur 18.08.2009, 17:51:25

[vokiel]

Odświeżasz tą wypowiedź i odświeżasz;)

[SQL] pobierz, plaintext 
INSERT  INTO `goods` (`date`,`ilosc`,`status`,`id`) VALUES 
('2009-08-12 12:28:20',0,'activ',1),
('2009-08-12 12:28:20',0,'activ',2),
('2009-06-22 21:23:49',5,'deactive',3),
('2009-06-22 21:25:30',4,'deactive',4),
('2009-08-16 00:18:57',13,'activ',5),
('2009-08-19 15:59:54',20,'deactive',6),
('2009-07-21 22:36:39',1,'activ',7),
('2009-08-05 00:26:13',1,'deactive',8);
[SQL] pobierz, plaintext 

(IMG:http://img43.imageshack.us/img43/6643/tabelac.png)

[SQL] pobierz, plaintext 
SELECT * FROM `goods` WHERE DATEDIFF(CURDATE(),`date`)>=10 OR `ilosc`=0;
[SQL] pobierz, plaintext 

(IMG:http://img26.imageshack.us/img26/9848/wynikid.png)

[SQL] pobierz, plaintext 
UPDATE `goods` SET `status`='deactive' WHERE DATEDIFF(CURDATE(),`date`)>=10 OR `ilosc`=0;
[SQL] pobierz, plaintext 

(IMG:http://img22.imageshack.us/img22/3905/updatev.png)

No nie wiem, ale wg mnie wszystko działa prawidłowo...