--- android/MarketStats/src/dk/thoerup/marketstats/ShowStats.java	2010/04/23 14:13:39	662
+++ android/MarketStats/src/dk/thoerup/marketstats/ShowStats.java	2010/07/05 09:28:40	957
@@ -1,16 +1,14 @@
 package dk.thoerup.marketstats;
 
 import java.io.IOException;
-import java.io.PrintWriter;
 import java.net.InetSocketAddress;
 import java.util.ArrayList;
-import java.util.Collections;
-import java.util.Date;
-import java.util.Formatter;
 import java.util.Locale;
+import java.util.concurrent.TimeUnit;
 import java.util.logging.Logger;
 
 import javax.servlet.ServletException;
+import javax.servlet.annotation.WebServlet;
 import javax.servlet.http.HttpServlet;
 import javax.servlet.http.HttpServletRequest;
 import javax.servlet.http.HttpServletResponse;
@@ -18,22 +16,24 @@
 import net.spy.memcached.MemcachedClient;
 
 import com.gc.android.market.api.MarketSession;
-import com.gc.android.market.api.MarketSession.Callback;
-import com.gc.android.market.api.model.Market.App;
 import com.gc.android.market.api.model.Market.AppsRequest;
-import com.gc.android.market.api.model.Market.AppsResponse;
 import com.gc.android.market.api.model.Market.CommentsRequest;
-import com.gc.android.market.api.model.Market.ResponseContext;
 
 
+@WebServlet(urlPatterns={"/ShowStats"})
 public class ShowStats extends HttpServlet {
 	private static final long serialVersionUID = 1L;
 
-	final int TIMEOUT = 30*60;
+	static final int MAXCOMMENTS = 30;
+	static final int APP_TIMEOUT = 30*60;
+	static final int COMMENT_TIMEOUT = APP_TIMEOUT * 4;
+	
 	static final Logger log = Logger.getLogger(ShowStats.class.getName());
 
 	String login;
 	String password;
+	
+	MemcachedClient memcache = null;
 
 
 	@Override
@@ -42,112 +42,134 @@
 
 		login = getServletContext().getInitParameter("login");
 		password = getServletContext().getInitParameter("password");
+		
+		try {
+			memcache = new MemcachedClient(new InetSocketAddress("localhost", 11211));
+		} catch (IOException e) {
+			throw new ServletException(e);
+		}
 	}
 
-	protected String doLookup(String appId) throws IOException {
-		MemcachedClient c = new MemcachedClient(new InetSocketAddress("localhost", 11211));
+	@Override
+	public void destroy() {
+		super.destroy();
+		
+		memcache.shutdown(3, TimeUnit.SECONDS);
+		memcache = null;
+	}
 
+	protected AppBean lookupApp(String query) throws IOException {
+		
 
-		String key = "marketstats:" + appId;
-		String response = (String) c.get(key);
+		String key = "marketstats:" + query.replace(' ', '_');
+		//String response = (String) memcache.get(key);
+		AppBean response = (AppBean) memcache.get(key);
 
 		if (response == null) {
-			response = doLookupWorker(appId);
-			c.set(key, TIMEOUT, response); 
-			response += "<!-- new lookup -->";
-		} else {
-			response += "<!-- from memcached -->";
-		}
+			response = lookupAppWorker(query);
+			if (response != null) {
+				memcache.set(key, APP_TIMEOUT, response);
+			}
+		} 
 		return response;
 	}
 
