Why should I use the command design pattern while I can easily call required methods? [closed]
Asked Answered
O

8

45

I am studying the command design pattern, and I am quite confused with the way of using it. The example that I have is related to a remote control class that is used to turn lights on and off.

Why should I not use the switchOn() / switchOff() methods of Light class rather than having separate classes and methods that eventually call switchOn / switchOff methods?

I know my example is quite simple, but that is the point. I could not find any complex problem anywhere on the Internet to see the exact usage of the command design pattern.

If you are aware of any complex real world problem that you solved that can be solved using this design pattern please share that with me. It helps me and future readers of this post to better understand the usage of this design pattern. Thanks

//Command
public interface Command {
  public void execute();
}

//Concrete Command
public class LightOnCommand implements Command {

  //Reference to the light
  Light light;

  public LightOnCommand(Light light) {
    this.light = light;
  }

  public void execute() {
    light.switchOn();        //Explicit call of selected class's method
  }
}

//Concrete Command
public class LightOffCommand implements Command {

  //Reference to the light
  Light light;

  public LightOffCommand(Light light) {
    this.light = light;
  }

  public void execute() {
    light.switchOff();
  }
}

//Receiver
public class Light {
  private boolean on;

  public void switchOn() {
    on = true;
  }

  public void switchOff() {
    on = false;
  }
}

//Invoker
public class RemoteControl {
  private Command command;

  public void setCommand(Command command) {
    this.command = command;
  }

  public void pressButton() {
    command.execute();
  }
}

//Client
public class Client {
  public static void main(String[] args) {
    RemoteControl control = new RemoteControl();
    Light light = new Light();
    Command lightsOn = new LightsOnCommand(light);
    Command lightsOff = new LightsOffCommand(light);

    //Switch on
    control.setCommand(lightsOn);
    control.pressButton();

    //Switch off
    control.setCommand(lightsOff);
    control.pressButton();
  }
}

Why should not I easily use code like the following?

 Light light = new Light();
 switch(light.command) {
  case 1:
    light.switchOn();
    break;
  case 2:
    light.switchOff();
    break;
 }
Omaromara answered 16/9, 2015 at 0:20 Comment(9)
Using the Command interface it's easier to plug your commands to new Buttons, Menus, Shortcuts etc.Fluecure
@Fluecure would you please give me an example. At the end, I need to explicitly call the method of the selected class so whats the difference?Omaromara
You're looking at it from a perspective of a very simple problem. For simple problems, simple solutions will always win. It's like when you were learning about object-oriented programming for the first time. At the beginning, I bet you were like "but why can't I just write all my code in one class?" (at least I was).Supersonic
@ThiagoPorciúncula you are right, my question is how would it be used for complex problems. I need an example to clarify on this.Omaromara
I find that design-type questions like this are hard to illustrate in a small example; it's exactly when you have a complex app that their usefulness starts outweighing their complexity, and it takes experience to build an intuition about it. I suggest you keep the pattern in the back of your mind, but go with what's simpler for now. If and when you need the more complex pattern, refactor. See also: YAGNI.Smith
@Smith I just learned it and I can use it but I just posted the question to see if anyone has ever used it to solve any complex problem. That would help to learn it better. What do you mean by YAGNI?Omaromara
en.m.wikipedia.org/wiki/You_ain%27t_gonna_need_itSmith
For a real world example of command pattern usage, see Activiti. Activiti is a very powerful workflow engine. It's heavily based on the command pattern. As it's opensource, you can download the code and give it a glance.Yi
Basically, you can do more things with the command objects. You could store them in a list for example; you can't store method calls in a list. If you're not going to do those things then you're right, it's silly to use objects here.Counselor
B
41

The main motivation for using the Command pattern is that the executor of the command does not need to know anything at all about what the command is, what context information it needs on or what it does. All of that is encapsulated in the command.

This allows you to do things such as have a list of commands that are executed in order, that are dependent on other items, that are assigned to some triggering event etc.

In your example, you could have other classes (e.g. Air Conditioner) that have their own commands (e.g. Turn Thermostat Up, Turn Thermostat Down). Any of these commands could be assigned to a button or triggered when some condition is met without requiring any knowledge of the command.

So, in summary, the pattern encapsulates everything required to take an action and allows the execution of the action to occur completely independently of any of that context. If that is not a requirement for you then the pattern is probably not helpful for your problem space.

Here's a simple use case:

