Zastanawiam się jakie podejście będzie najwłaściwsze dla komunikatów dla użytkownika (typu dodano artykuł, usnięto produkt, zły login lub hasło itd.).

Myślałem nad jakąś klasą Logger'a ale nie wiem czy to przypadkiem nie za dużo, pozatym Loggera o ile się nie myle stosuję się głownie do błędów aplikacji.

Inny sposób o którym tylko słyszałem to zrobić kolekcje obiektów, ale nie bardzo wiem na czym to by miało polegać...

Jezeli ktoś ma jakiś pomysł to będe wdzięczny?

Ostatnio czytałem, że fajnie pracuje się z wyjątkami. Może do Twojego celu by się nadawały?

Co do "kolekcje obiektów" to nawet nie słyszałem.

Wyjątki przy informacji "Artykuł został dodany"

Może poprostu wykorzystaj javascriptowy alert(), A potem wrzucaj do debuggera. Coś na modłe

[PHP] pobierz, plaintext 
<?php
function ShowAlert( $msg, $error = false )
{
 
# Tu wywolanie alerta, niestety nie chce mi sie myslec :)
 
if( $error )
{
 
Debug::addMessage( $msg );
 
}
 
}
?>
[PHP] pobierz, plaintext 

Z wyjątkami pracuje się doskonale a i owszem, ale przy wyjątkach a nie komunikatach.

Hmmm pomysł z java jest całkiem ok, ale wydaje mi się troche na wyrost, napewno musi istnieć jakiś prosty przyjemny sposób na rozwiązanie problemu.

Myślałem zeby zrobić coś takiego:
dodawanie do loggera tych wiadomości, oczywiscie oznaczając je jako wiadomosc dla użytkownika i potem je wyswietlać w miejscu wywolania np. funkcji $logger->showMsg(), tylko jak oznaczać że wiadomość było już wyświetlona? Ma ktoś jakiś pomysł? Zwykla zmienna flagowa?

Ale z drugiej strony czy Logger nie powinien złużyć do rapoertowania bardziej błędów dla admina niż wiadomości dla użytkownika?

To jak ci nazwa Logger nie pasuje to nadaj tej klasie nazwe: Messager i juz chyba bedzie ok po co na sile tworzyc se problemy. nie pasuje nazwa to ją zmien i po sprawie.

Heh...

Tym tropem to mogę sobie zmienić wyjątki na powiadamiacze ;-) Logger to ogolnie przyjeta nazwa dla klasy która odpowiada za logowanie zdarzeń, nie ważne czy się nazywa Loging, Logger, Messager, Log itd. Chodzi mi o to że mieszać jedno z drugim, ale mogę sie przecież mylić.

Nie, zdecydowanie idziemy innym tropem
Raz mowisz ze logger to bardziej dla admina niz wiadomosc dla usera,raz ze to ogolnie przyjeta nazwa dla czegos tam.... no sie zdecyduj. zrob klase dla logowania zdarzen, zrob klasy dla wiadomosci dla usera i juz. A jak chcesz polaczyc to polacz tylko rozdzielaj typami.
Ja przykladowo mam polaczone, tj. w jednej klasie mam metody do komunikatow dla usera oraz logowanie do pliku. sa to metody podobne i czesciowo zalezne wiec je polaczym i jest mi z tym dobrze

java !== java script !

A co do problemu.

U mnie jest klasa Template która dziedziczy po Smarty.
Posiada ona metode displayMessage('Dodano news');

Cała filozofia

Ja rozwiązuje to bardzo prosto. Więc chętnie wysłucham jak robią to inni.

@nospor

OK,

Co do Loggera chodziło mi po prostu o rozwiązanie, nie wiedziałem czy mam takie komunikaty po prostu logować czy może inaczej je jakoś exportować.

Wpadłem na jeszcze taki pomysł aby po prostu zapisywać komunikaty w obiekcie w zmiennej tablicy z którego się korzysta a potem je wyciągać 'po najświeższej', taki zalążek, co wy na to?

