PEP8

LICENSE

@@ -0,0 +1,30 @@
+sshch
+
+sshch is released under the DWTW license
+
+This program is free software; you can redistribute it and/or modify it under the terms of the Do What Thou Wilt License.
+
+DO WHAT THAU WILT
+TO PUBLIC LICENSE
+
+Version 2.5
+
+Everyone is permitted to copy and distribute verbatim or modified copies of this license document, and changing it is allowed as long as the name is changed.
+
+TERMS AND CONDITIONS FOR COPYING, DISTRIBUTION AND MODIFICATION
+
+0. Do what thou wilt shall be the whole of the Law.
+
+Anyone is allowed to copy and distribute the copies of this license agreement in whole or in part, as well as modify it without any other limitations. 
+
+DWTW – a license with a single requirement: DO WHAT THOU WILT 
+
+The license provides more freedom than any other one (such as GPL or BSD) and does not require saving the license text on copying. 
+
+DWTW – an accomplished and eligible license for free text (including the software, documentation and artwork). 
+
+The license does not contain "no warranty" clause. DWTW can be used in countries that do not legally acknowledge the transition to public domain. 
+
+Summary: 
+
+An author-creator gives his or her source code to the world for free, without becoming distracted by worldly thinking regarding how and why the others will use it.

README.md

