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. 22:17.
Grunn: Automatisk sammenslåing med etterfølgende innlegg.