Tidy up comments and syntax, also add main() func.

Author: Jared Manning

daily_checks.py

@@ -1,41 +1,42 @@
-# Script Name		: daily_checks.py
-# Author				: Craig Richards
-# Created				: 07th December 2011
-# Last Modified		: 01st May 2013
-# Version				: 1.4
+# Script Name	: daily_checks.py
+# Author		: Craig Richards
+# Created		: 07th December 2011
+# Last Modified	: 01st May 2013
+# Version		: 1.5
+#
+# Modifications	: 1.1 Removed the static lines for the putty sessions, it now reads a file, loops through and makes the connections.
+#				: 1.2 Added a variable filename=sys.argv[0] , as when you use __file__ it errors when creating an exe with py2exe.
+#				: 1.3 Changed the server_list.txt file name and moved the file to the config directory.
+#				: 1.4 Changed some settings due to getting a new pc
+#				: 1.5 Tidy comments and syntax
+#
+# Description	: This simple script loads everything I need to carry out the daily checks for our systems.
 
-# Modifications		: 1.1 Removed the static lines for the putty sessions, it now reads a file, loops through and makes the connections.
-#							: 1.2 Added a variable filename=sys.argv[0] , as when you use __file__ it errors when creating an exe with py2exe.
-#							: 1.3 Changed the server_list.txt file name and moved the file to the config directory.
-#							: 1.4 Changed some settings due to getting a new pc
-
-# Description			: This simple script loads everything I need to carry out the daily checks for our systems.
-
-import platform					# Load the Library Module
-import os							# Load the Library Module
-import subprocess				# Load the Library Module 
-import sys							# Load the Library Module
+import platform		# Load Modules
+import os
+import subprocess
+import sys
 
 from time import strftime		# Load just the strftime Module from Time
 
-def clear_screen():								# Function to clear the screen
-	if os.name == "posix":					# Unix/Linux/MacOS/BSD/etc
-		os.system('clear')						# Clear the Screen
+def clear_screen():				# Function to clear the screen
+	if os.name == "posix":		# Unix/Linux/MacOS/BSD/etc
+		os.system('clear')		# Clear the Screen
 	elif os.name in ("nt", "dos", "ce"):	# DOS/Windows
-		os.system('CLS')						# Clear the Screen
+		os.system('CLS')					# Clear the Screen
 
-def print_docs():									# Function to print the daily checks automatically
+def print_docs():							# Function to print the daily checks automatically
   print "Printing Daily Check Sheets:"
   # The command below passes the command line string to open word, open the document, print it then close word down
   subprocess.Popen(["C:\\Program Files (x86)\Microsoft Office\Office14\winword.exe", "P:\\\\Documentation\\Daily Docs\\Back office Daily Checks.doc", "/mFilePrintDefault", "/mFileExit"]).communicate() 
 
-def putty_sessions():										# Function to load the putty sessions I need
-  for server in open(conffilename):					# Open the file server_list.txt, loop through reading each line - 1.1 -Changed - 1.3 Changed name to use variable conffilename
-    subprocess.Popen(('putty -load '+server))		# Open the PuTTY sessions - 1.1
+def putty_sessions():						# Function to load the putty sessions I need
+  for server in open(conffilename):			# Open the file server_list.txt, loop through reading each line - 1.1 -Changed - 1.3 Changed name to use variable conffilename
+    subprocess.Popen(('putty -load '+server))	# Open the PuTTY sessions - 1.1
 
 def rdp_sessions():
   print "Loading RDP Sessions:"
-  subprocess.Popen("mstsc eclr.rdp")				# Open up a terminal session connection and load the euroclear session
+  subprocess.Popen("mstsc eclr.rdp")		# Open up a terminal session connection and load the euroclear session
 
 def euroclear_docs():
   # The command below opens IE and loads the Euroclear password document
@@ -44,15 +45,20 @@ def euroclear_docs():
 # End of the functions			
 
 # Start of the Main Program
+def main():
+	filename = sys.argv[0]							# Create the variable filename
+	confdir = os.getenv("my_config")				# Set the variable confdir from the OS environment variable - 1.3
+	conffile = ('daily_checks_servers.conf')		# Set the variable conffile - 1.3
+	conffilename = os.path.join(confdir, conffile)	# Set the variable conffilename by joining confdir and conffile together - 1.3
+	clear_screen()									# Call the clear screen function
+
+	# The command below prints a little welcome message, as well as the script name, the date and time and where it was run from.
+	print "Good Morning " + os.getenv('USERNAME') + ", " + filename, "ran at", strftime("%Y-%m-%d %H:%M:%S"), "on",platform.node(), "run from",os.getcwd()
+
+	print_docs()									# Call the print_docs function
+	putty_sessions()								# Call the putty_session function
+	rdp_sessions()									# Call the rdp_sessions function
+	euroclear_docs()								# Call the euroclear_docs function
 
-filename=sys.argv[0]									# Create the variable filename
-confdir = os.getenv("my_config")					# Set the variable confdir from the OS environment variable - 1.3
-conffile = ('daily_checks_servers.conf')			# Set the variable conffile - 1.3
-conffilename=os.path.join(confdir, conffile)	# Set the variable conffilename by joining confdir and conffile together - 1.3
-clear_screen()											# Call the clear screen function
-# The command below prints a little welcome message, as well as the script name, the date and time and where it was run from.
-print "Good Morning " + os.getenv('USERNAME') + ", " + filename, "ran at", strftime("%Y-%m-%d %H:%M:%S"), "on",platform.node(), "run from",os.getcwd()
-print_docs()												# Call the print_docs function
-putty_sessions()										# Call the putty_session function
-rdp_sessions()											# Call the rdp_sessions function
-euroclear_docs()										# Call the euroclear_docs function
+if __name__ == '__main__':
+	main()