[php]Return Array in funtie

[Speedener]

Ik ben aan het stoeien met funties, ff de does en dont's aan het uitzoeken.

Nu vroeg ik me af.

Ik heb deze functie:

PHP:

// MySQL fetch array
    function db_array($database_query)
    {
        $database_result = mysql_query($database_query) or die(mysql_error());        
        $database_array = mysql_fetch_array($database_result, MYSQL_ASSOC);
        return $database_array;
    }

En er komt dus een array uit.
Als ik het nu probeer weer te geven dmv een while, dan komt er een loop van hier tot tokio.

Hoe kan ik dat oplossen?

[D2k]

ik gok dat je je returnwaarde niet goed afvangt?
wat is je functie aanroep?

[Speedener]

PHP:

    while($row = db_array("SELECT * FROM frontpage ORDER BY posttime DESC LIMIT 10"))
    {
        echo $row['id'] . '<br>';
    }

[ Voor 17% gewijzigd door Speedener op 14-01-2003 10:05 ]

[D2k]

hmmz
en wat levert print_r($row); op?

[Verwijderd]

ja gek he dat ie oneindig loopt.... kijk eens wat er gebeurd!

[D2k]

Verwijderd schreef op 14 January 2003 @ 10:09:
ja gek he dat ie oneindig loopt.... kijk eens wat er gebeurd!

ik ziet um

* D2k needs coffee

[Janoz]

Let je wel ff op dat de teruggegeven array alleen maar een lijstje van de kolomwaarden van 1 row is? Als je vervolgens je result weer met een neiuwe query vult heb je natuurlijk weer een nieuwe result set. Vandaar dat dat bovenstaande lusje gewoon een oneindige lijst met kopienen van de eerste row weergeeft..

[Speedener]

Hij vraagt hem elke keer aan ofzo?

edit: hoe los ik dat op

[ Voor 32% gewijzigd door Speedener op 14-01-2003 10:12 ]

[Verwijderd]

Speedener schreef op 14 januari 2003 @ 10:05:

PHP:

    while($row = db_array("SELECT * FROM frontpage ORDER BY posttime DESC LIMIT 10"))
    {
        echo $row['id'] . '<br>';
    }

zoiets?

PHP:

    $arr = db_array("SELECT * FROM frontpage ORDER BY posttime DESC LIMIT 10");
    while($row = each($arr)))
    {
        echo $row['id'] . '<br>';
    }

[Speedener]

Verwijderd schreef op 14 januari 2003 @ 10:14:
[...]


zoiets?

PHP:

    $arr = db_array("SELECT * FROM frontpage ORDER BY posttime DESC LIMIT 10");
    while($row = each($arr)))
    {
        echo $row['id'] . '<br>';
    }

Nee dat is het ook niet want dan toont hij alleen maar Record 1.

[Verwijderd]

owja sorry niet echt naar je functie gekeken.
Komt omdat je een array krijgt per record. Normaar lus je niet door een array heen, maar door een resultset en krijg je een array terug.
In dit geval zul je in je functie denk ik door die resultset heen moeten gaan. Dat in een array zetten en die returnen.

vind die or die niet zo netjes in je functie, kun je dan niet beter false returnen?

[[ash]]

ehhh... misschien je array 'global' maken

[Speedener]

Verwijderd schreef op 14 januari 2003 @ 10:23:
owja sorry niet echt naar je functie gekeken.
Komt omdat je een array krijgt per record. Normaar lus je niet door een array heen, maar door een resultset en krijg je een array terug.
In dit geval zul je in je functie denk ik door die resultset heen moeten gaan. Dat in een array zetten en die returnen.

vind die or die niet zo netjes in je functie, kun je dan niet beter false returnen?

heb je wat documentatie voor dat result set? het ie geen phpfunctie toch?

[LuCarD]

Verwijderd schreef op 14 January 2003 @ 10:14:
[...]


zoiets?

PHP:

    $arr = db_array("SELECT * FROM frontpage ORDER BY posttime DESC LIMIT 10");
    while($row = each($arr)))
    {
        echo $row['id'] . '<br>';
    }

try this

PHP:

    $arr = db_array("SELECT * FROM frontpage ORDER BY posttime DESC LIMIT 10");
    foreach ($arr as $row)
    {
        echo $row['id'] . '<br>';
    }

[Bosmonster]