A powiedz jeszcze, jak w swoim rozwiązaniu rozwiązujesz kwestie wyświetlenia odpowiedniej wiadomości? Bo wiadomo że nie wyświetlisz całego 'loga', wyświetlasz po prostu ostatnio dodaną czy w np. przypisujesz wydarzenia do wiadomości i potem to jakoś rozwiązujesz przy wyświetlaniu?

@Strzałek

Hehe, myślę że nie będe miał już problemu z wyświetleniem komunikatu ;-)
Bardziej mi chodzi o sposób jego zapisywania...

Wiadomosci moga byc roznych typow. Moze byc kilka wiadomosci dla danego typu. W zaleznosci od typu roznie wyswietlam wiadomosci. jedne na czerwono (bledy), inne na zielono (komunikaty) i inne na inny kolor. Wyswietlam tyle wiadomosci ile bylo.
sa tez wiadomosci ktore sa czystym logiem. W zaleznosci od ustawionych parametrow takie wiadomosci wyswietlam albo tylko do logow pisze

To ja poproszę wtedy o szybką lekcje php bo nie potrafie dojść do jednego.

Powiedz masz obiekt klasy news która ma funkcje wyświetlenie formularza do dodawania i po wysłaniu danych dodaje go oraz ma metode wyświetlania wiadomości (dla uproszczenia załużmy że w tym samej klasie a nie osobnej). Mając powiedzmy prostą stronę:

[PHP] pobierz, plaintext 
<?php
$news = new News();
<body>
<h1>dodawanie newsa</h1>
$news->show_message();
$news->print_addForm();
</body>
?>
[PHP] pobierz, plaintext 

wysylasz formularz, dodaje się news, tworzy się wiadomość o tym fakcie, strona przeładowuje się i informacja umiera, chyba że zapisujesz ją w sesji, pliku lub w DB?

Ten fakt że tak powiem mnie najbardziej zastanawia.

W ktorym momencie napisalem ze mi sie strona przeladowuje po dodaniu wiadomosci?
wiadomosc jest dodawana juz po przeladowaniu strony. wysylasz formualrz (strona sie przeladowuje), zapisujesz do bazy i w zaleznosci od tego czy sie udalo czy nie to tworzysz taka czy siaka wiadomosc. Potem masz jakies cudo co ci dodaje do wygenerowanej strony wiadomosci ktore sa zapisane.

Jesli jednak z jakiegos powodu musisz znowu przeladowac strone to wiadomosci zapisujesz w sesji

Ok, a co z problemem przeładowania?

Po takiej operacji jak ty mówisz, weźmy dla przykładu naszego newsa, bez przeładowania nie czyści się ani POST ani GET. Wieć po dodaniu tego newsa, użytkownik widzi nasz komunikat, pusty formularz na nowego newsa, ale wystarczy że odświeży strone i nawet się nie zoreintuje kiedy dodał drugi raz ten sam rekord.

Strasznie komplikujesz proste rzeczy.

masz na stronie 3 akcje:
- formularz (zawsze)
- zapis newsa do bazy, nastepnie przekierowanie, zeby nie wyslac 2 raz posta
- wyswietlasz komunikat, ktory trzymasz np. w sesji, po czym go usuwasz.

Wczesniej przy programowaniu strukturalnym zastosowalem pewnien prosty mechanizm, ktory dzialal bardzo dobrze i calkiem wygodnie.

Mialem np. dodajNowoscFormularz i dodajNowosc. Pierwszy wyswietlal form, a drugi zapisywal w bazie - form kierowany byl do tego drugiego.

Jezeli wszystko wyszlo ok robilem na koncu przekierowanie (header('Location: xyz.php?msg=Dodano Nowosc')).

W pliku xyz.php mialem

[PHP] pobierz, plaintext 
<?php
if(!empty($_GET['msg']))
  echo $_GET['msg'];
?>
[PHP] pobierz, plaintext 

Adrian.

Tu masz cos jak zapobiegac odswierzaniu i to bez przekierowania:
http://forum.php.pl/index.php?showtopic=44383
Pozatym problem z odswierzaniem ma sie nijak do komunikatow.

