[rfc][icedtea-web] bringing applets out of browser (part1)

[gitne at gmx.de]

> Sent: Friday, December 12th, 2014 at 05:12 PM
> From: "Jie Kang" <jkang at redhat.com>
> To: "Jiri Vanek" <jvanek at redhat.com>
> Cc: "IcedTea Distro List" <distro-pkg-dev at openjdk.java.net>
> Subject: Re: [rfc][icedtea-web] bringing applets out of browser (part1)
> 
> […]
> +            s.append("  </resources>\n");
> +            s.append("  <applet-desc\n");
> +            s.append("    name='").append(getTitle()).append("'\n");
> +         s.append("    main-class='").append(getStrippedMain()).append("'\n");
> +         s.append("    width='").append(getApplet().getWidth()).append("'\n");
> +         s.append("    height='").append(getApplet().getHeight()).append("'>\n");

Try merging strings into one string where possible here.

> +         if (!getApplet().getParameters().isEmpty()){
> +             Set<Map.Entry<String,String>>  prms = getApplet().getParameters().entrySet();
> +             for (Map.Entry<String, String> entry : prms) {
> +                 s.append("    <param name='").append(entry.getKey()).append("' value='").append(entry.getValue()).append("'/>\n");                 
> +             }
> +         }
> +           s.append("  </applet-desc>\n");
> +            s.append("</jnlp>\n");

The strings in these last two statements here can be merged into one too.

Also, please concatenate the append operations with the derefenrence operator where possible. You can still group the string building process into logical chunks using line breaks for better readability. ;-) This produces much better code quality utilizing the stack which leads to higher performance. StringBuilder and the append() method have been developer exactly for this kind of purpose.

Jacob