Speedener schreef op 14 januari 2003 @ 10:27:
[...]

heb je wat documentatie voor dat result set? het ie geen phpfunctie toch?

[edit.. heb moeite met deze functie.. wat is het nut!]

Misschien moet je je afvragen wat het nut is van een dergelijke functie.. aangezien het geen dubbel werk voorkomt..

Je kunt misschien een functie maken (al zou een volledige db-class meer op zn plek zijn) om een resultset op te vragen (dus wat een mysql_query() teruggeeft), om errorhandling te centraliseren..

[ Voor 94% gewijzigd door Bosmonster op 14-01-2003 10:37 ]

[Verwijderd]

Speedener schreef op 14 January 2003 @ 10:27:
[...]

heb je wat documentatie voor dat result set? het ie geen phpfunctie toch?

nee het is niet een functie
het is een resultaat. die jij dmv functies aan kan roepen.
Php houdt voor die result set (of record set, wat je het ook wil noemen) een interne pointer bij waar hij is gebleven als je functies als mysql_fetch_array($result) gebruikt.
Omdat jij steeds opnieuw die functie uitvoert zal je steeds weer vooraan beginnen en dus nooit eindigen.

Lees mijn 2e reactie, daar kun je meer mee denk ik

[thomaske]

de fout zit in je functie.. van je hele recordset geef je alleen je eerste record terug..

[edit]
je moet de functionaliteit dus splitsen in 2 functies: een die de recordset aanmaakt, en een die een array van het huidige record teruggeeft.

[ Voor 47% gewijzigd door thomaske op 14-01-2003 10:38 ]

[Speedener]

Het lukt me echt ff niet

heb nu dit in mysql.php:

PHP:

<?php

// MySQL connection maken
    function db_connect($database_host, $database_username, $database_password, $database_databasename)
    {
        $database_connection = mysql_connect($database_host, $database_username, $database_password) or die(mysql_error());
        mysql_select_db($database_databasename, $database_connection);
        return TRUE;
    }

// MySQL query 
    function db_query($database_query)
    {
        $database_result = mysql_query($database_query) or die('<div style="width: 500px"><b>Error:</b><hr>' . mysql_error() . '<br><hr><br><b>Query:</b><hr><pre style="margin: 0px;">' . $database_query . '</pre><hr></div>');
        
        return $database_result;
    }
    
// MySQL fetch array
    function db_array($database_result)
    {
        //$count = 0;
        //if($count == 0){ $database_result = db_query($database_query); $count++; }
        //$database_array = ;
        return mysql_fetch_array($database_result, MYSQL_ASSOC);
    }
?>

en dit in mijn test.php:

PHP:

    require_once('mysql.php');
    db_connect('localhost', 'Speedener', 'xxx', 'xxx');

    $result = db_query("SELECT * FROM counter_name_country");
    $array = db_array($result);
    
    while($a = each($array))
    {
        echo $a['country_id'];
        echo '<br>';
    }

[Verwijderd]

luisteren naar woeitje je krijgt na een query, als die gelukt is, een resultset terug en daar kun je per row dmv mysql_fetch_array($resultset,$row) arrays uithalen.

[Verwijderd]

db_array geeft alleen het eerste record terug, daar gaat het fout!
het heeft ook geen zin om dit te doen:

PHP:

    $result = db_query("SELECT * FROM counter_name_country"); 
    $array = db_array($result);

daar kopieer je gewoon array 1 in array 2

loop in je functie door je result heen met while($record=mysql_fetch_array())
vul de array in die functie en return die.
Dan gaat het wel goed.

Qua performance......... is dat nie een goeie methode want je loopt 2 keer door je resultset terwijl je maar 1 van de 2 keer er echt iets mee doet.

[ Voor 2% gewijzigd door Verwijderd op 14-01-2003 10:54 . Reden: typo ]

[Speedener]

Dank allen,

Ik ga dus gewoon lekker mysql_fetch_array direct gebruiken

[thomaske]

Een mogelijkheid is ook om gebruikt te maken van een database-class

Lijstje van mogelijke classen

[DiveZoner]

thomaske schreef op 14 January 2003 @ 11:21:
Een mogelijkheid is ook om gebruikt te maken van een database-class

Lijstje van mogelijke classen

kan ook maar zelf bouwen is leuker en leerzamer, toch?