Come lo rifattorizzeresti? Il codice è quasi uguale, ci sono pochissime diferenze (un else e un save)

new_gist_file

package JE.AdvertSearchEngine;

import JE.Advert;
import JE.AdvertLifecycle.ProgressCategory;
import JE.AdvertLifecycle.Scheduling.Data.AdvertSchedule;
import Util.DateTime;

public class AdvertScheduleService {

  public AdvertSchedule scheduleAdvert(Advert advert) {
		
    	AdvertSchedule advertSchedule = AdvertSchedule.GetByAdvertId(advert.Id());
        String progressCategory = ProgressCategory.GetCategoryForAdv(advert);

        if (advertSchedule==null) {
            advertSchedule = AdvertSchedule.Create(advert.Website, progressCategory, advert.Id(), DateTime.Now());
        }
        else
        {
        	advertSchedule.LifecycleCategory = progressCategory;
        }

        advertSchedule.$Save();
        
        return advertSchedule;
	}

	public AdvertSchedule createAdvertSchedule(Advert advert) {
		AdvertSchedule advertSchedule = AdvertSchedule.GetByAdvertId(advert.Id());
		String progressCategory = ProgressCategory.GetCategoryForAdv(advert);
		
		if (advertSchedule == null) {
			advertSchedule = AdvertSchedule.Create(advert.Website, progressCategory, advert.Id(), DateTime.Now());
		}
		return advertSchedule;
	}
}

Mah, i 2 metodi mi sembrano molto simili, tenterei di mettere in comune il codice.

Non capisco perche' progressCategory debba essere passato 2 volte ad advertSchedule.