Python Ladebalken funktioniert nicht?

1 Antwort

Deine ProgressBar benutzt zwei float-Variablen namens saved und line.

Bei saved solltest Du in Zeile 35

saved += 1

schreiben.

Und line darfst Du nicht für die Textzeilen verwenden. Versuch's mal so:

with open("src.mbf", "r") as file:
    lines = file.readlines()

global saved, total
saved = 0
total = len(lines)

for line in lines:
    try:
        copy(line.rstrip(), dest + dat)
        saved += 1
    except Exception ex:
        print("ERROR! Cannot copy file '%s': %s"
              % (line.rstrip(),ex))

pb = ttk.Progressbar(..., maximum=total, value=saved)

Außerdem schreibst Du in Zeile 19 mkdir(dest + "/" + dat), lässt den '/' beim Kopieren aber weg. So klappt es besser und spart unnötige Variablen:

with open("dst.mbf", "r", encoding='utf-8') as fdst:
    dest = fdst.read() + 
           dt.now().strftime("/%d.%m.%Y %H-%M-%S")
    try:
        os.mkdir(dest)
    except ex:
        print("ERROR! Cannot create folder '%s': %s."
              %(dest,ex))
...
        copy(line.rstrip(), dest)

Noch'n Tipp: Verwende os.path.join() für dest.

anonymia1 
Fragesteller
 21.05.2018, 08:54

Ich bin irgendwie gerade raus... :D

Vielleicht könntest du mir nochmal den gesamten Code schicken weil ich gerade einfach nicht mehr durchblicke...
Bin noch Anfänger.

Lg McCrafterIV

0
ralphdieter  21.05.2018, 17:35
@anonymia1

Ich habe Deinen Code mal zum Laufen gebracht (es waren noch einige Fehler drin):

#!/usr/bin/env python3

import os
from time import sleep
from datetime import datetime as dt
from tkinter import *
from shutil import copy
from tkinter import ttk
import locale


def savedata():
    locale.setlocale(locale.LC_ALL, '')
    fenster = Tk()

    ordner = StringVar()
    ttk.Label(textvariable = ordner).pack()

    saved = IntVar()
    pb = ttk.Progressbar(fenster, length=300, variable=saved)
    pb.pack()

    datei = StringVar()
    ttk.Label(textvariable = datei).pack()
    fenster.update()

    with open("dst.mbf", "r", encoding='utf-8') as fdst:
        dest = fdst.read().strip() + dt.now().strftime("/%d.%m.%Y %H-%M-%S")
        try:
            os.mkdir(dest)
            ordner.set('Kopiere nach "%s"' % dest)
        except Exception as ex:
            print("ERROR! Cannot create folder '%s': %s." % (dest,ex))
            return

    with open("src.mbf", "r") as file:
        lines = file.readlines()

    pb.configure(maximum = len(lines))

    for line in lines:
        try:
            datei.set(line.rstrip())
            copy(line.rstrip(), dest)
            saved.set(saved.get()+1)
            fenster.update()
            sleep( 0.2 ) ### Pause nur zum Testen
        except Exception as ex:
            print("ERROR! Cannot copy file '%s': %s"
                  % (line.rstrip(),ex))

    datei.set("%d Dateien kopiert" % saved.get())
    fenster.mainloop() # Anwendung nicht beenden

if __name__ == "__main__":
    savedata()
1
anonymia1 
Fragesteller
 21.05.2018, 18:54
@ralphdieter

Nun habe ich noch 1 Problem...

Es kommt folgender Error:

ERROR! Cannot copy file 'C:\Users\Tim PC\AppData\Roaming\.minecraft': [Errno 13] Permission denied: 'C:\\Users\\PC\\AppData\\Roaming\\.minecraft'

Das die Datei keine berechtigung hatt kann ich selber herauslesen, schaffe jedoch nicht an die behebung des Problems. Ich habe schon probiert der Datei per cmd mehr Rechte zu geben aber dann geht es immer noch nicht...

0
ralphdieter  21.05.2018, 19:42
@anonymia1

Ist .minecraft eine normale Datei (nicht: Ordner/Systemdatei/Verknüpfung), und kannst Du sie von Hand kopieren?

copy C:\Users\Tim PC\AppData\Roaming\.minecraft nul:
0
ralphdieter  22.05.2018, 17:28
@anonymia1

Den Fall kannst Du mit os.path.isdir() abfangen und z.B. shutil.copytree() verwenden.

0
anonymia1 
Fragesteller
 23.05.2018, 14:16
@ralphdieter

Gut danke. Jetz habe ich nurnoch das Problem dass der Ordner mit Datum und Uhrzeit zwar erstellt wird(und auch leer ist) aber trotzdem dieser Error kommt:

ERROR! Cannot copy file 'C:\Users\PC\AppData\Roaming\.minecraft': [WinError 183] Eine Datei kann nicht erstellt werden, wenn sie bereits vorhanden ist: 'E:\\minecraft/23.05.2018 14-13-33'
1
ralphdieter  23.05.2018, 15:00
@anonymia1

Bei shutil.copy() reicht es, als Ziel ein Verzeichnis anzugeben. Die Quelldatei wird dort unter gleichem Namen abgelegt.

Bei shutil.copytree() musst Du als Ziel explizit einen Namen angeben, der noch nicht existiert. Wahrscheinlich willst Du den selben Namen wie das Quellverzeichnis (hier also ".minecraft/"):

from os import path # isdir(), join()

# ...

  for line in lines:
    try:
      line = line.rstrip()
      # ...
      if path.isdir(line):
        copytree(line, path.join(dest, 
                                 path.basename(line)))
1
anonymia1 
Fragesteller
 23.05.2018, 17:17
@ralphdieter

Danke jetzt gehts.

Allerdings noch etwas "kleines", wenn der Ordner den ich Kopieren will sehr groß ist, fängt es an zu hängen. Kennst du vielleicht ein e kleine Möglichkeit dies zu unterbinden indem jede Datei einzeln kopiert wird wenn ein Ordner kopiert wird. Was dann auch den vorteil hätte das wenn nur 1 Ordner kopiert wird das der Ladebalken nicht von 0 auf 100 springt.

0