[C#] Problema con list directory

[fermat]

ho creato questa classe:

using System;
using System.Collections.Generic;
using System.Text;
using System.IO;

namespace sharpFtpHell
{
    class ListFile
    {
        public string scansionaDirectory(string path)
        {
            DirectoryInfo dir = new DirectoryInfo(path);
            string result = "";
            if (path == null || path == "")
            {
                path = "E:\\";
            }
            FileInfo[] files = dir.GetFiles(path, SearchOption.AllDirectories);
            foreach (FileInfo fi in files)
            {
                Console.Write(fi.Name);
            }
            result = path + files;
            return result;
        }
    }
}

questa deve essere richiamata in un form in modo tale quando clicco sul bottone mi lista tutti i file e directory della dir selezionata.
la di iniziale è ricavata da una textbox:

    public partial class Form1 : Form
    {
        sharpFtpHell.ListFile objFile = new sharpFtpHell.ListFile();

        public Form1()
        {
            InitializeComponent();
            txtLocalPath.Text = "E:\\";
        }

        private void btnList_Click(object sender, EventArgs e)
        {
            txtLocal.Text = objFile.scansionaDirectory(txtLocalPath.Text);
        }
....

il problema è che al momento del click mi da questo errore:

Non iniziare la seconda parte del percorso con un nome di unità o un nome UNC.
Nome parametro: path2.

ci stanno poi tutti i dettagli che però sono lunghi.
se volete li posto.

[Pirelli72]

Se istanzi la variabile dir che e di tipo DirectoryInfo con la directory ad es. C:\ non puoi chiedergli la lista dei file ancora in C:\

[fermat]

scusami e allora che dovrei fare?
cioè, dov'è che sbaglio di preciso??

[albgen]

prova cosi'

class ListFile
    {
        public string scansionaDirectory(string path)
        {
            if (path == null || path == "")
            {
                path = "E:\\";
            }
            DirectoryInfo dir = new DirectoryInfo(path);
            string result = "";
           
            FileInfo[] files = dir.GetFiles();
            foreach (FileInfo fi in files)
            {
                Console.Write(fi.Name);
            }
            result = path + files;
            return result;
        }
    }

[fermat]

mi da un risultato strano:
E:\System.IO.FileInfo[]

come se nn leggesse le info.

[Pirelli72]

Per forza hai creato una funzione che ti restituisce una stringa singola e tu hai bisogno che ti ritorni una lista di stringhe e in aggiunta hai valorizzato una variabile stringa con una matrice di stringhe.

 result = path + files;

result è una stringa.
path è una stringa.

files è invece una matrice di stringhe.

FileInfo[] files = dir.GetFiles();

per cui la variabile result risulta essere:

 result = path + files.ToString ();

[fermat]

sicuramente sbaglio ma nn penso che il ToString() sia fondamentale.
infatti mi da lo stesso errore.

nn potrebbe essere il fatto che il result sta fuori dal ciclo?
ho provato a creare un ciclo for del genere:

class ListFile
    {
        public string scansionaDirectory(string path)
        {
            if (path == null || path == "")
            {
                path = "E:\\";
            }
            DirectoryInfo dir = new DirectoryInfo(path);
            string result = "";
            FileInfo[] files = dir.GetFiles();
            for (int i = 0; i < files[i].Length; i++)
            {
                result = path + files;
            }  
            return result;
        }
    }

ma mi da un errore di Indici fuori dalla matrice.

[Pirelli72]

Il .ToString() è ovvio che non risolve il tuo problema era solo per farti capire come mai ti viene fuori quel risultato: E:\System.IO.FileInfo[], per cui scrivere:

result = path + files;

o

result = path + files.ToString ();

è lo stesso perchè NON puoi valorizzare una variabile stringa con una matrice di stringhe.

Per cui per risolvere in maniera veloce puoi scrivere:

  public void scansionaDirectory(string path)
        {
            if (path == null || path == "")
            {
                path = "E:\\";
            }
            DirectoryInfo dir = new DirectoryInfo(path);
         
            FileInfo[] files = dir.GetFiles();
            foreach (FileInfo fi in files)
            {
                listBox1.Items.Add(path + fi.Name);
            }
        }

e hai la tua Lista di File, ma potrebbe non andar bene per i tuoi scopi.

[fermat]

Originariamente inviato da Pirelli72
Il .ToString() è ovvio che non risolve il tuo problema era solo per farti capire come mai ti viene fuori quel risultato: E:\System.IO.FileInfo[], per cui scrivere:

result = path + files;

o

result = path + files.ToString ();

è lo stesso perchè NON puoi valorizzare una variabile stringa con una matrice di stringhe.

Per cui per risolvere in maniera veloce puoi scrivere:

  public void scansionaDirectory(string path)
        {
            if (path == null || path == "")
            {
                path = "E:\\";
            }
            DirectoryInfo dir = new DirectoryInfo(path);
         
            FileInfo[] files = dir.GetFiles();
            foreach (FileInfo fi in files)
            {
                listBox1.Items.Add(path + fi.Name);
            }
        }

e hai la tua Lista di File, ma potrebbe non andar bene per i tuoi scopi.

questo codice però dovrei metterlo nel form direttamente giusto?

[Pirelli72]

Si esatto, per quello che ti dicevo prima potrebbe non andare bene.

Se la tua necessità è quella di incorporare la tua funzione in una classe allora potresti scrivere:

 public List<string> scansionaDirectory(string path)
        {
            List<string> myList = new List<string>();
            DirectoryInfo dir = new DirectoryInfo(path);
            string result = "";
            if (path == null || string.IsNullOrEmpty(path))
            {
                path = "E:\\";
            }
            FileInfo[] files = dir.GetFiles();
            foreach (FileInfo fi in files)
            {
                myList.Add(string.Concat(path, fi.Name));
            }
            return myList;
        }

e poi richiamare dalla Form:

List<string> myList = null;
            myList = scansionaDirectory("C:\\");

Come puoi vedere, siccome è un esempio, non ho creato l'istanza di nessuna classe.