Arch Linux

[python] Ordner auslesen.

danbruegge

Hallo Leute.

Ich bastel gerade an einem kleinen Skript das mir den Inhalt eines Ordners (genauer, nur die .mp3 Dateien) wieder geben soll.

Das wieder geben des gesamten Ordner Inhaltes klappt auch, nur beim aussortieren der nicht .mp3 Dateien hängt es irgendwie.

Eigentlich sollte es recht einfach und verständlich sein aber ... naja.

#!/usr/bin/python
# -*- coding: utf-8 -*-
# read.py

import os

dirList = os.listdir('.')
dirList.sort()

for sFile in dirList:
    if sFile.find('.mp3') == -1:
        dirList.remove(sFile)

for sFile in dirList:
    print sFile

Hier mal der Code.

Dadurch sollten eigentlich die Dateien "read.py" und "read.py~" aussortiert werden, "read.py" wird auch aussortiert, nur halt nicht die versteckte "read.py~". Das verstehe ich nicht.

Ich hoffe mir kann da einer Helfen. 🙂

//Edit.

Ok, hat sich erledigt. Ich habe einfach eine neue Liste erstellt und diese dann mit den .mp3's befühlt. Ich habe anstelle von

dirList.remove(sFile)

ein

continue

gesetzt.

#!/usr/bin/python
# -*- coding: utf-8 -*-
# read.py

import os

dirList = os.listdir('.')
dirList.sort()
newList = []

for sFile in dirList:
    if sFile.find('.mp3') == -1:
        continue

    newList.append(sFile)
    
for sFile in newList:
    print sFile

Dirk

Ich habe mir mal erlaubt, dein Script zu optimieren …

#!/usr/bin/env python
# -*- coding: utf-8 -*-
# read.py

from glob import glob
print glob('/pfad/zum/verzeichnis/*.mp3')

danbruegge

Ich danke dir. 🙂

Hier mal meine komplette Version mit Dirks verbesserung:

#!/usr/bin/python
# -*- coding: utf-8 -*-
# read-mp3.py
# -----------------------------------------------------------------------------
# Read all .mp3 files of an dir and save it to a .txt file in the same dir.
# -----------------------------------------------------------------------------

from glob import glob

dirList = glob('*.mp3')
dirList.sort()
mp3List = ''
lineBreak = '\n'

for number, sFile in enumerate(dirList):
#---        
    sFile = sFile.replace('_', ' ')
    sFile = sFile.replace('-', ' - ')
    sFile = sFile.replace('.mp3', '')
    
    if number == len(dirList)-1:
    #---
        lineBreak = ''
    #---
    
    mp3List = mp3List + sFile + lineBreak
#---

handle = open("mp3-list.txt", "w")
handle.write(mp3List)
handle.close()

Geht der Python Style so? Dirk, du als Python Fan kannst doch bestimmt noch was zu sagen? 😉

Dirk

Ich muss gleich mal wohin, und komme nachher nicht mehr ganz nüchtern wieder, aber ich guck's mir morgen mal an 🙂

Ganz kurz nur: Ich habe nicht aus Spaß #!/usr/bin/env python anstelle von #!/usr/bin/python verwendet:

http://mail.python.org/pipermail/tutor/2007-June/054816.html

Bis Morgen 🙂

danbruegge

Lass dir Zeit, bin bis Sonntag auf nem Festival. 😉

Das mit /usr/bin/env... habe ich über sehen, Entschuldigung.

acoolon

Moin,

(ungetestet)

#!/usr/bin/python
# -*- coding: utf-8 -*-
# read-mp3.py
# -----------------------------------------------------------------------------
# Read all .mp3 files of an dir and save it to a .txt file in the same dir.
# -----------------------------------------------------------------------------

from glob import glob

