[PHP] verlies van data

[Verwijderd]

ik had even een vraagje over het verlies van data buiten een while-loop.

code is als volgt:

PHP:

$sql = "SELECT * FROM winkel_orders WHERE ordernummer='".$_GET["ordernummer"];
$result = mysql_query($sql,$db);
while($row=mysql_fetch_array($result)){
  echo "1. Contract: ".$row["winkel_contract"]."<br />";
}
echo "2. Contract: ".$row["winkel_contract"]."<br />";

dit heeft als resultaat:
1. Contract: testcontract
2. Contract:

Ik vroeg mij af hoe dit kwam? De while loop haalt maar 1 rij op en de array $row wordt nergens gereset.

bij voorbaat dank!

[RobIII]

Ik weet geen fluit van PHP, maar is de scope van $row niet enkel binnen de while geldig gezien je 'm daar 'declareert' (of: voor het eerst gebruikt)?

Oh, en zowieso wordt bij de laatste iteratie $row natuurlijk "false" omdat mysql_fetch_array dan geen resultaten meer geeft.

quote: http://de.php.net/mysql_fetch_array

Returns an array of strings that corresponds to the fetched row, or FALSE if there are no more rows.

En nu weet je ook waarom ik geen trek heb me in PHP te verdiepen

[ Voor 64% gewijzigd door RobIII op 16-05-2007 10:13 ]

[Koppensneller]

Check dit even, een stuk op PHP.net over de scope van variabelen:

http://us2.php.net/variables.scope

[Verwijderd]

Je doet de while sluiten met een }

In de while declareer je em en dan sluit je
Daarna zal PHP niet meer weten wat die var inhoud en geeft dus nix weer

[Gonadan]

Bij elke iteratie van de loop wordt $row opnieuw gedeclareerd.
De $row die je gebruikt na de loop is dus de laatste die je gedeclareerd hebt.

De scope zou het niet mogen zijn omdat die per functie gaat, en niet per loop. (foreach apart geval)

De row wordt echter wel gereset. De functie mysql_fetch_array faalt namelijk.
Dus het 'falend resultaat' wordt in $row gezet.

[ Voor 21% gewijzigd door Gonadan op 16-05-2007 10:10 ]

[Martinspire]

Inderdaad, die eerste regel haalt ie uit die database ( uit $row )
En vervolgens sluit je hem af ( } ) waarna je dus een lege waarde krijgt.

Misschien handig om wat tutorials door te lezen? ( phphulp.nl of phpfreakz.nl ?)

[CodeCaster]

Van dit:

PHP:

while($row=mysql_fetch_array($result)){

Moet hij dit:

PHP:

$row=mysql_fetch_array($result)

toch echt twee keer uitvoeren, de eerste keer staat je testcontract erin, de tweede keer levert mysql_fetch_array false terug (en wordt $row dus ook false), en zal er uit de while-loop gesprongen worden.

Met reset() kun je de resultset weer op 0 zetten trouwens, hierdoor kun je er nogmaals doorheen lopen.

[ Voor 3% gewijzigd door CodeCaster op 16-05-2007 10:11 ]

[MueR]

Allereerst kan deze query niet uitgevoerd worden, omdat je een ' niet afsluit, bij je WHERE statement. Ook is deze query nogal gevoelig voor SQL injection. Vertrouw nooit data ingevoerd door gebruikers. Je zou die GET variabele bijvoorbeeld eens door intval($_GET['ordernummer']) kunnen laten omzetten naar een echt nummer.

Als je toch maar 1 rij verwacht, gebruik dan LIMIT 1 aan het einde van je query, zodat MySQL stopt met zoeken nadat het 1 regel heeft gevonden. En waarom de while loop? Allemaal extra laadtijd die je kan besparen.

Verder, zie de 2 hierboven over scope.

[Verwijderd]

Dank voor de reacties het is mij helemaal duidelijk!

[doxick]

mooi zeg dit topic:
de eerste die het goed heeft is de enige die geen php zegt te kunnen =]
A) het heeft niks te maken met de scope
while ($row = mysql_fetch_array( $result )); werkt ook gewoon terwijl hij niet wordt afgesloten door }

Waar ligt het aan:
while ($row = mysql_fetch_array( $result ))
de eerste keer is er een resultaat dus komt dit resultaat in $row te staan
de tweede keer is er geen resultaat, dus false. dat is ook het moment waarmee de condition van de while loop verandert in false (immers: $row = mysql_fetch_array( $result ) = false)

niks geen scope, gewoon een return value van de mysql_fetch_array

probeer anders maar:

PHP:

<?
    while ($i < 5)
    {
        $a = "blaat";
        $i++;
    }
    echo $a;
