C# / VB.NET - Copia file

Forum - C# / VB.NET - Copia file - Pagina 2

Pagine: [ 1 2 3 4 5 ]

nessuno (Normal User)
Guru^2

Messaggi: 6403
Iscritto: 03/01/2010

Postato alle 19:13
Lunedì, 19/06/2017

Anche se non è un forum per la lingua ciò non vuol dire che non si debba scrivere in italiano corretto. Fra l'altro bastava una semplice correzione e non tutta questa polemica che fai.

Del resto ti ho aiutato con la soluzione per la parte codice, che vuoi ancora? Chiudiamola qui ...

Ricorda che nessuno è obbligato a risponderti e che nessuno è perfetto ...
---
Il grande studioso italiano Bruno de Finetti ( uno dei padri fondatori del moderno Calcolo delle probabilità ) chiamava il gioco del Lotto Tassa sulla stupidità.

lupetto77 (Normal User)
Pro

Messaggi: 75
Iscritto: 04/12/2011

Postato alle 19:43
Lunedì, 19/06/2017

Se ti ho offeso mi scuso, non voleva essere una polemica, infatti ho inserito anche le faccine che ridono, e ti ho sempre ringraziato per il tuo aiuto.

Premetto ho lavorato poco con la classe FileStream per i byte, quindi ho fatto delle ricerche e ho trovato questo, che ho adattato al mio codice, ma c'è un problema ho una cartella di prova con due cartelle al suo interno, in una ci sono 4 file da pochi byte e nell'altra 1 file superiore a 4 gb, con questo codice mi ha copiato solo la cartella con i 4 file, e l'altra cartella vuota.

Nel metodo DoWork()

Codice sorgente - presumibilmente C#

string newDir = "";
            foreach (string file in files)
            {
                fi = new FileInfo(file);
                newDir = targetDir  + fi.DirectoryName.Substring(sourceDir.Length) + "\\"; // + "\\"
                if (!Directory.Exists(newDir))
                    Directory.CreateDirectory(newDir);
 
                //FileInfo fiDir = new FileInfo(newDir + fi.Name);
 
                totaleFile += 1; //Conto i file
                nomeFile = "Copia file: " + file.Substring(0, 3) + "...\\" + fi.Name; // Preleva i nomi dei file
                //int tmpBytes = Convert.ToInt32((((progressBar1.Value) / (progressBar1.Maximum)) * 100));
 
                try
                {
                    using (FileStream fsSource = new FileStream(file, FileMode.Open, FileAccess.Read))
                    {
                        // Read the source file into a byte array.
                        byte[] bytes = new byte[fsSource.Length];
                        int numBytesToRead = (int)fsSource.Length;
                        int numBytesRead = 0;
                        while (numBytesToRead > 0)
                        {
                            // Leggi può restituire qualsiasi cosa da 0 a numBytesToRead.
                            int n = fsSource.Read(bytes, numBytesRead, numBytesToRead);
 
                            // Interrompe quando viene raggiunta la fine del file.
                            if (n == 0)
                                break;
 
                            numBytesRead += n;
                            numBytesToRead -= n;
                        }
                        numBytesToRead = bytes.Length;
 
                        // Scrivere l'array byte nell'altro FileStream.
                        using (FileStream fsNew = new FileStream(newDir + fi.Name, FileMode.Create, FileAccess.Write))
                        {
                            fsNew.Write(bytes, 0, numBytesToRead);
                        }
                    }
                }
                catch (FileNotFoundException ioEx)
                {
                    MessageBox.Show(ioEx.Message);
                }
            }

e ancora grazie per l'aiuto che mi hai dato dall'inizio ad ora, e scusami se ti ho offeso. :hail:

Ultima modifica effettuata da lupetto77 il 19/06/2017 alle 19:44

lupetto77 (Normal User)
Pro

Messaggi: 75
Iscritto: 04/12/2011

Postato alle 2:09
Martedì, 20/06/2017

Ho provato anche usando BinaryReader e BinaryWriter:

Codice sorgente - presumibilmente C# / VB.NET

byte[] allBytes;
                        Stream str = File.OpenRead(file);
                        Stream str2 = File.OpenWrite(newDir + fi.Name);
                        using (BinaryReader br = new BinaryReader(str))
                        {
                            allBytes = br.ReadBytes((int)str.Length);
                        }
 
                        using (BinaryWriter bw = new BinaryWriter(str2))
                        {
                            bw.Write(allBytes);
                        }

Il risultato è questo:
copiati i 4 file
il file superiore di 4gb, dimensione finale è 1.7gb, quindi inutilizzabile.

Cos'è che sbaglio?

nessuno (Normal User)
Guru^2

Messaggi: 6403
Iscritto: 03/01/2010

