Question

I have to implement a few action. Each of they have some common elements, so they should inherit from one class. Lets say, that they are do some web actions, so base class is

public class WebAction
{
    private ServerData _serverData;
    public WebAction(ServerData serverData)
    {
        _serverData = serverData;
    }
}

Also, each of this action need to be execute, so in base class I also should have some sort of Execute() Method. Problem is, that each of this object do a little bit different thing, so It should take different parameters. My question is – witch way is best for reach this?

At first I was thinking abut some configuration object, e,g. Execute(IWebActionConfig config). In this case, I can use object like:

 LoginWebAction loginWebAction = LoginWebAction(staticServerConfig);
 LoginWebActionConfig = new LoginWebActionConfig { someData = "foo" };
 loginWebAction.Execute(loginWebActionConfig);

But is there any sense to create an additional tree of objects? Maybe will be better to configure each child of WebAction by properties?

LoginWebAction loginWebAction = LoginWebAction(staticServerConfig);
loginWebAction.someData = "foo";
// Execute will throw exception when someData not set
loginWebAction.Execute();

With of these (if any) way is better?

Answer 1

It is possible to do in more OOP style. look at NVI pattern

public class WebAction
{
  private ServerData _serverData;
  public WebAction(ServerData serverData)
  {
    _serverData = serverData;
  }
  public virtual ExecuteAction{};
  public Execute()
  {
     ExecuteAction();
     //to do the rest of base code 

  }
}
public class WebActionDerived1: WebAction
{
   WebActionDerived1(ServerData serverData, object arg1/*any unique param*/)
   :base(serverData)
   {
      _my_arg1 = arg1;
   }
   public override ExecuteAction
   {
    //call any procedure with your _my_arg1;
   }
}
public class WebActionDerived2: WebAction
{
   WebActionDerived1(ServerData serverData, string arg1,
   string arg2/*otherunique param*/)
   :base(serverData)
   {
      _my_arg1 = arg1;
      _my_arg2 = arg2;
   }
   public override ExecuteAction
   {
    //call any procedure with your _my_arg1, and _my_arg2;
   }
}