CODEWITHSUNDEEP
phpoopmodel-view-controllerdesign-patterns
Bv202
Bv202

Reputation: 4054

MVC - functionality in the controller/model

I'm trying to improve some PHP scripts using a OO design (it's procedural now with some OO parts). Just to be clear, I'm not trying to build a full MVC application, but I am trying to seperate parts as much as possible. I've never used MVC before in PHP (only a little bit in Java).

When using Google, I find 100 different MVC approaches for PHP and I can't find a good book on this subject. If anyone could suggest me a good book on OO design in PHP, it would be much appreciated.

Currently, a part to add a user to the database (assuming a user only contains a firstname for now), looks like this (users.php):

$validator = new UserValidator();

if ($validator->validate($_POST['user_firstname']))
  $result = $db->execute("INSERT INTO `users` (`user_firstname`) VALUES (?)", $_POST['user_firstname']);

Knowing that adding users may be done at multiple places and I don't want code repeat, I will create a usermodel. This class will contain a method addUser(). The thing I'm a bit stuck with is the validation. The UserValidator will check if all fields are filled in correctly.

I could do this:

$validator = new UserValidator();

if ($validator->validate($_POST['user_firstname']))
  $result = $user->addUser($_POST['user_firstname']);

But I could also do this:

$result = $user->adduser($_POST['user_firstname'];

Now the User-class will contain the validator and the addUser()-method will perform this validation. Assuming the above code is the controller, what option is the best? Delegating the validation functionality to the model or doing it in the controller?

The same problem applies when getting information of a certain user. Not everyone may get this info, so my code could look like this:

if ($user->hasAccess($_SESSION['id'], $_GET['id'])
  $user->getUserById($_GET'id']);

(The hasAccess()-method will check if the user who is logged in can view details of a certain user id)

But I could also just call getUserById() and check if you have access in that method. Which option is the best?

Thank you!

Upvotes: 0

Views: 506

Answers (2)

Fred Wilson
Fred Wilson

Reputation: 2207

In your first example, putting the validation logic in the call $result = $user->adduser($_POST['user_firstname']; is the cleaner way to go. Keep your controllers thin and let your models handle as much of the logic as possible. Ideally, your controller is coordinating data that will be passed to your "view", whatever that may be.

Your second example is less clear. You might have logic in hasAccess() that doesn't belong in getUserById() or you might be creating more work for getUserById() than makes sense for the method. It's always best to keep similar functionality as close as possible, but there are some assumptions that can't be made just by looking at the two lines you posted.

Upvotes: 1

Scott Harwell
Scott Harwell

Reputation: 7465

I user CakePHP pretty frequently, an MCV framework for PHP, and in such a case, the UserValidator would be broken out into a separate entity called a "component." The controller would call the component to do the validation before the save and then, if all passes, the data would be sent to the model for a save.

I think creating a separate validator class that can check your data in the controller may be the way to go.

Upvotes: 0

Related Questions