@@ -1,3 +1,25 @@
-Simpe python script for fast access to ssh hosts.
-
-under DWTWL 2.5 license: https://soundragon.su/license/license.html
+SSH connection manager with curses interface
+======
+sshch is released under DWTWL 2.5 license
+### Screenshot
+![sshch](https://raw.githubusercontent.com/zlaxy/sshch/master/sshch_screenshot.png)
+### Installing
+To install for all users:
+```
+sudo python setup.py install
+```
+To install just for current user:
+```
+mkdir ~/.local/bin
+cp sshch/sshch ~/.local/bin/
+```
+### Using
+To run curses interface:
+```
+sshch
+```
+To run command line help:
+```
+sshch -h
+```
+**If you want to use unsafe 'password' feature you must install 'sshpass' first.**

setup.py

@@ -0,0 +1,35 @@
+#!/usr/bin/env python
+
+"""Setup script for sshch"""
+
+def main():
+    from distutils.core import setup
+
+    setup(name='sshch',
+          author='zlaxy',
+          url='https://github.com/zlaxy/sshch/',
+          description='Ssh connection manager',
+          license='DWTWL 2.5',
+          version='0.55',
+          py_modules=['sshch'],
+          scripts=['sshch/sshch'],
+
+          # http://pypi.python.org/pypi?%3Aaction=list_classifiers
+          classifiers=[
+              'Development Status :: 4 - Beta',
+              'Environment :: Console :: Curses',
+              'Intended Audience :: System Administrators',
+              'License :: Freeware',
+              'Natural Language :: English',
+              'Operating System :: POSIX',
+              'Programming Language :: Python :: 2.7',
+              'Topic :: Internet',
+              'Topic :: System :: Networking',
+              'Topic :: System :: Systems Administration',
+              'Topic :: Utilities'])
+
+    return 0
+
+if __name__ == '__main__':
+    import sys
+    sys.exit(main())

sshch/sshch

@@ -4,20 +4,21 @@
 from __future__ import division
 from os import path
 from sys import argv
-from math import *
+from math import ceil
 from getpass import getpass
 from optparse import OptionParser
-from curses import textpad, panel
+from curses import textpad
 import ConfigParser
 import subprocess
 import base64
 import curses
 
 # https://github.com/zlaxy/sshch
-version="0.5"
+version = "0.55"
 # path to conf file, default: ~/.config/sshch.conf
 conf_file = path.expanduser("~") + '/.config/sshch.conf'
 
+
 def AddNewAlias(alias):
     if not conf.has_section(alias):
         conf.add_section(alias)
@@ -26,21 +27,25 @@ def AddNewAlias(alias):
     else:
         return "error: '" + alias + "' already exists"
 
+
 def SetAliasString(alias, string):
     conf.set(alias, "exec_string", string)
     conf.write(open(conf_file, "w"))
 
+
 def SetPassword(alias, string):
     string = base64.b64encode(base64.b16encode(
                               base64.b32encode(string)))
     conf.set(alias, "password", string)
     conf.write(open(conf_file, "w"))
 
+
 def RemoveAliases(aliases):
     for alias in aliases:
         conf.remove_section(alias)
         conf.write(open(conf_file, "w"))
 
+
 def ConnectAlias(alias, command=False):
     print "Connecting to " + alias + "..."
     exec_string = ""
@@ -51,47 +56,52 @@ def ConnectAlias(alias, command=False):
     exec_string = exec_string + conf.get(alias, "exec_string")
     if command:
         exec_string = exec_string + " " + command
-    p = subprocess.Popen(exec_string, shell=True)
-    streamdata = p.communicate()[0]
+    # Variables bellow is newer used
+    subprocess.Popen(exec_string, shell=True)
+    # p.communicate()[0]
     print "... " + alias + " session finished."
 
+
 def CMDAdd(alias):
     result = AddNewAlias(alias)
-    if result == True:
-        prompt_add = ("Enter connection string for new alias " +
-                      "(example: ssh user@somehost.com):\n")
+    if result:
+        prompt_add = ("".join(["Enter connection string for new alias ",
+                               "(example: ssh user@somehost.com):\n"]))
         string = ""
         while string == "":
-            string = raw_input (prompt_add)
+            string = raw_input(prompt_add)
         SetAliasString(alias, string)
     else:
         print result
 
+
 def CMDEdit(alias):
     if conf.has_section(alias):
-        prompt_edit = ("Enter connection string for existing alias " +
-                       "(example: ssh user@somehost.com):\n")
+        prompt_edit = ("".join(["Enter connection string for existing alias ",
+                       "(example: ssh user@somehost.com):\n"]))
         string = ""
         while string == "":
-            string = raw_input (prompt_edit)
+            string = raw_input(prompt_edit)
         SetAliasString(alias, string)
     else:
         print "error: '" + alias + "' alias is not exists"
 
+
 def CMDPassword(alias):
     if conf.has_section(alias):
         prompt_pass = ("[UNSAFE] Enter password for sshpass: ")
         string = ""
-        string = getpass (prompt_pass)
-        if not string == "": SetPassword(alias, string)
+        string = getpass(prompt_pass)
+        if not string == "":
+            SetPassword(alias, string)
     else:
         print "error: '" + alias + "' alias is not exists"
 
+
 def CMDRemove(alias):
     if conf.has_section(alias):
-        prompt_remove = ("Type 'yes' if you sure to remove '" +
-                         alias + "' alias: ")
-        string = raw_input (prompt_remove)
+        prompt_remove = ("Type 'yes' if you sure to remove '" + alias + "' alias: ")
+        string = raw_input(prompt_remove)
         if string == "yes":
             RemoveAliases([alias])
         else:
@@ -99,6 +109,7 @@ def CMDRemove(alias):
     else:
         print "error: '" + alias + "' alias is not exists."
 
+
 def CMDConnect(aliases, command=False):
     for alias in aliases:
         if conf.has_section(alias):
@@ -106,13 +117,17 @@ def CMDConnect(aliases, command=False):
         else:
             print "error: '" + alias + "' alias is not exists"
 
+
 def CMDList(option, opt, value, parser):
     print ', '.join(str(p) for p in conf.sections())
 
+
 def CMDFullList(option, opt, value, parser):
-    print '\n'.join((str(p) + " - " + conf.get(p, "exec_string") +
-        (" [password]" if conf.has_option(p, "password") else ""))
-        for p in conf.sections())
+    for p in conf.sections():
+        to_print = "".join([str(p), " - ", conf.get(p, "exec_string"),
+                            (" [password]" if conf.has_option(p, "password") else ""), "\n"])
+        print(to_print)
+
 
 def CursesConnect(screen, aliases, command=False):
     curses.endwin()
@@ -121,21 +136,25 @@ def CursesConnect(screen, aliases, command=False):
     print "Press 'enter' to continue."
     screen.getch()
 
+
 def CursesExit(error=False):
     curses.endwin()
-    if error: print error
+    if error:
+        print error
     exit()
 
+
 def CursesTextpadConfirm(value):
     if value == 10:
         value = 7
     return value
 
+
 def CursesTextpad(screen, h, w, y, x, title="", value="",
                   text_colorpair=0, deco_colorpair=0):
     new_window = curses.newwin(h + 3, w + 2, y - 1, x - 1)
-    title_window = new_window.subwin(1, w , y , x)
-    title_window.addnstr(0, 0, title, w , text_colorpair)
+    title_window = new_window.subwin(1, w, y, x)
+    title_window.addnstr(0, 0, title, w, text_colorpair)
     title_window.refresh()
     sub_window = new_window.subwin(h, w, y + 1, x)
     textbox_field = textpad.Textbox(sub_window, insert_mode=True)
@@ -143,10 +162,11 @@ def CursesTextpad(screen, h, w, y, x, title="", value="",
     new_window.box()
     new_window.attroff(deco_colorpair)
     new_window.refresh()
-    sub_window.addnstr(0, 0 , value, h * w, text_colorpair)
+    sub_window.addnstr(0, 0, value, h * w, text_colorpair)
     sub_window.attron(text_colorpair)
     return textbox_field
 
+
 def CursesPanel(screen, h, w, y, x, text,
                 text_colorpair=0, deco_colorpair=0, confirm=0):
     new_window = curses.newwin(h, w, y, x)
@@ -154,10 +174,9 @@ def CursesPanel(screen, h, w, y, x, text,
     new_window.attron(deco_colorpair)
     new_window.box()
     new_window.attroff(deco_colorpair)
-    sub_window = new_window.subwin(h - 2, w - 2 , y + 1 , x + 1)
+    sub_window = new_window.subwin(h - 2, w - 2, y + 1, x + 1)
     sub_window.insstr(0, 0, text)
-#    sub_window.addnstr(0, 0, text, ((h - 2) * (w - 2) - 1))
-    panel = curses.panel.new_panel(new_window)
+    curses.panel.new_panel(new_window)
     curses.panel.update_panels()
     screen.refresh()
     if confirm == "password":
@@ -194,67 +213,76 @@ def CursesPanel(screen, h, w, y, x, text,
     else:
         screen.getch()
 
+
 def CMDOptions():
     class FormatedParser(OptionParser):
+
         def format_epilog(self, formatter):
             return self.epilog
     usage = "usage: %prog [options] [aliases]"
     progname = path.basename(__file__)
-    epilog = ("Examples:\n  " + 
-        progname + " existingalias\n  " +
-        progname + " -a newremoteserver\n  " + 
-        progname + " --edit=newremoteserver -p newremoteserver\n  " +
-        progname + ' -c "ls -l" newremoteserver\n  ' + 
-        progname + " -c reboot existingalias newremoteserver\n"
-        "Examples of connection string:\n  " +
-        "ssh user@somehost.com\n  " +
-        "ssh gates@8.8.8.8 -p 667\n  " +
-        "ssh root@somehost.com -t tmux a\n" +
-        "Also, you can edit config file manually: " + conf_file + "\n")
+    epilog = ("".join(["Examples:\n  ",
+                       progname, " existingalias\n  ",
+                       progname, " -a newremoteserver\n  ",
+                       progname, " --edit=newremoteserver -p newremoteserver\n  ",
+                       progname, ' -c "ls -l" newremoteserver\n  ',
+                       progname, " -c reboot existingalias newremoteserver\n",
+                       "Examples of connection string:\n  ",
+                       "ssh user@somehost.com\n  ",
+                       "ssh gates@8.8.8.8 -p 667\n  ",
+                       "ssh root@somehost.com -t tmux a\n",
+                       "Also, you can edit config file manually: ", conf_file, "\n"]))
     opts = FormatedParser(usage=usage, version="%prog " + version,
                           epilog=epilog)
-    opts.add_option('-l', '--list', action = "callback",
-        callback=CMDList, help="show list of all existing aliases")
-    opts.add_option('-f', '--fulllist', action = "callback",
-        callback=CMDFullList, help=("show list of all existing " +
-        "aliases with connection strings"))
+    opts.add_option('-l', '--list', action="callback",
+                    callback=CMDList, help="show list of all existing aliases")
+    opts.add_option('-f', '--fulllist', action="callback",
+                    callback=CMDFullList, help=("show list of all existing " +
+                                                "aliases with connection strings"))
     opts.add_option('-a', '--add', action="store", type="string",
-        dest="add", metavar="alias", default=False,
-        help="add new alias for connection string")
+                    dest="add", metavar="alias", default=False,
+                    help="add new alias for connection string")
     opts.add_option('-c', '--command', action="store", type="string",
-        dest="command", metavar="command", default=False,
-        help="add command for executing alias")
+                    dest="command", metavar="command", default=False,
+                    help="add command for executing alias")
     opts.add_option('-e', '--edit', action="store", type="string",
-        dest='edit', metavar="alias", default=False,
-        help="edit existing connection string")
+                    dest='edit', metavar="alias", default=False,
+                    help="edit existing connection string")
     opts.add_option('-p', '--password', action="store", type="string",
-        dest='password', metavar="alias", default=False,
-        help="set and store password for sshpass [UNSAFE]")
+                    dest='password', metavar="alias", default=False,
+                    help="set and store password for sshpass [UNSAFE]")
     opts.add_option('-r', '--remove', action="store", type="string",
-        dest='remove', metavar="alias", default=False,
-        help="remove existing alias of connection string")
+                    dest='remove', metavar="alias", default=False,
+                    help="remove existing alias of connection string")
     options, alias = opts.parse_args()
