C# Action<T>, de Passer de la fonction en tant que paramètre

Je veux passer une condition que l'Action pour une autre méthode. Première ligne dans "ComputerPriceGenerator" œuvres", mais comment faire pour que le tableau de travail (deuxième ligne)?.. Toutes les idées

Je suis à la recherche de conseils..., CalculateAllPrice n'est pas conçu encore

public void ComputerPriceGenerator()
{
    //Below line Works
    PriceMachine.CalculatePrice(cart.Computers[0],() => ComputerConverter(cart.Computers[0]));           
    //How to make this work, i don't want to loop it???
    PriceMachine.CalculateAllPrice(cart.Computers,() => ComputerConverter(??));
}

public void ComputerConverter(Computer comp)
{
    if (comp.Memory <= 2)
        comp.Discount = 10;
}
Votre code aurait été un candidat idéal pour l'utilisation de LINQ, cependant, il est plein d'effets secondaires. J'avais envisager de refactoring.
Sans montrer les signatures de CalculatePrice et CalculateAllPrice il est difficile de savoir comment vous aider.
Skeet: Il me semblait qu'il était partie à la recherche pour obtenir des conseils sur ce que la signature de CalculateAllPrice devrait être. Je peux me tromper, cependant.
Morgan... je suis à la recherche de conseils..., CalculateAllPrice n'est pas encore conçu, mais pour CalculatePrice il prend type d'Action CalculatePrice(Ordinateur comp, Action myAction)

OriginalL'auteur kayak | 2011-01-31

  1. 10

    Votre CalculatePrice méthode ne devrait pas prendre juste Action, OMI - les deux méthodes doivent prendre Action<Computer>. Je voudrais donc avoir des méthodes comme ceci:

    public static void CalculatePrice(Computer computer, Action<Computer> action)
public static void CalcuateAllPrices(IEnumerable<Computer> computers,
                                     Action<Computer> action)

    et de les appeler comme ceci:

    PriceMachine.CalculatePrice(cart.Computers[0], ComputerConverter);
PriceMachine.CalculateAllPrice(cart.Computers, ComputerConverter);
    Cela a fonctionné merci beaucoup
    Jon, j'ai une question, non résolue(cette question n'est pas liée à celui-ci, je vais supprimer ce) .., Laissez-moi savoir si vous avez des suggestions, " stackoverflow.com/questions/4444972/... "
    Les réponses dire à peu près ce que je voudrais. Vous avez presque certainement ne voulez pas aller en bas de la route.
    Merci @Jon SKeet
    Jon, est votre deuxième méthode, un exemple d'une monade? J'ai essayé d'envelopper ma tête autour de ce qu'ils sont et comment les utiliser.

    OriginalL'auteur Jon Skeet

  2. 1

    Que vous souhaitez appliquer la méthode à tous les éléments du tableau, vous n'obtiendrez pas autour d'une itération sur elle.

    Vous pouvez définir PriceMachine.CalculateAllPrice en tant que tel:

    public void CalculateAllPrice(IEnumerable<Computer> data, Action<Computer> action)
{
  foreach(Computer c in data)
    action(c);
}
    je suis d'accord, mais comment passer à l'Action de la principale méthode " PriceMachine.CalculateAllPrice(cart.Ordinateurs,() => ComputerConverter(??));"

    OriginalL'auteur Femaref

  3. 1
    PriceMachine.CalculateAllPrice(cart.Computers, (Computer x) => ComputerConverter(x));

    Puis faire CalculateAllPrice itérer cart.Computers et passer de l'un à la fonction anonyme.

    Semble un peu exagéré. (cart.Computers, (Computer x) => ComputerConverter(x)) est le même que (cart.Computers, ComputerConverter).
    Bien sûr, j'étais juste garder avec le lambda de style, il a posté.

    OriginalL'auteur Justin Morgan