Cleaned Up the code

I made some modifications to the code,
I mainly made the README better With Usage and prerequisites.

Signed-off-by: Aditya Prasad <adityaprasadiskool@gmail.com>

Author: anantham

.gitignore

@@ -52,3 +52,5 @@ docs/_build/
 
 # PyBuilder
 target/
+# thumb files are used by windows to cache a image for thumbnails
+[Tt]humbs.db

README.md

@@ -1,8 +1,16 @@
-##Python-Scripts
-====================
+## Python-Scripts
 
 **A _motley_ collection of python scripts. These are the different programs I wrote for fun.**
 
+### Usage
+
+* Install python 2.7.9 in your system. You can find it [here][5].
+
+* Just download this file to your computer OR copy the code into a text editor and save it as filename.py
+
+* Double click on the file to run OR open the command line there and type 
+	'python filename.py'
+
 * **Root.py**
 	This is a program which takes a integer greater than or equal to one and 
 	finds its root to the specified accuracy. It was made mainly to compute 
@@ -43,4 +51,5 @@
 [1]: https://superuser.com/questions/330297/automate-logging-in-through-sonicwall/785792?noredirect=1#comment1023176_785792
 [2]: http://relativetoaditya.blogspot.in/2014/12/maximum-subarray.html
 [3]: https://en.wikipedia.org/wiki/Minimum_spanning_tree
-[4]: https://stackoverflow.com/questions/14369739/creating-adjacency-lists-from-dicts-in-python/27380835#27380835
+[4]: https://stackoverflow.com/questions/14369739/creating-adjacency-lists-from-dicts-in-python/27380835#27380835
+[5]: https://www.python.org/ftp/python/2.7.9/python-2.7.9.msi

root.py

@@ -1,23 +1,44 @@
+# btw I KNOW that import math.sqrt() and sqrt(<number>) can do this :p
 resp="n"
 while(not(resp=="y" or resp=="Y")):
-    print "enter the number of which u want to find root (greater than or equal to 1)"
+    print "Enter the number of which u want to find root (greater than 1)"
     userinput=raw_input()
+    
     length=len(userinput)
-    no=int(userinput)
-    print "enter the no of decimals to which u want to find root"
-    deci=int(raw_input())
+    try:
+        no=int(userinput)
+    except ValueError:
+        print "INVALID INPUT!!!! Exiting program"
+        break
+    if(no<1):
+        print "Please enter number greater than 1"
+        break
+    
+    print "Enter the no of decimals to which u want to find root, Maximum 11 digits"
+    try:
+        deci=int(raw_input())
+    except ValueError:
+        print "INVALID INPUT!!!! Exiting program"
+        break
+    
+    if(deci>11):
+        print "sorry This script cannot perform that function, sorry"
+        # We can increase this using the decimal module.
+        break
+    
     #now we need to get the intial amount by which ans should be varied if no is a single
     #digit no then by .1 if its a n digit no then by 10 to the power n-1
     if length==1:
         multi=10**(-2)
     elif length>1:
         multi=10**(length-3)
     ans=no/2
+    
     #to find root(no) to a high degree of precision
     while(not(len(str(ans))==deci+length+1)):
         #if the point at which if we add prec*opti we overshoot and if we subtract we undersell..
         #then we optimize the number that is added, ie make it smaller
-
+        
         #basically better way to do this, use patterns
         if( ans**2<no and (ans+multi)**2>no):
             multi=multi/10
@@ -41,3 +62,5 @@
     print "Do You want to exit the program??! y/n"
     resp=raw_input()
 
+dummy = raw_input()
+

scrape_results.py

@@ -11,6 +11,9 @@
 # because otherwise python doesnt recognise the error I specified and throws a NameError: name 'NoSuchElementException' is not defined
 from selenium.common.exceptions import NoSuchElementException
 
