Question

Given that there is file selection widget on the view and controller need to handle event of selecting file, should I rather write controller method:

public void fileSelected(String filePath){
  //process filePath
}

or

public void fileSelected(){
  String filePath = view.getSelectedFilePath();
  //process filePath
}

The first approach seems to introduce less coupling between C and V: C don’t know what exactly data does C need while handling given event.

But it requires creating a lot of verbose methods similar to getSelectedFile on V side.

On the other hand, second approach may lead to cluttered controller methods in more complex cases than in example (much more data to pass than just filePath).

From your own experience, which approach do you prefer?

Answer 1

The first approach is my favourite. The only difference is I would rather use an object (like Mario suggested) to pass arguments to the method. This way method’s signature will not change when you add or remove some of the arguments. Less coupling is always good 🙂

One more thing:
If You want to try the second solution I recommend using a ViewFactory to remove view logic from the controller.