Postato alle 9:21
Martedì, 20/06/2017

Non devi leggere e scrivere il file in un solo passaggio. Devi allocare un buffer più piccolo e leggere/scrivere in un ciclo passando il valore di byte trattati alla bar ad ogni giro

lupetto77 (Normal User)
Pro

Messaggi: 75
Iscritto: 04/12/2011

Postato alle 17:11
Martedì, 20/06/2017

Grazie per il tuo continuo aiuto.

Scusami cosa c'è di sbagliato qui, perchè io passo i byte alla progressbar ad ogni ciclo:

Codice sorgente - presumibilmente C#

string newDir = "";
            foreach (string file in files)
            {
                fi = new FileInfo(file);
                newDir = targetDir + "\\" + fi.DirectoryName.Substring(sourceDir.Length) + "\\"; // + "\\"
                if (!Directory.Exists(newDir))
                    Directory.CreateDirectory(newDir);
 
                //FileInfo fiDir = new FileInfo(newDir + fi.Name);
 
                totaleFile += 1; //Conto i file
                nomeFile = "Copia file: " + file.Substring(0, 3) + "...\\" + fi.Name; // Preleva i nomi dei file
                                                                                      
 
try
                {
                    using (FileStream fsSource = new FileStream(file, FileMode.Open, FileAccess.Read))
                    {
                        // Leggere il file di origine in un array di byte.
                        byte[] bytes = new byte[fsSource.Length];
                        numBytesToRead = (int)fsSource.Length;
                        int numBytesRead = 0;
                        while (numBytesToRead > 0)
                        {
                            // Questo può restituire qualsiasi cosa da 0 a numBytesToRead.
                            int n = fsSource.Read(bytes, numBytesRead, numBytesToRead);
 
                            // Interrompe quando viene raggiunta la fine del file.
                            if (n == 0)
                                break;
 
                            numBytesRead += n;
                            numBytesToRead -= n;
                        }
                        numBytesToRead = bytes.Length;
 
                        // Scrivo l'array byte nell'altro FileStream.
                        using (FileStream fsNew = new FileStream(newDir + fi.Name, FileMode.Create, FileAccess.Write))
                        {
                            fsNew.Write(bytes, 0, (int)numBytesToRead);
                        }
 
                        backgroundWorker1.ReportProgress(numBytesToRead); //Passo alla progressBar il valore dei byte del file
                    }
                }
                catch (FileNotFoundException ioEx)
                {
                    MessageBox.Show(ioEx.Message);
                }
            }

Io qui leggo un file alla volta per poi passarlo al FileStream ad ogni ciclo da dove prelevo i byte del file, infine copiarli e passare il valore alla progressbar.

Non devi leggere e scrivere il file in un solo passaggio. Devi allocare un buffer più piccolo e leggere/scrivere in un ciclo passando il valore di byte trattati alla bar ad ogni giro

Non ho capito bene quello che mi vuoi dire.
Dovrei leggere il file passato per poi inserire una parte di byte in un buffer e passarli alla progressbar?
Mi faresti un piccolo esempio se puoi.

Ultima modifica effettuata da lupetto77 il 20/06/2017 alle 17:27

nessuno (Normal User)
Guru^2

Messaggi: 6403
Iscritto: 03/01/2010

Postato alle 18:51
Martedì, 20/06/2017

Leggi questa discussione

https://stackoverflow.com/questions/6044629/file-copy-with- ...

e l'esempio che usa il buffer di 1024 * 1024 bytes

lupetto77 (Normal User)
Pro

Messaggi: 75
Iscritto: 04/12/2011

Postato alle 0:51
Mercoledì, 21/06/2017

Ciao grazie per link, mi Ã¨ stato di ispirazione per capire come funziona.
Ho scritto questo codice:

Codice sorgente - presumibilmente C#

string newDir = "";
            foreach (string file in files)
            {
                fi = new FileInfo(file);
                newDir = targetDir + "\\" + fi.DirectoryName.Substring(sourceDir.Length) + "\\"; // + "\\"
                if (!Directory.Exists(newDir))
                    Directory.CreateDirectory(newDir);
 
                totaleFile += 1; //Conto i file
                nomeFile = "Copia file: " + file.Substring(0, 3) + "...\\" + fi.Name; // Preleva i nomi dei file
 
                int BUFFER_SIZE = 1024; //1 KB
                byte[] buffer = new byte[BUFFER_SIZE];
                FileStream fsr = File.Open(file, FileMode.Open, FileAccess.Read);
                FileStream fsw = File.Create(newDir + fi.Name, BUFFER_SIZE, FileOptions.Asynchronous);
                int readBytes = fsr.Read(buffer, 0, BUFFER_SIZE);
                Task t = new Task(() =>
                {
                    while (readBytes > 0)
                    {
                        double persentage = (double)fsr.Length * 100.0 / readBytes;
                        backgroundWorker1.ReportProgress((int)persentage);
 
                        fsw.Write(buffer, 0, BUFFER_SIZE);
 
                        readBytes = fsr.Read(buffer, 0, BUFFER_SIZE);
                    }
                });
                //t.Wait();
                t.RunSynchronously();
}

