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:

Mountain's man said no!

()
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

Mountain's man said no!

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.

"Infelici sono quelli che hanno tanto cervello da vedere la loro stupidità."
(Fligende Blatter)

"Dubitare di se stessi è il primo segno d'intelligenza."
(Ugo Ojetti)

()
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: