[PHP] System prywatnych wiadomości - Masz () nową wiadomość Opcje

[Kerth]

Witajcie,
otóż tak jak w temacie. Jak mam zrobić, żeby pojawiało mi się Masz () nową wiadomość i w miejsce nawiasów wpisało liczbę (sumę - COUNT) wiadomości(czyli liczbę pobranych wartości[zer] z bazy danych). Mam pole `przeczytane_pw` i jak jest 0 to jest nieprzeczytane jak 1 to przeczytane. Czy ja to dobrze robię, czy może problem jest gdzieś indziej? Bo nie pobiera mi nic i ciągle wykonuje się tylko else.

[PHP] pobierz, plaintext 
$pobierz_czy_read = $db->prepare("SELECT COUNT(`przeczytane_pw`) FROM `wiadomosci_pw` WHERE `wiadomosc_do` = :nick_odbiorcy");
		$pobierz_czy_read->bindValue(":nick_odbiorcy", $_SESSION['nick'], PDO::PARAM_STR);
 
		if(!$pobierz_czy_read->execute()){
			throw new Exception("Błąd zapytania");
			}else{
			$data_read_pw = $pobierz_czy_read->fetch();
			$wynik2_read_pw = $data_read_pw['przeczytane_pw'];
 
			if($wynik2_read_pw == 1){
				echo '<span class = "tab"><span class="gray">Nie masz wiadomości</span></span>';
				}else{
				echo '<span class = "tab"><span class="lightgreen">Masz () <span class = "red">nieprzeczytaną</span> wiadomość!</span></span>';
			}
		}
[PHP] pobierz, plaintext 

Jakieś rady?

[tzm]

No to patrz... prosta sprawa. (sorki nie dziala mi Y)

Kazda nowa wiadomosc niech updateuje pole message_for_read +1 jesli jest to wiadomosc nie przecztana. Nie jest to doskonale rozwiazanie bo moglbs sie pokusic o serializacje tablic z id message w celu ustalenia jeszcze ktora wiadomosc jest przecztana a ktora nie.
Kazde odcztanie wiadomosci niech Ci updatuje pole message_for_read -1 jesli ja ktos przecztal.
Tm sposobem nie musisz robic count['row'] == 1 // nieprzecztana tlko masz wszstko w jednej tablic z baz tabeli user.
Mozesz sobie po ajaxie ustawic automatczne odswiezanie co iles tam milisekund tlko to jedno pole w bazie i jesli message_for_read > 0 to zmieniasz wartosc z Nie masz nowch... na Masz message_for_read(bo int) nowch wiadomosci.
Nie rob echo w funkcjach - korzstaj z return.

Tak to rozwiazalem jak przepiswalem facebooka kieds i zdawalo egzamin. Wdajnosciowo tez wdaje sie uzasadnione bo zamiast pobierac X wpisow wiadomosci co jedna sekunde - pobierasz jeden i jesli spelnia dan warunek to dopiero podejmujesz akcje odcztania ich. Tle.

[adrianpl20]

w zapytaniu po

[SQL] pobierz, plaintext 
COUNT(`przeczytane_pw`)
[SQL] pobierz, plaintext 

dodaj

[SQL] pobierz, plaintext 
AS wynik
[SQL] pobierz, plaintext 

czyli

[SQL] pobierz, plaintext 
COUNT(`przeczytane_pw`) AS wynik
[SQL] pobierz, plaintext 

i w tablicy jako nazwe pola daj wynik

[aras785]

Cześć.

Kolega wyżej już odpowiedział na pytanie odnośnie zliczanie, a ja mam małą sugestie:
Jeśli możesz to sprawdzaj wiadomości dla użytkownika na podstawie jego ID, a nie nicku.

Pozdrawiam

[Turson]

Błąd leży gdzie indziej, w logice. Mianowicie, jeżeli kolumna `przeczytane_pw` przyjmuje wartość 0 lub 1, to

