View Single Post
I see you...
NAPse's Avatar
380
Et par ting som kan være greit å tenke på:
-Ikke importer mer en du har behov for. Begrens det til de delene du faktisk bruker.
-Du har en unødvendig løkke. Og det er generelt god praksis å ikke ha "evige" løkker.

Funkjsoner i et såpass enkelt skript ser jeg ikke den store nytteverdien i, så det mener jeg du ikke trenger.

Forøvrig fungerer skriptet dårlig på min maskin, da den ikke åpner neste fane før jeg lukker hele vinduet.

Mitt løsningsforslag:
SPOILER ALERT! Vis spoiler

Kode

# -*- coding: cp1252 -*-
#Imports
from time import sleep
from webbrowser import get as chrome
     
#Definerer URLs
chrome_path = 'C:/Program Files (x86)/Google/Chrome/Application/chrome.exe %s'

versions = {"f":#Full
                ["freak.no", "vg.no", "db.no", "facebook.com", "yr.no"],
            
            "l":#Lite
                ["freak.no", "vg.no"],
            
            "pr0n":#Im Twelve Years Old and What is This?
                ["freak.com"]
            }

version = "" #Bare for å ha definert en variabel som kan brukes i while-løkka

#Start
print ("""Web Launcher

Full versjon:
VG + FaceBook + DB + YR

Lite versjon:
Freak + VG
""")
 
while version != 'x':
    version = input("Velg versjon (f)ull / (l)ite: \nLukk programmet ved å skrive e(x)it ").lower()
    if versions.has_key(version): #Hvis dictionary "versions" har gyldig nøkkel ifht. input
        for url in versions.get(version): #For hver url i lista til nøkkel gjør dette...
            chrome(chrome_path).open(url)
            break

print("Programmet lukkes")
sleep(2)


Oops. Den break'en skulle enten ikke vært der i det hele tatt, evt. vært en "indent" bakover.
Sist endret av NAPse; 12. februar 2018 kl. 23:17. Grunn: Automatisk sammenslåing med etterfølgende innlegg.