[HTML][PHP]Optymalizacja wyświetlania dynamicznego HTML'a Opcje

[kyez]

Witajcie.

Piszę sobie taki skrypt mailingowy i zastanawiam się jak tu zopytmalizować wyświetlanie HTML'a. Chodzi o to że PHP kontroluje dane w prowadzane przez użytkownika i wyświetla odpowiedni komunikat po wysłaniu formularza. Ale wydaje mi się że rozwiązanie które zastosowałem nie jest zbyt profesjonale i optymalne.
Proszę was o podpowiedzi.
Przy okazji, jak oceniacie napisany przezemnie kod?

[PHP] pobierz, plaintext 
<?php
$formtresc = "
 
<style type='text/css'>
 
input{
	display: block;
}
 
</style>
 
<div form-wrapper style='width: 400px; height: 500px; margin: 0 auto;'>
 	<center>
	<form action='mail.php' method='get'>
			<div class='error' id='imie'></div>
		Your name: <input type='text' name='imie' />
			<div class='error' id='email'></div>
		Your e-mail: <input type='text' name='email' />
			Select subject of your message: <div class='error' id='temat'></div>
			<select name='subject'>
 
				<option>Default</option>
				<option>Bug report</option>
				<option>Grats</option>
 
 
			</select></br>
 
		Your message: <textarea name='message' cols='50' rows='10'></textarea></br></br>
 
		<input type='submit' name='send' value='Dalej'/>
	</form>
	</center>
	</div>
";
 
$formtrescE1 = "<form action='mail.php' method='get'>
			<div class='error' id='imie'>Podaj imie!</div>
		<input type='text' name='imie' /></br>
			<div class='error' id='email'></div>
		<input type='text' name='email' />
			<div class='error' id='temat'></div>
			<select name='subject'>
 
				<option>Default</option>
				<option>Bug report</option>
				<option>Grats</option>
 
			</select>
 
		<textarea name='message' cols='50' rows='10'>Tresc wiadomosci</textarea>
 
		<input type='submit' value='Dalej'/>
	</form>";
 
$formtrescE2 = "<form action='mail.php' method='get'>
			<div class='error' id='imie'></div>
		<input type='text' name='imie' /></br>
			<div class='error' id='email'>Podaj email</div>
		<input type='text' name='email' />
			<div class='error' id='temat'></div>
			<select name='subject'>
 
				<option>Default</option>
				<option>Bug report</option>
				<option>Grats</option>
 
			</select>
 
		<textarea name='message' cols='50' rows='10'>Tresc wiadomosci</textarea>
 
		<input type='submit' value='Dalej'/>
	</form>";
 
$formtrescE3 = "<form action='mail.php' method='get'>
			<div class='error' id='imie'></div>
		<input type='text' name='imie' /></br>
			<div class='error' id='email'></div>
		<input type='text' name='email' />
			<div class='error' id='temat'>Podaj temat wiadomosci</div>
			<select name='subject'>
 
				<option>Default</option>
				<option>Bug report</option>
				<option>Grats</option>
 
 
			</select>
 
		<textarea name='message' cols='50' rows='10'>Tresc wiadomosci</textarea>
 
		<input type='submit' value='Dalej'/>
	</form>";
 
$formtrescE4 = "<form action='mail.php' method='get'>
			<div class='error' id='imie'></div>
		<input type='text' name='imie' /></br>
			<div class='error' id='email'></div>
		<input type='text' name='email' />
			<div class='error' id='temat'></div>
			<select name='subject'>
 
				<option>Default</option>
				<option>Bug report</option>
				<option>Grats</option>
 
			</select>
			<div class='error' id='message'>Podaj tresc wiadomosci</div>
		<textarea name='message' cols='50' rows='10'></textarea>
 
		<input type='submit' value='Dalej'/>
	</form>";
 
 
#--------------Temporary Settings--------#
	# error id:
	# 1 - brak imienia
	# 2 - brak emaila
	# 3 - nie wybrany temat
	# 4 - brak tresci maila
 
 
	#$_GET['error']['id'] = '4';
 
