Sprawdzanie wpisu - porównywanie - usuwanie Opcje

[majesgrudziadz]

Borykam się z takim problemem i nie wiem - nie mogę sobie przypomnieć.
Może ktoś z was mi przypomni jak tego dokonać.

Mianowicie jest tabela zawierająca następujące wpisy:

id_aut
email
kod_autoryzacji
data_wygasniecia ---> zapisana w formacie Y.m.d - H:i:s czyli 2012.02.02 - 18:02:45


mój kod wygląda tak w zasadzie jego początek:

[PHP] pobierz, plaintext 
 
<?
 
require 'config.php';
 
// Tworzy aktualną datę w formacie RRRR.MM.DD - GG.MM
	$data_aktualna = date('Y.m.d - H:i:s');
 
// Sprawdza czy istnieje autoryzacja która miała ważność do daty aktualnej
	$data_sprawdzona = mysql_query("SELECT * FROM `autoryzacja` WHERE `data_wygasniecia`='$data_aktualna'");
	$data_sprawdzona = mysql_fetch_array($data_sprawdzona);
 
// Jeżeli jest taka autoryzacja to ją usuwa.
	if ($data_sprawdzona['data_wygasniecia'] <= $data_aktualna) {
	echo "Jest autoryzacja do usunięcia";
	}else{
	echo "Nie ma autoryzacji do usunięcia";
	}
?>
 
[PHP] pobierz, plaintext 

Nie ważne jaka jest ta data to zawsze pokazuje ze jest autoryzacja do usuniecia

Moje pytanie: Jak sformułować zapytanie if aby brał pod uwagę że data wygaśnięcia jest mniejsza lub równa od daty aktualnej ? bo w ten sposób coś mi nie działa

Ten post edytował majesgrudziadz 2.02.2012, 18:37:34

[viking]

Wykorzystaj moc bazy. Zobacz manual NOW() zamiast bawić się w $data_aktualna. Potrzebujesz tego Selecta? Bo możesz wykorzystać

[SQL] pobierz, plaintext 
DELETE FROM autoryzacje WHERE data_wygasniecia<=NOW();
[SQL] pobierz, plaintext 

Ten post edytował viking 2.02.2012, 18:15:25

[majesgrudziadz]

Kolego viking a jak było by lepiej ?

twój sposób czy na wykorzystanie mocy bazy czy mój sposób który właśnie poprawiłem i powinien działać ... jeszcze to sprawdzę

[PHP] pobierz, plaintext 
<?
 
require 'config.php';
 
// Pobiera aktualną datę w formacie RRRR.MM.DD - GG.MM
	$data_aktualna = date('Y.m.d - H:i:s');
 
// Pobiera daty aktywacji i wyswietla je
	$data_sprawdzona = mysql_query("SELECT * FROM `autoryzacja`");
 
	while($data_sprawdzona = mysql_fetch_array($data_sprawdzona)) {
 
		if ($data_sprawdzona['data_wygasniecia'] <= $data_aktualna) {
		//Tutaj usuwamy jeżeli data wygaśnięcia jest starsza niż data aktualna	
		}
 
	}
 
?>
[PHP] pobierz, plaintext 

Bo z tego co mi się wydaje to NOW() zapisze date w formacie RRRR-MM-DD GG:MM:SS a ja mam format daty RRRR.MM/DD - GG:MM:SS

Ten post edytował majesgrudziadz 2.02.2012, 18:41:34

[viking]

Twoje pole w bazie musi być typu daty więc nie ma możliwości żeby nie porównało. Chyba że jakimś cudem jako string wcisnąłeś. To co robisz jest bezsensowne. Obciążasz bazę niepotrzebnym selectem, obciążasz serwer parsowaniem by w końcu zrobić to, co ci podałem.

[majesgrudziadz]

Moja wersja sprawdzona i działa w 100%

[PHP] pobierz, plaintext 
<?
 
require 'config.php';
 
// Pobiera aktualną datę w formacie RRRR.MM.DD - GG.MM
	$data_aktualna = date('Y.m.d - H:i:s');
 
// Pobiera daty aktywacji i wyswietla je
	$data_sprawdzona = mysql_query("SELECT * FROM `autoryzacja`");
 
	while($data_sprawdzona = mysql_fetch_array($data_sprawdzona)) {
		echo "Aktualna data ".$data_aktualna." jest większa od dat:<br><br>";
		if ($data_sprawdzona['data_wygasniecia'] <= $data_aktualna) {
		//Wyświetla daty mniejsze bądź równe dacie aktualnej
		echo $data_sprawdzona['data_wygasniecia'];
		}
 
	}
 
?>
[PHP] pobierz, plaintext 

Co do obciążania to zadaje pytanie select czyli twoja wersja również zada pytanie select
Moja wersja usuwa wpisy jeżeli tak samo jak twoja

Więc nie widzę różnicy w obciążalności serwera jednakże z ciekawości wypróbuję twoją wersję gdyż jak już pisałem w poprzednim poście ... dawno nie pisałem scryptów, dużo się pozmieniało, więc nie dość, że muszę sobie wszystko przypomnieć to i pouczyć się nowych możliwości więc każda rada i propozycja jest dla mnie przydana więc sprawdzę to


Hmm jednak moja wersja nie działa tak do końca ... wyświetla tylko 1 zapytanie reszty nie chce ... może ktoś podpowie dlaczego ?

Ten post edytował majesgrudziadz 2.02.2012, 19:14:58

[viking]

A gdzie ja robię selecta? Jak będziesz miał milion rekordów to wszystkie wybierzesz, po czym je ziterujesz w PHP żeby kilka skasować? Naprawdę zrób jak Ci mówię bo nie potrzebujesz inaczej.

[majesgrudziadz]

Zrobiłem po swojemu ... takich rekordów w bazie to będzie koło 2 na tydzień jak dobrze pójdzie...

Jak próbowałem zrobić tak jak pokazałeś to nie generuje mi daty :/

Btw w późniejszej fazie kodu wyciągam jeszcze z tabeli adres email i wg niego usuwam usera który nie aktywował konta w przeciągu 24h więc już totalnie nie wiem jak zrobić to po twojemu bo usunie wszystkie wpisy ... a jak mam wtedy usunąć userów ?

Dlatego zostałem przy mojej wersji.

[viking]

A pokaż mi jak wiążesz z tabelą użytkowników? Trochę schemat rozjaśnij.