Sí, bindParam
vincula un parámetro a un nombre de variable (referencia), no un valor, como dice el manual
.
Sin embargo, hay una sintaxis más simple para su situación. PDOStatement::execute
puede tomar una matriz de valores.
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);
}