C# / VB.NET - [C#] Download Files

Forum - C# / VB.NET - [C#] Download Files

()
Newbie

Messaggi:
Iscritto:

Postato alle 1:00
Giovedì, 01/01/1970

Ciao.

Sto realizzando un programma per scaricare file di una qualunque estensione.
Ho 2 textbox, una dove immettere l'URL e l'altra dove immettere la directory di salvataggio e il nome del file.
Il problema dove sta? E' che quando faccio fare il download, si, viene compiuto alla grande ma il file viene salvato senza estensione... :nono:

Vi posto il codice:

Codice sorgente - presumibilmente C++

private void SelezionaDirectoryDownload()
        {
            SaveFileDialog SalvaFile = new SaveFileDialog();
            SalvaFile.Filter = "Tutti i tipi di file (*.*)|*.*";
            if (SalvaFile.ShowDialog() == DialogResult.OK)
            {
                SalvaFile.Dispose();
                TextBoxDirectory.Text = SalvaFile.FileName;
            }
        }

Codice sorgente - presumibilmente C++

private void DownloadFile()
        {
            try
            {
                WebClient Client = new WebClient();
                Uri URL = new Uri(TextBoxURL.Text);
                Client.DownloadFileAsync(URL, TextBoxDirectory.Text);     
            }
 
            catch (Exception)
            {
                MessageBox.Show("URL non valido o problema di connessione!", "Errore!", MessageBoxButtons.OK, MessageBoxIcon.Error);
            }             
        }

riseofapocalypse (Ex-Member)
Pro

Messaggi: 150
Iscritto: 08/07/2009

Postato alle 15:01
Sabato, 11/07/2009

Innanzitutto perchè esegui il Dispose prima di utilizzare SaveFile?

e comunque il Dispose puoi anche evitarlo, credo che venga effettuato automaticamente grazie al meccanismo di Garbage Collection!

Codice corretto:

Codice sorgente - presumibilmente C++

private void SelezionaDirectoryDownload()
        {
            SaveFileDialog SalvaFile = new SaveFileDialog();
            SalvaFile.Filter = "Tutti i tipi di file (*.*)|*.*";
            if (SalvaFile.ShowDialog() == DialogResult.OK)
                TextBoxDirectory.Text = SalvaFile.FileName;
        }

Il resto mi sembra corretto!

P.S. Forse non inserisce l'estensione perchè SaveFile non inserisce l'estensione in TextBoxDirectory :k:

()
Newbie

Messaggi:
Iscritto:

Postato alle 21:06
Sabato, 11/07/2009

A parte il fatto che manca una { dopo l'if (condizione)... :hail:

Non so nemmeno io perchè ho messo il Dispose(). :-|

Infatti, il problema di fondo è quello ma come faccio a far salvare le ultime 4 lettere della textbox prima (link) come ad esempio .txt ?

Ultima modifica effettuata da il 11/07/2009 alle 21:16

riseofapocalypse (Ex-Member)
Pro

Messaggi: 150
Iscritto: 08/07/2009

Postato alle 21:30
Sabato, 11/07/2009

Prima una precisazione

non manca una parentesi graffa, devi sapere che se l'If (ma anche il For, il While, ecc...) è composto da una sola istruzione, si possono omettere le sue graffe

Dopodichè

io ti consiglio di impostare la seconda parte così:

Codice sorgente - presumibilmente C++

private void DownloadFile()
        {
            try
            {
                WebClient Client = new WebClient();
                Uri URL = new Uri(TextBoxURL.Text);
                Client.DownloadFileAsync(URL, TextBoxDirectory.Text & TextBoxURL.Text.Substring(TextBoxURL.Text.LastIndexOf('.')););    
            }
 
            catch (Exception)
            {
                MessageBox.Show("URL non valido o problema di connessione!", "Errore!", MessageBoxButtons.OK, MessageBoxIcon.Error);
            }            
        }

Prova e fammi sapere perchè non l'ho provato :k:

Ultima modifica effettuata da riseofapocalypse il 11/07/2009 alle 21:34

Il Totem (Admin)
Guru^2

Messaggi: 3635
Iscritto: 24/01/2006

Postato alle 14:38
Domenica, 12/07/2009

IO.Path.GetExtension("url/file") funziona anche con indirizzi url. Comunque non è male usare Dispose. Anzi, sarebbe bene avere cura di distruggere gli oggetti dopo l'uso, anche se la posizione di questo metodo nel primo esempio era sbagliata.

()
Newbie

Messaggi:
Iscritto:

Postato alle 20:57
Domenica, 12/07/2009

@ riseofapocalypse : Grazie per la dritta dell'if ma il tuo metodo dell'estensione era un pò "grezzo". :k:

@ Il Totem : Grazie, era proprio quello che cercavo. :k: