:
Code: Alles auswählen
# ...
for block in iter(partial(in_file.read, blocksize), ''):
sha.update(block)
# ...BlackJack hat geschrieben:@xeike: Man muss das ja nicht in einer so aufwändigen Klasse machen. Und wieder einmal kommt die `iter()`-Funktion mit zwei Argumenten zum Einsatz
Ja, over-engineeredDas was die so schrecklich macht ist nicht das sie sauber sind, sondern das sie so oft komplex bis „over engineered” sind.
war das Wort, das ich gesucht habe.Na ja, im Prinzip ist das das, was ich auch schon mal versucht hatte dir nahe zu legen: deine Beiträge (sowohl die, in denen es um deinen eigenen Code geht, als auch die, in denen du anderen Ratschläge gibst), erwecken den Eindruck, als hättest du einen Kenntnisstand, der sich in deinen Code-Beispielen leider so nicht widerspiegelt; und noch immer beherzigst du beispielsweise die Vermeidung von Code auf Modulebene nicht konsequent. Da musst du dich dann halt auch nicht wundern, wenn solche Antworten kommenGary123456 hat geschrieben:Ich habe mein Bestes gegeben. Was der "Chef" dazu sagen würde, wäre mir egal, da ich in Python noch übe. Ich finde diese Aussage nicht fair. Dennoch versuche ich den Code zu verbessern.
- insbesondere auch für dich selbst.Jap, das ist mir klar (http://www.winfaq.de/faq_html/Content/t ... ip1390.htm). Dazu hat sich ein anderen Programmierprofi gemeldet und mir da was erklärt.Darin wird der Inhalt des Arbeitsspeichers ausgelagert wenn du deinen Computer in den Standby modus versetzt.
Ist die Sache nun geklärt?Dann schau dir bspw. mal shasum an, du brichst eine Endlosschleife via break ab statt über die Kopfbedingung zu gehen, warum gibts die wohl?
Deine Einrückung beim return ist auch so ein Ding.
Ich schaffe oft einen Code, der funktioniert, jedoch scheint dieser oft nicht euren Ansprüchen genug zu sein, da der entweder keine vernünftigen Datenstrukuren besitzt oder vom Aufbau her falsch ist. Ich habe in der Theorie viel gemacht. Da aber das praktisch genau so wichtig ist, will ich es mit diesem Programm üben. Und ich persönlich finde, dass ich mich im Gegensatz zu den alten Threads weiterentwickelt habe. Wer anderer Meinung ist, kann das gerne hier sagen. Dann der zweite Punkt, ihr alle sagt, dass ich Code auf Modulebene schreibe. Dann frage ich mich, was das bedeuten soll.erwecken den Eindruck, als hättest du einen Kenntnisstand, der sich in deinen Code-Beispielen leider so nicht widerspiegelt; und noch immer beherzigst du beispielsweise die Vermeidung von Code auf Modulebene nicht konsequent.
Code auf Modulebene heißt, dass du deinen Code nicht in Funktionen packst und ihn in einer Funktion an passender Stelle ausführst.Gary123456 hat geschrieben:Dann der zweite Punkt, ihr alle sagt, dass ich Code auf Modulebene schreibe. Dann frage ich mich, was das bedeuten soll.
Code: Alles auswählen
# test.py
class TolleKlasse:
pass
def meine_neue_funktion(parameterliste):
pass
def main():
meine_neue_funktion("blah")
noch(was)
viel(mehr)
if __name__ == "__main__":
# hier kommt der Code rein
main()
Code: Alles auswählen
if __name__ == "__main__":
main()Code: Alles auswählen
if __name__ = "__main__":
print "Test"Code: Alles auswählen
print "Test"Code: Alles auswählen
import os
import sha
class Lister(object):
def __init__(self, verzeichnisse, geschuetzt):
self.verzeichnisse = verzeichnisse
self.geschuetzt = geschuetzt
def listen(self):
for self.path in self.verzeichnisse:
for self.entry in os.listdir(self.path):
return os.path.join(self.path, self.entry)
def shasum(self, filepath, blocksize = 1024 * 1024):
self.h = sha.new()
with open(filepath, "rb") as f:
try:
while True:
self.part = f.read(blocksize)
if not part:
break
h.update(part)
finally:
f.close()
return h.hexdigest()
def main():
verzeichnisse = ["C:\\"]
geschuetzt = []
Klasse = Lister(verzeichnisse, geschuetzt)
print Klasse.listen()
#COde kommt noch!
if __name__ == "__main__":
main()
Code: Alles auswählen
C:\$Recycle.BinCode: Alles auswählen
import os
import sha
class Lister(object):
def __init__(self, verzeichnisse, geschuetzt):
self.verzeichnisse = verzeichnisse
self.geschuetzt = geschuetzt
def listen(self):
for self.path in self.verzeichnisse:
for self.entry in os.listdir(self.path):
print os.path.join(self.path, self.entry)
def shasum(self, filepath, blocksize = 1024 * 1024):
self.h = sha.new()
with open(filepath, "rb") as f:
try:
while True:
self.part = f.read(blocksize)
if not part:
break
h.update(part)
finally:
f.close()
return h.hexdigest()
def main():
verzeichnisse = ["C:\\"]
geschuetzt = []
Klasse = Lister(verzeichnisse, geschuetzt)
Klasse.listen()
#COde kommt noch!
if __name__ == "__main__":
main()
Code: Alles auswählen
C:\$Recycle.Bin
C:\AMD
C:\Documents and Settings
C:\Dokumente und Einstellungen
C:\hiberfil.sys
C:\MSOCache
C:\pagefile.sys
C:\PerfLogs
C:\Program Files
C:\Program Files (x86)
C:\ProgramData
C:\Programme
C:\Put a directory on PYTHONPATH here
C:\Python25
C:\Python27
C:\Recovery
C:\System Volume Information
C:\test.exe
C:\Users
C:\VS_EXPBSLN_x64_deu.CAB
C:\VS_EXPBSLN_x64_deu.MSI
C:\Windows
C:\XProfanX2
C:\zoek-results07.04.2013-2116.log
>>>
Du kannst nur ein Objekt zurückgeben. Dieses Objekt kann allerdings auch eine Liste sein, ein Klassenexemplar, sogar Klassen oder Funktionen. Lister.listen müsste also die Einträge in einer Liste oder einer ähnlichen Datenstruktur sammeln und diese dann zurückgeben oder allerdings du machst daraus einen Generator, dafür einfach return durch yield ersetzen.Gary123456 hat geschrieben:Return gibt Werte zurück. Das heist Du kannst mit der Ausgabe was machen. Aber kann dieses return nur einen Wert zurück liefern, oder wie? *verwirrt*
Code: Alles auswählen
for self.path in self.verzeichnisse: # path solltest du nicht an sel binden
for self.entry in os.listdir(self.path): # entry auch nicht.
print os.path.join(self.path, self.entry)In Bayrisch: Des wor deppad von miar. Zufrieden?Edit: Warum machst du path zu einem Attribut? Das ist doch unsinnig, da es keine andere Methode gibt, die damit was anfangen kann (und außerdem ist der Wert mehr oder weniger Zufall, …).
Code: Alles auswählen
def listen(directories):
for path in directories:
for entry in os.listdir(path):
yield os.path.join(path, entry)
Code: Alles auswählen
for filename in listen(["C:\\"]):
print filenameDazu fallen mir nur zwei Fragen ein:Gary123456 hat geschrieben:Ich komme der Lösung immer näher
Eine wesentliche Eigenschaft von Python ist, dass viele Dinge, die in anderen Sprachen erzwungen werden, per Konvention geregelt sind. Es ist nicht unbedingt nötig, dass du dich an pep8 hälst, oder idiomatischen Code schreibst, damit deine Programme laufen.Gary123456 hat geschrieben:Bedeutet, dass ich das alles per main() steuern soll? Und so bessere Kontrolle über Funktionen habe? Verstehe es immer noch nicht ganz ...
Code: Alles auswählen
for char in "Hello World": print(char)
Code: Alles auswählen
program HelloWorld;
var i: integer;
msg: string;
begin
msg := 'Hello World!';
for i := 1 to length(msg) do writeln(msg[i])
end.
Code: Alles auswählen
import os
import sha
import filecmp
def shasum(filepath, blocksize = 1024 * 1024):
h = sha.new()
with open(filepath, "rb") as f:
try:
while True:
part = f.read(blocksize)
if not part:
break
h.update(part)
finally:
f.close()
return h.hexdigest()
def listen(verzeichnisse):
for path in verzeichnisse:
for entry in os.listdir(path):
yield os.path.join(path, entry)
def main():
with open("Testdatei.txt", "w") as textdatei:
verzeichnisse = ["C:\\"]
geschuetzt_files = ["C:\hiberfil.sys", "C:\pagefile.sys"]
k = 0
for files in listen(verzeichnisse):
if os.path.isfile(files):
if filecmp.cmp(files, geschuetzt_files[k]):
k += 1
if k > len(geschuetzt_files):
k = 0
else:
continue
else:
print files
textdatei.close()
#Code kommt noch!
if __name__ == "__main__":
main()
Das wäre auch mein erster Versuch gewesen. Leider gibt es dann, wo ich files printen lasse, die Dateinamen immer doppelt aus.Wenn Du über `geschuetzt_files` (ebenfalls ein schlechter Name) iterieren willst, warum nimmst Du dann nicht ganz einfach eine ``for``-Schleife über die Elemente dieser Liste statt diesem reichlich komischen Code mit dem `k`, der so überhaupt keinen Sinn macht.
Um nicht immer diese textdatei öffnen zu müssen, habe ich print benutzt. Daher ist das meiner Sicht nach nicht sinnbefreit.Ebenfalls sinnfrei ist die Datei `textdatei` (schlechter Name), die überhaupt nicht benutzt wird.