Czasami jesteś zajęty i przekazujesz małe zadania młodszym programistom. Ale jeśli nie zwrócisz wystarczającej uwagi, znajdziesz się przy tego rodzaju kodzie w produkcji:
class DivtoggleController extends Zend_Controller_Action {
public function closeAction() {
/* ... code removed for brevity ... */
$req = $this->getRequest();
$formData = $req->getPost();
$d = $formData['div'];
$i = $formData['id'];
$dm = new Model_DivtoggleManager();
$rs = $dm->setDivToggleById($d, $i);
}
}
class Model_DivtoggleManager extends Zend_Db_Table {
public function setDivToggleById($div, $id) {
$result = $this->getAdapter()->query(
"update div_toggle set " . $div . "=1 where id=" . $id
);
}
}
Biorąc pod uwagę, że usunąłem logikę zarządzania uwierzytelnianiem / sesjami dla zwięzłości, kto może mi powiedzieć, jaki może być problem z tą próbką?