just formatted code better..

files/foldermenu.py

@@ -44,6 +44,7 @@ def setup_options():
     parser.add_argument("--version",action='version', version=VERSION)
     parser.add_argument("args",type=str,action="store",default="",nargs="?",
                       help="Arguments for the command, if -c used. The string will be re-parsed with shutils. Use '--' to skip local parsing.")
+
     options = parser.parse_args()
     if not os.path.exists(DEFAULTFILE):
         options.defaults = False
@@ -58,13 +59,13 @@ def ichr(i):
 
     if i < 10:
         return str(i)
-    i=i + 87
+    i += 87
     if i > 112:
-        i=i+1
+        i += 1
     if i > 119:
-        i=i+1
+        i += 1
     if i > 122:
-        i=i-122+64
+        i += 64 - 122
     return chr(i)
 
 class bc:
@@ -89,11 +90,12 @@ class bc:
         self.END = ''
 
     def pos(self,y,x):
-        return "\033["+str(y)+";"+str(x)+"H"
+        return "\033[%s;%sH"%( y, x )
 
     def posprint(self, y, x, s):
         sys.stdout.write( self.pos(y, x) + str(s) )
 
+
 class getch:
     def __init__(self):
         import sys, tty, termios
@@ -109,11 +111,9 @@ class getch:
             termios.tcsetattr(fd, termios.TCSADRAIN, old_settings)
         return ch
 
+
 class launch_item:
     ''' Class for launchable items '''
-    description=''
-    command=''
-    launcher=''
     def __init__(self, command, description, launcher):
         self.command = command
         self.description = description
@@ -121,18 +121,13 @@ class launch_item:
 
 class entry_collection:
     ''' Object containing the list items, and the printing methods '''
-    options=[]
-    entries=[]
-    dirs=[]
-    banner=[]
-    menu_keys=[ichr(i+1) for i in range(60)]
-    args=''
-    dir_mode=False
-    co=bc()
-    max_length=0
-
     def __init__(self, options):
         self.options = options
+        self.menu_keys=[ichr(i+1) for i in range(60)]
+        self.args = ''
+        self.co = bc()
+        self.dir_mode = False
+        self.max_length = 0
         self.initialize()
         if not self.options.colors:
             self.co.disable()
@@ -151,17 +146,19 @@ class entry_collection:
             self.max_length = max([len(e.description) for e in self.entries]) + 1
 
     def set_args(self, args):
-        self.args=args;
+        self.args = args
 
     def read_menu(self, menu_file = MENUFILE):
         ''' Read the menu file '''
         if os.path.exists(menu_file):
-            f=file(menu_file,'r')
+            with open(menu_file, 'rt') as f:
                 for row in f:
                     if row.strip() == '':
                         continue
                     if row[0:2] == '##':
-                    self.banner.append(row[2:].replace("\n","").replace("\r",""))
+                        self.banner.append(
+                            row[2:].replace("\n","").replace("\r","")
+                        )
                         continue
                     if row[0] == '#':
                         continue
@@ -170,15 +167,24 @@ class entry_collection:
                         row = [ "$" + row[0].strip(), row[0] ]
                     else:
                         row = [ row[0].strip(), row[1] ]
-                self.entries.append(launch_item(command=row[1],
+                    self.entries.append(
+                        launch_item(
+                            command = row[1],
                             description = row[0],
-                                           launcher="menu"))
+                            launcher = "menu"
+                        )
+                    )
+
     def read_folder(self):
         ''' Read folder contents, return executable files and dirs '''
 
-        self.dirs.append(launch_item(command='..',
+        self.dirs.append(
+            launch_item(
+                command = '..',
                 description = '..',
-                                launcher="dir"))
+                launcher = "dir"
+            )
+        )
         dirs = []
         executables = []
         for f in os.listdir('.'):
@@ -189,17 +195,23 @@ class entry_collection:
                 dirs.append(f)
         dirs.sort()
         for d in dirs:
-            self.dirs.append(launch_item(
+            self.dirs.append(
+                launch_item(
                     command = d,
                     description = d,
-                                launcher="dir"))
+                    launcher = "dir"
+                )
+            )
         if self.options.executables:
             executables.sort()
             for e in executables:
