refactored code for better readability

[meviz.git] / src / main / java / eu / svjatoslav / meviz / htmlindexer / layouts / MixedLayout.java

diff --git a/src/main/java/eu/svjatoslav/meviz/htmlindexer/layouts/MixedLayout.java b/src/main/java/eu/svjatoslav/meviz/htmlindexer/layouts/MixedLayout.java
index f336ad1..4e6b5bf 100755 (executable)
--- a/src/main/java/eu/svjatoslav/meviz/htmlindexer/layouts/MixedLayout.java
+++ b/src/main/java/eu/svjatoslav/meviz/htmlindexer/layouts/MixedLayout.java
@@ -21,6 +21,7 @@ import eu.svjatoslav.meviz.htmlindexer.Constants;
 import eu.svjatoslav.meviz.htmlindexer.IndexingContext;
 import eu.svjatoslav.meviz.htmlindexer.Layout;
 import eu.svjatoslav.meviz.htmlindexer.metadata.Dimension;
+import eu.svjatoslav.meviz.htmlindexer.metadata.DirectoryMetadata;
 import eu.svjatoslav.meviz.htmlindexer.metadata.fileTypes.AbstractFile;
 import eu.svjatoslav.meviz.htmlindexer.metadata.fileTypes.GeneralFile;
 import eu.svjatoslav.meviz.htmlindexer.metadata.fileTypes.Picture;
