svn commit: r1416766 - /cxf/trunk/osgi/karaf/commands/src/main/java/org/apache/cxf/karaf/commands/ListBussesCommand.java

[ffang]

Author: ffang
Date: Tue Dec  4 03:54:49 2012
New Revision: 1416766

URL: http://svn.apache.org/viewvc?rev=1416766&view=rev
Log:
[CXF-4654]cxf:list-buses - Table layout should be aligned if bus name is long

Modified:
    
cxf/trunk/osgi/karaf/commands/src/main/java/org/apache/cxf/karaf/commands/ListBussesCommand.java

Modified: 
cxf/trunk/osgi/karaf/commands/src/main/java/org/apache/cxf/karaf/commands/ListBussesCommand.java
URL: 
http://svn.apache.org/viewvc/cxf/trunk/osgi/karaf/commands/src/main/java/org/apache/cxf/karaf/commands/ListBussesCommand.java?rev=1416766&r1=1416765&r2=1416766&view=diff
==============================================================================
--- 
cxf/trunk/osgi/karaf/commands/src/main/java/org/apache/cxf/karaf/commands/ListBussesCommand.java
 (original)
+++ 
cxf/trunk/osgi/karaf/commands/src/main/java/org/apache/cxf/karaf/commands/ListBussesCommand.java
 Tue Dec  4 03:54:49 2012
@@ -31,9 +31,10 @@ import org.apache.karaf.shell.console.Os
  */
 @Command(scope = "cxf", name = "list-busses", description = "Lists all CXF 
Busses.")
 public class ListBussesCommand extends OsgiCommandSupport {
-    protected static final String HEADER_FORMAT = "%-40s %-20s";
-    protected static final String OUTPUT_FORMAT = "[%-38s] [%-18s]";
-
+    protected static final int DEFAULT_BUSID_LENGTH = 38;
+    protected String headerFormat = "%-40s %-20s";
+    protected String outputFormat = "[%-38s] [%-18s]";
+   
     private CXFController cxfController;
 
     public void setController(CXFController controller) {
@@ -42,15 +43,29 @@ public class ListBussesCommand extends O
 
     protected Object doExecute() throws Exception {
         List<Bus> busses = cxfController.getBusses();
-        System.out.println(String.format(HEADER_FORMAT, "Name", "State"));
+        renderFormat(busses);
+        System.out.println(String.format(headerFormat, "Name", "State"));
 
         for (Bus bus : busses) {
             String state = "";
             if (bus instanceof CXFBusImpl) {
                 state = ((CXFBusImpl)bus).getState().toString();
             }
-            System.out.println(String.format(OUTPUT_FORMAT, bus.getId(), 
state));
+            System.out.println(String.format(outputFormat, bus.getId(), 
state));
         }
         return null;
     }
+
+    private void renderFormat(List<Bus> busses) {
+        int longestBusId = DEFAULT_BUSID_LENGTH;
+        for (Bus bus : busses) {
+            if (bus.getId().length() > longestBusId) {
+                longestBusId = bus.getId().length();
+            }
+        }
+        if (longestBusId > DEFAULT_BUSID_LENGTH) {
+            headerFormat = "%-" + (longestBusId + 2) + "s %-20s";
+            outputFormat = "[%-" + longestBusId + "s] [%-18s]";
+        }
+    }
 }