def list_filetype(filetype):
    dir_list = glob('*%s' %filetype).sort()
    for (old, new) in (('_', ' '), ('-', ' - '), (filetype, ''):
        dir_list = [entry.replace(old, new) for entry in dir_list]

    return '\n'.join(dir_list)

def main():
    mp3_list = list_filetype('.mp3')

    try:
        handle = open('mp3-list.txt', 'w')
        handle.write(mp3_list)
        handle.close()
    except IOError:
        print mp3_list

if __name__ == '__main__':
    main()

Und ich moechte noch ein paar Worte zu deinem Code verlieren, man soll ja lernen.

#!/usr/bin/env python
# coding: utf-8 # ieeeeh emacs (es muss nur ein 'coding: <coding>' dort stehen)
# read-mp3.py
# -----------------------------------------------------------------------------
# Read all .mp3 files of an dir and save it to a .txt file in the same dir.
# -----------------------------------------------------------------------------

from glob import glob

dirList = glob('*.mp3')
dirList.sort()
mp3List = ''
lineBreak = '\n'

for number, sFile in enumerate(dirList): # besser: for (number, sFile) ...
#---        
    sFile = sFile.replace('_', ' ')
    sFile = sFile.replace('-', ' - ')
    sFile = sFile.replace('.mp3', '')
    
    if number == len(dirList)-1: # besser: len(dirList) - 1
    #---
        lineBreak = ''
    #---
    
    mp3List = mp3List + sFile + lineBreak # besser mp3List += sFile + lineBreak
#---

handle = open("mp3-list.txt", "w") # niemals, niemals ohne
handle.write(mp3List) # exception-handling
handle.close()

Ich wuerde dir empfehlen einmal den PEP8 Guide durchzulesen. Ausserdem sind Funktionen und List-Comprehension etwas ganz tolles 😉.

@Dirk: Sehe ich nicht so, man lernt nur durchs selber machen, nicht durchs importieren von Modulen. Und wir konnten ja beobachten, wie er selbst seinen Fehler gefunden und sogar eine Loesung praesentiert hat.
Und bitte antworte erst wenn du wieder nuechtern bist 😉

aco

Dirk

acoolon schriebman lernt nur durchs selber machen, nicht durchs importieren von Modulen.

Das ist halt die Frage: Geht es hier darum, etwas zu lernen, oder darum, eine gute Lösung für ein Problem zu finden? Eine (imho) gute Lösung für dsa Problem „Lese ein Verzeichnis ein, und schreibe alle gefundenen Dateien mit der Endung ‚.mp3' in eine Datei in diesem Verzeichnis" könnte zum Beispiel dieses hier sein (und zwar inklusive pydoc-Strings):

#!/usr/bin/env python
# -*- coding: utf-8 -*-
# vim: ts=4:sw=4
# Read given directory and write it to file
# 2010 Dirk Sohler, WTFPL 2 or later

"""This script reads all given directories and writes all containing
MP3 files (determined by file suffix mp3, not by MIME type) into
"fileslist" in the directories. It is possible to commit more than
one directory when calling the script"""


import sys
import os
from glob import glob


def getdirlist(argv):
    """Checks if the parameters given are directories or not. If not,
    write an error message to STDERR and discard the item."""
    dirs = []
    for item in argv[1:]:
        if os.path.isdir(item):
            dirs.append(item)
        else:
            sys.stdout.write('The item "%s" is not a directory\n' % item)
    if len(dirs) < 1:
        sys.stdout.write('There are no directorys to process\n')
        sys.exit(1)
    return dirs


def getfiles(directory):
    """Opens the given directory and reads all MP3 files by using the
    glob() function. Then it extracts the filename only and returns
    all detected files in a sorted list."""
    files = []
    for f in glob(os.path.join(directory, '*.mp3')):
        files.append(os.path.splitext(os.path.basename(f))[0])
    return sorted(files)


def writelist(files, d):
    """Creates and opens a file in the given directory and writes the
    given files list into that file. If it could not close the file
    handle it will print an error message"""
    charset = sys.getfilesystemencoding()
    with open(os.path.join(d.decode(charset), 'fileslist'), 'w+') as f:
        write_items = f.write('\n'.join(files))
    if not f.closed:
        sys.stdout.write('Could not close the file handle')


def main():
    """Commits all given command line parameters to the getdirlist()
    function and processes it's output via the writelist() function."""
    for d in getdirlist(sys.argv):
        writelist(getfiles(d), d)


if __name__ == '__main__':
    main()

Und da das Forum mit dem eingegebenen Code scheiße baut, hier nochmal unverändert:

https://p.0x7be.de/i90amibl5/plain

Ich sehe gerade: Die Replaces sind nicht drin, aber das sollte ein leichtes für dich sein, die in dem for-Loop in der Funktion getfiles() einzubauen 🙂

acoolon

Moin,

also mir geht es ja ums lernen:

WTFPL

Mach ich!

sys.stdout.write('')

Nicht ganz so gut, es sei denn es folgt ein sys.stdout.flush(), denn sonst kommt die Ausgabe erst nach Beenden des Programms. Jedoch ist Feedback ein zentraler Aspekt guten Designs.
Du schreibst, es wuerde eine Ausgabe nach sys.stderr geben. mit print geht das auch:

print >> sys.stderr, ''

for item in argv[1:]
...
if len(dirs) < 1
...
for d in getdirlist(sys.argv)

Warum uebergibst du der Funktion alle Parameter? Denn wenn du das Slicing in der bei mir letzten Zeile machst, kannst du in der mittleren

if dirs

schreiben.

Warum importierst du glob wenn du eh os importierst? Ich meine glob haette ich verstanden, wenn dafuer os wegfaellt.

Danke, aco

Dirk

acoolon schriebNicht ganz so gut,

Doch, sogar SEHR gut. Fehler kommen auf STDERR, basta.

acoolon schriebes sei denn es folgt ein sys.stdout.flush()

Ja, denn häng das noch mit dran. Das Programm ist allerdings so schnell, dass das vom Effekt her keinen Unterschied machen würde. Es sei denn, du übergibst hunderte von Verzeichnissen mit Dutzenden von Dateien 🙂

acoolon schriebJedoch ist Feedback ein zentraler Aspekt guten Designs.

Sinnvolles Feedbak, und dieses zur richtigen Zeit. Nichts ist nerviger, unnötiger, und dämlicher als wenn ein Programm ständig und zu jeder unnötige Informationen ausgibt. Was interessiert mich direkt nach dem Anstoßen des Programms mit – sagen wir wirklich mal – hunderten von Verzeichnissen mit Dutzenden von Dateien, ob einer der übergebenen Parameter kein Pfad zu einem Verzeichnis ist? Soll ich den Prozess dann abbrechen, die Befehlszeile nochmal bearbeiten, und dann noch mal von vorne anfangen?

So hat man während des Durchgangs keinen unnötigen Spam, und hinterher eine Zusammenfassung darüber, ob – was – nicht geklappt hat.

acoolon schriebDu schreibst, es wuerde eine Ausgabe nach sys.stderr geben. mit print geht das auch:

Und warum sollte ich umständlich die Ausgabe einer Funktion umleiten, wenn es für das, was ich machen will, eine Betriebssystemunabhängige, andere Funktion genau für diesen Einsatzzweck gibt?

acoolon schriebWarum uebergibst du der Funktion alle Parameter?

Weil ich's erst anders gemacht, und dann noch mal umgebaut habe. Vor allem habe ich da was im Hinterkopf gehabt, dass man sich so ein try/except spart, wenn sys.argv[1:] mal einen IndexError produziert. Aber stimmt, hier könnte man auch sys.argv sogar direkt benutzen. Im Zweifelsfall ist sys.argv[1:] dann ja einfach 'ne leere Liste 🙂 Und ob die Funktion nun weiß, wie das Script heißt, ist letztendlich aber auch egal …

acoolon schriebWarum importierst du glob wenn du eh os importierst?

Weil glob im Modul glob ist, und nicht im Modul os.

acoolon

Moin,

hoere nur ich diesen agressiven Unterton?

- Bitte nichts aus dem Zusammenhang reissen
- Wer sagt, dass das Script nicht eines Tages ueber einen sehr grossen Verzeichnisbaum laeft
- Ja moeglichweise abbrechen, genau dann, wenn er ueber ein grosses Verzeichnis laeuft.
- Ausgabe waehrend des Durchlaufs ist Spam, nachdem keiner?
- Seit wann ist print Betriebssystemabhaengig? Print macht im Endeffekt ein str(), <file>.write(), <file>.flush().. und du kannst halt angeben was <file> sein soll
- Nein, es ist nicht egal, das ist schlechter Stil, um dich zu zitieren "basta". Jede Funktion sollte die Daten bekommen die sie braucht, nicht mehr.
- Ja, glob ist in glob (haha -.-) aber die gleiche Funktionalitaet laesst sich auch ohne glob in moeglichweise einer Zeile mehr erreichen. (Kennst du os.path.walk()?)

aco

BTW; Slicing produziert nie IndexError

Dirk

acoolon schriebhoere nur ich diesen agressiven Unterton?

Soll ich hinter jede meiner Aussagen einen Smilie malen?

acoolon schrieb- Ja moeglichweise abbrechen, genau dann, wenn er ueber ein grosses Verzeichnis laeuft.

… was ja kein Verzeichnis ist, wenn es eine Fehlermeldung gibt. Fehlermeldungen gibt es nur, wenn es sich nicht um Verzeichnisse handelt, ansonsten läüft das Script ganz normal durch, und auch weiter, wenn sich unter den Angaben ein Nicht-Verzeichnis verbirgt, das wird dann am Ende ausgegeben, und man kann sich dessen dann immer noch annehmen, während alle Verzeichnisse dann schon fertig sind.

acoolon schrieb- Ausgabe waehrend des Durchlaufs ist Spam, nachdem keiner?

Exakt. Wenn das Programm macht, was es soll, bedarf es keiner Ausgabe. Hinterher eine Zusammenfassung, dagegen habe selbst ich nichts. Aber währenddessen? Muss nicht sein.

acoolon schrieb- Seit wann ist print Betriebssystemabhaengig? Print macht im Endeffekt ein str(), <file>.write(), <file>.flush().. und du kannst halt angeben was <file> sein soll

PEP-8 schriebCode should be written in a way that does not disadvantage other implementations of Python (PyPy, Jython, IronPython, Pyrex, Psyco, and such).

Wenn „print 'blubb' >> sys.stderr" überall gleich funktioniert, und „sys.stderr.write('blubb')" nicht, dann spricht nichts dagegen „print 'blubb' >> sys.stderr" zu verwenden – auch wenn ich es für quatsch halte, die Ausgabe einer Funktion umzuleiten, wenn es eine andere Funktion gibt, die genau für das vorgesehen ist, was mit der Umleitung erreicht wird.

acoolon schriebJede Funktion sollte die Daten bekommen die sie braucht, nicht mehr.

Dann pass' das Programm an, die Lizenz ist dir bekannt.

acoolon schrieb- Ja, glob ist in glob (haha -.-) aber die gleiche Funktionalitaet laesst sich auch ohne glob in moeglichweise einer Zeile mehr erreichen.

Warum über Umwege die Funktionalität von Python-Standardmodulen nachbauen?

acoolon schrieb(Kennst du os.path.walk()?)

Baue mir mittels os.path.walk() dieses hier nach:

files =[]
for f in glob(os.path.join(directory, '*.mp3')):
    files.append(os.path.splitext(os.path.basename(f))[0])
return sorted(files)

acoolon schriebSlicing produziert nie IndexError

Na denn 🙂

acoolon

Moin,

diesen will ich noch loswerden, ist dann aber der letzte:

- Feedback. Anzeige, das das Programm arbeitet und nicht in einer Schleife gefangen oder sonst etwas ist (*empfiehlt "The Desingn of Everyday Things"*)
- PEP 214: print statement (print macht mehr (und macht es besser) als *.write())
- es geht mir noch immer um den Lerneffekt.
- Um zu Lernen und zu schauen wie welche Probleme geloest werden
- Bitte zeige mir ein Stueck Code, ich will sehen wann Slicing einen IndexError wirft. (In dem Fall werde ich doch noch einen Beitrag schreiben und um entschuldigung bitten)

Sorry, os.walk(), da ist die Erregung mit mir durchgegangen.

def foo(path, ext='.mp3'):
    files = []
    for (dirpath, dirnames, filenames) in os.walk(path):
        files += [f.rstrip(ext) for f in filenames if f.endswith(ext)]
    return sorted(files)

aco

Dirk

acoolon schrieb- Feedback. Anzeige, das das Programm arbeitet und nicht in einer Schleife gefangen oder sonst etwas ist (*empfiehlt "The Desingn of Everyday Things"*)

„Rule of Silence: When a program has nothing surprising to say, it should say nothing.", Eric S. Raymond, The Art of Unix Programming. Dass das Programm sich melden sollte, wenn's nicht funktioniert (warum auch immer), versteht sich von selbst. Die Ausgabe von Nicht-Verzeichnissen in einer übergebenen Verzeichnisliste gehört meiner Meinung nach nicht dazu. Denn hier trifft die Rule of Silence nämlich genau so: Dass Nicht-Verzeichnisse ausgelassen werden, wenn Verzeichnisse erwartet werden ist nämlich „nothing surprising to say".

acoolon schrieb- PEP 214: print statement (print macht mehr (und macht es besser) als *.write())

Und wie kann ich damit print wie das benutzen, was es ist – eine Funktion? „print('bla') >> sys.stdout" und print('bla' >> sys.stdout) funktionieren jedenfalls nicht.

piet

Dirk Sohler schrieb
acoolon schrieb- PEP 214: print statement (print macht mehr (und macht es besser) als *.write())
Und wie kann ich damit print wie das benutzen, was es ist – eine Funktion? „print('bla') >> sys.stdout" und print('bla' >> sys.stdout) funktionieren jedenfalls nicht.

Früher (python 2) war es ein statement, heute oder in Zukunft (python 3) ist es eine Funktion. Siehe auch: http://diveintopython3.org/porting-code-to-python-3-with-2to3.html#print und http://docs.python.org/release/3.0.1/whatsnew/3.0.html

import sys

#python2
#print >>sys.stderr, "fatal error"

#python3
print("fatal error", file=sys.stderr)

Dirk

piet schrieb

import sys

#python2
#print >>sys.stderr, "fatal error"

#python3
print("fatal error", file=sys.stderr)

Schade, dass man, wenn man print anstatt sys.stderr.write() benutzt, den Code beim (hoffentlich bald vorgenommenem) Wechsel von Python 2 auf Python 3 anpassen muss, wenn man keine Abfrage einbaut. Noch einer der Vorteile von sys.stderr.write(): Es funktioniert unter den meisten Python-Versionen 🙂

acoolon

Ahhhhhhhhhhhhhhhhhhhhhhhhhhrght!

(Onomatopoesie sehe ich nicht als Beitrag)

danbruegge

Oh man, da hab ich ja was angerichtet. 😃

Danke für die Diskussion und die Hilfe.

piet

Dirk Sohler schrieb
piet schrieb
import sys

#python2
#print >>sys.stderr, "fatal error"

#python3
print("fatal error", file=sys.stderr)
Schade, dass man, wenn man print anstatt sys.stderr.write() benutzt, den Code beim (hoffentlich bald vorgenommenem) Wechsel von Python 2 auf Python 3 anpassen muss, wenn man keine Abfrage einbaut. Noch einer der Vorteile von sys.stderr.write(): Es funktioniert unter den meisten Python-Versionen 🙂

Machst du viel mit Python?
Ich hatte bis jetzt nur wenige Berührungspunkte, aber irgendwie fühlt sich python sehr kryptisch an, oder kommt mir das nur so vor?

cu

Dirk

piet schriebMachst du viel mit Python?

Ich bevorzuge Python gegenüber anderen Sprachen, wenn es darum geht, kleine Programme/Scripte zu erstellen, ja 🙂

Dirk

deadshox schriebOh man, da hab ich ja was angerichtet. 😃

Ein bisschen Reibung erhält die Wärme 🙂

Nächste Seite »