我有三张桌子,
Business:
id
name
Office:
id
name
business_id
Employee:
id
name
office_id
员工将office_id作为外键,办公室将business_id作为外键。
我有一个与每个域对象/实体相关的域对象/实体以及与每个域对象相关的数据库映射器对象。
现在,当我被公司名称,办公室名称和员工姓名提供时,如何插入新的员工?
最初我认为逻辑应该是这样的:
$businessMapper = new businessMapper();
$business = $businessMapper->findBusinessByName($business_name);
if ($business == false) {
$business = new businessEntity(array(
'businessName' => $business_name,
));
$businessMapper->save($business);
}
$officeMapper = new officeMapper();
$office = $officeMapper->getOfficeByName($office_name, $business);
.......etc.......
但后来我才意识到,如果我必须保存一项新业务,我就无法拥有办公室或员工,因此尝试获取它们是一种浪费的查询。所以我想我应该创建一个if / else结构。
get business entity by business_name
if ($business == false) {
create business entity
save business entity
create office entity
save office entity
create employee entity
save employee entity
} else {
get office entity by office_name + business_id
if (office == false) {
create office entity
save office entity
create employee entity
save employee entity
} else {
......etc......
}
}
但是有太多重复的逻辑,它非常不可扩展/脏。
那应该怎样实现呢?
其次,逻辑应该放在哪里?它应该放在员工的映射器中吗?或“添加员工”操作的控制器,还是应该有新模型?
我使用Zend作为我的框架,但我认为这个问题适用于所有MVC样式结构,所以无论您的框架偏好如何,请随时回复:)
答案 0 :(得分:3)
如果在if()块期间使用OR语句并将查询放在if语句中,那么只有在IF语句的第一部分失败时才会执行它 - 确保您不会不必要地执行查询。就个人而言,我们做了类似的事情,但是当我们创建一个新域时,我们在域new = 1中设置了一个标志,我们检查了这个,相同的方法只是消除了对这些脏变量的需要; - )
$businessMapper = Factory::getMapper('business');
if (!$business = $businessMapper->findByName("Business Name"))
{
$business = $businessMapper->create();
$business->setName("Business Name");
$businessMapper->save($business);
$newBusiness = true;
}
$officeMapper = Factory::getMapper('office');
if (isset($newBusiness) || !$office = $officeMapper->findByName("Office Name"))
{
$office = $officeMapper->create();
$office->setName("Office Name");
$office->setBusiness($business->getId());
$officeMapper->save($office);
$newOffice = true;
}
$employeeMapper = Factory::getMapper('employee');
if (isset($newOffice) || !$employee = $employeeMapper->findByName("Employee"))
{
$employee = $employeeMapper->create();
$employee->setName("Employee Name");
$employee->setOffice($office->getId());
$employeeMapper->save($employee);
}