-    if options.add: CMDAdd(options.add)
-    if options.edit: CMDEdit(options.edit)
-    if options.password: CMDPassword(options.password)
-    if options.remove: CMDRemove(options.remove)
-    if alias: CMDConnect(alias, options.command)
+    if options.add:
+        CMDAdd(options.add)
+    if options.edit:
+        CMDEdit(options.edit)
+    if options.password:
+        CMDPassword(options.password)
+    if options.remove:
+        CMDRemove(options.remove)
+    if alias:
+        CMDConnect(alias, options.command)
 
 # curses template from: https://stackoverflow.com/a/30828805/6224462
+
+
 def CursesMain():
-    help_screen = (" Press:\n" +
-    "  'z'/'x' or arrows - navigation\n" +
-    "  'a'/'F2' - add new alias\n" +
-    "  'e'/'F4' - edit existing alias\n" +
-    "  'p'/'F6' - set alias's password for sshpass [UNSAFE]\n" +
-    "  'space'/'insert' - select\n" +
-    "  'r'/'F8' - remove selected alias/aliases\n" +
-    "  'c'/'F3' - execute specific command with selected alias/aliases\n" +
-    "  'enter'/'F9' - connect to selected alias/aliases\n" +
-    "  'q'/'F10' - quit\n" +
-    " Run program with '--help' option to view command line help.\n" +
-    " Also, you can edit config file manually:\n" +
-    "  " + conf_file)
+    help_screen = ("".join([" Press:\n",
+                            "  'z'/'x' or arrows - navigation\n",
+                            "  'a'/'F2' - add new alias\n",
+                            "  'e'/'F4' - edit existing alias\n",
+                            "  'p'/'F6' - set alias's password for sshpass [UNSAFE]\n",
+                            "  'space'/'insert' - select\n",
+                            "  'r'/'F8' - remove selected alias/aliases\n",
+                            "  'c'/'F3' - execute specific command with selected alias/aliases\n",
+                            "  'enter'/'F9' - connect to selected alias/aliases\n",
+                            "  'q'/'F10' - quit\n",
+                            " Run program with '--help' option to view command line help.\n",
+                            " Also, you can edit config file manually:\n",
+                            "  ", conf_file]))
     strings = conf.sections()
     row_num = len(strings)
     selected_strings = [" " for i in range(0, row_num + 1)]
