python/pin.py: modularize

python/pin.py

@@ -1,15 +1,31 @@
 #!/usr/bin/env python
+"""
+Takes desired PIN length as an argument and generates a PIN code of that
+length.
+"""
 import secrets
 import sys
 
-try:
-    wantedCount = int(sys.argv[1])
-except IndexError as noarg:
-    print("Enter a digit as an argument!")
 
-try:
-    for i in range(int(wantedCount)):
-        print(secrets.randbelow(10), end="")
-except NameError as noWantedCount:
+def main():
+    """
+    This is where the magic happens.
+    """
+    try:
+        desired_length = int(sys.argv[1])
+    except IndexError:
+        print("Enter a digit as an argument!")
+
+    try:
+        for i in range(int(desired_length)):
+            print(secrets.randbelow(10), end="")
+            # We satisfy pylint by having the variable here.
+            # TODO:make this a while loop?
+            i += 1
+    except NameError:
+        print()
     print()
-print()
+
+
+if __name__ == "__main__":
+    main()

python/ssid_check_valid_length.py

@@ -4,7 +4,7 @@ This script asks for a SSID name, counts it and tells whether it's of valid
 length
 """
 # Request input or assume "test" on empty.
-givenssid = str(
+GIVENSSID = str(
     input(
         "Please enter the SSID you are thinking of (preferably 26 chars to fit _nomap): ",
     )
@@ -13,28 +13,32 @@ givenssid = str(
 
 # Ensure it's UTF-8 and store the size. E.g. åäö are longer than oao
 # TODO: make this a function too
-givenssidlen = len(givenssid.encode("utf8"))
+GIVENSSIDLEN = len(GIVENSSID.encode("utf8"))
 
 
 # Checking the SSID length is done twice, so thus a function
-def checkssidlen(givenssidlen):
+def checkssidlen(GIVENSSIDLEN):
+    """
+    This function checks that the given string is below 32 characters long and
+    can thus be used as an SSID.
+    """
     # 32 bytes should be the maximum SSID length
-    if givenssidlen <= 32:
-        print(givenssid, "is", givenssidlen, "long and thus valid length")
+    if GIVENSSIDLEN <= 32:
+        print(GIVENSSID, "is", GIVENSSIDLEN, "long and thus valid length")
     else:
         print(
-            givenssid,
+            GIVENSSID,
             "is",
-            givenssidlen,
+            GIVENSSIDLEN,
             "to long, please select a different SSID",
         )
 
 
 # Checking the SSID without _nomap
-checkssidlen(givenssidlen)
+checkssidlen(GIVENSSIDLEN)
 # TODO: make this a function too
-givenssidlen = len(givenssid.encode("utf8"))
+GIVENSSIDLEN = len(GIVENSSID.encode("utf8"))
 
 # Continuing with _nomap to ensure it also fits
-givenssid = givenssid + "_nomap"
-checkssidlen(givenssidlen)
+GIVENSSID = GIVENSSID + "_nomap"
+checkssidlen(GIVENSSIDLEN)