--- android/TrainInfoService/src/dk/thoerup/traininfoservice/banedk/DepartureFetcher.java 2010/02/02 18:42:38 580 +++ android/TrainInfoService/src/dk/thoerup/traininfoservice/banedk/DepartureFetcher.java 2010/07/10 10:53:44 978 @@ -1,14 +1,12 @@ package dk.thoerup.traininfoservice.banedk; -import java.sql.Connection; -import java.sql.ResultSet; -import java.sql.Statement; -import java.util.ArrayList; + import java.util.Collections; import java.util.List; import java.util.Map; import java.util.logging.Logger; +import com.gargoylesoftware.htmlunit.BrowserVersion; import com.gargoylesoftware.htmlunit.WebClient; import com.gargoylesoftware.htmlunit.html.DomNodeList; import com.gargoylesoftware.htmlunit.html.HtmlElement; @@ -16,97 +14,120 @@ import dk.thoerup.circuitbreaker.CircuitBreaker; import dk.thoerup.circuitbreaker.CircuitBreakerManager; -import dk.thoerup.traininfoservice.DBConnection; +import dk.thoerup.traininfoservice.StationBean; +import dk.thoerup.traininfoservice.StationDAO; +import dk.thoerup.traininfoservice.Statistics; public class DepartureFetcher { + enum TrainType{ + STOG, + REGIONAL + } + Logger logger = Logger.getLogger(DepartureFetcher.class.getName()); - Map> cache = new TimeoutMap>(120 * 1000); + Map cache; + + StationDAO stationDao = new StationDAO(); private boolean useTempSite; - public DepartureFetcher(boolean tempSite) { + public DepartureFetcher(boolean tempSite, int cacheTimeout) { useTempSite = tempSite; + cache = new TimeoutMap(cacheTimeout); } - public List cachedLookupDepartures(int stationID) throws Exception { + public DepartureBean cachedLookupDepartures(int stationID, boolean arrival) throws Exception { + final String key = "" + stationID + ":" + arrival; + /* + DepartureBean departureBean = cache.get(key); - List list = cache.get(stationID); - if (list == null) { - list = lookupDepartures(stationID); - cache.put(stationID, list); + if (departureBean == null) { + departureBean = lookupDepartures(stationID,arrival); + cache.put(key, departureBean); } else { - logger.info("Departure: Cache hit " + stationID); //remove before production + Statistics.getInstance().incrementDepartureCacheHits(); + logger.info("Departure: Cache hit " + key); //remove before production } - return list; + return departureBean;*/ + + return lookupDepartures(stationID,arrival); } - public List lookupDepartures(int stationID) throws Exception { - List departureList = new ArrayList(); + public DepartureBean lookupDepartures(int stationID, boolean arrival) throws Exception { - Connection conn = null; - try - { - conn = DBConnection.getConnection(); - - String SQL = "SELECT stationcode_fjrn, stationcode_stog FROM trainstations WHERE id=" + stationID; - Statement stmt = conn.createStatement(); - ResultSet rs = stmt.executeQuery(SQL); - - if (rs.next()) { - String code = rs.getString( 1 ); - if (! rs.wasNull() ) { - List list = lookupDepartures(code, "FJRN"); - departureList.addAll(list); - } - - code = rs.getString(2); - if (! rs.wasNull() ) { - List list = lookupDepartures(code, "S2"); - departureList.addAll(list); - } - Collections.sort( departureList ); - - } - - } finally { - if (conn != null && !conn.isClosed() ) { - conn.close(); - } + DepartureBean departureBean = new DepartureBean(); + + StationBean station = stationDao.getById(stationID); + + if (station.getRegional() != null) { + DepartureBean tempBean = lookupDepartures(station.getRegional(), TrainType.REGIONAL, arrival); + departureBean.departureEntries.addAll( tempBean.departureEntries ); + departureBean.notifications.addAll(tempBean.notifications); } - return departureList; + if (station.getStrain() != null) { + DepartureBean tempBean = lookupDepartures(station.getStrain(), TrainType.STOG, arrival); + departureBean.departureEntries.addAll( tempBean.departureEntries ); + departureBean.notifications.addAll(tempBean.notifications); + } + + Collections.sort( departureBean.departureEntries ); + + + return departureBean; } - public List lookupDepartures(String stationcode, String type) throws Exception { + public DepartureBean lookupDepartures(String stationcode, TrainType type, boolean arrival) throws Exception { if (useTempSite == false) { - return lookupDeparturesNormalSite(stationcode, type); + return lookupDeparturesNormalSite(stationcode, type, arrival); } else { - return lookupDeparturesFromTemporarySite(stationcode, type); + //return lookupDeparturesFromTemporarySite(stationcode, type); + //TODO: find out what to to if they ever put a temp site up on trafikinfo.bane.dk + return null; + } + } + + private String getTypeString(TrainType type) { + switch (type) { + case STOG: + return "S-Tog"; + case REGIONAL: + return "Fjerntog"; + default: + return ""; //Can not happen } } - public List lookupDeparturesNormalSite(String stationcode, String type) throws Exception { + public DepartureBean lookupDeparturesNormalSite(String stationcode, TrainType type, boolean arrival) throws Exception { - List departureList = new ArrayList(); + DepartureBean departureBean = new DepartureBean(); - final WebClient webClient = new WebClient(); + final WebClient webClient = new WebClient( BrowserVersion.FIREFOX_3 ); webClient.setTimeout(2500); webClient.setJavaScriptEnabled(false); + + + String typeString = getTypeString(type); + String arrivalDeparture = (arrival==false) ? "Afgang" : "Ankomst"; - String uri = "http://www.bane.dk/visStation.asp?ArtikelID=4275&W=" + type + "&S=" + stationcode; - BanedkInvocation wrapper = new BanedkInvocation(webClient, uri); + //String uri = "http://www.bane.dk/visStation.asp?ArtikelID=4275&W=" + type + "&S=" + stationcode; + String uri = "http://trafikinfo.bane.dk/Trafikinformation/AfgangAnkomst/" + arrivalDeparture + "/" + stationcode + "/" + typeString + "/UdvidetVisning"; + + //logger.info("URI: " + uri); + HtmlunitInvocation wrapper = new HtmlunitInvocation(webClient, uri); CircuitBreaker breaker = CircuitBreakerManager.getManager().getCircuitBreaker("banedk"); HtmlPage page = (HtmlPage) breaker.invoke(wrapper); - HtmlElement table = page.getElementById("afgangtabel"); + String tableName = arrival == false ? "afgangtabel" : "ankomsttabel"; + HtmlElement table = page.getElementById(tableName); if (table != null) { DomNodeList tableRows = table.getElementsByTagName("tr"); @@ -116,7 +137,7 @@ if (rowClass != null && rowClass.toLowerCase().contains("station") ) { DomNodeList fields = currentRow.getElementsByTagName("td"); - DepartureBean departure = new DepartureBean(); + DepartureEntry departure = new DepartureEntry(); String time = fields.get(0).asText(); if (time.equals("")) @@ -127,7 +148,7 @@ departure.setUpdated(updated); String trainNumber = fields.get(2).asText(); - if (type.equalsIgnoreCase("S2")) //If it is S-train we need to extract the trainNumber + if (type == TrainType.STOG) //If it is S-train we need to extract the trainNumber trainNumber = trainNumber + " " + extractTrainNumber(fields.get(2)); departure.setTrainNumber(trainNumber); @@ -140,34 +161,55 @@ String location = fields.get(5).asText(); departure.setLocation(location); - String status = fields.get(6).asText(); + String status = fields.get(6).asText().trim(); departure.setStatus(status); String note = extractNote( fields.get(7) ); departure.setNote(note); - departureList.add(departure); + departure.setType(typeString); + + departureBean.departureEntries.add( departure ); } } } else { logger.warning("No departures found for station=" + stationcode + ", type=" + type); } - return departureList; + HtmlElement notifDiv = page.getElementById("station_planlagte_text"); + if (notifDiv != null) { + + DomNodeList tables = notifDiv.getElementsByTagName("table"); + for (HtmlElement tab : tables) { + + DomNodeList anchors = tab.getElementsByTagName("a"); + if (anchors.size() == 2) { + departureBean.notifications.add( anchors.get(1).getTextContent() ); + } + } + + } + + + webClient.closeAllWindows(); + + return departureBean; } + /* + @Deprecated public List lookupDeparturesFromTemporarySite(String stationcode, String type) throws Exception { List departureList = new ArrayList(); - final WebClient webClient = new WebClient(); + final WebClient webClient = new WebClient(BrowserVersion.FIREFOX_3); webClient.setTimeout(2500); webClient.setJavaScriptEnabled(false); String uri = "http://bane.dk/lite/station.asp?w=" + type + "&s=" + stationcode; - BanedkInvocation wrapper = new BanedkInvocation(webClient, uri); + HtmlunitInvocation wrapper = new HtmlunitInvocation(webClient, uri); CircuitBreaker breaker = CircuitBreakerManager.getManager().getCircuitBreaker("banedk"); HtmlPage page = (HtmlPage) breaker.invoke(wrapper); @@ -190,15 +232,13 @@ DepartureBean departure = new DepartureBean(); String time = fields.get(0).asText().trim(); - logger.info("time:" + time); + if (time.equals("")) time = "0:00"; //Bane.dk bug work-around departure.setTime(time); String trainNumber = fields.get(1).asText(); - if (type.equalsIgnoreCase("S2")) //If it is S-train we need to extract the trainNumber - trainNumber = trainNumber + " " + extractTrainNumber(fields.get(2)); departure.setTrainNumber(trainNumber); String destination = fields.get(2).asText(); @@ -210,7 +250,7 @@ String status = fields.get(4).asText(); departure.setStatus(status); - String note = extractNote( fields.get(5) ); + String note = fields.get(5).asText(); departure.setNote(note); departureList.add(departure); @@ -218,9 +258,11 @@ } else { logger.warning("No departures found for station=" + stationcode + ", type=" + type); } + webClient.closeAllWindows(); + return departureList; - } + }*/ private int extractUpdated(HtmlElement updatedTd) { //extract the digit (in this case: 4) from "media/trafikinfo/opdater4.gif" @@ -252,20 +294,11 @@ } private String extractTrainNumber(HtmlElement trainTd) { - String number = ""; HtmlElement anchorElement = trainTd.getElementsByTagName("a").get(0); String href = anchorElement.getAttribute("href"); - String argstring = href.substring( href.indexOf('?') + 1); - - String args[] = argstring.split("&"); - for (String arg : args) { - String pair[] = arg.split("="); // Key=pair[0], Value=pair[1] - - if (pair[0].equalsIgnoreCase("TogNr")) - number = pair[1]; - } - + int pos = href.lastIndexOf('/'); + String number = href.substring(pos+1); return number; }