🚀 go-pugleaf

Jak wiadomo przykłady są istotą nauki, napisałem zatem przykładowy
skrypt do zmiany tapety pulpitu. Proszę Was o komentarz, wytknięcie
błędów stylistycznych, poprawki, opis co można zrobić lepiej, jak
lepiej wykorzystać pythona itp.

#v+
from os 	import access,getenv,listdir,execv,spawnv,P_WAIT
from random 	import Random
from math 	import floor
import sys

# --- customize this section --- #
images	 	= getenv('HOME') + '/obrazy/tapety/'
imgchanger	= '/usr/local/bin/icewmbg'
imgchanger_flags = ' -s '
ending		= '.chosen'
# --- DONT'T TOUCH below --- #

wallpapers 	= listdir(images)
recordfile 	= images + ending

# delete our record file from filelist, so that it isn't chosen as
# wallpaper :) (unless it's absent). Touch it if it doesn't exists
if ending in wallpapers:	
    wallpapers.remove(ending)
else:
    if spawnv( P_WAIT, "/bin/touch", ("WTF/bezsensu", recordfile) ):
	print "wanted to create %s, but failed" % (recordfile)
	sys.exit(1)

# check which wallpaper is currently displayed
if access(recordfile, 6):		#WTF? Gdzie są opisane te tryby ?
    f = open(recordfile, "r")
    current = f.readline()
    if "\n" in current:			#get rid of fsckin \n
	current = current[:-1] 	
    f.close()
    # remove current wallpaper from list of wallpapers
    if current in wallpapers:		#needed in case, file was deleted
	wallpapers.remove(current)
else:
    print "need +r access to %s! touch it at least" % (recordfile)
    sys.exit(1)


seed = Random()
chosen = wallpapers[int(floor(len(wallpapers) * seed.random()))]

# record what was chosen, so that we have true change allways
# TODO: add swich - user might don't care
if access(recordfile, 6):
    f = open(recordfile, "w")
    f.write(chosen)
    f.close()
else:
    print "need +w access to %s!" % (recordfile)
    sys.exit(1)


#change wallpaper
cmd = ( imgchanger, 
	imgchanger_flags, 
	images + chosen
	)
execv(cmd[0], cmd[1:])
#v-

Tam gdzie jest 'WTF' kończą się moje zdolności pojmowania tego co python
robi z argv dla komend. Nie dałem import * przy os, dlatego, że nadpisałoby
mi to funkcję open().

Skrypt działa (zmienia mi tapetę w gnome ;). Uprasza się o odpalanie z &.
Nie wiem czemu icewmbg, chodź umiera, to jednak nie oddaje klawiatury.

Dziękuję za uwagę i proszę o pomoc.
-- 
    [ http://www.stringi.com/viper/ ] [ GG 1513591 ]
	All good things come to those who wait