#-------------End of temp. settings------#
 
 
 
#--------Configuration File Info---------#
#--------------READ THIS!----------------#
#
#	First Line: There you have an e-mail
#	address. Messages will be send to 
#	that e-mail!
#	
#	Second Line: There you have color
#	of border of the inuts
#
#	
#	
#
#-----End of conf. file info-------------#
 
 
 
#-------------Functions------------------#
function errorExist(){
 
	$error = isset($_GET['error']);
	#var_dump($a);
	#print_r($_GET);
	return $error;
}
 
function errorIdChecker(){ # funkcja zamiast 'echo' ma zwracac id bledu dla innej funkcji
	if ($_GET['error']['id'] == 1) {
 
		#echo 'brak imienia';
		return 1;
 
	} elseif ($_GET['error']['id'] == 2) {
 
		#echo "brak emaila";
		return 2;
 
	} elseif ($_GET['error']['id'] == 3) {
 
		#echo "brak tematu";
		return 3;
 
	} elseif ($_GET['error']['id'] == 4) {
 
		#echo "wpisz tresc wiadomosci";
		return 4;
 
	}
 
} #END OF errorIdChecker function
 
function makePage($errorID, $trescE1, $trescE2, $trescE3, $trescE4){ // wyswietlanie odpoiedniej tresci
 
	switch ($errorID) {
		case 1:
			echo $trescE1;
			break;
 
		case 2:
		 	echo $trescE2;
			break;	
 
		case 3:
			echo $trescE3;
			break;
 
		case 4:
			echo $trescE4;
			break;
 
		default:
			echo "Wystapił niesnany błąd";
			echo "<a href='mail.php'>Go Back</a>";
			break;
 
	}# Switch end
 
} #END OF makePage function
 
 
function catchError(){
 
	if (empty($_GET['imie'])){
		$_GET['error']['id'] = 1;
		return;
	}
	if (empty($_GET['email']) or !preg_match("/^[_\.0-9a-zA-Z-]+@([0-9a-zA-Z][0-9a-zA-Z-]+\.)+[a-zA-Z]{2,6}$/i", $_GET['email'])) {
		$_GET['error']['id'] = 2;
		return;
	}
	if ($_GET['subject'] == 'Default') {
		$_GET['error']['id'] = 3;
		return;
	}
	if (empty($_GET['message'])) {
		$_GET['error']['id'] = 4;
		return;
	}
 
}
 
function loadConfigFile(){
 
	$DOCUMENT_ROOT = $_SERVER['DOCUMENT_ROOT']; 
	@ $file = fopen('config.txt', 'r');
 
	if (!$file) {
		echo "Blad otwarcia pliku konfiguracyjnego";
		exit;
	} else {
		for ($i=0; !feof($file); $i++) { 
 
			$config[] = fgetcsv($file, 999, "]");
			#print_r($config);
		}
 
	}
	return $config;
}
 
 
 
 
	#var_dump($errorE);
 
if(isset($_GET['send'])){
	catchError();
	$errorE = errorExist(); //sprawdzanie czy przy ladowaniu strony jest jakis error $errorE = false/true
	if ($errorE == true) {
 
		$errorId = errorIdChecker();
 
		makePage($errorId, $formtrescE1, $formtrescE2, $formtrescE3, $formtrescE4);
		#print_r($a);
	} else{
		$configf = loadConfigFile();
		echo '<pre>';
			print_r($configf);
		echo "</pre>";
		$message = $_GET['message']."
 
 
 
 
		Author e-mail addres: ".$_GET['email'];
		mail($configf[0][0], $_GET['subject'], /*$_GET['message']*/$message); //not working on localhost
	}
} else {
	echo $formtresc;
	#echo $_SERVER['DOCUMENT_ROOT'];
}
 
// Imie 
// email
// kategoria	
// tresc
// dodawanie wyrazow/liczb
 
 
 
