isset und versteckte Formularfelder

[antman]

Hallo Board,

also ich habe ein verstecktes Formularfeld auf value="1" gesetzt.
Ich frage es nun mit if(isset($_POST['hiddenfield']) ab. Zeigt aber keine Wirkung.

Es soll gestet werden, welches Formular von einem Skript verarbeitet wird. Woran kann das liegen ?

[mrhappiness]

code?
print_r($_POST);

[Kropff]

if(isset($_POST['hiddenfield'])

ist die falsche bedingung. sobald du ein formularelement hast trifft isset immer zu auch wenn kein wert angebene ist. prüfe mit

if(!empty($_POST['hiddenfield']) 

oder mit

if(($_POST['hiddenfield']) 

gruss
peter

[antman]

OKAI!

Thx!

[antman]

Aber wie kann man testen, ob eine mysql Abfrage ein Ergebnis geliefert hat? if(issset($row))? Oder einfach if($row). Das ist ja nun ein Array.

[mrhappiness]

mysql_num_rows ist kaputt bei dir?

[antman]

Nein ich benutze die PEAR DB Klasse:
http://pear.php.net/manual/en/packag...lt.numrows.php

Ich möchte nicht jedes mal abfragen if($res->numRows() == 1);
Gibt es keine Funktion die false oder true zurückliefert ?

[mrhappiness]

Original geschrieben von antman
nicht jedes mal abfragen

das heißt?

einmal pro abfrage musst du ja zwangsläufig abfragen und das ergebnis kannst du dir dann ja zur not in einer variable speichern oder?

[antman]

Stimmt auch wieder.

[antman]

Also ich habe mal eine Frage:
Ich habe eine Funktion geschrieben, die fügt einer Tabelle(Warenkorb) ein Produkt hinzu oder ändert die Menge eines bereits existierenden.

    function addToCard($anzahl, $artikelnummer)
    { 
        for($i=0; $i < count($artikelnummer); $i++) {
            if (is_int($anzahl) && is_int($artikelnummer)) {
                $res =& $this->db->query("
                SELECT *
        FROM   warenkorb
                WHERE (sid = '".$this->sid."') 
                AND   (artikelnummer = '".$artikelnummer[$i]."')");

                $row =& $res->fetchRow(DB_FETCHMODE_ASSOC);
                if($res->numRows() == 1) { // item already in db
                    $row['anzahl'] = $anzahl[$i];     // if no change       
                    $neuAnzahl = $row['anzahl'];      // don't change value
                    $res =& $this->db->query("
                    UPDATE warenkorb
                SET anzahl = '".$neuAnzahl."'
            WHERE (sid = '".$this->sid."') 
                AND (artikelnummer = '".$artikelnummer[$i]."')"); 
                } elseif ($anzahl[$i] != 0 && $anzahl[$i] != "") {        // item not yet in db
                                                                    $res =& $this->db->query("
                INSERT INTO warenkorb (sid, anzahl, artikelnummer)
        VALUES('".$this->sid."', '".$anzahl[$i]."', '".$artikelnummer[$i]."')");
                } // end elseif
            } // end for
           } // end addToCard()
    } 

Sehen erfahrene Programmierer hier "komische" Programmiertechniken ? Würde mich über Verbesserungsvorschläge freuen.

[mrhappiness]

Original geschrieben von antman
Also ich habe mal eine Frage:
Ich habe eine Funktion geschrieben, die fügt einer Tabelle(Warenkorb) ein Produkt hinzu oder ändert die Menge eines bereits existierenden.

toll, aber warum postest du nicht die, sondern irgendwas komisches?

du hast eine schleife, die du so oft durchläufst, wie du artikel im array $artikelnummer hast und dann prüfst du, ob $artikelnummer vom typ integer ist?

und warum liest du nicht nur den wert im feld anzahl aus der datenbank sondern komplett alles?

und warum hast du so viele klammern bei deinem WHERE?

und wenn die funktion für mehrere artikel auf einmal genutzt wird, warum liest du dann nicht die anzahl aller artikel aus dem array auf einmal aus?

[antman]

toll, aber warum postest du nicht die, sondern irgendwas komisches?

???

du hast eine schleife, die du so oft durchläufst, wie du artikel im array $artikelnummer hast und dann prüfst du, ob $artikelnummer vom typ integer ist?

if (is_int($anzahl) && is_int($artikelnummer)) { 

Das muss in die for Schleife, weil doch ein Array von Arrays übergeben wird, oder was meinst du ? Muss doch prüfen, ob korrekte Werte an die Funktion übergeben werden.

und warum liest du nicht nur den wert im feld anzahl aus der datenbank sondern komplett alles?

Hmm, mache ich immer so. Der einfachheit halber. Hat das große Nachteile ?

und warum hast du so viele klammern bei deinem WHERE?

Der Übersicht wegen. Besser weglassen ?

und wenn die funktion für mehrere artikel auf einmal genutzt wird, warum liest du dann nicht die anzahl aller artikel aus dem array auf einmal aus?

Verstehe ich nicht.

[mrhappiness]

Original geschrieben von antman
???

genau

if (is_int($anzahl) && is_int($artikelnummer)) { 

Das muss in die for Schleife, weil doch ein Array von Arrays übergeben wird, oder was meinst du ? Muss doch prüfen, ob korrekte Werte an die Funktion übergeben werden.

count($artikelnummer);
liefert dir zurück, wieviele werte im array $artikelnummer stehen. wenn $artikelnummer keinarray ist, wird 1 zurückgeliefert

is_int($artikelnummer)
liefert immer dann, wenn $artikelnummer keine ganze zahl ist (z.b. wenn 4artikelnummer ein array ist)

du sagst also:
gehe das array X eintrag für eintrag durch
wenn X (was ein array ist) eine ganze zahl ist, dann mache was
funktioniert das wirklich für mehr als eine artikelnummer?

Hmm, mache ich immer so. Der einfachheit halber. Hat das große Nachteile ?

jein, mach doch einfach SELECT anzahl

Der Übersicht wegen. Besser weglassen ?

deine sache, ich find sie überflüssig

Verstehe ich nicht.

warum fragst du nicht mit einem SELECT ab, welchen artikel aus dem array der kunde schon wie oft in seinem warenkorb hat?
das wäre eine abfrage anstatt 10 (bei 10 artikelnummern)

[antman]

Hallo,

ich werde die Funktion gleich mal so abändern, wie du es vorgeschlagen hast und dann die abgeänderte Funktion posten.

[antman]

du sagst also:
gehe das array X eintrag für eintrag durch
wenn X (was ein array ist) eine ganze zahl ist, dann mache was
funktioniert das wirklich für mehr als eine artikelnummer?

Das stimmt nicht. Nicht "wenn x eine ganze Zahl ist" sondern x[1],x[2], etc.

Dern letzten Punkt verstehe ich nicht. Wie soll das mit einer Abfrage gehen ?