don't repeat yourself

This commit is contained in:
Florent
2013-07-08 09:44:10 +02:00
parent 89f1e64ad0
commit b858179dd9

View File

@@ -74,12 +74,7 @@ class UserMapper
} }
$row = $result->current(); $row = $result->current();
$user = new User(); return $this->mapObject($row);
$user->setUserID($row['userid']);
$user->setUsername($row['username']);
$user->setEmail($row['email']);
return $user;
} }
/** /**
@@ -94,15 +89,26 @@ class UserMapper
$entries = array(); $entries = array();
foreach ($resultSet as $row) { foreach ($resultSet as $row) {
$entries[] = $this->mapObject($row);
$entry = new User();
$entry->setUserID($row['userid']);
$entry->setUsername($row['username']);
$entry->setEmail($row['email']);
$entries[] = $entry;
} }
return $entries; return $entries;
} }
}
/**
* Maps a table row to an object
*
* @param array $row
*
* @return \DesignPatterns\DataMapper\User
*/
protected function mapObject(array $row)
{
$entry = new User();
$entry->setUserID($row['userid']);
$entry->setUsername($row['username']);
$entry->setEmail($row['email']);
return $entry;
}
}