Saya pikir itu terlihat cukup bagus; Saya suka itu. Saya berasumsi secure()
melindungi dari injeksi sql. Saya tidak benar-benar melihat apa pun yang akan saya ubah.
Anda bisa keluarkan tanda kurung kurawal dari for karena satu pernyataannya, tapi itu bukan masalah besar
foreach ($qWhere as $key=>$value)
$fields[] = sprintf("%s = '%s'", $key, secure($value));