ok, can you change that into inner class? I did the same way before, just thought seperated might be clearer to see then I changed again. 
So maybe you can put it back. 

发自我的 iPhone

> 在 2014年3月2日，下午10:50，yeonni <notifications@github.com> 写道：
> 
> In src/Controller/editActivityController.java:
> 
> >      public EditActivityController(AgendaModel model, EditActivityFrame view) {
> > +    	
> > +    	this.model = model;
> > +    	this.view = view;
> > +		addEventsListener(); // seperated the Listener into another class 
> > +
> > +     } // inner EditActivityController(...,...)
> > +
> > +    
> > +	private void addEventsListener()
> 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.
> 
> 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
> }
> }
> —
> Reply to this email directly or view it on GitHub.