Jesli jednak koniecznie chcesz robic przekierowanie to zapisz komuniakty do sesji, po przekierowaniu je odbierz, wyswietl i usun z sesji. No tu nie ma zadnej filozofii


@Prph taka metoda jest beznadziejna. A jak bedziesz chcial przekazac kilka komunikatow i beda za dlugie na url? Pomijam juz fakt ze ktos ci moze takiego linka spreparowac ze na Twojej stronie bedzie napisa:
"Ta strona jest do.... kitu " albo jeszcze gorsze rzeczy i porazka totalna

edit: z php5 to ma malo wspolnego. przenosze

Jak dla mnie to klasa metoda dla wyswietlania. Ostatni serwis jaki robilem byl tak zaprojektowany(graficznie) ze w pliku index.php nad calym includowaniem itp moglem umiescic metode wyswietlajaca wiadomosci. Jesli wiadomosci nie bylo wszystko wygladalo w porzadku. Takze mala klasa i do wyswietlania i przekazywania wiadomosci.

[PHP] pobierz, plaintext 
<?php
if(akcja()) { /// jesli jakas tam operacja jest udana
		  $msg->AddMessage("Operacja udana");
	  } else { // jesli niedudana
		   $msg->AddMessage("Operacja nieudana");  // mozna dodawac jeszcze parametry e.g flagi bledow itp.
	  }
?>
[PHP] pobierz, plaintext 

A gdziekolwiek tam gdzie chcesz dodac tylko

[PHP] pobierz, plaintext 
<?php
echo $msg->ShowMessage() // ewentualnie flaga jako parametr.
?>
[PHP] pobierz, plaintext 

Przewagatego uwazam jest taka, ze taka wiadomosc mozesz wyprintowac sobie gdzie tylko chcesz

To ja ciągle czegoś nie rozumiem, jeżeli np. wyswietlasz wiadomosc w kodzie przed wywolaniem akcji to co wtedy? Wtedy nie ma jeszcze wiadomości...
Więc chyba trzeba najpierw wywołać akcje, zapisać wiadomość do sesji, przeładować stronę żeby wyczyścić REQUESTa i dopiero potem wyswietlic wiadomosc z sesji...

teraz ja nie rozumiem. do kogo to pytanie? do mnie czy do thornag'a? W zasadzie to i tak bez roznicy.
Ty chyba mylisz pojecia "dodanie wiadomosci" i "wyswietlenie wiadomosci". Dodawac to se mozesz kiedy chcesz, przed zapisem formualrza, po zapisie formularza, ale wyswietlasz wtedy kiedy potrzebujesz. robisz przekierowanie - wyswietlaj po przekierowaniu.

Pytanie do zainteresowanych: ;-)

No ale jak, przecież po przeładowaniu obiekt tworzy się na nowo i wszystkie wartości zmiennych idą w las. Więc nie można sobie przekierować a potem wyśwetlić zmiennej, chyba że jest to zmienna sesyjna, ale odwoływałem się do jego sposobu thornag'a:

że wystarczy zmienić kolejność w jego przypadku i wszystko się posypię:

tj.

wpierw zrobię w kodzie:

[PHP] pobierz, plaintext 
<?php
echo $msg->ShowMessage() // ewentualnie flaga jako parametr.
?>
[PHP] pobierz, plaintext 

a dopiero potem

[PHP] pobierz, plaintext 
<?php
if(akcja()) { /// jesli jakas tam operacja jest udana
		  $msg->AddMessage("Operacja udana");
	  } else { // jesli niedudana
		   $msg->AddMessage("Operacja nieudana");  // mozna dodawac jeszcze parametry e.g flagi bledow itp.
	  }
?>
[PHP] pobierz, plaintext 

To jeżeli nie zapiszę zmiennej do sesji i nie przeładuję strony to nic się nie pokaże.

I druga kwestia w tym wypadku.