interface Command {
    void execute();
}

class Light {
    public Command turnOn();
    public Command turnOff();
}

class AirConditioner {
    public Command setThermostat(Temperature temperature);
}

class Button {
    public Button(String text, Command onPush);
}

class Scheduler {
    public void addScheduledCommand(Time timeToExecute, Command command);
}

Then you can do things such as:

new Button("Turn on light", light.turnOn());
scheduler.addScheduledCommand(new Time("15:12:07"), airCon.setThermostat(27));
scheduler.addScheduledCommand(new Time("15:13:02"), light.turnOff());

As you can see the Button and Scheduler don't need to know anything at all about the commands. Scheduler is an example of a class that might hold a collection of commands.

Note also that in Java 8 functional interfaces and method references have made this type of code even neater:

@FunctionalInterface
interface Command {
    void execute();
}

public Light {
    public void turnOn();
}

new Button("Turn On Light", light::turnOn);   

Now the methods that are turned into commands don't even need to know about commands - as long as they have the correct signature you can quietly create a anonymous command object by referencing the method.

Bloc answered 16/9, 2015 at 0:30 Comment(4)
I am studying it and do not have any specific problem space yet. I would like to get more information on Command DP and its usage.Omaromara
To add air conditioner I can do similar to what I did to light. Why should I have separate command class to do that for me? I am a bit confused.Omaromara
@DanielNewtown if you assign the command to a button (for example) the button would have to know if the command was for a light or air conditioner. The command pattern ensures it doesn't need to know anything about the command to execute it.Bloc
@sprinter: that's exactly why the OP's example assigns commands to buttons. To illustrate the fact that proper function of the button click only requires to be able to call a method with no parameters. (i.e. have access to a Command object - no more specific - and call its execute method)Tadich
P
23

Let us focus on the non-implementation aspect of the command design, and some main reasons for using the Command desing pattern grouped in two major categories:

  • Hiding actual implementation of how the command is executed
  • Allow methods to be built around command, aka command extensions

Hide implementation

In most programming, you'll want to hide away implementation so that when looking at the top-most problem, it consists of a comprehensible subset of commands/code. I.e. You don't need/want to know the gory details of how a light is switched on, or a car is started. If your focus is to get the car started, you don't need to understand how the engine works, and how it needs fuel to enter the engine, how the valves work, ...

Indicating the action, not how it is done

A command gives you this kind of view. You'll immediately understand what the TurnLightOn command does, or StartCar. Using a command you'll hide the details of how something is done, whilst clearly indicating the action which is to be executed.

Allow changing of inner details

In addition, lets say you later on rebuild your entire Light class, or a Car class, which requires you instantiate several different objects, and possibly you need to something else before actually doing your wanted operation. In this case if you had implemented the direct access method, in a lot of places, you would need to change it in all places where you've coded it beforehand. Using a command, you can change the inner details of how to do stuff without changing the calling of the command.

Possible command extensions

Using a Command interface gives you an extra layer between the code using the command, and the code doing the actual action of the command. This can allow for multiple good scenarios.

Security extension, or interface exposure

Using a command interface, you can also limit access to your objects, allowing you to define another level of security. It could make sense to have a module/library with a fairly open access so that you could handle internal special cases easily.

From the outside, however, you might want to restrict the access to the light so that it is only to be turned on or off. Using commands gives you the ability to limit the interface towards a class.

In addition if you want, you could build a dedicated user access system around the commands. This would leave all of your business logic open and accessible and free of restrictions, but still you could easily restrict access at the command level to enforce proper access.

Common interface to executing stuff

When building a sufficient large system, commands gives a neat way to bridge between different modules/libraries. Instead of you needing to examine every implementation detail of any given class, you can look into which commands are accessing the class.

And since you are leaving out implementation details to the command itself, you could use a common method to instantiate the command, execute it and review the result. This allows for easier coding, instead of needing to read up on how to instantiate that particular Light or Car class, and determining the result of it.

Sequencing of commands

Using commands, you can also do stuff like sequencing of commands. That is since you don't really matter if you are executing the TurnOnLight or StartCar command, you can execute sequences of theses as they are executed the same way. Which in turns can allow for chains of commands to be executed, which could be useful in multiple situation.

You could build macros, execute sets of commands which you believe are grouped together. I.e. the command sequence: UnlockDoor, EnterHouse, TurnOnLight. A natural sequence of commands, but not likely to made into a method as it uses different objects and actions.

