[xml2js] provide the shape name and dimensions as comments (#30)

Even if not directly used in the js code, this information can be helpful to get
context.

Author: tbouffard

README.md

@@ -111,6 +111,9 @@ java -cp target/mxgraph-svg2shape-*-jar-with-dependencies.jar com.mxgraph.xml2js
 
 The tool writes the generated code in the console
 ``` javascript
+// shape: shape-01
+// width: 30.5
+// height: 40.2
 // foreground
 canvas.begin();
 canvas.moveTo(19.87, 0);

src/main/java/com/mxgraph/xml2js/Xml2Js.java

@@ -1,9 +1,6 @@
 package com.mxgraph.xml2js;
 
-import org.w3c.dom.Document;
-import org.w3c.dom.Element;
-import org.w3c.dom.Node;
-import org.w3c.dom.NodeList;
+import org.w3c.dom.*;
 import org.xml.sax.InputSource;
 
 import javax.xml.parsers.DocumentBuilder;
@@ -84,7 +81,13 @@ private static Document parseXml(String xml) {
     }
 
     public void parseShape(Node shape) {
-        logInfo(format("Parsing shape '%s'", shape.getAttributes().getNamedItem("name").getTextContent()));
+        AttributesTextContent shapeAttributes = new AttributesTextContent(shape.getAttributes());
+        String name = shapeAttributes.text("name");
+        logInfo(format("Parsing shape '%s'", name));
+        generateComment(format("shape: %s", name));
+        generateComment(format("width: %s", shapeAttributes.text("w")));
+        generateComment(format("height: %s", shapeAttributes.text("h")));
+
         NodeList shapeChildren = shape.getChildNodes();
         for (int i = 0; i < shapeChildren.getLength(); i++) {
             Node item = shapeChildren.item(i);
@@ -96,9 +99,21 @@ public void parseShape(Node shape) {
         }
     }
 
+    private static class AttributesTextContent {
+        private final NamedNodeMap attributes;
+
+        public AttributesTextContent(NamedNodeMap attributes) {
+            this.attributes = attributes;
+        }
+
+        public String text(String attributeName) {
+            return attributes.getNamedItem(attributeName).getTextContent();
+        }
+    }
+
     private void parseForeground(Node foreground) {
         logDebug("Parsing foreground");
-        generateCode("// foreground");
+        generateComment("foreground");
         NodeList children = foreground.getChildNodes();
 
         for (int i = 0; i < children.getLength(); i++) {
@@ -126,6 +141,10 @@ private void generateCode(String code) {
         codeLines.add(code);
     }
 
+    private void generateComment(String comment) {
+        generateCode("// " + comment);
+    }
+
     private void parsePath(Node path) {
         logDebug("Parsing path");
         generateCanvasMethodCall("begin()");

src/test/java/com/mxgraph/xml2js/Xml2JsTest.java

@@ -21,6 +21,9 @@ void parse_code_lines() {
         String xml = builder.build();
 
         assertThat(xml2Js.parseCodeLines(xml)).containsExactly(
+                "// shape: shape-01",
+                "// width: 284.03",
+                "// height: 327.7",
                 "// foreground",
                 "canvas.begin();",
                 "canvas.moveTo(141.9, 0);",