Prepared Statements icm. array als input

[Sgrovert]

Sinds een paar dagen ben ik aan het uitzoeken hoe prepared Statements vo SQL werken. Ik ben een simpele functie aan het maken welke een array aan data in een tabel kan toevoegen.

Mijn input ziet er als volgt uit :

PHP:

    //$data moet er als volgt uitzien :
    // array('veldnaam' => array('data type','data'),'veld 2' => array('type','data'),etc)
    //Mogelijke types zijn :
    // i = integer;
    // s = string;
    // d = double;
    // b = blob;
$data = array('field1' => array('s','string'), 'field2' => array('s','value'));
insertData(table',$data);

En de functie zelf :

PHP:

function insertData($tableName,$data) {
    $mysqli = new mysqli("localhost", "userName", "Pass", "dbName");

    $fieldName=array(); //Veld Namen
    $fieldType=array(); //Data Type
    $fieldValue=array(); //Data
    $values=array(); //Generen van ? in Query
    foreach($data as $field => $content) {
        array_push($fieldName,$field);
        array_push($fieldType,$content[0]);
        array_push($fieldValue,$content[1]);
        array_push($values,'?');
    }
    $preparedStatement='INSERT INTO `'.$tableName.'` (`'.implode($fieldName,'`,`').'`) values ('.implode($values,',').')';
    if ($stmt = $mysqli->prepare("$preparedStatement")) {
        /* Bind our params */

        $stmt->bind_param(implode($fieldType), $fieldValue);
    
    } else {
        echo'Error 473';
    }
}

Deze code lever mij de volgende warning op : "PHP Warning: Parameter 2 to mysqli_stmt::bind_param() expected to be a reference, value given in .."

Ik heb een redelijk vermoeden waarom ik de warning krijg, maar hoe los ik dit netjes op? De grootte van mijn array is namelijk variabel.

[Grijze Vos]

Dan genereer je een variabel aantal queries.
Het punt van parametrized queries is dat je je query niet dynamisch met concatenatie opbouwt. Iets dat je hier dus precies wel aan het doen bent..

[Merethil]

Is het niet zo simpel als:

PHP:

for($i = 0 ; $i < count($fieldType) ; $i++)
{
    $stmt->bind_param($fieldType[$i], $fieldValue[$i]);
}

Of is dit niet wat je zoekt?

[Janoz]

Waarom maak je weer een nieuwe wrapper om iets dat eigenlijk al een wrapper is? Je voegt een stuk eigen syntax toe welke nauwelijks winst oplevert tov gewoon rechtstreeks de prepared statement code uitschrijven. Voegt een berg potentiele bugs toe, en maakt het onderhoud voor anderen (want eigen wiel), maar ook jezelf (niet gevalideerde string configuratie welke je in een zelf bedacht arrayboom probeert te plaatsen) een stuk lastiger.

In mijn opinie ben je vanuit de verkeerde kant aan het generaliseren. Ga niet proberen je database toegang nog makkelijker te maken, want dat kan eigenlijk niet zonder generiekheid te verliezen. Redeneer vanuit je eigen entiteiten. Jij wil een persoon opslaan? Maak dan een persoon methode welke gewoon de parameters mee krijgt die je nodig hebt, zoals bijvoorbeeld naam enzo (of maak een persoon class welke alle benodigde gegevens erin heeft zitten)

[Sgrovert]

Allemaal bedankt voor de reacties. Ik heb besloten om voor de optie van Janoz te gaan. Losse Methodes zijn op de lange termijn inderdaad makkelijker.

[Barryvdh]

Je kan ook naar een ORM kijken, bijvoorbeeld Doctrine of Eloquent van Laravel (Ook als los project), dan kan je heel simpel een model maken en je data aanpassen.

Voorbeeldjes:
Fluent Query Builder

PHP:

$user = DB::table('users')->first();
DB::table('users')->insert(array('email' => '[e-mail]'));
$affected = DB::table('users')
    ->where('id', '=', 1)
    ->update(array('email' => '[e-mail]'));

Eloquent ORM

PHP:

$user = User::find(1);
//Of voor een nieuwe user
$user = new User;

$user->email = '[e-mail]';
$user->password = 'new_secret';

$user->save();

[RobIII]

Even een kick naar PRG (Waar hoort mijn topic?)
WEB » PRG