Vielen Dank für die Tipps. Ich werde sie mir gleich mal zu Gemüte führen ...

Ich habe noch eine Frage:
In meinem PHP Buch wird bei einem Formularfeld immer auf eine andere PHP Datei verwiesen. Ist dann meine Umsetzung, alles in eine Datei zu schrieben, sinnvoll?! Oder sollte das lieber getrennt sein?
Für den Fall, dass meine Lösung so ok ist: Ist die Verwenung von ID und isset auch sinnvoll? Oder kann man das noch "eleganter" lösen?

Daaaanke schonmal für Antwort und Hilfe!!

Mister Ad

Ja, die HTML ist noch nicht schön .. aber das soll erstmal seitens PHP funktionieren und auch "elegant" gelöst sein. Das Bunte drum rum, kommt später mal .. :wink:

Und Code einrücken?!? Wie mach ich das am besten?!
Ich bin Angänger und mache es eigentlich so, dass es mirgut gefällt und ich damit klarkomme.

Ich nehm immer 4 leerstellen wenn was neues kommt also z.B. so:

also nach { (if, switch, for, etc.) welches hinter der function steht neue Zeile mit 4 leerstellen mehr als in der davor.

Ich hoff ich drück mich richtig aus

Zu deinem Code noch, du sprichst 2mal die Datenbank an. Einmal im If than und einmal im else teil. Wenn du es nur einmal vor dem If machst, dann vermeidest du redundanzen (wiederholungen) und somit wird der Quelltext wiederrum ein wenig übersichtlicher.

Ausserdem würde ich anstatt print "<input type='button'>" die ' und " anders rum schreiben. Also print '<input type="button">'. Aber ich glaub ist beides erlaubt.

______________________
edit
in deim Fall so

Ich hoff ich hab alles geändert und keine Fehler eingebaut aber is ja nnur als Beispiel damit du es verstehst.

MFG

Basti

In der Regel ist der erste Schritt fü besseren Code eben die Aufteilung von Code in verschiene Dateien. Beispiel:

• alle Anfragen der Anwendung/Site gehen an index.php;
  Hier wird alles erledigt, was jeden Aufruf betrifft;
• dort wird eine zentrale Konfigurations-Datei eingebunden - in deinem Fall würde die einfach die Datenbank-Verbindungsdaten enthalten;
• In der index.php wird die Anfrage analysiert und z.B. eine Datei action/deluser.php oder action/show_deluser_fom.php (also die auszuführende Aktion) eingebunden.
• Diese Action-Dateien analysieren die übergebenen Daten und führen die nötigen Aktionen aus, um a) zu entscheiden, welche "Seite", besser "View" der Benutzer ausgeliefert bekommen soll und b) alle Daten, die in diese Sicht gepackt werden müssen zusammenzustellen. Im einfachsten Fall schreibst du eben einfach alle Daten in ein Array und den Namen der Sicht in eine Variable.
• Wieder zurück in der index.php (da dieser Mechanismus wieder bei jedem Seitenaufruf identisch ist) wird view/$view.php eingebunden, also z.B. view/deluser_form.php oder view/deluser_success.php.
• Dort grifst du dann auf das Daten-Array zu.
• Da es auch in den Views, wenn du jede als eigene HTML-Seite konzipierst viele Redundanzen gibt, beschränkst du das HTML, das in den View-Dateien ausgegeben wird auf z.B. den Inhalt eines DIV-Containers der später fertigen Seite und sorgst auch in der index.php dafür, dass das Drumherum, das bei allen Seiten gleich ist überall ausgegeben wird. Entweder mit einem header- und einem footer-Template oder mit einem Haupttemplate, dass dann an der richtigen Stelle ein require "view/$view.php"; ausführt. (Mit Template meine ich hier eine HTML-Seite mit eingebettetem PHP-Code)

Das ganze ist ohne Objeke natürlich nicht wirklich doll zu lösen, aber ich hab so auch schon erfolgreich eine Webanwendung umgesetzt und das ist in jedem Fall ein Schritt in Richtung eleganteres Design.

Die Bezeichnung "ID" ist nicht gut getroffen. Ich verwende bei sowas, wenn überhaupt, "sent". Anstatt "true" würde ich auch eher eine 1 nehmen, da hier ja die Zeichenkette "true" ankommt, anstatt der boolsche Wert true.

Wenn du allerings im action-Attribut des Forulars die Aktion eindeutig definierst, kannst du einfach anhand dieses Wertes prüfen, welches Formular da ankommt. Die Daten selbst musst du ja eh noch prüfen.

Was das einrücken angeht, so benutze ich inzwischen immer Tabs. Wichtig ist letztlich, dass du dich auf ein einheitliches Bild festlegst - das für dich übersichtlich ist und auf allgemein Üblichem aufbaut. Schau dir dazu z.B. mal die Coding Conventions/Guidelines von PEAR oder coWIKI oder ... an.

Basti

Bis auf die angenehme Eigenschaft, dass echo mehrere Parameter aufnehmen kann.
Statt vielen, unnötigen Zeichenkettenverknüpfungen bei print('a'.'b'.'c'.'d'.'e'); benutze ich lieber echo 'a', 'b', 'c', 'd', 'e';

Andereseits kann man echo nicht wie print als Funktion "benutzen".
zB array_map('echo', $a); geht nicht , array_map('print', $a); dagegen schon.
edit: stimmt nicht mal. Auch mit print geht das nicht.

N!cKY

was ist daran besser, ist doch egal ob man "." oder "," schreibt!
aber zB beim tenären operator gibts unterschiede:
isset($var) ? echo 'hallo' : echo 'nö'; // geht nicht
isset($var) ? print('hallo') : print('nö'); //geht
ok man kanns auch so machen:
echo isset($array) ? 'hallo' : 'nö'; //geht
einklich isses wurschd was man nimmt, ich benutz echo! glaub ich hab mal irgandwo gehört das das sogar schneller ist, aber weis nicht mehr genau!

Basti

Na, mit dem Punkt-Operator muss jedesmal eine neue Zeichenkette aufgebaut werden und mit dem Komma, schmeißt du einfach eine nach dem anderen raus. (@"bruchpilot": Ich hab mich auf den Fall hier bezogen - natürlich gibts Fälle, wo es relevant ist, was man nimmt)

Das ist genauso albern, wie die Diskussion um single oder double quotes. Lediglich bei dem oben angesprochenen Punkt-Operator könnte das relevant werden. Aber auch nur in Verbindung mit seeehr langen Zeichenketten - und wer gibt überhaupt lange Zeichenketten mit echo aus?

Basti

also es gab mal nen Test, is auch irgendwo der Link hier im forum, wo getestet wurde was schneller ist.

1) echo ''
2) print('')
3) print ''

(glaub ich dass es die 3 Varianten waren)

und rauskam --> schnellste 1), dann 3) und dann 2). Aber das alles spielt sich im ms bereich ab also net wirklich tragisch was man nimmt.

Links:
http://www.drweb.de/php/php_schneller_gemacht.shtml