Was sagt Ihr zu diesem Beispiel? Kann man das noch verwenden, oder ist es veraltet?

  • Hallo,


    das hat mir jemand zugesendet, darum möchte ich mal fragen, ist das noch der heute Stand, das man so programmiert?


    Zitat

    [/code]

  • Als erstes Thema Sicherheit:
    Benutzereingaben maskieren: mysqli_real_escape_string (http://php.net/manual/de/mysqli.real-escape-string.php)
    hier am Bsp:

    PHP
    foreach($felder as $feld)
    {
        $sql .= mysqli_real_escape_string($_POST[$feld]).", ";
    }


    Sonst sind Sql-Injections möglich!


    Prüfung:

    PHP
    if(empty($_POST["angemeldetesgewerbe"]) OR empty($_POST["plz"]) OR empty($_POST["ort"]) OR empty($_POST["email"]) OR empty($_POST["stundenlohn"]) OR empty($_POST["handy"]))


    hier passiert keine wirkliche Prüfung. Es gehört eigtl mehr dazu.
    Bsp:
    $_POST["plz"] ist numerisch. Es lohnt sich also schon mal zu prüfen, ob:

    PHP
    if(is_numeric($_POST["plz"])){ ... }


    BTW:
    Postleitzahlen fangen auch schon mal mit einer 0 an. Außerdem bekommt man via $_POST und $_GET immer Strings, auch wenn im Eingabefeld eine Zahl eingegeben wurde.
    Heißt: Eingabefeld 123 führt zu $_POST/$_GET '123'.
    Die Funktion is_numeric() (oder besser gesagt PHP) kommt damit allerdings klar. Es wird versucht den String '123' als Zahl zu verwenden, was auch funktioniert.
    So ist zum Bsp:

    PHP
    if(is_numeric(02828))


    wahr, also true.
    (Man kann hier auch noch die Länge der PLZ prüfen.)


    Bsp eMail:

    PHP
    if(empty($email) === false and filter_var($email, FILTER_VALIDATE_EMAIL) === true){ ... }


    Usw. Am besten immer nach den Schlagworten via Suchmaschiene suchen. Zb "php email prüfen".


    Was den Rest betrifft: alles Ansichtssache.
    Hier zB wurde das Erstelen des Inputfelds über eine Funktion erledigt:

    HTML
    <td><?php eingabe("selbststaendigseit"); ?></td>


    So lange man Redundanz (zu viel sich wiederholender Code) vermeidet - ok.

  • Gibt nicht mehr viel zu sagen, aber warum wird das Input feld über ne Funktion erstellt, wenn man trotzdem noch den ganzen Rest schreibt..?!


    Ansonnsten .. Design und Struktur miteinander zu mischen ist veraltet

    • (Zeile 9) cellspacing="[...]" cellpadding="[...]" border="[...]"
    • (Zeile 83) colspan="[..]" und <center>

    Sowas regelt man über CSS..

Jetzt mitmachen!

Sie haben noch kein Benutzerkonto auf unserer Seite? Registrieren Sie sich kostenlos und nehmen Sie an unserer Community teil!