[MySQL][PHP] Pomoc przy optymalizacji 1 kodu PHP Opcje

[poramancza]

Witam serdecznie,

Na wstępie chciałbym prosić o wyrozumiałość oraz "nieszydzenie" z mojego kodu jako, że jest to pierwszy kod który napisałem w PHP. Po kilku kursach video i przeczytanych poradnikach, postawiłem sobie za zadanie, stworzyć prosty formularz w który użytkownik wpisze dane przepisu kulinarnego a ten zapisze go w bazie danych. Dane mają być również sprawdzane (minimum 1 znak,A-Za-z0-9), jeżeli przejdą walidacje, system ma dodać je do bazy ale jeżeli któryś z nich się nie zgadza ma wyświetlić się odpowiednia wiadomość informująca użytkownika co wpisał źle.

Wszystko działa poprawnie ale mam wrażenie że można tu dużo zmienić, więc proszę doświadczonych programistów o jakieś wskazówki dla początkującego . Nie proszę o gotowe rozwiązanie tylko o nakierowanie mnie na odpowiedni trop.

Dodam jeszcze że mój cel nie do końca został zrealizowany, ponieważ chciałem aby wiadomość wyświetlała się użytkownikowi np. 'Tytuł powinien zawierać minimum 1 znak będący...' ale nie kasowała wcześniej wpisanej zawartości w formularzu. Niestety moja nieudolna metoda wykonania tego kodu odświeża stronę po komunikacie i kasuje wszystko.

To mój plik klasa.php

