Lista z przedmiotami zwraca pusty

Question

Utworzono prostą funkcję listy, ale jeśli przepuszczę przez listę, jest pusta. Nie powinno być!

// 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() nie zamierza powrócić żadnej znaczącej wartości. Zamiast tego wypróbuj coś takiego:

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

Dobrze jest to problem na początek:

catch { } 

Jeśli coś pójdzie nie tak, będziesz tylko cicho przerwania.

Może to właśnie się dzieje? (EDYCJA: To jest.) Zobacz później.)

Kolejnym problemem jest to, że twoja "lista" zawiera tylko jeden ciąg ... czy to naprawdę zamierzałeś? Wątpię, czy lista, którą widzisz, jest rzeczywiście pusta - ale będzie zawierała pojedynczy pusty ciąg.

(Na marginesie, chciałbym silnie sugerujemy rozpocząć następujące konwencje nazewnictwa .NET i unikać zmiennych globalnych, takich jak ten.)

EDIT: Aargh - Właśnie sobie sprawę, co zrobiłeś . Prawdopodobnie dostaniesz NullReferenceException w list_all_pocesses, którą złapałeś i zignorowałeś.

Twoje połączenie z pro_hook.Add jest wykonywane przed przypisaniem wartości do pro_hook. Zasadniczo masz inicjator zmiennej, który używa metody, która z kolei używa zmiennej. Nie rób tego. Jeśli przejdziesz przez kod w debugerze, możesz uzyskać więcej informacji na temat tego, co się dzieje, ale w zasadzie stworzyłeś dla siebie dużą kulkę spaghetti.

Dlaczego list_all_pocesses nie zwróci tylko List<string>? Dlaczego w ogóle używasz StringBuilder?

Answer 3

Cóż ... zwracasz pustego konstruktora sznurków. To Twój problem. Twój kod robi to, co chcesz. :)

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; 
    } 
}