+START_ROLL_NUMBER = 110113006;
+END_ROLL_NUMBER = 110113099
+
 # open a window of firefox
 driver = webdriver.Firefox()
 
@@ -46,7 +49,7 @@ class student:
 
 	GPA = '7.57'
 
-for roll_number in range(110113000,110113100):
+for roll_number in range(START_ROLL_NUMBER,END_ROLL_NUMBER+1):
 
     # intialize a empty list in which the students data will be stored
     temp_data=[]

sonicwall_login.py

@@ -13,7 +13,8 @@
 driver.get("https://192.168.20.1/auth1.html")
 # The next line is an assertion to confirm that title has “Sonic” word in it: (not really neccesary :p)
 # This is used to confirm that the webpage is the right one
-assert "Sonic" in driver.title
+# some sites may not have it
+# assert "Sonic" in driver.title
 # we use the 'name' tag to get a handle to the username and password  this finds the appropriate box.
 user = driver.find_element_by_name("userName")
 passwd = driver.find_element_by_name("pwd")

xkcd.py

@@ -3,40 +3,63 @@
 from bs4 import BeautifulSoup
 
 
-
-print "Enter from which xkcd comic onwards should be downloaded"
+print "\nEnter from which xkcd comic onwards should be downloaded"
 start=int(raw_input())
-print "Enter till which xkcd comic should be downloaded (latest is 1445)"
+print "\nEnter till which xkcd comic should be downloaded \n(latest is somewhere around 1475)"
+print "If you want till the latest one just enter a large value \nbut keep a eye on whether you see a lot of 404's cause that signifies the end."
 end=int(raw_input())
 
 def download(url, filepath):
-    page=urllib2.urlopen(url)
-
-    soup=BeautifulSoup(page)
+    try:
+        page = urllib2.urlopen(url)
+    except urllib2.HTTPError:
+        print "lol, not funny joke, No comic for 404 guys! "
+        print "if u see this a lot, all available comics have been downloaded"
+        return
 
-    finalurl=soup.find("div", {"id": "comic"}).contents[1].get("src")
-
-    image=urllib.urlopen(finalurl)
+    soup = BeautifulSoup(page)
+    
+    finalurl = soup.find("div", {"id": "comic"}).contents[1].get("src")
 
+    try:
+        check = "to check, the url ="+finalurl
+    except TypeError:
+        # Now as finalurl is not a string, we have not scraped the image url,
+        # the tag in this page is 'href'!
+        finalurl = soup.find("div", {"id": "comic"}).contents[1].get('href')
+        print "rare image, we somehow got the URL -> "+soup.find("div", {"id": "comic"}).contents[1].get('href')
+        
+        # what if we didnt get a image!?
+        if(finalurl[:4:-1][0:4] != 'gnp.'):
+            print "Not a .png image!!"
+            print "backup plan, we use this ->"+soup.find("div", {"id": "comic"}).contents[1].img.get("src")
+            finalurl = soup.find("div", {"id": "comic"}).contents[1].img.get("src")
+            
+    try:
+        image = urllib.urlopen(finalurl)
+    except AttributeError:
+        print "Anomaly - final url was not properly set"
+        
     if image.headers.maintype=="image":
 
-        buf=image.read()
+        buf = image.read()
 
         print "Image stored to "+filepath
 
-        download=file(filepath,"wb")
+        download = file(filepath,"wb")
 
         download.write(buf)
 
         download.close()
         image.close()
+    print "\n"
+
+base="http://xkcd.com/"
 
-base="https://xkcd.com/"
-#change the path here, to whereever you want the directory to be
-path='C:\\Users\\Aditya\\Desktop\\Work\\xkcd\\'
+# Change the path here, to whereever you want the directory to be
+path = "F:\\Work\\xkcd\\"
 
 for i in range(start,end+1):
-    
     tempurl=base+str(i)+"/"
     temppath=path+str(i)+".png"
     download(tempurl,temppath)