-                self.entries.append(launch_item(
+                self.entries.append(
+                    launch_item(
                         command = e,
                         description = e,
-                                    launcher="exec"))
+                        launcher = "exec"
+                    )
+                )
 
     def entry_color(self, launcher):
         if launcher == "dir":
@@ -212,10 +224,14 @@ class entry_collection:
     def menu(self):
         """ draws the menu at the top of the screen """
         if self.dir_mode:
-            helptext=".:executables"
+            helptext = "[.]executables"
             my_entries = self.dirs
         else:
-            helptext='-:args ('+self.co.END+self.args+self.co.YEL+')  .:folders'
+            helptext = '[.]folders [-]args (%s%s%s)'%(
+                self.co.END,
+                self.args,
+                self.co.YEL
+            )
             my_entries = self.entries
 
         maxrows, maxcolumns = termsize()
@@ -231,14 +247,34 @@ class entry_collection:
         else:
             pars = float(self.options.columns)
         if self.options.banner:
-            b=self.banner
+            banner = self.banner
         else:
-            b=[]
+            banner = []
-        blen=len(b)
+        blen = len(banner)
         cwd = os.path.basename( os.getcwd() )[0:15]
-        self.co.posprint(1,3,self.co.END+self.co.CLR+self.co.WHI+cwd+self.co.YEL+' Menu q/x:exit '+helptext+self.co.END)
+
-        for i,e in enumerate(b):
+        self.co.posprint(
-            self.co.posprint(i+2,0,e)
+            1,
+            1,
+            self.co.END + self.co.CLR
+        )
+        self.co.posprint(
+            1,
+            3,
+            "%s%s%s [q/x]exit %s%s"%(
+                self.co.WHI,
+                cwd,
+                self.co.YEL,
+                helptext,
+                self.co.END
+            )
+        )
+        for i, e in enumerate(banner):
+            self.co.posprint(
+                i+2,
+                0,
+                e
+            )
         rows = int( math.ceil( len(my_entries) / pars ) )
         while rows > maxrows:
             pars += 1
@@ -246,19 +282,38 @@ class entry_collection:
         maxcolumns = int(math.ceil(maxcolumns / pars))
         r = 1 + blen
         par = 1
-        for e,i in zip(my_entries,self.menu_keys):
+        for entry, key in zip(my_entries, self.menu_keys):
             if r - blen > rows:
                 par += 1
                 r = 1 + blen
-            printline=e.description
+            printline = entry.description
             if len(printline) > maxcolumns:
                 printline = printline[:maxcolumns] + "..."
             if par == 1:
-                self.co.posprint(r+1,2,self.co.WHI+i+self.co.END+' '+self.entry_color(e.launcher)+printline+self.co.END)
+                column = 2
+                border = ""
             else:
-                self.co.posprint(r+1,maxcolumns*(par-1),'| '+self.co.WHI+i+self.co.END+' '+self.entry_color(e.launcher)+printline+self.co.END)
+                column = maxcolumns * ( par - 1 )
+                border = '| '
+            self.co.posprint(
+                r+1,
+                column,
+                "%s%s%s%s %s%s%s"%(
+                    border,
+                    self.co.WHI,
+                    key,
+                    self.co.END,
+                    self.entry_color(entry.launcher),
+                    printline,
+                    self.co.END
+                )
+            )
             r += 1
-        self.co.posprint(rows+2+blen,0,"")
+        self.co.posprint(
+            rows + 2 + blen,
+            0,
+            "#"
+        )
 
     def list(self):
         """ draws the list at cursor """
@@ -267,15 +322,19 @@ class entry_collection:
         maxcolumns -= 10
         # heuristics for guessing column count
         if self.options.columns == 0:
-            pars=float(1)
+            pars = 1.0
             if len(self.entries) > 9:
-                pars=math.floor(maxcolumns/float(self.max_length))
+                pars = max(
-                pars=max(1,pars)
+                    1.0,
-                if len(self.entries)/pars < pars:
+                    math.floor(
+                        maxcolumns / float(self.max_length)
+                    )
+                )
                 while len(self.entries) / pars < pars:
                     pars -= 1
         else:
             pars = float(self.options.columns)
