why did you seperated listeners like this? I don't think anonymous inner class is not a good idea because it makes code override same method several times. you can just add listeners and override methods for them.

```java
public class EditActivityController implements ActionListener {
...

cancelBtn.addActionListener(this);
saveBtn.addActionListener(this);

... 
public void actionPerformed(ActionEvent e) {
 if (e.getSource() == cancelBtn) {
 //action for cancelBtn
} else if (e.getSource() == saveBtn) {
 //action for saveBtn
}
}
```