Christian Kruse: Wiki-Artikel Reloadsperre

Beitrag lesen

problematische Seite

Hallo Matthias,

Die beiden anderen Methoden funktionieren weiterhin,

Kannst du bitte noch mal über den Teil 1 schauen?

Autowert… urghs. ;-)

$id_check -> bindParam(':BName', $_POST['bname']);
$id_check -> execute();

Ich würde hier lieber die Parameter an execute() übergeben, das ist einfacher lesbar:

$id_check -> execute(array(':BName' => $_POST['bname']));

Das gleiche weiter unten. Weiter:

if(isset($benutzer['ID'])) array_push($fehler,'Dieser Benutzer existiert schon.');

Diese Notation ist gefährlich, siehe goto fail - ich würde immer Klammern nutzen. In dem INSERT hast du einen Quoting-Fehler: "Ich", es müsste aber \"Ich\" sein (SQL embedded in PHP).

Bzgl des Inhalts: ich würde mir die Bemerkung über natürliche Primärschlüssel sparen: es macht den Satz schlechter lesbar, tut hier nichts zur Sache und der Wahrheitsgehalt ist mindestens strittig (um natürliche vs künstliche Primärschlüssel existiert seit Ewigkeiten eine leidenschaftliche Diskussion). Bei den Error-Codes würde ich explizit MySQL erwähnen, vllt im Stile von „so lauten der Error-Code und die entsprechende Fehlermeldung bei MySQL“ oder so.

Ansonsten sieht das gut aus.

LG,
CK

0 48

Wiki-Artikel Reloadsperre

Matthias Apsel
  • php
  • programmiertechnik
  • selfhtml-wiki
  1. 0
    Christian Kruse
    1. 0
      Matthias Apsel
      1. 0
        Christian Kruse
        1. 0
          Matthias Apsel
          1. 0
            Christian Kruse
            1. 0
              Matthias Apsel
              1. 0
                Christian Kruse
                1. 0
                  Matthias Apsel
                  1. 0
                    Christian Kruse
                    1. 0
                      Matthias Apsel
                    2. 0
                      Matthias Apsel
                      1. 0
                        Matthias Apsel
                        1. 0
                          Christian Kruse
                          1. 0
                            Matthias Apsel
                            1. 0
                              Christian Kruse
                              1. 0
                                Matthias Apsel
                                1. 0
                                  Christian Kruse
                                  1. 0
                                    Matthias Apsel
                                    1. 0
                                      Der Martin
                                      1. 1
                                        Auge
                                        1. 0
                                          Christian Kruse
                                    2. 0
                                      Christian Kruse
        2. 0
          Mitleser
          1. 0
            Christian Kruse
    2. 0
      Matthias Apsel
      1. 1
        Christian Kruse
        1. 0
          Matthias Apsel
          1. 1
            Christian Kruse
            1. 0
              Matthias Apsel
              1. 0
                Christian Kruse
                1. 0
                  Matthias Apsel
                  1. 0
                    Christian Kruse
                2. 0
                  Matthias Apsel
                  1. 0
                    Christian Kruse
                    1. 0
                      Matthias Apsel
                3. 0
                  Matthias Apsel
                  1. 0
                    Christian Kruse
                    1. 0
                      Matthias Apsel
                      1. 0
                        Christian Kruse
                        1. 0
                          Matthias Apsel
                        2. 0
                          Matthias Apsel
                          1. 0
                            Christian Kruse
                            1. 0
                              Matthias Apsel
                              1. 0
                                Christian Kruse
                                1. 1
                                  Matthias Apsel
                                  1. 0
                                    Christian Kruse
  2. -1
    pl