@@ -279,21 +307,20 @@ def CursesMain():
     page = 1
     for i in range(1, max_row + 1):
         if row_num == 0:
-            box.addnstr(1, 1, "There aren't any aliases yet. Press 'a'" +
-                       " to add new one.", width - 6, highlight_text)
+            box.addnstr(1, 1, "There aren't any aliases yet. Press 'a' to add new one.",
+                        width - 6, highlight_text)
         else:
-            if (i == position):
-                box.addnstr(i, 2, "[" + selected_strings[i] + "] " +
-                    str(i) + " " + strings[i - 1] + " (" +
-                    conf.get(strings[i - 1], "exec_string") + ")" +
-                    (" [password]" if conf.has_option(strings[i - 1],
-                    "password") else ""), width - 6, highlight_text)
+            if conf.has_option(strings[i - 1], "password"):
+                password = " [password]"
             else:
-                box.addnstr(i, 2, "[" + selected_strings[i] + "] " +
-                    str(i) + " " + strings[i - 1] + " (" +
-                    conf.get(strings[i - 1], "exec_string") + ")" +
-                    (" [password]" if conf.has_option(strings[i - 1],
-                    "password") else ""), width - 6, normal_text)
+                password = ""
+            exec_string = ["[", selected_strings[i], "] ", str(i), " ",
+                           strings[i - 1], " (", conf.get(strings[i - 1], "exec_string"),
+                           ")", password]
+            if (i == position):
+                box.addnstr(i, 2, "".join(exec_string), width - 6, highlight_text)
+            else:
+                box.addnstr(i, 2, "".join(exec_string), width - 6, normal_text)
             if i == row_num:
                 break
     screen.refresh()