Ora riesco a copiare i file.

Ma ci sono 3 problemi:
1. Non viene rispettata la dimensione finale dei file, questo perche'¨ si scrive la dimensione del BUFFER_SIZE, pero' dovra' pure copiare la parte del file letta.

2. La copia risulta piu' lenta di quella fatta con il copia e incolla di windows.

3. Riprende il passo 1, i file di dimensioni piu' piccoli risultano sempre di 1kb, e anche quelli piu' grossi di dimensione possono risultare un pochino piu' grandi.

Idee su come posso risolvere?
Intanto continuo con le mie prove, grazie ancora.

Ultima modifica effettuata da lupetto77 il 21/06/2017 alle 22:53

lupetto77 (Normal User)
Pro

Messaggi: 75
Iscritto: 04/12/2011

Postato alle 17:58
Giovedì, 22/06/2017

Ho fatto altri progressi risolvendo i tre problemi elencati sopra, in questo modo:

Codice sorgente - presumibilmente C#

string[] files;
        FileInfo fi;
        long bytesTotali;
 
        private void GetFiles(string path)
        {
            files = Directory.GetFiles(path, "*.*", SearchOption.AllDirectories);
 
            foreach (string f in files)
            {
                FileInfo fi = new FileInfo(f);
                bytesTotali += fi.Length;
            }
 
            progressBar1.Value = 0;
            progressBar1.Maximum = (int)bytesTotali;
        }
 
        private async Task CopyFilesAsync(FileStream Source, FileStream Destination)
        {
            byte[] buffer = new byte[1024 * 1024];
            int numRead;
            while ((numRead = await Source.ReadAsync(buffer, 0, buffer.Length)) != 0)
            {
                int percentuale = Convert.ToInt32(((numRead / bytesTotali) * 100));
                backgroundWorker1.ReportProgress(percentuale);
                await Destination.WriteAsync(buffer, 0, numRead);
            }
        }
 
        private void button1_Click(object sender, EventArgs e)
        {
            backgroundWorker1.RunWorkerAsync();
        }
 
        private async void backgroundWorker1_DoWork(object sender, DoWorkEventArgs e)
        {
            string UserDirectory = @"C:\Users\nomeUtente\Desktop\1";
            string targetDir = @"C:\Users\nomeUtente\Desktop\2";
 
            GetFiles(UserDirectory);
 
            string newDir = "";
            foreach (string file in files)
            {
                fi = new FileInfo(file);
                newDir = targetDir + "\\" + fi.DirectoryName.Substring(UserDirectory.Length) + "\\";
                if (!Directory.Exists(newDir))
                    Directory.CreateDirectory(newDir);
 
                using (FileStream SourceReader = File.Open(file, FileMode.Open))
                {
                    using (FileStream DestinationWriter = File.Create(newDir + fi.Name))
                    {
                        await CopyFilesAsync(SourceReader, DestinationWriter);
                    }
                }
            }
 
            MessageBox.Show("Copia effettuata con successo!");
        }
 
        private void backgroundWorker1_ProgressChanged(object sender, ProgressChangedEventArgs e)
        {
            progressBar1.Value = e.ProgressPercentage;
        }

Ma è rimasto un solo problema, quello della progressBar, mi da la seguente eccezione:

Codice sorgente - presumibilmente Plain Text

System.InvalidOperationException: 'Operazione cross-thread non valida: è stato eseguito l'accesso al controllo 'progressBar1' da un thread diverso da quello da cui è stata eseguita la creazione.'

Se si decommentano le seguenti righe funziona ma senza progressBar ovviamente:

Codice sorgente - presumibilmente Plain Text

Nel metodo GetFiles
progressBar1.Value = 0;
progressBar1.Maximum = (int)bytesTotali;

Codice sorgente - presumibilmente C# / VB.NET

Nella funzione CopyFilesAsync()
int percentuale = Convert.ToInt32(((numRead / bytesTotali) * 100));
backgroundWorker1.ReportProgress(percentuale);

Spero che qualcuno mi possa dare una mano, ci ho provato e riprovato ma niente :grr:

Continuerò a provare ma la vedo dura questa volta, spero in un aiuto da voi esperti.

Ultima modifica effettuata da lupetto77 il 22/06/2017 alle 18:00

Pagine: [ 1 2 3 4 5 ]