Serialisation of commands

Commands being rather small in nature, also allows for serialisation quite nicely. This is useful in server-client context, or program-microservice context.

The server (or program) could trigger a command, which then serialises the command, sends it over some communication protocol (i.e. event queue, message queue, http, ... ) to someone actually handling the command. No need to first instantiate the object at the server, i.e. a Light which could be light-weight (pun intended), or a Car which could be a really large structure. You just need the command, and possibly a few parameters.

This could be a good place to introduce to the CQRS - Command Query Responsibility Separation pattern for further studies.

Tracking of commands

Using the extra layer of commands in your system could also allow for logging or tracking commands, if that is a business need. Instead of doing this all over the place, you can gather tracking/logging within the command module.

This allows for easy changes of logging system, or addition of stuff like timeing, or enabling/disabling logging. And it's all easily maintained within the command module.

Tracking of commands also allows for allowing undo actions, as you can choose to re-iterate the commands from a given state. Needs a little bit of extra setup, but rather easily doable.


In short, commands can be really useful as they allow connection between different parts of your soon-to-be-large program, as they are light-weight easily remembered/documented and hides the implementation details when so needed. In addition the commands allows for several useful extensions, which when building a larger system will come in handy: i.e. common interface, sequencing, serialisation, tracking, logging, security.

Poky answered 16/9, 2015 at 10:29 Comment(0)
S
12

The possibilities are many, but it'd typically be something like:

  • building a command-line framework that abstracts out the parsing of options from the action. Then you can register an action with something like opts.register("--on", new LightOnCommand()).
  • letting users drag and drop a sequence of actions to execute as a macro
  • registering a callback when some event is triggered, like on(Event.ENTER_ROOM, new LightOnCommand())

The general pattern here is that you've got one piece of the code responsible for figuring out that some action needs to be taken without knowing what that action is, and another piece of the code knows how to do an action but not when to do it.

For instance, in that first example, the opts instance knows that when it sees a command line option --on, it should turn the lights on. But it knows this without actually knowing what "turn the lights on" means. In fact, it could well be that the opts instance came from a third-party library, so it can't know about lights. All it knows about is how to accociate actions (commands) with command-line options that it parses.

Smith answered 16/9, 2015 at 0:49 Comment(9)
Great, would you elaborate on your first example?Omaromara
Also, with the last example, why wont you just call light.switchOn method? why would you use LightOnCommand method!!?Omaromara
@DanielNewtown Because whoever calls the callback, doesn't (statically) know, which function to call. He just has a reference to some Command object and calls its execute method without knowing what that does. This way someone else can decide, what should happen in that situation, just by registering an object of a specific subclass of Command.Fluecure
Because that on method was declared on some very general class, like Button or UserInput, where other instances of that class might perform other options. Maybe for an office building, you also want on(Event.ENTER_ROOM, new TriggerAlarm()).Smith
@Smith I think got you now, in that case Light should be an interface or an abstract class that its subclasses can implement its switchon, switchoff methods in their own way. am I right?Omaromara
That could also be useful, but it's orthogonal. The thing to keep in mind is that the command line parser, macro framework, etc might not even know that any lightbulb-type-thing even exists. All it knows about is that Commands can be executed. It might even be a separate code base from the thing that knows about lights.Smith
@Smith but LightOnCommand constructor needs a light object, you would need to pass that as well otherwise it does not know on which type of light it should apply the commandOmaromara
Right. You have basically three pieces of code: one defines Commands and events; another knows about Lights and such. These two pieces don't know about each other, and may be in different projects. And then a third piece, the "main" component, knows about both of those pieces and stitches them together.Smith
Another situation where this pattern is useful is implementing multi-level undo. If the Command interface has an undo() method as well as the execute() method, and the concrete classes implement undo() in such a way that they perfectly reverse the effects of execute(), then you can push commands onto a stack as they are executed, and pop and undo() to undo an arbitrary number of times.Pushcart
D
7

You don't have to. Design patterns are simply guidelines that some people in the past have found useful when writing applications of significant complexity.

In your case, if what you have to do is turn the light switch on and off, and not much else, the second option is a no brainer.

Less code is almost always better than more code.

