main.py

@@ -1,6 +1,5 @@
 import math
 import random
-
 def intToDotted(ip):
     # input: ip
 
@@ -18,12 +17,13 @@ def intToDotted(ip):
 def printDotted(octets):
     # input: list containing octets
 
-    # returns out the dotted decimal format
-
+    # prints out the dotted decimal format
     # hint: use '.'.join()
-    #returns prettyfied DD format as string
+    #returns nothing
+    dottedIP = [str(octets[0]), str(octets[1]), str(octets[2]), str(octets[3])]
+    x = ".".join(dottedIP)
 
-    return ".".join(str(x) for x in octets) 
+    return x    
 
 def generateNetwork():
     # input: none
@@ -32,13 +32,12 @@ def generateNetwork():
     # to avoid getting too large and too small networks    
     # returns dict: {ip:int, prefix:int}
     ip = random.randint(0, 2**32-1)
-    prefix = random.randint(8, 31) #max prefix generated is 
-    # 30, allowing for at least 1 bit to be subnetted and still have hostbits left
+    prefix = random.randint(8, 31)
     ipandprefix = {"ip":ip, "prefix":prefix}
     return ipandprefix
 
     
-def getNetId(ip, prefix):
+def getnetid(ip, prefix):
     snm=(2**prefix-1)<<(32-prefix)
     netid = ip&snm
     return netid
@@ -52,6 +51,7 @@ def analyseSubnet(ip, prefix):
     # and amount of addresses and amount of
     # usable adresses for a given subnet 
 
+    
     wmask = 0xffffffff >> prefix
     bid = ip|wmask
     
@@ -81,17 +81,19 @@ def newSubnetExercises(ip, prefix):
 
     # returns dict: {ip: int, prefix:int, newprefix:int}
     # newprefx contains the new subnet mask that solves the exercise
-
+    print("Generating new Subnet exercise...")
     aufgabentyp = random.randint(0,1)
     bonusaufgabe = random.randint(0,10)
-    newprefix = random.randint(prefix+1, prefix+5)
+    newprefix = random.randint(prefix, 32)
 
-    print("Network: {}/{}".format(printDotted(intToDotted(ip)), prefix))
-    print("newprefix: {}".format(newprefix))
+
+    
+    print(printDotted(intToDotted(ip)))
+    print("Prefix: ", prefix)
     if aufgabentyp == 0:
-        print("Split the Network into {} subnets".format(2**(newprefix-prefix)))
+        print("You will have to split the subnet into {} subnets".format(random.randint(2,32-prefix-2)))
     else:
-        print("Split the network into subnets with {} Hosts".format(2**(32-newprefix)))
+        print("Split subnet into subnet with {} Hosts".format(random.randint(8,2**(32-prefix)/2)))
     
     if bonusaufgabe > 5:
         print("What is the Net ID of the given subnet?")
@@ -110,15 +112,14 @@ def listSubnets(ip, prefix, newprefix):
     # within the given range, showing net id and bc DONE        
     # as well as amount of adresses and usable hosts per network DONE
     # returns: nothing, this just prints all possible subnets DONE
-    nid = getNetId(ip, prefix)
+    nid = getnetid(ip, prefix)
     print(newprefix)
     hostbits = 32 - newprefix
     newnetbits = newprefix - prefix
     newnetbits = 2**newnetbits
     for i in range(0,newnetbits):
-        print("subnet", i,":", printDotted(intToDotted(nid +i*2**hostbits)))
+        print(printDotted(intToDotted(nid +i*2**hostbits)))
         if i > 25:
-            print("and many many more...")
             break