日期:2014-05-16  浏览次数:20890 次

asp.net mvc3:自己编写的一些垃圾代码,删前分享(2013-01-18)
不小心发现自己编写的一些垃圾代码,删前留念。可以作为面试题使用。
有时候一个程序员会或者不会什么具体的技术差别不大,因为看看书几分钟就能学会;但编写垃圾代码一旦养成习惯,不是一天两天能改好的。
1. 这段代码等同于?
            return vc.RequestContext.HttpContext.Request.Params["view"] != null
                       ? vc.RequestContext.HttpContext.Request.Params["view"]
                       : null;

2. 如何写成一行代码?
            if (String.IsNullOrEmpty(originalPage))
            {
                AnaylizeUrl(isInHomeArea, page.Request.RawUrl);
            }
            else
            {
                AnaylizeUrl(isInHomeArea, HttpUtility.UrlDecode(originalPage));
            }

3. 这些代码实际上不存在,但差点被写出来,请看哪里可以优化?
名字很长,可能不太容易看懂,但不用看懂就能发现有问题的地方。
                if (team.MethodologyValueOf(Methodology.TeamManagementTypeName) == Methodology.TeamManagementTypeKanban)
                {
                    @RenderPage("MyItems/_MyItemsKanban.cshtml", myItemsViewModel, team, userName)
                }
                else if (team.MethodologyValueOf(Methodology.TeamManagementTypeName) == Methodology.TeamManagementTypeScrum)
                {
                    @RenderPage("MyItems/_MyItemsScrum.cshtml", myItemsViewModel, team, userName)
                }
                else if (....)
                {
                    ...
                }
Mehthodology在此定义:
        public static string TeamManagementTypeName = "TeamManagementType";
        public static string TeamManagementTypeKanban = "Kanban";
        public static string TeamManagementTypeScrum = "Scrum";
提示:修改后那些if-else-if代码只有一行,无论日后是否增加新的分支,都无需改动。