Ja, bindParam
binder en parameter til et variabelnavn (reference), ikke en værdi, som manualen siger
.
Der er dog en enklere syntaks til din situation. PDOStatement::execute
kan tage en række værdier.
public function insert($table, $cols, $values){
$placeholder = array();
for ($i = 0; i < count($values); $i++)
$placeholder[] = '?';
$sql = 'INSERT INTO '. $table . ' (`' . implode("`, `", $cols) . '`) ';
$sql.= 'VALUES (' . implode(", ", $placeholder) . ')';
$stmt = $this->dbh->prepare($sql);
$stmt->execute($values);
}