public class Foo<T>

{

    static protected T StartValue { get; private set; }

​

    public Foo(T startValue)

    {

        StartValue = startValue;

    }

//I don't see the reason arg2 here being T, if we are going to keep the same calculation. It might just be an object

    public static int DoCalc(object arg1, T arg2)

    {

        //no 0 check for StartValue

        //no decimal check for arg1 

        //no check for all if the value is too large (e.g. Int64)

        //no check if it is a string

        return (int)arg1 + Convert.ToInt32(arg2) / Convert.ToInt32(StartValue);

    }

​

    public static int higherOrderCalculation(Func<int, int, int> Func, int one, int two)

    {

        return Func(one, two);

    }

​

    public void LogToConsole(string message)

    {

        Console.WriteLine(string.Format(message));//this can be message, no need for string.Format()

    }

// It's an overall bad practice to use void on an async method, it should be Task instead

    public async void LogtoFile(string msg)

    {

        await File.WriteAllTextAsync($"logfile{DateTime.Now.ToString("yyyy-mm-dd")}.log", "msg");//"msg" should be msg in order to log the actual message

    }

}

​