[PHP] pobierz, plaintext 
<?php
class wpis {
 
var $connect;
 
 
function __construct($dbuser,$dbpass,$dbhost,$dbname){
 
	$this->connect = mysql_connect($dbhost,$dbuser,$dbpass) or die('Połączenie z MySQL nieudane!!!');
	mysql_select_db($dbname,$this->connect) or die('Wybrana baza danych nie istnieje!!!');
 
}
 
function dodaj($title,$kraj,$czas,$vege,$ingrid,$opis) {
 
	//zmienna status okresla czy walidacja pola zakonczyla sie pozytywnie (1) czy bledna (0)
	$status_t = 0;
	$status_k = 0;
	$status_i =  0;
	$status_o = 0;
 
 
 
//Sprawdza wymagane wartości w polach tekstowych 
 
 
	if(preg_match('/^[a-zA-Z0-9 ]{1,40}$/',$title)){
 
	$status_t = 1;
 
	}
	else{
 
		echo('<script type="text/javascript">
		window.alert ("Tytuł powinien zawierać minimalnie jedną literę bądź cyfrę, bez znaków specjalnych");
		</script>');
	}
 
 
 
	if(preg_match('/^[a-zA-Z0-9 ]{1,40}$/',$kraj)){
 
	$status_k = 1;
 
 
	}
	else{
 
		echo('<script type="text/javascript">
		window.alert ("Kraj powinien zawierać minimalnie jedną literę bądź cyfrę, bez znaków specjalnych");
		</script>');
	}
 
 
 
 
	if(preg_match('/^[a-zA-Z0-9 ]{1,200}$/',$ingrid)){
 
	$status_i = 1;
 
 
	}
	else{
 
		echo('<script type="text/javascript">
		window.alert ("Składniki powinny zawierać minimalnie jedną literę bądź cyfrę, bez znaków specjalnych");
		</script>');
	}
 
 
 
	if(preg_match('/^[a-zA-Z0-9 ]{1,1000}$/',$opis)){
 
	$status_o = 1;
 
 
	}
	else{
 
		echo('<script type="text/javascript">
		window.alert ("Opis powinien zawierać minimalnie jedną literę bądź cyfrę, bez znaków specjalnych");
		</script>');
	}
 
 
 
 
 
//Sprawdza czy wszystkie warunki zostały spełnione ($status), jezeli tak realizuje dodawanie pliku do bazy.
 
	if($status_t == 1 & $status_k == 1 & $status_i ==1 & $status_o == 1) {
 
 
 
		mysql_query('insert into wpisy values(null,\''.mysql_real_escape_string($title,$this->connect).'\',
\''.mysql_real_escape_string($kraj,$this->connect).'\',\''.mysql_real_escape_string($czas,$this->connect).'\'
,\''.mysql_real_escape_string($vege,$this->connect).'\',\''.mysql_real_escape_string($ingrid,$this->connect).'\'
,\''.mysql_real_escape_string($opis,$this->connect).'\');');
 
 
//Wyświetla informacje o dodaniu wpisu
 
echo('<b>Wpis: '.'\''.$title.'\' '.'został dodany pomyślnie!!!<br /><br />');
 
 
}
}
}
 
 
?>
[PHP] pobierz, plaintext 

To część kodu do mojego index.php:

[PHP] pobierz, plaintext 
<?php
 
 
require_once('./klasa.php');
 
 
$wpis = new wpis('phpkurs','kursik','localhost','WPISY');
 
 
if (isset($_POST['dodaj'])){	
$title = $_POST['title'];
$kraj = $_POST['kraj'];
$czas = $_POST['czas'];
$ingrid = $_POST['ingrid'];
$opis = $_POST['opis'];
$vege = $_POST['vege'];
 
$wpis -> dodaj(''.$title.'',''.$kraj.'',''.$czas.'',''.$vege.'',''.$ingrid.'',''.$opis.'');
 
}
 
 
?>
[PHP] pobierz, plaintext 

[com]

No to tak bez przeładowywania zainteresuj się technologią AJAX

Skoro dopiero się uczysz to od razu przerzuć się na mysqli albo PDO bo lada moment mysql_* zniknie z php raz na zawsze...

co do statusu to jak już tak bardzo chcesz to zrób sobie z tego jeden array :

[PHP] pobierz, plaintext 
$status = array('t'=>0,'k'=>0,'i'=>0,'o'=>0);
[PHP] pobierz, plaintext 

zmiana statusu to np:

[PHP] pobierz, plaintext 
$status['i'] = 1;
[PHP] pobierz, plaintext 

i wtedy zamiast:

[PHP] pobierz, plaintext 
	if($status_t == 1 & $status_k == 1 & $status_i ==1 & $status_o == 1) {
#whatever
}
[PHP] pobierz, plaintext 

mam po prostu:

[PHP] pobierz, plaintext 
if (!in_array(0, $status, true)) {
#whatever
}
[PHP] pobierz, plaintext 

Ponadto napisałbym funkcje dla preg_match skoro tylko zmieniasz zakres znaków i content

[PHP] pobierz, plaintext 
$wpis -> dodaj(''.$title.'',''.$kraj.'',''.$czas.'',''.$vege.'',''.$ingrid.'',''.$opis.'');
[PHP] pobierz, plaintext 

why?

[PHP] pobierz, plaintext 
$wpis -> dodaj($title,$kraj,$czas,$vege,$ingrid,$opis);
[PHP] pobierz, plaintext 

[phpion]

Wszystko działa poprawnie

Ten fragment wygląda podejrzanie:

[PHP] pobierz, plaintext 
if($status_t == 1 & $status_k == 1 & $status_i ==1 & $status_o == 1) {
[PHP] pobierz, plaintext 

Nie powienieneś zastosować tutaj && zamiast &? Nawet jeśli w tym przypadku Ci to działa to (sądząc po komentarzu) raczej chodziło Ci o &&.

[Turson]

Obiektówki można dołączyć więcej. Jedna metoda walidacji z parametrami jaki wzór w preg_replace ma sprawdzić i zmienną do sprawdzenia. Potem tylko $wpis->walidacja($pattern,$zmienna). Zwracasz true, jeśli jest ok i działasz dalej

[poramancza]

Dzięki wielkie za pomoc wszystkie rady bardzo pomocne i napewno skorzystam.

A co do && to faktycznie pomyliłem się, ale jakimś cudem działa tak jak jest

No i szczególnie dziękuje za podpowiedź z użyciem opcji array. Uporządkowało mi to sporą część kodu.

Ten post edytował poramancza 30.10.2013, 18:21:38