[PHP] Komunikaty po wykonaniu danej operacji + osobna strona transakcji, Pytanie odnośnie rozwiązania. Opcje

[nait]

Pisze mały panel admina od jakiegoś czasu, ciągle coś zmieniam, ulepszam i chyba mniej więcej mam to co sobie zaplanowałem i chciałbym zapytać bardziej doświadczonych ludzi czy to dobre rozwiązanie.
Chodzi o wyświetlanie komunikatów podczas dodania, edycji itd, także o puste pola itp.
Tak więc mam powiedzmy link.php i w niej mam odpowiedni formularz, z niego dane przesyłane są transact.php i tam przetwarzane, sprawdzanie czy poprawnie zostało wszystko wypełnione itd i w zależności od powodzenia operacji i przesyłam spowrotem getem id komunikatu jaki powinien wyświetlić się w tym wypadku na link.php <-- odpowiednią funkcję sobie napisałem do tego bo chciałem taki system wykorzystać do każdej podstrony.

np report_l=1,2, a później to rozbijam na odp komunikaty i wyświetlam je.
Skrócony przykład funkcji:

[PHP] pobierz, plaintext 
function OperationReport() {    
	if ((!empty($_GET['p']) && isset($_GET['p'])) && 
		!empty($_GET['report_l']) && isset($_GET['report_l'])) {
 
// ---- Links Start ------------------------------------------------------------------------------ //			
 
		$reports = explode(',', $_GET['report_l']);		
		foreach($reports as $value) {	 
			switch ($value) {
 
				// --- Dodaj --------------------------------------------------------------------- //
 
				case '1': // 
					$report[1][] = "Pole link nie może być puste";    
					$report[1][] = 0;  // 0 - niepowodzenie operacji, 1 - operacja pomyślnie wykonana
				break;
				case '2': // 
					$report[2][] = "Pole opis nie może być puste";    
					$report[2][] = 0;
				break;
				case '3': // 
					$report[3][] = "Link nie może przekroczyć 350 znaków";    
					$report[3][] = 0;
				break;
				case '4': // 
					$report[4][] = "Opis nie może przekroczyć 350 znaków";    
					$report[4][] = 0;
				break;
				case '5': // 
					$report[5][] = "Błąd podczas ustalania pozycji linku";    
					$report[5][] = 0;
				break;
				case '6': // 
					$report[6][] = "Nie można dodać linku do bazy";    
					$report[6][] = 0;
				break;
				case '7': // 
					$report[7][] = "Link dodany do bazy";    
					$report[7][] = 1;
				break;; 
 
				// --- Edytuj --------------------------------------------------------------------- //
 
			}
		}     	
        foreach ($report as $value){
			echo "<div style=\"width:100%;background-color:" . ($value[1] ? '#628f3d' : '#b02f2f') . ";padding:4px;\">$value[0]</div>\n";
		}
	}
}
[PHP] pobierz, plaintext 

i przykład strony transakcji:

[PHP] pobierz, plaintext 
// Zamiana kluczy z $_POST na zmienne
// $_POST['$klucz'] na $klucz
foreach ($_POST as $key => $value) {
	$$key = $value;
}
 