[PHP] pobierz, plaintext 
	if($wynik2_read_pw == 1){
[PHP] pobierz, plaintext 

nie ma sensu

Masz policzyć ilość rekordów, które są nieprzeczytane, gdzie przeczytane_pw = 0

[PHP] pobierz, plaintext 
$pobierz_czy_read = $db->prepare("SELECT COUNT(`przeczytane_pw`) AS ilosc FROM `wiadomosci_pw` WHERE `wiadomosc_do` = :nick_odbiorcy AND przeczytane_pw = 0");
$pobierz_czy_read->bindValue(":nick_odbiorcy", $_SESSION['nick'], PDO::PARAM_STR);
 
if(!$pobierz_czy_read->execute()){
	throw new Exception("Błąd zapytania");
}
else{
	$data_read_pw = $pobierz_czy_read->fetch();
	$wynik2_read_pw = $data_read_pw['ilosc'];
 
	if($wynik2_read_pw==0){
		echo '<span class = "tab"><span class="gray">Nie masz wiadomości</span></span>';
		}
		else{
			echo '<span class = "tab"><span class="lightgreen">Masz ('.$wynik2_read_pw.') <span class = "red">nieprzeczytaną</span> wiadomość!</span></span>';
		}
}
[PHP] pobierz, plaintext 

[aras785]

-- Jednak masz rację

Ten post edytował aras785 6.07.2014, 12:59:42

[Kerth]

Rzeczywiście działa. Dzięki za pomoc.

Podepnę się jeszcze raz do tego tematu. Mam zrobione wszystko z systemu prywatnych wiadomości oprócz linku z wiadomością. Jak taki link zrobić? Obecnie mam tak, że gdy kliknie się w odebrane:
http://iv.pl/images/82667606302760991926.png

są to po prostu tylko wiadomości pobierane zapytaniem z bazy danych. Ale jak zrobić link, że jak się w niego kliknie to ustawi na odczytane? Zapytanie wiem jak zrobić tylko z tym linkiem nie bardzo.

[Turson]

index.php?action=message&setRead=id_wiadomości
Link jest dowolny

[Kerth]

Ale nie chodzi mi o link w przeglądarce tylko o wyświetlenie zamiast gotowych wiadomości sformatowany <a href... </a> np.
<a href = ""> Tu temat wiadomości </a>

[Turson]

Nie rozumiem problemu

[Kerth]

U mnie wiadomości wyświetlają się tak:
http://iv.pl/images/82667606302760991926.png
a chcę, żeby zamiast wyświetlania od razu wszystkich wiadomości w słupku pojawił się link przenoszący do tej wiadomości.
Tak jak na forum: http://iv.pl/images/28556360714258001314.png

Ten post edytował Kerth 6.07.2014, 14:34:07

[Wazniak96]

Nom to tworzysz kolejny plik, który będzie pobierał z tablicy GET id wiadomości i ją wyświetlał. A wtedy tylko dajesz odnośnik do example.com/odczytaj.php?id=1

Oczywiście nie zapomnij zweryfikować w tym pliku czy osoba czytając wiadomość jest jej odbiorcą

[Kerth]

Nie wiem jak sformułować zapytanie, które będzie pobierało GET`em dane z bazy. Kombinuję tak, ale nie działa:

[PHP] pobierz, plaintext 
$id = $_GET['id'];
$pobierz_pw = $db->prepare("SELECT `wiadomosc_od`, `temat_pw`, `wiadomosc_pw`, `data_pw` FROM `wiadomosci_pw` WHERE `id` = :id");
$pobierz_pw->bindValue(":nick_odbiorcy", $_SESSION['nick'], PDO::PARAM_STR);
$pobierz_pw->bindValue(":id", $id, PDO::PARAM_INT);
$pobierz_pw -> closeCursor();
if(!$pobierz_pw->execute()){
throw new Exception("Wystąpił błąd podczas pobierania prywatnych wiadomości. Skontaktuj się z programistą aplikacji!");
} 
[PHP] pobierz, plaintext 

Mam błędy typu:

[PHP] pobierz, plaintext 
Warning: PDOStatement::execute(): SQLSTATE[HY093]: Invalid parameter number: number of bound variables does not match number of tokens in C:\WebServ\httpd\strona4\get_pw_message.php on line 106
 
Fatal error: Uncaught exception 'Exception' with message 'Wystąpił błąd podczas pobierania prywatnych wiadomości. Skontaktuj się z programistą aplikacji!' in C:\WebServ\httpd\strona4\get_pw_message.php:107 Stack trace: #0 {main} thrown in C:\WebServ\httpd\strona4\get_pw_message.php on line 107
[PHP] pobierz, plaintext 

Co robię nie tak?

[Pyton_000]

Chcesz bindować 2 elementy a w zapytaniu masz 1

[Kerth]

No racja ale jak usunę :nick_odbiorcy i tak zostaje mi:

[PHP] pobierz, plaintext 
Fatal error: Uncaught exception 'Exception' with message 'Wystąpił błąd podczas pobierania prywatnych wiadomości. Skontaktuj się z programistą aplikacji!' in C:\WebServ\httpd\strona4\get_pw_message.php:107 Stack trace: #0 {main} thrown in C:\WebServ\httpd\strona4\get_pw_message.php on line 107
[PHP] pobierz, plaintext 

[Turson]

Wyświetlaj błędy zapytań na poziomie dev

[PHP] pobierz, plaintext 
if(!$pobierz_pw->execute()){
print_r(!$pobierz_pw->errorInfo());
} 
[PHP] pobierz, plaintext 

[Kerth]

Ten kod nie wyświetla nic. Żadnej reakcji.

[Turson]

CloseCursor() powinno być chyba za execute()

[Pyton_000]

[PHP] pobierz, plaintext 
$id = $_GET['id'];
try {
$pobierz_pw = $db->prepare("SELECT `wiadomosc_od`, `temat_pw`, `wiadomosc_pw`, `data_pw` FROM `wiadomosci_pw` WHERE `id` = :id");
$pobierz_pw->bindValue(":nick_odbiorcy", $_SESSION['nick'], PDO::PARAM_STR);
$pobierz_pw->bindValue(":id", $id, PDO::PARAM_INT);
$pobierz_pw -> closeCursor();
$pobierz_pw->execute();
}
catch(PDOException $e) {
echo $e->getMessage();
}
[PHP] pobierz, plaintext 

Ten post edytował Pyton_000 7.07.2014, 15:26:47

[Kerth]

Nic niestety nie wyświetla się. Może pokażę jak to wygląda:

[PHP] pobierz, plaintext 
$id = $_GET['id'];
 
try {
	$pobierz_pw = $db->prepare("SELECT `wiadomosc_od`, `temat_pw`, `wiadomosc_pw`, `data_pw` FROM `wiadomosci_pw` WHERE `id` = :id");
	$pobierz_pw->bindValue(":id", $id, PDO::PARAM_INT);
	$pobierz_pw -> closeCursor();
	$pobierz_pw->execute();
}
catch(PDOException $e) {
	echo $e->getMessage();
}
 
foreach($pobierz_pw as $row_pw)
{
	echo '<a href = "get_pw_message.php"> Temat: '.$row_pw['temat_pw'].'</a> </br>';
 
	echo "</br>Nadawca: \t";
	echo('<span class = "lightgreen">');
	echo ''.$row_pw['wiadomosc_od'].'</br>' ;
	echo('</span>');
 
	echo "Temat: \t";
	echo('<span class = "lightgreen">');
	echo ''.$row_pw['temat_pw'].'</br>';
	echo('</span>');
 
	echo "Treść: \t";
	echo('<span class = "lightgreen">');
	echo ''.$row_pw['wiadomosc_pw'].'</br>';
	echo('</span>');
 
	echo "Otrzymano: \t";
	echo('<span class = "lightgreen">');
	echo ''.$row_pw['data_pw'].'</br>';
	echo('</span>');
	echo '________________________________</br>';
}
[PHP] pobierz, plaintext