Czy używać jednej zmiennej sesyjnej i przy każdym wydarzeniu napisywać ją? Ale wtedy możemy wyświetlić tylko jedną wiadomość.
A jeżeli dla każdego komunikatu dawać inna nazwę zmiennej sesyjnej to jak z jej czyszczeniem?

------
Edit:

OK, już mam rozwiązany problem ;-)
Dzięki wszystkim za pomoc. Jakby co to rozwiązałem to tak:
Przykładowa metoda dodawania danych:

[PHP] pobierz, plaintext 
<?php
public function form_addRekord() {
		if($this->form->validate()) {
			try {
				// dodawanie do DB.
				$this->addRekordFromPost(array('id', 'MAX_FILE_SIZE'));
				// sprawdzam czy sa jakies pliki do uploadu
				if($_FILES) {
					$this->handleFileUpload();
				}
 
				$this->addMsg('Dodano nowy rekord.'); // ustawiam komunikat
 
				$tuJestem = $_SERVER['PHP_SELF'];			
				header("Location: $tuJestem"); // robie przekierowanie
 
			} catch (Exception $e) {
				MyErrorHandling::handle_error($e);
			}
		} else {
			$this->form->addElement('submit', null, 'Dodaj');
			$this->form->display();
		}
	}
?>
[PHP] pobierz, plaintext 

Metody obsługujące komunikaty.

[PHP] pobierz, plaintext 
<?php
public function showMsg() {
		session_start();
		echo $_SESSION['userMsg'];
		$_SESSION['userMsg'] = null;
	}
 
public function addMsg($string) {
		session_start();
		$_SESSION['userMsg'] = $string;
	}
?>
[PHP] pobierz, plaintext 

No i potem na stronie:

[PHP] pobierz, plaintext 
<?php
$cms->showMsg();
$cms->form_addRekord(); 
?>
[PHP] pobierz, plaintext 

no to przeciez caly czas mowie, ze jesli robisz przekierowanie to wal to do sesji.
niech Twoje metoda AddMessage() zapisuje dane do sesji.
Metoda GetMessage() niech pobiera z sesji i usuwa powiedzmy od razu

edit: nie widzialem jak edytowales poprzedniego posta (byl na innej stronie)

No i już sobie przypomniałem dlaczego zadałem wlaściwie to pytanie, otóż to rozwiązanie której opisałem powyżej już kiedyś zastosowałem, i jest problem, już tłumacze jaki.

Mam liste rekordów, do każdej link usun np. usun.php?id=10 (dla ułatwienia).

Klikam link
Usuwam rekord
Ustawiam komunikat: usunieto rekord
Przeladowuje strone (ale wiadomo header location wczytuje do konca dokument i dopiero sie przeladowuje)
Komunikat wyswietla sie mimo wszystko (tylko tego nie widac bo zaraz znika)
Zmienna z SESYJNA z komunikatem czysci sie (bo ma sie czyscic po pokazaniu).
Strona sie przeladowala.
I komunikat sie nie wyswietla bo jest pusty.

I co tu robic?

[PHP] pobierz, plaintext 
<?php
header('Location: ....');
exit;
?>
[PHP] pobierz, plaintext 

Sprawdź...
exit zatrzymuje wogóle cały proces, nie tylko wykonywanie skryptu, strona po prostu "zawiesi się" i nie przeładuję się wogole.

To ty sprawdź
exit powoduje zatrzymanie wysylania juz czegokolwiek do przegladarki. ale juz header poszedl i ten header zalatwia ci sprawe. Tak sie wlasnie robi: puszcza sie headera przekierowujacego i wali sie exit bo nie ma sensu juz generowanie czego innego

ps: ty pracujesz dla tej firmy co masz w nicku? bo sie tak zastanawiam czytajac ich oferte

ad 1) Sprawdziłem zanim to napisałem i jest tak jak mówie, ale wieczorem sprawdze jeszcze raz dla pewności.

ad 2) Pomagam czasami.

Jesli ja dla Ciebie nie jestem osobą wiarygodną to moze manual Cię przekona:
http://www.php.net/manual/pl/function.header.php