Tôi nghĩ rằng nó trông khá tốt; Tôi thích nó. Tôi giả sử secure()
bảo vệ khỏi tiêm sql. Tôi thực sự không thấy bất cứ điều gì tôi sẽ thay đổi.
Bạn có thể lấy dấu ngoặc nhọn ra khỏi vòng lặp for kể từ một câu lệnh của nó, nhưng điều đó không có gì to tát cả
foreach ($qWhere as $key=>$value)
$fields[] = sprintf("%s = '%s'", $key, secure($value));