是否更好地测试是否需要在其内部或外部使用函数？

Question

什么是最佳做法？调用一个函数然后返回,如果你测试一些东西,或测试一些东西然后调用？

我更喜欢函数内部的测试,因为它可以更容易地查看调用的函数.

例如:

protected void Application_BeginRequest(object sender, EventArgs e)
        {
            this.FixURLCosmetics();
        }

和

private void FixURLCosmetics()
        {
            HttpContext context = HttpContext.Current;
            if (!context.Request.HttpMethod.ToString().Equals("GET", StringComparison.OrdinalIgnoreCase))
            {
                // if not a GET method cancel url cosmetics
                return;
            };

            string url = context.Request.RawUrl.ToString();
            bool doRedirect = false;

            // remove > default.aspx
            if (url.EndsWith("/default.aspx", StringComparison.OrdinalIgnoreCase))
            {
                url = url.Substring(0, url.Length - 12);
                doRedirect = true;
            }

            // remove > www
            if (url.Contains("//www"))
            {
                url = url.Replace("//www", "//");
                doRedirect = true;
            }

            // redirect if necessary
            if (doRedirect)
            {
                context.Response.Redirect(url);
            }
        }

这个好吗:

if (!context.Request.HttpMethod.ToString().Equals("GET", StringComparison.OrdinalIgnoreCase))
            {
                // if not a GET method cancel url cosmetics
                return;
            };

或者应该进行该测试Application_BeginRequest？

什么是更好的？

日Thnx

Answer 1

我觉得测试里面的功能更好.如果你在函数之外测试,你将不得不测试可以调用函数的所有地方(并且会导致很多重复的代码).

将所有东西放在一个地方然后到处散布是更好的.

Answer 2

如果一个方法在它可以执行其功能之前绝对需要满足某个条件,那么是的,你应该将验证放在该函数中.另一方面,如果您的调用代码说"只在这组条件下执行此操作",则调用代码中的条件更好,因为下次要调用该方法时,您可能不希望包含该条件.