[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. (IMG:style_emoticons/default/smile.gif)
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 

[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)