Uso variable en lugar del nombre de la función

Question

estoy usando el siguiente momento:

foreach (string file in files) { 
    switch (filetype.Value) { 
     case "ReadFile": 
      ReadFile(file); 
      break; 
     case "ReadMSOfficeWordFile": 
      ReadMSOfficeWordFile(file); 
      break; 
     case "ReadMSOfficeExcelFile": 
      ReadMSOfficeExcelFile(file); 
      break; 
     case "ReadPDFFile": 
      ReadPDFFile(file); 
      break; 
    } 
} 

Funciona, pero se siente un poco mal. La forma en Python sería algo más parecido a esto:

foreach string file in files: 
    filetype.Value(file) 

tengo un momento muy difícil imaginar que C# no se puede hacer algo como esto. Puede ser que mis habilidades de Google sean malas, pero parece que no puedo resolverlo.

SOLUCIÓN

public static readonly IDictionary<string, Action<string>> FileTypesDict = new Dictionary<string,Action<string>> { 
    {"*.txt", ReadFile}, 
    {"*.doc", ReadMSOfficeWordFile}, 
    {"*.docx", ReadMSOfficeWordFile}, 
    {"*.xls", ReadMSOfficeExcelFile}, 
    {"*.xlsx", ReadMSOfficeExcelFile}, 
    {"*.pdf", ReadPDFFile}, 
}; 

 

foreach (KeyValuePair<string, Action<string>> filetype in FileTypesDict) { 
    string[] files = Directory.GetFiles(FilePath, filetype.Key, SearchOption.AllDirectories); 
    //System.Reflection.MethodInfo ReadFileMethod = ReadFile.GetType().GetMethod(filetype.Value); 
    foreach (string file in files) { 
     FileTypesDict[filetype.Key](file); 
    } 
} 

Answer 1

Puede hacerlo con un poco de preparación utilizando los delegados, así:

private static readonly IDictionary<string,Action<string>> actionByType = 
    new Dictionary<string,Action<string>> { 
     {"ReadFile", ReadFile} 
    , {"ReadMSOfficeWordFile", ReadMSOfficeWordFile} 
    , {"ReadMSOfficeExcelFile", ReadMSOfficeExcelFile} 
    , {"ReadPDFFile", ReadPDFFile} 
    }; 

Cuando es el momento de llamar a su acción , hazlo de la siguiente manera:

actionByType[actionName](file); 

Answer 2

se podía mantener un Dictionary de delegados, como la forma más sencilla:

Dictionary<string, Action<string>> fileReaders = new Dictionary<string, Action<string>>() { 
    {"ReadFile", ReadFile}, 
    {"ReadOfficeWordFile", ReadOfficeWordFile}, 
    {"ReadOfficeExcelFile", ReadOfficeExcelFile}, 
    {"ReadPDFFile", ReadPDFFile} 
}; 

A continuación, llame así:

fileReaders[fileType.Value](file); 

Dependiendo de lo que regresan a sus métodos, es posible que tenga para cambiar el tipo de delegado (Action<string> significa void something(string someparam) como firma de método, por ejemplo) también.

Answer 3

Puede utilizar un delegado:

Action<string> action; 
switch (filetype.Value) { 
    case "ReadFile": 
    action = ReadFile; 
    break; 
    case "ReadMSOfficeWordFile": 
    action = ReadMSOfficeWordFile; 
    break; 
    case "ReadMSOfficeExcelFile": 
    action = ReadMSOfficeExcelFile; 
    break; 
    case "ReadPDFFile": 
    action = ReadPDFFile; 
    break; 
    default: 
    throw new NotImplementedException("Unhandled file type '"+filetype.Value+"'."); 
} 
foreach (string file in files) { 
    action(file); 
} 

Answer 4

También podría utilizar la reflexión (si estás bien con los diferentes sobrecarga que puede traer) Marque esta solution

esperanza de que ayudó

Answer 5

Creo que lo que está buscando necesitará alguna refacturación de su código.

Todos sus "casos" (TextFile, MSOfficeWordFile, MSOfficeExcelFile, PdfFile) deben ser sus propias clases que implementan una interfaz única.

Su interfaz debe denominarse como "IReadableFile" y especificar un método llamado "ReadFile()".

Cada clase debe tener su propia implementación de "ReadFile()"

Ejemplo:

public interface IReadableFile 
{ 
    void ReadFile(); 
} 

---

public class MSOfficeWordFile : IReadableFile 
{ 
    public void ReadFile() 
    { 
     ReadMSOfficeWordFile(file); 
    } 
} 

---

foreach(IReadableFile file in files) 
    file.ReadFile(); 

Código puede contener algunos errores, pero Espero que entiendas la idea.

Answer 6

Si usted está buscando una manera de evitar la asignación explícita de los nombres de los métodos a los valores de cadena que puede utilizar la reflexión para hacer la invocación método dinámico (esto supone filetype.Value es de tipo String)

String method_name = String.Empty; 
foreach (string file in files) { 
    method_name = filetype.Value; 
    System.Reflection.MethodInfo method = this.GetType().GetMethod(method_name); 
    method.Invoke(this, new object[]{file}); 
}