?>

[Gonadan]

doxick schreef op vrijdag 18 mei 2007 @ 04:46:
mooi zeg dit topic:
de eerste die het goed heeft is de enige die geen php zegt te kunnen =]

Al heeft hij dat er wel achteraf bij ge-edit.
Maar hij heeft gelijk.

Een leuk punt want aansluit op het scope verhaal is het volgende:
Bij het gebruik van een foreach om door een array te lopen wordt er een kopie van het array gebruikt.

Note: Unless the array is referenced, foreach operates on a copy of the specified array and not the array itself. Therefore, the array pointer is not modified as with the each() construct, and changes to the array element returned are not reflected in the original array. However, the internal pointer of the original array is advanced with the processing of the array. Assuming the foreach loop runs to completion, the array's internal pointer will be at the end of the array.

[Grijze Vos]

RobIII schreef op woensdag 16 mei 2007 @ 10:06:
En nu weet je ook waarom ik geen trek heb me in PHP te verdiepen

Nee, want while(!rs.eof) is niet precies hetzelfde ofzo.

[pistole]

Grijze Vos schreef op vrijdag 18 mei 2007 @ 12:04:
[...]

Nee, want while(!rs.eof) is niet precies hetzelfde ofzo.

Ik denk dat je RobIII niet helemaal hebt begrepen (iets met strings in een resultset enzo)

[Grijze Vos]

pistole schreef op vrijdag 18 mei 2007 @ 12:09:
[...]

Ik denk dat je RobIII niet helemaal hebt begrepen (iets met strings in een resultset enzo)

die functie geeft gewoon een pointer terug naar een record, en dat blijft ie doen totdat de recordset leeg is, dan geeft ie null (false) terug. Dat die records arrays van strings zijn maakt echt geen ene ruk uit, PHP is dynamically typed.

Zijn verkapte flame is gewoon op hetzelfde niveau als "ik heb een hekel aan M$".
Je mag best een hekel hebben aan PHP, maar persoonlijk hoef ik dat niet in every other PHP topic op dit forum te lezen eigenlijk.

[ Voor 11% gewijzigd door Grijze Vos op 18-05-2007 12:18 ]

[Creepy]

Misschien begrijp ik je verkeerd maar RobIII bedoelt naar mijn idee het mixed resulttype (en niet de array van Strings). En naar mijn idee begrijpelijk als iemand dit vies vindt.