@@ -301,35 +328,36 @@ def CursesMain():
     key_pressed = screen.getch()
     while True:
         if key_pressed == ord('q') or key_pressed == ord(
-                              'Q') or key_pressed == (
-                               27) or key_pressed == curses.KEY_F10:
+            'Q') or key_pressed == (
+                27) or key_pressed == curses.KEY_F10:
             CursesExit()
         if key_pressed == ord('h') or key_pressed == ord(
-                              'H') or key_pressed == curses.KEY_F1:
+                'H') or key_pressed == curses.KEY_F1:
             CursesPanel(screen, height - 4, width - 6, 2, 3,
                         help_screen, normal_text, highlight_text)
         if key_pressed == ord('a') or key_pressed == ord(
-                              'A') or key_pressed == curses.KEY_F2:
+                'A') or key_pressed == curses.KEY_F2:
             new_alias_textpad = CursesTextpad(screen, 1, width - 8,
-                (height // 2) - 1, 4, "Enter new alias:", "",
-                normal_text, highlight_text)
+                                              (height // 2) - 1, 4, "Enter new alias:", "",
+                                              normal_text, highlight_text)
             add_alias = new_alias_textpad.edit(CursesTextpadConfirm)
             if not add_alias.rstrip() == "":
                 add_result = AddNewAlias(add_alias.rstrip())
-                if not add_result == True: CursesPanel(screen, 3,
-                    width - 6, (height // 2) - 1, 3, add_result,
-                    normal_text, highlight_text)
+                if not add_result:
+                    CursesPanel(screen, 3,
+                                width - 6, (height // 2) - 1, 3, add_result,
+                                normal_text, highlight_text)
                 else:
                     add_string = ""
                     while add_string.rstrip() == "":
                         string_textpad = CursesTextpad(screen, 3,
-                            width - 8, (height // 2) - 1, 4,
-                            "Enter full execution string:",
-                            "ssh ", normal_text, highlight_text)
+                                                       width - 8, (height // 2) - 1, 4,
+                                                       "Enter full execution string:",
+                                                       "ssh ", normal_text, highlight_text)
                         add_string = string_textpad.edit(
                             CursesTextpadConfirm)
                     SetAliasString(add_alias.rstrip(),
-                        add_string.replace("\n", "").rstrip())
+                                   add_string.replace("\n", "").rstrip())
                     strings = conf.sections()
                     row_num = len(strings)
                     selected_strings.append(" ")
@@ -340,20 +368,20 @@ def CursesMain():
             edit_string = ""
             while edit_string.rstrip() == "":
                 string_textpad = CursesTextpad(screen, 3, width - 8,
-                    (height // 2) - 1, 4, "Enter new execution string:",
-                    conf.get(strings[position - 1], "exec_string"),
-                    normal_text, highlight_text)
+                                               (height // 2) - 1, 4, "Enter new execution string:",
+                                               conf.get(strings[position - 1], "exec_string"),
+                                               normal_text, highlight_text)
                 edit_string = string_textpad.edit(CursesTextpadConfirm)
             SetAliasString(strings[position - 1],
-                edit_string.replace("\n", "").rstrip())
+                           edit_string.replace("\n", "").rstrip())
             strings = conf.sections()
         if (key_pressed == ord('p') or key_pressed == ord(
                 'P') or key_pressed == curses.KEY_F6) and row_num != 0:
             set_password = ""
             set_password = CursesPanel(screen, 4, width - 6,
-                (height // 2) - 1, 3,
-                " Enter user password for sshpass and press 'enter':" +
-                "\n>", normal_text, highlight_text, "password")
+                                       (height // 2) - 1, 3,
+                                       " Enter user password for sshpass and press 'enter':\n>",
+                                       normal_text, highlight_text, "password")
             if not set_password == "":
                 SetPassword(strings[position - 1], set_password)
         if (key_pressed == ord('r') or key_pressed == ord(
@@ -364,15 +392,16 @@ def CursesMain():
                 if selected_strings[i] == "*":
                     selected.append(strings[i - 1])
             if len(selected) > 0:
-                remove_confirm = ("Are you sure to remove " +
-                    str(len(selected)) + " selected aliases? (y/N)")
+                remove_confirm = (
+                    "".join(["Are you sure to remove ",
+                             str(len(selected)), " selected aliases? (y/N)"]))
             else:
-                remove_confirm = ("Are you sure to remove '" +
-                    strings[position - 1] + "' alias? (y/N)")
+                remove_confirm = ("".join(["Are you sure to remove '",
+                                           strings[position - 1], "' alias? (y/N)"]))
                 selected.append(strings[position - 1])
             remove_result = CursesPanel(screen, 4, width - 6,
-                (height // 2) - 1, 3, remove_confirm, normal_text,
-                highlight_text, "remove")
+                                        (height // 2) - 1, 3, remove_confirm, normal_text,
+                                        highlight_text, "remove")
             if remove_result == "confirm":
                 RemoveAliases(selected)
                 strings = conf.sections()
@@ -391,12 +420,14 @@ def CursesMain():
             if not len(selected) > 0:
                 selected.append(strings[position - 1])
             command_textpad = CursesTextpad(screen, 3, width - 8,
-                (height // 2) - 1, 4,
-                "Enter specific command to execute with selected " +
-                "alias/aliases:", "", normal_text, highlight_text)
+                                            (height // 2) - 1, 4,
+                                            "".join([
+                                                "Enter specific command to execute with selected ",
+                                                "alias/aliases:"]
+                                            ), "", normal_text, highlight_text)
             command_string = command_textpad.edit(CursesTextpadConfirm)
             CursesConnect(screen, selected,
-                command_string.replace("\n", "").rstrip())
+                          command_string.replace("\n", "").rstrip())
         if (key_pressed == ord("\n") or key_pressed == (
                 curses.KEY_F9)) and row_num != 0:
             selected = []
@@ -410,7 +441,8 @@ def CursesMain():
                 curses.KEY_IC)) and row_num != 0:
             if selected_strings[position] == ' ':
                 selected_strings[position] = '*'
-            else: selected_strings[position] = ' '
+            else:
+                selected_strings[position] = ' '
             if page == 1:
                 if position < i:
                     position = position + 1
@@ -428,7 +460,7 @@ def CursesMain():
                     page = page + 1
                     position = 1 + (max_row * (page - 1))
         if key_pressed == curses.KEY_DOWN or key_pressed == ord(
-                                     'x') or key_pressed == ord('X'):
+                'x') or key_pressed == ord('X'):
             if page == 1:
                 if position < i:
                     position = position + 1
@@ -446,7 +478,7 @@ def CursesMain():
                     page = page + 1
                     position = 1 + (max_row * (page - 1))
         if key_pressed == curses.KEY_UP or key_pressed == ord(
-                                   'z') or key_pressed == ord('Z'):
+                'z') or key_pressed == ord('Z'):
             if page == 1:
                 if position > 1:
                     position = position - 1
@@ -457,12 +489,12 @@ def CursesMain():
                     page = page - 1
                     position = max_row + (max_row * (page - 1))
         if key_pressed == curses.KEY_LEFT or (key_pressed ==
-                          curses.KEY_PPAGE):
+                                              curses.KEY_PPAGE):
             if page > 1:
                 page = page - 1
                 position = 1 + (max_row * (page - 1))
         if key_pressed == curses.KEY_RIGHT or (key_pressed ==
-                          curses.KEY_NPAGE):
+                                               curses.KEY_NPAGE):
             if page < pages:
                 page = page + 1
                 position = (1 + (max_row * (page - 1)))
@@ -472,25 +504,22 @@ def CursesMain():
         for i in range(1 + (max_row * (page - 1)), max_row + 1 +
                        (max_row * (page - 1))):
             if row_num == 0:
-                box.addnstr(1, 1, "There aren't any aliases yet. " +
-                    "Press 'a' to add new one.",  width - 6,
-                    highlight_text)
+                box.addnstr(1, 1, "There aren't any aliases yet. Press 'a' to add new one.",
+                            width - 6, highlight_text)
             else:
-                if (i + (max_row * (page - 1)) == (position +
-                    (max_row * (page - 1)))):
-                    box.addnstr(i - (max_row * (page - 1)), 2, "[" +
-                        selected_strings[i] + "] " + str(i) + " " +
-                        strings[i - 1] + " (" + conf.get(strings[i - 1],
-                        "exec_string") + ")" + (" [password]" if 
-                        conf.has_option(strings[i - 1], "password") else
-                        ""), width - 6, highlight_text)
+                if conf.has_option(strings[i - 1], "password"):
+                    password = " [password]"
                 else:
-                    box.addnstr(i - (max_row * (page - 1)), 2, "[" +
-                        selected_strings[i] + "] " + str(i) + " " +
-                        strings[i - 1] + " (" + conf.get(strings[i - 1],
-                        "exec_string") + ")" + (" [password]" if 
-                        conf.has_option(strings[i - 1], "password") else
-                        ""), width - 6, normal_text)
+                    password = ""
+                exec_string = ["[", selected_strings[i], "] ", str(i), " ",
+                               strings[i - 1], " (", conf.get(strings[i - 1], "exec_string"),
+                               ")", password]
+                if (i + (max_row * (page - 1)) == (position + (max_row * (page - 1)))):
+                    box.addnstr(i - (max_row * (page - 1)), 2, "".join(
+                        exec_string), width - 6, highlight_text)
+                else:
+                    box.addnstr(i - (max_row * (page - 1)), 2, "".join(
+                        exec_string), width - 6, normal_text)
                 if i == row_num:
                     break
         screen.refresh()
@@ -498,6 +527,7 @@ def CursesMain():
         key_pressed = screen.getch()
     CursesExit()
 
+
 if __name__ == "__main__":
     conf = ConfigParser.RawConfigParser()
     if not path.exists(conf_file):
@@ -509,8 +539,7 @@ if __name__ == "__main__":
         except KeyboardInterrupt:
             exit()
         except ConfigParser.Error:
-            print ("Error: can't parse your config file, please check" +
-                " it manually or make new one")
+            print ("Error: can't parse your config file, please check it manually or make new one")
             exit()
     else:
         try:
@@ -518,8 +547,8 @@ if __name__ == "__main__":
         except KeyboardInterrupt:
             CursesExit()
         except ConfigParser.NoOptionError:
-            CursesExit("Error: can't parse your config file, please " +
-                "check it manually or make new one")
+            CursesExit("".join(["Error: can't parse your config file, please ",
+                       "check it manually or make new one"]))
         except curses.error:
-            CursesExit("Error: can't show some curses element, maybe " +
-                "your terminal is too small")
+            CursesExit("".join(["Error: can't show some curses element, maybe ",
+                       "your terminal is too small"]))