Unübersichtlich und „aufgeblähter" Code?

[MudGuard]

Hi,

> 	if(isset($_GET['hersteller'])) {
> 		if(empty($_POST['hersteller'])) {
> 	if(isset($_GET['hersteller'])) {
> 				$h_titel 	= $_POST["hersteller"];
> 	}} // Ende $_GET['hersteller']
> 				$h_titel 	= $_POST["hersteller"];

steckt der 'hersteller' in GET oder in POST, oder gar in beiden? Wenn das zwei unterschiedliche Werte sind, dann wäre es übersichtlicher, wenn die zwei Parameter auch unterschiedliche Namen hätten ...

>        // Datensatz aus der Datenbank auslesen
>       $stmt = $mysqli->prepare("SELECT h_id, h_titel, h_titelURL, h_status FROM >web_hersteller WHERE h_id=?");
>        $stmt->bind_param("s", $hersteller);
>        $stmt->execute();
>        $stmt->bind_result($h_id, $h_titel, $h_titelURL, $h_status);
>        $stmt->fetch();
>        $stmt->close();

wäre etwas, was ich in eine Funktion auslagern würde - es wird ja vermutlich im gesamten Projekt mehrere DB-Abfragen geben, Parameter sql-Statement, die bind-params und die bind_results (letztere zwei als Array/Objekt) ...

cu,
Andreas a/k/a MudGuard