:: Buchempfehlung ::

yonthebeach · #1 18.06.2006, 10:29:32

Hi,

wie kann hier meinen Code besser gestalten?

1. Wie kann ich den Aufruf der Funktion

"<?php getlang('%%str_sample1%%')?>"

abkürzen?

2. Und wie würde man unter PHP so was wegpacken, ich meine in eine "Userklasse" oder so. Damit es wiederverwendbar wird und die Seite sauberer wird. Was wäre professionell.

Code:

<?php 
	$servername = "localhost";
	$username = "root";
	$password = "poroot";
	$port = "3306";
	$db = "mm";

	$verbidung = mysql_connect($servername, $username, $password)
		or die ("Kann keine Verbindung zur Datenbank herstellen".mysql_error());
		
	mysql_select_db($db);
	
	function getlang($templatetext)
  {
		$array = explode ('%%str_', $templatetext);
		echo $array[0];
		for ($i=1; $i<count($array); $i++)
		{
		  list ($ident, $tail) = explode('%%', $array[$i]);
		  $q = mysql_query ("select EN from t_lang_objects where IDENT='$ident'");
		  if ($q && $r = mysql_fetch_row($q))
		    echo $r[0];
		  else
		    echo $ident;
		  echo $tail;
		}
  }
?>

<!DOCTYPE html PUBLIC "-//W3C//DTD XHTML 1.0 Transitional//EN" "http://www.w3.org/TR/xhtml1/DTD/xhtml1-transitional.dtd">
<html xmlns="http://www.w3.org/1999/xhtml">
<head>
<meta http-equiv="Content-Type" content="text/html; charset=iso-8859-1" />
<title>Untitled Document</title>
</head>

<body>
<?php getlang('%%str_sample1%%')?>
</body>
</html>

<?php
	/* Verbindung schließen */
	mysql_close($verbidung);
?>

gruß
yavuz

Opendix · #2 18.06.2006, 19:43:01

hmm... sauber wäre wohl wenn du die gesamte Programmierlogik einfach aus dem Script heraus nimmst.

Sprich du hast 2 files!
In dem einen hast du das ganze html mit dem funktionsaufruf und im anderen eben das mit dem mysql!

Persönlich mache ich es mit dem SQL immer so, dass ich mir eine eigene SQL-Klasse geschriben habe für eigene kleine Projekte wird was grösseres verwendet nehme ich die PEAR Klasse für die Datenbank!

Für templates benutze ich smarty (smarty.php.net)

ich weis nun nich ob dir das weiterhelfen konnte aber ich habs ma versucht *g*

weiter ist es aus meiner sicht gefährlich die Verbindungsdaten direkt da rein zu schreiben, sollte (aus irgendeinem Grund) mal der PHP-parser ausfallen kann das jeder sehen... schreib die besser in ein INI-File welches du bequem mit parse_ini_file() auslesen kann und leg das file unter den root-Ordner da kommt sicherlich niemand ran!

wie willst du den funktionsaufruf da abkürzen?? ausser das du da vielleicht noch ein ; machen solltes ;)

p.s. professionell wäre die PHP-Tags anstatt den CODE-Tags zu verwenden ;)

feuervogel · #3 18.06.2006, 19:45:46

und in einer funktion hat ein "echo" nichts zu suchen!

:: Buchempfehlung ::

:: Anbieterverzeichnis ::

Globale Branchen

Informieren Sie sich über ausgewählte Unternehmen im Anbieterverzeichnis von SELFPHP

:: Newsletter ::

Abonnieren Sie hier den kostenlosen SELFPHP Newsletter!