-
-	protected String doLookupWorker(String appId) {
-		final StringBuilder sb = new StringBuilder();
-
+	protected AppBean lookupAppWorker(String query) {
 
 		MarketSession session = new MarketSession();
 		session.login(login,password);
 
 
 		AppsRequest appsRequest = AppsRequest.newBuilder()
-		.setAppId(appId)
-		.setStartIndex(0).setEntriesCount(10)
+		.setQuery(query)
+		.setStartIndex(0)
+		.setEntriesCount(1)
 		.setWithExtendedInfo(true)
 		.build();
-
+		
+		AppsCallback appsCb = new AppsCallback();
+		session.append(appsRequest, appsCb);
+		session.flush();
+		
+		return appsCb.getResult();
+	}	
+	
+	CommentsRequest buildCommentRequest(String appId, int start, int count) {
 		CommentsRequest commentsRequest = CommentsRequest.newBuilder()
 		.setAppId(appId)
-		.setStartIndex(0)
-		.setEntriesCount(10)
+		.setStartIndex(start)
+		.setEntriesCount(count)
 		.build();
-
-
-
-
-		Callback<AppsResponse> appsCb = new Callback<AppsResponse>() {
-			@Override
-			public void onResult(ResponseContext context, AppsResponse response) {
-				//System.out.println("Response : " + response);
-				//sb.append("Response: " + response + "\n");
-
-				Formatter form = new Formatter(sb);
-				App app = response.getApp(0);
-				sb.append( "<h2>" + app.getTitle() + "</h2>");
-				sb.append("Ver: " + app.getVersion() + " (" + app.getVersionCode() + ")\n"  );
-				sb.append("Ratingcount: " + app.getRatingsCount() + "\n");
-
-				sb.append("Rating: "  );
-				double rating = Double.parseDouble( app.getRating() );
-				form.format("%.4f", rating);
-				sb.append("\n");
-
-				sb.append("Downloads: " + app.getExtendedInfo().getDownloadsCountText() + " (" + app.getExtendedInfo().getDownloadsCount() + ")\n" );
-
-				sb.append("\n");
-
-			}
-		};
-
+		
+		return commentsRequest;
+	}
+	
+	private ArrayList<CommentBean> loadComments(String appId) {
+		
 		ArrayList<CommentBean> commentBeans = new ArrayList<CommentBean>();
-
+		MarketSession session = new MarketSession();
+		session.login(login,password);
+		
 		CommentCallback commentsCb = new CommentCallback();
 		commentsCb.setList( commentBeans ); 
 
-		commentsCb.setLocale("en");
-		session.setLocale( Locale.ENGLISH );
-
-		session.append(appsRequest, appsCb);
-		session.append(commentsRequest, commentsCb);
-		session.flush();
-
-		commentsCb.setLocale( "da" );
-		session.setLocale( new Locale("da") );
-		session.append(commentsRequest, commentsCb);
-		session.flush();
-
-		Collections.sort(commentBeans);
-
-		sb.append("-----------------------------------------------------------------\n");
-		for (CommentBean c : commentBeans) {
-			sb.append("User: " + c.author + " ("  + c.locale + ")\n");
-			sb.append("Rating: " + c.rating + "\n");
-			sb.append("Time: " + new Date(c.time).toString() + "\n");
-			sb.append( c.text + "\n");
-			sb.append("\n");
-
-		}
-
+		session.setLocale( Locale.ROOT );
+		
+		
+		
+		int start = 0;
+		do {
+			int count = 10;
+			if (start > 0)				
+				count = Math.min(10, commentsCb.getEntryCount() );
+			
+			//log.warning("count=" + count + " start=" + start + " entryCount=" + commentsCb.getEntryCount() );
+			CommentsRequest commentsRequest = buildCommentRequest(appId,start,count);			
+			session.append(commentsRequest, commentsCb);
+			start +=10;
+			
+			CommentsRequest commentsRequest2 = buildCommentRequest(appId,start,count);			
+			session.append(commentsRequest2, commentsCb);
+			session.flush();
+			
+			start +=10;
+			
+			if (start >= MAXCOMMENTS)
+				break; //emergency brake
+			
+		} while ( start < commentsCb.getEntryCount() );
+		
+		return commentBeans;
+	}
 
-		return sb.toString();
+	
+	
+	@SuppressWarnings("unchecked")
+	public ArrayList<CommentBean> getComments(AppBean app) {		
+		String key = "marketstats:comments:" + app.getId();
+		ArrayList<CommentBean>  comments =  (ArrayList<CommentBean>) memcache.get(key);
+
+		if (comments == null) {
+			comments = loadComments(app.getId());
+			memcache.set(key, COMMENT_TIMEOUT, comments); 
+		} 
+		return comments;
 	}
 
 	protected void doGet(HttpServletRequest request, HttpServletResponse response) throws ServletException, IOException {
-		String appId = request.getParameter("appId");
-
-		response.setContentType("text/html");
-		PrintWriter out = response.getWriter();
-		out.print( "<html><body><pre>" + doLookup(appId) + "</pre></body></html>" );
+		String query = request.getParameter("query");
+		
+		AppBean app = lookupApp(query);
+		if (app != null) {
+			request.setAttribute("app", app);
+		
+			ArrayList<CommentBean> comments = getComments(app);
+			request.setAttribute("comments", comments);
+		
+			getServletContext().getRequestDispatcher("/statsview.jsp").forward(request, response);
+		} else {
+			response.getWriter().print("No app found with query=" + query);
+		}
 	}
 
 }