Dorey answered 16/9, 2015 at 0:29 Comment(6)
yes but the main reason of using design pattern is to have extendible codes. I am wondering what sort of issues would I have when I want to extend my code.Omaromara
I think you're looking at it backwards: you're trying to make the design pattern fit the problem, while it should be the other way around. As others have noted, it's hard to illustrate a use case here in the context of the example.Dorey
Of course you are right for this simple problem. But the question was different: What's the point of the design pattern? And in general there is a point, otherwise people wouldn't use it as much.Fluecure
Thats is just an example, please read the bold part of the question.Omaromara
@Fluecure Uh, no, the question was "why should not I use the switchOn() / switchOff() methods of Light class rather than having separate classes and methods that eventually call switchOn / switchOff methods"Dorey
He just said it himself: "Thats is just an example, please read the bold part of the question."Fluecure
S
5

The example ICommand you give is rather limited and is only of real use in a programming language that does not have lambda expressions. Sprinter covers this well in his answers showing the use of command factories.

Most cases of the command pattern include other methods for example, CanRun and/or Undo. These allows a button to update its enable state based on the state of the command, or a application to implement an undo stack.

Like most design patterns, the Command Pattern comes into its own where things get a little more complex. It is also well know, so helps make your code clearly to most programmers.

Studdingsail answered 16/9, 2015 at 9:18 Comment(0)
S
2

There're couple of benefits of Command pattern that I've experienced. But first of all, I would like to remind that as all other patterns this pattern is also to increase code readability and bring some common understanding to your (probably) shared code base.

I use this pattern to transition from a method-oriented interface to a command-oriented interface. That means, I'm encapsulating method calls into concrete commands along with necessary data. It makes the code more readable yes but more importantly I can treat methods as objects which lets you easily add/remove/modify commands without increasing the complexity of the code. So it makes it easy to manage as well easy to read.

Secondly, since you've your methods as objects, you can store/queue them to execute them later. Or to cancel them even after you've executed them. This is where this pattern helps you to implement "Undo".

Finally, command pattern also used to decouple command execution and commands themselves. It's like a waiter doesn't know about how to cook the orders that he received. He doesn't care. He doesn't have to know. If he'd know that, he'd be working as a cook as well. And if the restaurant owner wants to fire the waiter, he/she ends up with having no cook as well. Of course this definition is not special or related to command pattern only. That's explains why we need decoupling or dependency management in general.

Semolina answered 15/2, 2017 at 8:46 Comment(0)
C
1

You can do whatever you think but it is good to follow the pattern for usability of manageability of code.

In real life example, Light would be an interface. You have different implementations of Light like LEDLight, TubeLight

If you execute the concrete command through Inovker ( RemoteControl), you need not worry about changes in method name in Receiver. switchOn() in Light can be changed to switchOnDevice() in future. But it does not effect Invoker since ConcreteCommand (LightsOnCommand) will make relevant changes.

Assume the scenario, where you publish your interfaces to one service (Service A) and implementation in other services (Service B).

Now Service A should not know about changes in Receiver.

Invoker provides loose coupling between Sender & Receiver of the message.

Have a look at some more related SE questions:

Using Command Design pattern

Command Pattern seems needlessly complex (what am I failing to understand?)

Creight answered 11/2, 2016 at 11:56 Comment(0)
R
1

Design patterns are basically developed to solve complex problem or in other words we can say that they are used to prevent your solution to become complex. They provide us the flexibility to add new functionalities in future without making much changes in our existing code. Open for extension, closed for modification.

Command Pattern basically encapsulates a request as an object and thereby letting you parameterize other objects with different requests and support undoable operations.

When we see a remote control that is a best example of command pattern. In this case we have associated a concrete command to each button and that command has an information of the receiver to act upon.

In your example of switch case, suppose if you want to associate a button on remote control to fan instead of light then again you will have to change the existing code because you need to associate a button with a command. This is the essence of command pattern where it provides you a flexibility to associate any command with a button so that at run time you can change the functionality. Although in general TV remote you don't have this feasibility i.e. you can't make volume up button to work as volume down. But if you use any application to control your TV then you can assign any button any of the available command.

Further, using command pattern you can have a set of commands for a particular functionality to achieve, i.e. macro command. Hence if you think at a broader level then this pattern can get you the flexibility to extend functionalities.

Command pattern help us in decoupling invoker(remote control) and Receiver (Light,Fan,etc) with the help of command object(LightOnCommand, FanOffCommand, etc).

Refrigerator answered 6/4, 2018 at 17:58 Comment(0)

© 2022 - 2024 — McMap. All rights reserved.