In which class would you put these methods?

Question

If I have a class Userand its account can be suspended by adding an entry to the table suspensions, which of these class / method signatures, in your opinion, is more appropriate?

User::suspend($reason, $expiryDate);
Suspension::add($userid, $reason, $expiryDate);

This is a simple example, but I have this situation throughout my application. On the one hand, I would like to make it a method of the User object, since the action being performed is directly related to this user object, but, on the other hand, its method on the suspension object seems a little cleaner.

What do you think?

Answer 1

you pause the user.

User.Suspend()

User.Suspend "" . , , , .

Answer 2

. OO . , () ( ). , , , . , User.

: ? , , .

, / . , , User. , Suspension . , User.

Answer 3

, , , User.

, Suspensions, , :

class SuspensionManager
suspendUser(....)
getSuspendedUser(...)
....

Answer 4

* 100% , , /

. , OOAD. , User Suspension . User , ( ), Suspension , ( ). UserSuspention, .

OOAD SOLID. , Suspension , SRP ( )..., .

API :

  public class UserSuspension
  {
    public void SuspendUser(User user, Suspension suspension) {  ...   }
    public void SuspendUser(Guid userId, string reason, DateTime expiryDate) { ... }
  }

Answer 5

Suspension::add(), . , , , (.. , ), : User::suspend(), , a "suspended" User, - - .

, , User::suspend() , , , User suspended, , , .. , , User , .

Answer 6

-. , , .

, , :

User user = GetUser($userId); // unnecessary database hit?
user.suspend(reason, expiryDate);

Answer 7

Account, User Suspension

Answer 8

.

, . , , , .

, , . , , suspend - , .

? .

- , "AccountAction" . , AccountAction applyTo, .

, , persitance ORM .

. OO - , . , , .

, . , , , , , , , .

Answer 9

, .

User:: suspend() , ( ) , . .

Answer 10

, , , . ? ? , //, . , . , , . , , , .