checkstyle sweep

[fanfix.git] / src / jexer / TDirectoryTreeItem.java

diff --git a/src/jexer/TDirectoryTreeItem.java b/src/jexer/TDirectoryTreeItem.java
index 01c9b46a4170dad07854cc65898d46e80511d712..456ad9259c4f1acc641de437b7d88330cbbe8111 100644 (file)
--- a/src/jexer/TDirectoryTreeItem.java
+++ b/src/jexer/TDirectoryTreeItem.java
@@ -1,4 +1,4 @@
-/**
+/*
  * Jexer - Java Text User Interface
  *
  * License: LGPLv3 or later
@@ -42,51 +42,60 @@ import java.util.LinkedList;
 public class TDirectoryTreeItem extends TTreeItem {
 
     /**
-     * Directory entry corresponding to this list item.
+     * File corresponding to this list item.
      */
-    File dir;
+    private File file;
+
+    /**
+     * Get the File corresponding to this list item.
+     *
+     * @return the File
+     */
+    public final File getFile() {
+        return file;
+    }
 
     /**
      * Called when this item is expanded or collapsed.  this.expanded will be
      * true if this item was just expanded from a mouse click or keypress.
      */
     @Override
-    public void onExpand() {
-        // System.err.printf("onExpand() %s\n", dir);
+    public final void onExpand() {
+        // System.err.printf("onExpand() %s\n", file);
 
-        if (dir == null) {
+        if (file == null) {
             return;
         }
         getChildren().clear();
 
         // Make sure we can read it before trying to.
-        if (dir.canRead()) {
+        if (file.canRead()) {
             setSelectable(true);
         } else {
             setSelectable(false);
         }
-        assert (dir.isDirectory());
+        assert (file.isDirectory());
         setExpandable(true);
 
-        if ((isExpanded() == false) || (isExpandable() == false)) {
+        if (!isExpanded() || !isExpandable()) {
             getTreeView().reflow();
             return;
         }
 
-        for (File file: dir.listFiles()) {
+        for (File f: file.listFiles()) {
             // System.err.printf("   -> file %s %s\n", file, file.getName());
 
-            if (file.getName().startsWith(".")) {
+            if (f.getName().startsWith(".")) {
                 // Hide dot-files
                 continue;
             }
-            if (!file.isDirectory()) {
+            if (!f.isDirectory()) {
                 continue;
             }
 
             try {
                 TDirectoryTreeItem item = new TDirectoryTreeItem(getTreeView(),
-                    file.getCanonicalPath(), false, false);
+                    f.getCanonicalPath(), false, false);
 
                 item.level = this.level + 1;
                 getChildren().add(item);
@@ -109,7 +118,9 @@ public class TDirectoryTreeItem extends TTreeItem {
      * @throws IllegalArgumentException if this function is called
      */
     @Override
-    public final TTreeItem addChild(final String text, final boolean expanded) {
+    public final TTreeItem addChild(final String text,
+        final boolean expanded) throws IllegalArgumentException {
+
         throw new IllegalArgumentException("Do not call addChild(), use onExpand() instead");
     }
 
@@ -118,6 +129,7 @@ public class TDirectoryTreeItem extends TTreeItem {
      *
      * @param view root TTreeView
      * @param text text for this item
+     * @throws IOException if a java.io operation throws
      */
     public TDirectoryTreeItem(final TTreeView view,
         final String text) throws IOException {
@@ -131,6 +143,7 @@ public class TDirectoryTreeItem extends TTreeItem {
      * @param view root TTreeView
      * @param text text for this item
      * @param expanded if true, have it expanded immediately
+     * @throws IOException if a java.io operation throws
      */
     public TDirectoryTreeItem(final TTreeView view, final String text,
         final boolean expanded) throws IOException {
@@ -146,58 +159,59 @@ public class TDirectoryTreeItem extends TTreeItem {
      * @param expanded if true, have it expanded immediately
      * @param openParents if true, expand all paths up the root path and
      * return the root path entry
+     * @throws IOException if a java.io operation throws
      */
     public TDirectoryTreeItem(final TTreeView view, final String text,
         final boolean expanded, final boolean openParents) throws IOException {
 
         super(view, text, false);
 
-        List<String> parentPaths = new LinkedList<String>();
+        List<String> parentFiles = new LinkedList<String>();
         boolean oldExpanded = expanded;
 
         // Convert to canonical path
-        File rootPath = new File(text);
-        rootPath = rootPath.getCanonicalFile();
+        File rootFile = new File(text);
+        rootFile = rootFile.getCanonicalFile();
 
-        if (openParents == true) {
+        if (openParents) {
             setExpanded(true);
 
             // Go up the directory tree
-            File parent = rootPath.getParentFile();
+            File parent = rootFile.getParentFile();
             while (parent != null) {
-                parentPaths.add(rootPath.getName());
-                rootPath = rootPath.getParentFile();
-                parent = rootPath.getParentFile();
+                parentFiles.add(rootFile.getName());
+                rootFile = rootFile.getParentFile();
+                parent = rootFile.getParentFile();
             }
         }
-        dir = rootPath;
-        if (rootPath.getParentFile() == null) {
+        file = rootFile;
+        if (rootFile.getParentFile() == null) {
             // This is a filesystem root, use its full name
-            setText(rootPath.getCanonicalPath());
+            setText(rootFile.getCanonicalPath());
         } else {
             // This is a relative path.  We got here because openParents was
             // false.
-            assert (openParents == false);
-            setText(rootPath.getName());
+            assert (!openParents);
+            setText(rootFile.getName());
         }
         onExpand();
 
-        if (openParents == true) {
-            TDirectoryTreeItem childPath = this;
-            Collections.reverse(parentPaths);
-            for (String p: parentPaths) {
-                for (TWidget widget: childPath.getChildren()) {
+        if (openParents) {
+            TDirectoryTreeItem childFile = this;
+            Collections.reverse(parentFiles);
+            for (String p: parentFiles) {
+                for (TWidget widget: childFile.getChildren()) {
                     TDirectoryTreeItem child = (TDirectoryTreeItem) widget;
                     if (child.getText().equals(p)) {
-                        childPath = child;
-                        childPath.setExpanded(true);
-                        childPath.onExpand();
+                        childFile = child;
+                        childFile.setExpanded(true);
+                        childFile.onExpand();
                         break;
                     }
                 }
             }
             unselect();
-            getTreeView().setSelected(childPath);
+            getTreeView().setSelected(childFile);
             setExpanded(oldExpanded);
         }
         getTreeView().reflow();