@@ -44,7 +45,8 @@ public class MixedLayout implements Layout {
        private IndexingContext indexingContext;
 
        @Override
-       public void enlistDirectory(final AbstractFile directory) {
+       public void enlistDirectory(final AbstractFile directory,
+                       final DirectoryMetadata parentDirectoryMetadata) {
                dirHtml.append("<TR>");
                dirHtml.append("<TD><b><a href=\""
                                + indexer.getDirectoryUrl(directory, indexingContext)
@@ -54,7 +56,8 @@ public class MixedLayout implements Layout {
        }
 
        @Override
-       public void enlistFile(final AbstractFile file) {
+       public void enlistFile(final AbstractFile file,
+                       final DirectoryMetadata parentDirectoryMetadata) {
 
                final String fullFileName = file.fileName;
 
@@ -69,93 +72,100 @@ public class MixedLayout implements Layout {
        }
 
        @Override
-       public void enlistImage(final AbstractFile abstractFile) {
+       public void enlistImage(final Picture picture,
+                       final DirectoryMetadata parentDirectoryMetadata) {
                try {
+                       enlistImageInternal(picture);
+               } catch (final Exception e) {
+                       logger.error(
+                                       "Error generating thumbnails for image file: "
+                                                       + e.toString()
+                                                       + ", thumbnail generation skipped, enlisting as ordinary file instead.",
+                                       e);
+                       enlistFile(picture, parentDirectoryMetadata);
+               }
+       }
 
-                       final Picture picture = (Picture) abstractFile;
-
-                       final Dimension imageSize = picture.getDimensions();
-
-                       final List<Dimension> recommendedDimensions = Picture
-                                       .getThumbnailDimensions(imageSize);
-
-                       Dimension listingDimension = new Dimension(1500, 450);
+       private String enlistImageDimensions(final Picture picture) {
+               final StringBuilder html = new StringBuilder();
+
+               final List<Dimension> recommendedDimensions = Picture
+                               .getThumbnailDimensions(picture.getDimensions());
+
+               int count = 0;
+               for (final Dimension dimension : recommendedDimensions) {
+                       count++;
+                       if (count > 1)
+                               html.append(", ");
+                       html.append("<a href=\""
+                                       + indexer.getThumbnailUrl(picture, dimension,
+                                                       indexingContext) + "\">");
+                       html.append(dimension.width + "x" + dimension.height);
+                       html.append("</a>");
+               }
 
-                       if (listingDimension.width > imageSize.width)
-                               if (listingDimension.height > imageSize.height)
-                                       listingDimension = imageSize;
+               return html.toString();
+       }
 
-                       final String listingThumbnailUrl = indexer.getThumbnailUrl(picture,
-                                       listingDimension, indexingContext);
+       private void enlistImageInternal(final Picture picture) {
 
-                       filesHtml.append("<div style=\"float:left; margin: 10px;\">");
-                       filesHtml.append("<a href=\""
-                                       + indexingContext.getGlobalUrl()
-                                       + UrlParamEncoder.encode(indexingContext.getLocalUrl()
-                                                       + "/" + picture.fileName) + "\">\n");
+               filesHtml.append("<div style=\"float:left; margin: 10px;\">");
 
-                       filesHtml.append("<img border=\"0\" src=\"" + listingThumbnailUrl
-                                       + "\"/></a>\n");
-                       filesHtml.append("<br/><b>"
-                                       + FilePathParser
-                                                       .getFileNameWithoutExtension(picture.fileName)
-                                       + "</b><br/>(");
+               // clickable image thumbnail
+               filesHtml.append("<a href=\""
+                               + indexingContext.getGlobalUrl()
+                               + UrlParamEncoder.encode(indexingContext.getLocalUrl() + "/"
+                                               + picture.fileName) + "\">\n");
+               filesHtml.append("<img border=\"0\" src=\""
+                               + getImageThumbnailUrl(picture) + "\"/></a><br/>\n");
 
-                       int count = 0;
-                       for (final Dimension dimension : recommendedDimensions) {
-                               count++;
-                               if (count > 1)
-                                       filesHtml.append(", ");
-                               filesHtml.append("<a href=\""
-                                               + indexer.getThumbnailUrl(picture, dimension,
-                                                               indexingContext) + "\">");
-                               filesHtml.append(dimension.width + "x" + dimension.height);
-                               filesHtml.append("</a>");
-                       }
+               // file name
+               filesHtml.append("<b>"
+                               + FilePathParser.getFileNameWithoutExtension(picture.fileName)
+                               + "</b><br/>");
 
-                       filesHtml.append(")\n");
-                       filesHtml.append("</div>");
+               // links to thumbnails of various sizes
+               filesHtml.append("(" + enlistImageDimensions(picture) + ")\n");
 
-               } catch (final Exception e) {
-                       logger.error(
-                                       "Error generating thumbnails for image file: "
-                                                       + e.toString()
-                                                       + ", thumbnail generation skipped, enlisting as ordinary file instead.",
-                                       e);
-                       enlistFile(abstractFile);
-               }
+               filesHtml.append("</div>");
        }
 
        /** {@inheritDoc} */
        @Override
-       public void enlistOgv(final AbstractFile abstractFile)
+       public void enlistOgv(final AbstractFile abstractFile,
+                       final DirectoryMetadata parentDirectoryMetadata)
                        throws UnsupportedEncodingException {
 
                final GeneralFile file = (GeneralFile) abstractFile;
 
+               filesHtml.append("<div style=\"margin: 30px; clear: both;\">");
+
+               // watchable video
                filesHtml
-                               .append("<div style=\"margin: 30px; clear: both;\">"
-                                               + "<video preload=\"metadata\" controls=\"controls\" tabindex=\"0\">\n");
+                               .append("<video preload=\"metadata\" controls=\"controls\" tabindex=\"0\">\n");
                filesHtml
                                .append("<source type=\"video/ogg\" codecs=\"theora, vorbis\" src=\""
                                                + indexingContext.getGlobalUrl()
                                                + UrlParamEncoder.encode(indexingContext.getLocalUrl()
                                                                + "/" + file.fileName) + "\">\n");
                filesHtml.append("</source>\n");
-
                filesHtml.append("</video>\n");
-
                filesHtml.append("<br/>\n");
 
+               // video title
                filesHtml
                                .append("Video: <b>"
                                                + FilePathParser
                                                                .getFileNameWithoutExtension(file.fileName)
                                                + "<b/>  &nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;");
+
+               // video download link
                filesHtml.append("<a href=\""
                                + indexingContext.getGlobalUrl()
                                + UrlParamEncoder.encode(indexingContext.getLocalUrl() + "/"
-                                               + file.fileName) + "\">download as OGV</a></div>\n");
+                                               + file.fileName) + "\">download as OGV</a>");
+
+               filesHtml.append("</div>\n");
 
                filesHtml.append("\n");
 
@@ -205,6 +215,19 @@ public class MixedLayout implements Layout {
                return finalHtml.toString();
        }
 
+       private String getImageThumbnailUrl(final Picture picture) {
+               final Dimension imageSize = picture.getDimensions();
+
+               Dimension listingDimension = new Dimension(1500, 450);
+
+               if (listingDimension.width > imageSize.width)
+                       if (listingDimension.height > imageSize.height)
+                               listingDimension = imageSize;
+
+               return indexer.getThumbnailUrl(picture, listingDimension,
+                               indexingContext);
+       }
+
        @Override
        public void init(final String galleryTitle, final List<String> path,
                        final AbstractIndexer indexer, final IndexingContext indexingContext) {