+
         rows = int(math.ceil( len(self.entries) / float(pars) ))
         maxcolumns = int(math.floor(maxcolumns / pars)) - 2
         # If names won't fit the columns, make sure at least 3 characters are visible
@@ -284,15 +343,13 @@ class entry_collection:
             origmaxrows -= 5
             origmaxcolumns -= 10
             while maxcolumns < 6:
-                pars=pars-1
+                pars -= 1
                 rows = int(math.ceil( len(self.entries) / float(pars) ))
                 maxcolumns = int(math.floor( origmaxcolumns / pars )) - 2
 
         self.max_length = min(maxcolumns, self.max_length)
         if self.options.horizontal:
-            foo=pars
+            pars, rows = rows, pars
-            pars=rows
-            rows=foo
         formatted = []
         for r in range(int(rows)):
             formatted.append([])
@@ -302,14 +359,21 @@ class entry_collection:
                     formatted[r][p] += ' '
         r = 0
         par = 0
-        for e,i in zip(self.entries,self.menu_keys):
+        for entry,key in zip(self.entries, self.menu_keys):
             if r >= rows:
-                par=1+par
+                par += 1
                 r = 0
-            printline=e.description[:(maxcolumns-3)]
+            printline = entry.description[:(maxcolumns-3)]
-            printline=printline+' '*(self.max_length-len(printline)-1)
+            printline += ' '*(self.max_length-len(printline)-1)
-            formatted[r][par]=self.co.WHI+i+self.co.END+' '+self.entry_color(e.launcher)+printline+self.co.END
+            formatted[r][par]="%s%s%s %s%s%s"%(
-            r=1+r
+                self.co.WHI,
+                key,
+                self.co.END,
+                self.entry_color(entry.launcher),
+                printline,
+                self.co.END
+            )
+            r += 1
         if self.options.horizontal:
             # let's shuffle the deck, and print values in horizontal order:
             formatted = zip(*formatted)
@@ -323,8 +387,6 @@ class entry_collection:
         ''' launch the given entry '''
 
         bg = False
-        wait=True
-        #Run the program in background
         idx = self.menu_keys.index(key)
         # note, no error checking here
         if self.dir_mode:
@@ -335,27 +397,35 @@ class entry_collection:
         command_str = self.entries[idx].command
 
         if command_str[-1] == '&':
+            #Run the program in background
             command_str = command_str[:-1]
             bg = True
         if len(self.args) > 0:
-            command_str=command_str+" "+self.args
+            command_str += ' ' + self.args
         if self.entries[idx].launcher == 'exec':
             command_str = './' + command_str
 
         if not self.options.command:
-            print('#$ '+command_str)
+            print(command_str)
         try:
             if bg:
-                subprocess.Popen(command_str, stderr=subprocess.PIPE, shell=True,executable="/bin/bash")
+                subprocess.Popen(
+                    command_str,
+                    stderr = subprocess.PIPE,
+                    shell=True,
+                    executable="/bin/bash"
+                )
             else:
-                subprocess.call(command_str, stderr=subprocess.STDOUT, shell=True,executable="/bin/bash")
+                subprocess.call(
+                    command_str,
+                    stderr = subprocess.STDOUT,
+                    shell = True,
+                    executable = "/bin/bash"
+                )
         except:
             print('Non-zero exit code: "' + command_str + '"')
 
-        if (self.options.command or self.options.once or bg):
+        if not (self.options.command or self.options.once or bg):
-            wait=False
-        
-        if wait:
             print('Press any key...')
             ch=getch()
             inkey=ord(ch.get())
@@ -398,10 +468,11 @@ def start_engines():
         entries.menu()
         inkey = ord(ch.get())
         #print('-'+str((inkey))+'-')
-        if inkey in [113,120,27,3,24,4]:
+        if inkey in (113, 120, 27, 3, 24, 4): # esc, q, x
             print('Exited in: ' + os.getcwd())
             sys.exit(0)
         if inkey == 45: # -
+            print('')
             readline.set_startup_hook(lambda: readline.insert_text(entries.args))
             args = raw_input('args: ')
             entries.set_args(args)
@@ -415,6 +486,7 @@ def start_engines():
                 sys.exit(0)
             entries.initialize()
 
+if __name__ == "__main__":
     start_engines()