switch ($_GET['p']) {
// ================================================================================
=============== //
// ---- Links Start ------------------------------------------------------------------------------ //
 
	case 'links':  
		$report = 'report_l';
		$redirect = '../admin.php?p=links';
		$report_id = '';		    
		switch ($_POST['action']) { 
 
			// ---------------------------------------------------------------------------------- //
 
			case 'Dodaj': 
				FilterInputData($link);
				FilterInputData($description);
				if (empty($link))
					$report_id .= '1,';
				AddHttp($link);
				if (empty($description))
					$report_id .= '2,';
				if (strlen($link) > 350)
					$report_id .= '3,';
			 	if (strlen($description) > 350)
				 	$report_id .= '4,';         
				$query_p = "SELECT COUNT(id)+1 AS position FROM links";
				$result_p = mysqli_query($connect, $query_p);
				$position = mysqli_fetch_assoc($result_p);				
				extract($position);
				if(!$result_p)
					$report_id = '5,';     	   
				mysqli_free_result($result_p);
				if(empty($report_id)) {
					$query = "INSERT INTO links values (
								'NULL',
								'" .$position. "',
								'" .$link. "',
								'" .$description. "',
								'" .MYSQL_DATE. "')";
					$result = mysqli_query($connect, $query);				
					if(!$result)
						$report_id = '6,';
				} 
				if (empty($report_id))         
					$report_id = '7';
				echo $report_id;
				header("Location: $redirect&link=a&$report=$report_id");						             
			break;   
 
			// ---------------------------------------------------------------------------------- //
		}
	break;
}
 
[PHP] pobierz, plaintext 

Sorry za lekki chaos w tym Po prostu chciałbym się dowiedzieć czy te rozwiązania są dobre i czy w miare rozsądnie to napisalem.
Z góry dzieki!

Ten post edytował nait 15.09.2011, 14:25:44

[vokiel]

Ja do komunikatów (tzw flash'y) korzystam z sesji. Po wykonaniu jakiejś akcji komunikat dodaję do tablicy komunikatów. Wykonuję tak wszystkie akcje, niezbędne przekierowania, etc. Na finalnej stronie pobieram komunikaty i wyświetlam (z odpowiednim kolorowaniem), a przy tym usuwam z sesji.

W ten sposób nie przekazuję tego jawnie poprzez $_GET, nie wyświetla się nic podwójnie po odświeżeniu strony, nikt nikomu psikusa nie zrobi wysyłając spreparowany link, który wyświetli stronę stwierdzającą poprawne usunięcie konta w systemie.

[nait]

W sumie genialne Mogłem odrazu spytać.
A np. mam zrobione, że po kliknięciu na link czy tam button pojawia się formularz edycji, dodania itd sprawdzam czy w $_GET'cie jest dana zmienna np link=a i pokazuje formularz, czy link=d&id=1 pyta o potwierdzenie usunięcia rekordu. Takie rzeczy też lepiej zastąpić zmiennymi sesyjnymi czy w tych przypadkach to raczej bez różnicy?
Jak Wy to rozwiązujecie?

Ten post edytował nait 15.09.2011, 18:10:19

[vokiel]

Sprawdzasz przy takim ustaleniu linków do usuwania (link=d&id=1) czy użytkownik ma odpowiednie prawa? Bo jak nie, to wiesz, jeden skrypt i jest czysto

Jeśli usuwasz przez formularz, to usuwaj operując na $_POST, moim zdaniem adresy (zmienne $_GET) powinny służyć nawigacji.

[nait]

Zn w moim przypadku będzie tylko 1 użytkownik, admin więc chyba nie ma co sprawdzać i takie rozwiązanie chyba jest jak najbardziej ok? A tak z ciekawości jak w tym przypadku można by sprawdzać prawa?;p

[vokiel]

Jeśli masz tylko jednego użytkownika to uprawnień nie musisz sprawdzać, wystarczy, że sprawdzisz czy jest zalogowany.

[nait]

Ok rozumiem. Ale nawiązując do tego co pisałeś żeby $_GET'a używać tylko do nawigacji to jak to załatwić postem, mianowicie:
Mam tam wiersze rekordów z bazy i w nich odnośniki do edycji np. link=e&id=1 - jak to $_POST'em załatwić? Spekuluje że: sprawdzić jaki input został wciśnięty i ew hiddenami dodatkowe potrzebne dane po przesyłać do posta tak?

[cer98]

jakiego wzorca używasz ?

[nait]

yyy żadnego. Nie pisze tego obiektowo bo obiektówki dopiero się uczę. Piszę tak dla siebie w celu utrwalenia wiedzy