SystemInformation: show a progress bar

ctrueden · ctrueden · commit 045242f6a7d2 · 2014-07-18T15:55:13.000-05:00
For whatever reason, computing all the system information takes several
seconds. So let's report progress as we go.

This might still not be granular enough, depending on which loop(s) are
taking the most time. Can be improved further later as needed.
diff --git a/src/main/java/org/scijava/plugins/commands/debug/SystemInformation.java b/src/main/java/org/scijava/plugins/commands/debug/SystemInformation.java
@@ -45,6 +45,7 @@
 import org.scijava.ItemIO;
 import org.scijava.app.App;
 import org.scijava.app.AppService;
+import org.scijava.app.StatusService;
 import org.scijava.command.Command;
 import org.scijava.plugin.Parameter;
 import org.scijava.plugin.Plugin;
@@ -76,19 +77,29 @@ public class SystemInformation implements Command {
 	@Parameter
 	private AppService appService;
 
+	@Parameter
+	private StatusService statusService;
+
 	@Parameter(label = "System Information", type = ItemIO.OUTPUT)
 	private String info;
 
 	// -- Runnable methods --
 
 	@Override
 	public void run() {
+		int progress = 0, max = 8;
+		statusService.showStatus(0, max, "Gathering system information");
+
 		final StringBuilder sb = new StringBuilder();
 
 		// dump basic version information (similar to the status bar)
+
 		sb.append(appService.getApp().getInfo(false) + NL);
 
+		statusService.showProgress(++progress, max);
+
 		// dump information about available SciJava applications
+
 		final Map<String, App> apps = appService.getApps();
 		for (final String name : apps.keySet()) {
 			final App app = apps.get(name);
@@ -104,6 +115,8 @@ public void run() {
 			}
 		}
 
+		statusService.showProgress(++progress, max);
+
 		// dump all available Maven metadata on the class path
 
 		final List<POM> poms = POM.getAllPOMs();
@@ -125,6 +138,8 @@ public void run() {
 			}
 		}
 
+		statusService.showProgress(++progress, max);
+
 		// output libraries in sorted order
 		final ArrayList<POM> sortedPOMs = new ArrayList<POM>(poms);
 		Collections.sort(sortedPOMs);
@@ -151,15 +166,21 @@ public void run() {
 			if (orgURL != null) sb.append("\torganization URL = " + orgURL + NL);
 		}
 
+		statusService.showProgress(++progress, max);
+
 		// compute the set of known plugin types
+
 		final List<PluginInfo<?>> plugins = context.getPluginIndex().getAll();
 		final HashSet<Class<? extends SciJavaPlugin>> pluginTypeSet =
 			new HashSet<Class<? extends SciJavaPlugin>>();
 		for (final PluginInfo<?> plugin : plugins) {
 			pluginTypeSet.add(plugin.getPluginType());
 		}
 
+		statusService.showProgress(++progress, max);
+
 		// convert to a list of plugin types, sorted by fully qualified class name
+
 		final ArrayList<Class<? extends SciJavaPlugin>> pluginTypes =
 			new ArrayList<Class<? extends SciJavaPlugin>>(pluginTypeSet);
 		Collections.sort(pluginTypes, new Comparator<Class<?>>() {
@@ -171,17 +192,27 @@ public int compare(final Class<?> c1, final Class<?> c2) {
 
 		});
 
+		statusService.showProgress(++progress, max);
+
 		// dump the list of available plugins, organized by plugin type
+
 		for (final Class<? extends SciJavaPlugin> pluginType : pluginTypes) {
 			dumpPlugins(sb, pluginType);
 		}
 
+		statusService.showProgress(++progress, max);
+
 		// dump system properties
+
 		sb.append(NL);
 		sb.append("-- System properties --" + NL);
 		sb.append(getSystemProperties());
 
+		statusService.showProgress(++progress, max);
+
 		info = sb.toString();
+
+		statusService.clearStatus();
 	}
 
 	// -- Utility methods --
