Probleme mit ein Formular

blitz

Hallo,

ich möchte ein fertiges Skript anpassen und hier eine zusätzliche Checkbox einbauen, dieses klappte auch noch nun sollte aber geprüft werden ob der Hacken in der Box gestzt ist oder nicht.
Irgendwie bekomme ich dieses nicht hin, ich habe die Datei mal angehangen und hier ist ein link zu der Exchange: http://www.power-off-hosting.de/skripte/exchange/anmelden.php

Ich habe am Skript in den Zeilen 315,316 und 317 die Checkbox eingebaut nun fehlt halt die Abfrage.

Wer kann einen Anfänger da helfen?

Torty

Nunja, ich würde sagen -> "siehe dazu Dein bisheriger Thread"

Das sollten wir schon hinbekommen.

Kleiner Tip von meiner Seite:
Vermeide HTML-Code per Echo-Anweisung auszugeben, denn dadurch geht dir das Syntax-Highlighting
in den meisten Editoren verloren und der Code wird schlechter pflegbar.
Besser wäre dann eine PHP-Unterbrechung alla ?> ... <? - dadurch bleibt der Code viel besser lesbar.

6_freddy

Hi,
wäre es nicht besser, die Überprüfung ebenfalls mit PHP zu machen? Es wäre sonst möglich, dass, bei deaktiviertem Javascript, ungeprüfte Checkboxes übermittelt werden.
MfG
6_freddy

Torty

Ok, klar geht das, aber dadurch muss das Formular erst abgesandt - dann überprüft und notfalls das Formular (mit allen Daten)
erneut übertragen werden. Wen das nicht stört, kann es gern auch mit PHP machen.

Die Checkbox mit name="check1" ist dann unter PHP als $check1 ansprechbar. Hat diese Variable den Wert "checked",
so wurde der Hacken gesetzt.

6_freddy

Hi,

Zitat

Es wäre sonst möglich, dass, bei deaktiviertem Javascript, ungeprüfte Checkboxes übermittelt werden.

Das kannste damit eben verhindern. Und - ja, das muss über den Server laufen, aber das bietet doch auch weitere Vorteile, z. B. in Bezug auf die auszugebende Meldung, etc.
MfG
6_freddy

blitz

Das war ein fertiges Skript, ich passe es nur meinen bedürfnissen an...

ein neues Thema habe ich hier gestartet da ich nach reiflicher überlegung meine hier passt es besser rein, letzlich ist es ja ein php-skript.

ohh ich sehe gerade das ihr schon einen ansatz habt... nur wie muß es nun fertig aussehen... ich komme einfach nicht weiter damit

Rocco

Zitat von Torty

Besser wäre dann eine PHP-Unterbrechung alla ?> ... <? - dadurch bleibt der Code viel besser lesbar.

ganz meiner meinung, doch das ist gegen den sinn des "clean-code-gedanken", den einige kleinliche leute pflegen.... nebenbei mal angemerkt

Torty

Zum Thema "clean code PHP" hab ich mal folgenden Link gefunden: Cleaner PHP-Code (eng)

Nunja, ich bin aber schon der Meinung, dass getrennte PHP- & HTML-Sequenzen lesbarer sind.
Hier kleines Beispiel:

Code

// Variante 1
echo "<table width=\"100%\" border=\"0\" cellspacing=\"0\" cellpadding=\"0\">


// Variante 2
?>
  <table width="100%" border="0" cellspacing="0" cellpadding="0">
<?

Weiterer Nachteil der Variante 1 ist, dass die Editoren alles innerhalb der ""-Zeichen der echo-Anweisung grau (zumindest einfarbig)
darstellen und das nicht zur Lesbarkeit des Codes beiträgt.

Aber da muss auch jeder die für ihn passende Version verwenden. Ein Stückweit ist dies halt auch Geschmackssache.

blitz

Ich glaube es wird nun am Thema vorbei geredet, ich bruahce eine Lösung für die Checkbox das Skript war fertig Programmiert und ich will es nur um die Checkbox erweitern.

MrJava

Zitat von Rocco

ganz meiner meinung, doch das ist gegen den sinn des "clean-code-gedanken", den einige kleinliche leute pflegen.... nebenbei mal angemerkt

Clean-Code Gedanke?
Ich seh da kein Vorteil drin die komplette Ausgabe in PhP-Code zu packen.

Da bevorzuge ich eher das MVC-Modell. Also die Trennung von
persistenten Daten, der grafischen Darstellung und der Steuerungsschicht.
Aber das ist nicht die Stärke von PhP.

Tags