Rename CompetitorFinder --> CompetitorAnalyzer and add javadoc

This commit is contained in:
Jan Broß committed 2015-11-04 12:55:42 +01:00
1 parent 3ae4d99260
commit 0daf5d427e
5 files changed
+19 -11

No files matched your search

@@ -31,7 +31,7 @@ import com.sap.sailing.domain.abstractlog.race.tracking.RaceLogDeviceMarkMapping
import com.sap.sailing.domain.abstractlog.race.tracking.RaceLogStartTrackingEvent;
import com.sap.sailing.domain.abstractlog.race.tracking.analyzing.impl.RaceInformationFinder;
import com.sap.sailing.domain.abstractlog.race.tracking.analyzing.impl.RaceLogTrackingStateAnalyzer;
import com.sap.sailing.domain.abstractlog.race.tracking.analyzing.impl.RegisteredCompetitorsFinder;
import com.sap.sailing.domain.abstractlog.race.tracking.analyzing.impl.RegisteredCompetitorsAnalyzer;
import com.sap.sailing.domain.abstractlog.regatta.RegattaLog;
import com.sap.sailing.domain.abstractlog.regatta.RegattaLogEventVisitor;
import com.sap.sailing.domain.abstractlog.regatta.events.RegattaLogDefineMarkEvent;
@@ -481,7 +481,7 @@ public class RaceLogRaceTracker implements RaceTracker, GPSFixReceivedListener {
Set<Competitor> competitors;
try {
RegattaLog regattaLog = params.getRegattaLog();
competitors = new RegisteredCompetitorsFinder(raceLog, regattaLog).analyze();
competitors = new RegisteredCompetitorsAnalyzer(raceLog, regattaLog).analyze();
} catch (DoesNotHaveRegattaLogException e) {
logger.warning(params.getLeaderboard().getName()+" doesn't have an attached RegattaLog");
competitors = new RaceLogRegisteredCompetitorsAnalyzer(raceLog).analyze();
@@ -12,10 +12,18 @@ import com.sap.sailing.domain.abstractlog.regatta.RegattaLog;
import com.sap.sailing.domain.abstractlog.shared.events.RegisterCompetitorEvent;
import com.sap.sailing.domain.base.Competitor;
public class RegisteredCompetitorsFinder extends RaceLogAnalyzer<Set<Competitor>> {
/**
* Used to find competitors of a race. Checks whether the competitors for the given race are registered on the RaceLog
* via {@link RaceLogUsesOwnCompetitorsAnalyzer} or on the RegattaLog and fetches the competitors appropriately.
*
* @author D056848
*
*/
public class RegisteredCompetitorsAnalyzer extends RaceLogAnalyzer<Set<Competitor>> {
private RegattaLog regattaLog;
public RegisteredCompetitorsFinder(RaceLog raceLog, RegattaLog regattaLog) {
public RegisteredCompetitorsAnalyzer(RaceLog raceLog, RegattaLog regattaLog) {
super(raceLog);
this.regattaLog = regattaLog;
}
@@ -8,14 +8,14 @@ import com.sap.sailing.domain.abstractlog.AbstractLogEvent;
import com.sap.sailing.domain.abstractlog.BaseLogAnalyzer;
import com.sap.sailing.domain.abstractlog.race.analyzing.impl.RaceLogRegisteredCompetitorsAnalyzer;
import com.sap.sailing.domain.abstractlog.race.tracking.RaceLogUseCompetitorsFromRaceLogEvent;
import com.sap.sailing.domain.abstractlog.race.tracking.analyzing.impl.RegisteredCompetitorsFinder;
import com.sap.sailing.domain.abstractlog.race.tracking.analyzing.impl.RegisteredCompetitorsAnalyzer;
import com.sap.sailing.domain.abstractlog.shared.events.RegisterCompetitorEvent;
import com.sap.sailing.domain.base.Competitor;
/**
* This class searches for RegisterCompetitorEvents in the given log.
*
* TLDR: It's likely, that you shouldn't use this, but you might want to use {@link RegisteredCompetitorsFinder} or {@link RaceLogRegisteredCompetitorsAnalyzer}
* TLDR: It's likely, that you shouldn't use this, but you might want to use {@link RegisteredCompetitorsAnalyzer} or {@link RaceLogRegisteredCompetitorsAnalyzer}
*
* Note that solemnly analyzing a certain race or RegattaLog my not lead the correct registered competitors for the race
* corresponding to the RaceLog/the races corresponding to the RegattaLog.
@@ -12,7 +12,7 @@ import com.sap.sailing.domain.abstractlog.race.RaceLogRevokeEvent;
import com.sap.sailing.domain.abstractlog.race.impl.BaseRaceLogEventVisitor;
import com.sap.sailing.domain.abstractlog.race.tracking.RaceLogRegisterCompetitorEvent;
import com.sap.sailing.domain.abstractlog.race.tracking.RaceLogUseCompetitorsFromRaceLogEvent;
import com.sap.sailing.domain.abstractlog.race.tracking.analyzing.impl.RegisteredCompetitorsFinder;
import com.sap.sailing.domain.abstractlog.race.tracking.analyzing.impl.RegisteredCompetitorsAnalyzer;
import com.sap.sailing.domain.abstractlog.regatta.RegattaLog;
import com.sap.sailing.domain.abstractlog.regatta.RegattaLogEventVisitor;
import com.sap.sailing.domain.abstractlog.regatta.events.RegattaLogRegisterCompetitorEvent;
@@ -194,7 +194,7 @@ public class CompetitorProviderFromRaceColumnsAndRegattaLike {
}
private Set<Competitor> getCompetitorsForRaceAndRegisterAsRegattaLogListener(RaceLog racelog) {
Set<Competitor> viaLog = new RegisteredCompetitorsFinder(racelog, provider.getRegattaLike().getRegattaLog())
Set<Competitor> viaLog = new RegisteredCompetitorsAnalyzer(racelog, provider.getRegattaLike().getRegattaLog())
.analyze();
// note: adding listeners is idempotent; at most one occurrence of this listener exists in the regatta log's
// listeners set
@@ -83,7 +83,7 @@ import com.sap.sailing.domain.abstractlog.race.tracking.RaceLogUseCompetitorsFro
import com.sap.sailing.domain.abstractlog.race.tracking.analyzing.impl.RaceLogOpenEndedDeviceMappingCloser;
import com.sap.sailing.domain.abstractlog.race.tracking.analyzing.impl.RaceLogTrackingStateAnalyzer;
import com.sap.sailing.domain.abstractlog.race.tracking.analyzing.impl.RaceLogUsesOwnCompetitorsAnalyzer;
import com.sap.sailing.domain.abstractlog.race.tracking.analyzing.impl.RegisteredCompetitorsFinder;
import com.sap.sailing.domain.abstractlog.race.tracking.analyzing.impl.RegisteredCompetitorsAnalyzer;
import com.sap.sailing.domain.abstractlog.race.tracking.impl.RaceLogUseCompetitorsFromRaceLogEventImpl;
import com.sap.sailing.domain.abstractlog.regatta.RegattaLog;
import com.sap.sailing.domain.abstractlog.regatta.RegattaLogEvent;
@@ -2313,7 +2313,7 @@ public class SailingServiceImpl extends ProxiedRemoteServiceServlet implements S
boolean raceLogTrackerExists = raceLog == null ? false : getService().getRaceTrackerById(raceLog.getId()) != null;
boolean competitorRegistrationsExist;
try {
competitorRegistrationsExist = raceLog == null ? false : ! new RegisteredCompetitorsFinder(raceLog, getRegattaLogInternal(leaderboard.getName())).analyze().isEmpty();
competitorRegistrationsExist = raceLog == null ? false : ! new RegisteredCompetitorsAnalyzer(raceLog, getRegattaLogInternal(leaderboard.getName())).analyze().isEmpty();
} catch (DoesNotHaveRegattaLogException e) {
competitorRegistrationsExist = false;
}
@@ -5835,7 +5835,7 @@ public class SailingServiceImpl extends ProxiedRemoteServiceServlet implements S
String fleetName) throws DoesNotHaveRegattaLogException {
RaceLog raceLog = getRaceLog(leaderboardName, raceColumnName, fleetName);
RegattaLog regattaLog = getRegattaLogInternal(leaderboardName);
return convertToCompetitorDTOs(new RegisteredCompetitorsFinder(raceLog, regattaLog).analyze());
return convertToCompetitorDTOs(new RegisteredCompetitorsAnalyzer(raceLog, regattaLog).analyze());
}
@Override