En grappig detail: null is niet hetzelfde als false in PHP.(http://www.php.net/manual/en/language.types.null.php). De functie geeft dan ook FALSE terug en geen null.

PHP:

if (null == false) {
        echo "null == false\n";
}

if (null === false) {
        echo "null === false";
}

Geeft wel "null == false" en niet "null === false". Het is alleen in PHP zo dat NULL naar false wordt geconverteerd indien nodig zodat het in dit geval niet uitmaakt of er nu null of false wordt teruggegeven.

[RobIII]

Grijze Vos schreef op vrijdag 18 mei 2007 @ 12:04:
[...]

Nee, want while(!rs.eof) is niet precies hetzelfde ofzo.

Nee, want .eof geeft of true of false en niet een array van strings of false

En het is verder geen verkapte PHP flame, helemaal niet. Ik zeg alleen dat mijn voorkeur ergens anders ligt. Niet meer en niet minder.

Grijze Vos schreef op vrijdag 18 mei 2007 @ 12:13:
[...]
Je mag best een hekel hebben aan PHP, maar persoonlijk hoef ik dat niet in every other PHP topic op dit forum te lezen eigenlijk.

Granted, in de meeste PHP topics waarin ik post (als ik er al in post ) vermeld ik even dat ik er geen zak van weet; maar dat is puur omdat het kan zijn omdat ik dan onzin sta uit te kramen. Ik weet niet waar jij vandaan haalt dat er in "every other PHP topic" geflamed wordt

[ Voor 39% gewijzigd door RobIII op 18-05-2007 14:41 ]

[P.O. Box]

RobIII schreef op vrijdag 18 mei 2007 @ 14:39:
[...]

Nee, want .eof geeft of true of false en niet een array van strings of false

EOF geeft ook alleen aan of de recordset End Of File is.... mysql_fetch_array() heeft, zoals je uit de naam al kunt afleiden, een hele andere bedoeling....
het is dus appels en peren vergelijken...

[ Voor 6% gewijzigd door P.O. Box op 18-05-2007 14:44 ]

[Grijze Vos]

RobIII schreef op vrijdag 18 mei 2007 @ 14:39:
[...]

Nee, want .eof geeft of true of false en niet een array van strings of false

En het is verder geen verkapte PHP flame, helemaal niet. Ik zeg alleen dat mijn voorkeur ergens anders ligt. Niet meer en niet minder.

Komt gewoon op hetzelfde neer als een pointer list. Of je geeft een pointer, of null.
Dat ze hier `null` hebben geimplementeerd als `false`, is wellicht niet handig, maar het verandert niks aan de werkwijze.

Taal-filosofische debatten en voorkeuren voor andere talen hoeven wat mij betreft niet telkens in een php topic aan bod te komen. Ik stoor me gewoon aan het feit dat er te vaak in een php topic vermeldt moet worden dat de betreffende poster liever geen PHP gebruikt want <vul reden in>.

[Wim-Bart]

Waarom niet:

PHP:

$ordernummer= (isset($_GET["ordernummer"])?$_GET["ordernummer"]:-1); 
$sql = "SELECT `winkel_contract` FROM `winkel_orders` WHERE ordernummer='".$ordernummer."';";
unset($result);
$result = mysql_query($sql,$db); 

$rows= mysql_num_rows($result);
$row= array();

if ($rows>0)
{
  for($contract=0;$contract<$rows;$contract++)
  {
    $row= mysql_fetch_array($result);
    echo ($contract+1)." Contract: ".$row[winkel_contract]."<br />";
  } 
}
else
{
  echo "Sorry er zijn geen contracten.";
}

[Wim-Bart]

RobIII schreef op vrijdag 18 mei 2007 @ 14:39:
[...]

Nee, want .eof geeft of true of false en niet een array van strings of false
[...]

Persoonlijk vindt ik dat altijd vieze aannames. .eof is een leuke functie maar in asp een volledig lugubere functie. Heb je een lege tabel dan moet je zowel .eof en .bof testen. Doe je dat niet goed dan zijn de resultaten soms heel raar en zoek je uren naar een bug. Voor mij zou code er als volgt uit moeten zien (in pseudocode):

code:

Open tabel;
als tabel==open dan
(
  voer query uit;
  als query==succes dan
  (
    bepaal aantal records in query;
    als aantal records>0 dan
    (
       zet iteratie op 0;
       herhaal zolang iteratie<aantal records
       (
          lees record;
          als record goed gelezen
          (
            verwerk de record......................
          )
          anders
          (
             lees fout exceptie;
          )
       )
    )
   anders
   (
     Meld dat er 0 records zijn of 0 records exceptie;
   )
  )
  anders
  (
    query fout exceptie;
  )
) anders (
  tabel niet open exceptie;
)

Op deze wijze vang je heel wat ellende af. Bijvoorbeeld, er zit een memory leak in een (ODBC) driver en je test op .eof. Nu kan het zijn dat je itereerd tot .eof en er blind van uitgaan dat je leesacties altijd goed gaan, dan zit je opeens met een huge probleem, door voldoende testen in te bouwen ondervang je heel wat problemen waar je geen invloed op hebt. Maar ieder zijn eigen idee hierover natuurlijk. Zelf merk ik dat logica ongeveer 20% van de code is maar goede foutafhandeling 80% is. En met de juiste plaatsing van testen kost het weinig performance.

[PrisonerOfPain]

nevermind

[ Voor 92% gewijzigd door PrisonerOfPain op 18-05-2007 15:54 ]

[pistole]

Wim-Bart schreef op vrijdag 18 mei 2007 @ 15:40:
[...]


Persoonlijk vindt ik dat altijd vieze aannames. .eof is een leuke functie maar in asp een volledig lugubere functie. Heb je een lege tabel dan moet je zowel .eof en .bof testen. Doe je dat niet goed dan zijn de resultaten soms heel raar en zoek je uren naar een bug. Voor mij zou code er als volgt uit moeten zien (in pseudocode):

(...)

Op deze wijze vang je heel wat ellende af. Bijvoorbeeld, er zit een memory leak in een (ODBC) driver en je test op .eof. Nu kan het zijn dat je itereerd tot .eof en er blind van uitgaan dat je leesacties altijd goed gaan, dan zit je opeens met een huge probleem, door voldoende testen in te bouwen ondervang je heel wat problemen waar je geen invloed op hebt. Maar ieder zijn eigen idee hierover natuurlijk. Zelf merk ik dat logica ongeveer 20% van de code is maar goede foutafhandeling 80% is. En met de juiste plaatsing van testen kost het weinig performance.

Hoewel het zwaar offtopic is, heb ik in mijn inmiddels 10 jaar .asp ervaring nooit last gehad van deze memory leak en noodzaak gezien om op zowel eof als bof te checken...

[ Voor 21% gewijzigd door pistole op 18-05-2007 20:38 ]