CODEWITHSUNDEEP
.netdelegatesactioninline
Larry
Larry

Reputation: 18041

Is it a good practice to use Action delegates as Inline functions?

I often have some piece of code that repeats.

Usually, I put them in a function, but sometimes I hate to do this because :

So, to simulate Inline code which is missing from C#, I use Action delegates :

public void Display(DateTime from, DateTime to)
{
    var start = from.ToOADate();
    var end = to.ToOADate();

    [...]

    // This Action delegate helps me not to repeat the code.
    var removePoints = new Action<Series>(serie =>
    {
        var pointsToRemove = serie.Points.Where(pt => pt.XValue < start || pt.XValue > end).ToArray();

        foreach (var pt in pointsToRemove)
            serie.Points.Remove(pt);
    });

    removePoints(FlameTemperatureSerie);
    removePoints(BoshGasFlowRateSerie);
    removePoints(PercCOSerie);
    removePoints(PercH2Serie);

    [...]
}

This is quite helpful, especially because the Action delegate execution context can use the local variables.

I seems good to me, but I never saw nowhere Action delegates used this way. That's why I would like to know if this practice could be recommended, or if might causes issues I dont know.

Upvotes: 1

Views: 955

Answers (2)

Mike B
Mike B

Reputation: 2672

It is perfectly valid to have a function that is only called once if it makes the code easier to read and maintain.

Upvotes: 0

SLaks
SLaks

Reputation: 887777

As long as it doesn't get too confusing, there is nothing wrong with that.

Upvotes: 2

Related Questions