La lista con elementos devuelve vacío

Question

He creado una función de lista simple, pero si paso por la lista, está vacía. No debería ser!

// List function 
    public class process_hook 
    { 
     public static List<String> pro_hook = new List<String> 
               (new String[] { list_all_pocesses() }); 
     protected static string list_all_pocesses() 
     { 
      StringBuilder _list = new StringBuilder(); 
      foreach (Process i in Process.GetProcesses(".")) 
      { 
       try 
       { 
        foreach (ProcessModule pm in i.Modules) 
        { 
         pro_hook.Add(pm.FileName.ToString()); 
        } 
       } 
       catch { } 
      } 
      return _list.ToString(); 
     } 
    } 


     // call 
     private void button1_Click(object sender, EventArgs e) 
     { 
      foreach (String _list in process_hook.pro_hook) 
      { 
       Console.WriteLine(_list); 
      } 
     } 

Answer 1

_list.ToString() no va a devolver ningún valor significativo. Intente algo como esto en su lugar:

public static List<string> pro_hook = list_all_processes(); 

protected static List<string> list_all_processes() 
{ 
    var list = new List<string>(); 

    foreach (Process i in Process.GetProcesses(".")) { 
     try { 
      foreach (ProcessModule pm in i.Modules) { 
       list.Add(pm.FileName); 
      } 
     } catch { } 
    } 

    return list; 
} 

Answer 2

Bueno, esto es un problema para empezar:

catch { } 

Si algo va mal, vas a silencio aborto.

Quizás eso es lo que está pasando? (EDITAR: Sí. Ver más adelante.)

El siguiente problema es que su "lista" solo contendrá una sola cadena ... ¿es eso realmente lo que pretendía? Dudo que la lista que estás viendo esté realmente vacía, pero contendrá una sola cadena vacía.

(Como nota al margen, me fuertemente sugieren que se inicia siguiendo las convenciones de nomenclatura de .NET y evitar las variables globales como esta.)

EDIT: Aargh - Me acabo de dar cuenta de lo que has hecho . Probablemente obtenga un NullReferenceException en list_all_pocesses, que ha capturado e ignorado.

Su llamada a pro_hook.Add se realiza antes de que haya asignado un valor a pro_hook. Básicamente, tienes un inicializador variable que usa un método que a su vez usa la variable. No hagas eso. Si revisa su código en el depurador, puede hacerse una idea de lo que está sucediendo, pero básicamente ha creado una gran bola de espagueti para usted.

¿Por qué no list_all_pocesses acaba de devolver un List<string>? ¿Por qué estás usando un StringBuilder en absoluto?

Answer 3

Bueno ... estás devolviendo un generador de cadenas vacío. Ese es tu problema. Tu código está haciendo lo que le estás diciendo que haga. :)

return _list.ToString(); 

Answer 4

public class process_hook 
{ 
    public static List<string> pro_hook = list_all_pocesses(); 
    protected static List<string> list_all_pocesses() 
    { 
     List<string> list = new List<string>(); 

     foreach (Process i in Process.GetProcesses(".")) 
     { 
      foreach (ProcessModule pm in i.Modules) 
      { 
       list.Add(pm.FileName.ToString()); 
      } 
     } 
     return list; 
    } 
}