My OperaNhững kẻ "xuất chúng"
Friday, December 3, 2010 5:32:24 PM
function getBranchesOfCompany($orgId, $condition = '', $fields = array())
{
$defaultFields = array(
'foo' => 'foo',
'bar' => 'bar'
);
if ($fields) {
$defaultFields = array_merge($defaultFields, $fields);
}
$condtionString = "org_id = $orgId";
if ($condition) {
$condtionString .= " AND " . $condtionString;
}
.....
}
Giờ truyền mảng 2 phần tử, đại diện cho 2 field cho tham số $fields, nó vẫn bị merge với cái default field. Thay vì trả về 2, nó trả về tận 4 fields.
Truyền điều kiện cho tham số $condtion, nó... chẳng có tác dụng gì hết, ngoài việc làm biến $conditionString... bị lặp lại 2 lần
Với hàm tí hon kiểu này mà mất quá nhiều thời gian viết + viết mà cứ khăng khăng nó chạy ko sai đã là tội to rồi. Vậy mà còn chụp cho em cái mũ không chịu phối hợp, thấy nó lỗi thì cứ bảo anh để anh sửa chứ. Vâng, anh là người viết hàm để cho mọi người dùng, chẳng phải anh đẽo cày giữa đường mà đã đc chỉ rõ cần tham số nào, trả về cái gì rồi mà anh vẫn đếch làm được, xong anh ngoác mồm ra đổ lỗi cho người khác ko test hộ anh chỉ vì anh đang bị nói về cái tội trễ deadline đến 2 tuần??? Chưa kể câu SQL của anh còn thuộc dạng ngớ ngẩn nữa cơ, để đến nỗi khi chạy thì gây ra lỗi Fatal Error. Vậy mà khi em thử hỏi anh là anh test kỹ chưa thì anh cười tươi rói, nói là hàm chạy ngon lắm, anh vừa chạy xong.
Vâng, với những gì anh thể hiện & cách anh hành xử. Em thấy anh trình độ cực kỳ hạn chế nhưng lại không biết mình đang ở đâu. Vâng, thực sự anh là 1 trong những kẻ "xuất chúng". Nên nhớ đây là chữ xuất chúng trong ngoặc kép chứ ko phải với nghĩa xuất chúng của tác giả Malcolm Gladwell
Và cơ hội dành cho anh cũng không còn nữa đâu, anh ạ!
(38%)