?>
 
[PHP] pobierz, plaintext 

[timon27]

Robisz bałagan wrzucając 10 różnych stron w 1 plik. Po co?
Skoro masz 5 różnych treści to zrób 5 różnych plików.
Jak bardzo chcesz je mieć pod jednym adresem to np. użyj include();
Takie moje zdanie.

Ten post edytował timon27 18.09.2013, 23:21:15

[nospor]

ps: $_GET['error']['id'] = 1;
Tablica _GET nie sluzy do zapisuwania w niej jakis swoich danych..... w tej tablicy znajduje sie to co bylo w URL i tak ma pozostac.

[kyez]

Czyli tutaj:

[PHP] pobierz, plaintext 
 
 switch ($errorID) {
 
		case 1:
 
			echo $trescE1;
 
			break;
 
 
 
		case 2:
 
		 	echo $trescE2;
 
			break;	
 
 
 
		case 3:
 
			echo $trescE3;
 
			break;
 
 
 
		case 4:
 
			echo $trescE4;
 
			break;
 
 
 
		default:
 
			echo "Wystapił niesnany błąd";
 
			echo "<a href='mail.php'>Go Back</a>";
 
			break;
 
 
[PHP] pobierz, plaintext 

zamiast echo mam dać include ?

i pozmieniam wszystkie get'y na post. Tylko czy wszystkie?

Dziękuje wam bardzo za odpowiedzi. Czy macie jeszcze jakieś uwagi ?

Pozdrawiam, Przemek.

[nospor]

i pozmieniam wszystkie get'y na post.

A gdzie ja ci pisalem ze masz get na posty zamienic? Zarowno GET czy POST nie sluzy do przenoszenia wartosci miedzy funkcjami w aplikacji.... GET czy POST to tablice wypelniane danymi z przegladarki.

[kyez]

No ale musze jakoś przekazać informacje o błędzie po wysłaniu formularza.

[nospor]

Przeciez ty info o errorze zapisujesz i odbierasz w obrebie jednego żądania....
A nawet jakby bylo inaczej to zapisanie tego w GET czy POST by ci wcale nie pomoglo.

POczytaj prosze co to GET, co to POST i nie kloc sie w tak podstawowej kwestii

[kyez]

To nie tak, że chcę się kłócić. Poprostu nie wiem jak to inaczej rozwiązać.

[com]

a jak chcesz coś już na silę przechowywać to stwórz sobie tablice, albo trzymaj to w sesji, a reszta tak jak mówi nospor .

[kyez]

Czyli mój kod nie jest dobry? Szczerze się przyznam, że potrzebuję pieniędzy i chciałem napisać coś na codecanyon.net. Powiedzcie szczerze czy nadaje się na sprzedarz po wprowadzeniu wymienionych poprawek? Jeśli nie to czy długa jeszcze droga przedemną? Dodam, że mam zamiar jeszcze dodać wybór stylu w jakim ma być cały formularz przez prostą edycję pliku config.txt (kupujący nie musi znać php, aby konfigurować skrypt)

[Greg0]

Jeśli nauka PHP ma być motywowana zarabianiem pieniędzy na "tworzeniu" (czyt. nauce tworzenia) i bublach to daruj sobie to na starcie i idź rozdawać ulotki.

[kyez]

Raz. Piszę w PHP nie od 5 miesięcy a od 2 lat. Dwa. Nie motywuję nauki zarabianiem pieniędzy. Poprostu chcę się dowiedzieć jak dobry jest mój kod. Jak Ci się nie podoba nie musisz dawać złośliwych komentarzy. Nie piszę tu bo jestem chytry i chcę zbić miliony, tylko chcę się dowiedzieć o jakości mojego kodu, to źle ?

[com]

przecież jak już powiedziano twój "kod" zawiera błędne zastosowanie GET i POST, wiec czego jeszcze oczekujesz? poza tym czym on się wyróżnia od milionów przykładów w necie ?