你会如何重构页面成员的ASPX代码？

我已经继承了一个项目，该项目使用以下模式将代码后面的参数传递给aspx页面。我知道这是错误的，但是我正在重构最佳方式。你会如何重构页面成员的ASPX代码？

代码背后：

using System; 
using System.Web; 

namespace BadPassing 
{ 
    public partial class _Default : System.Web.UI.Page 
    { 
     private Random rnd = new Random(); //Is this bad? 

     protected int numberOne; //this is bad 
     protected int numberTwo; //this is bad 

     protected void Page_Load(object sender, EventArgs e) 
     { 
      numberOne = rnd.Next(100); 
      numberTwo = rnd.Next(100); 
     } 
    } 
}

ASPX页面：

<%@ Page Language="C#" AutoEventWireup="true" CodeBehind="Default.aspx.cs" Inherits="BadPassing._Default" %> 

<!DOCTYPE html PUBLIC "-//W3C//DTD XHTML 1.0 Transitional//EN" "http://www.w3.org/TR/xhtml1/DTD/xhtml1-transitional.dtd"> 

<html xmlns="http://www.w3.org/1999/xhtml" > 
<head runat="server"> 
    <title>Bad Page</title> 
</head> 
<body> 
    <form id="form1" runat="server"> 
    <div> 
     <a href="http://www.google.com/search?q=<%=this.numberOne %>%2B<%=this.numberTwo %>"> 
      Add <%=this.numberOne %> and <%=this.numberTwo %> using google. 
     </a> 
    </div> 
    </form> 
</body> 
</html>

我的理解是，的Numberone和numberTwo不是线程安全的，并可能导致不正确的行为，如果两个人在加载页面时同一时间。此外，如果页面依赖于numberOne和numberTwo在回发之间存储值，则多个同时发生的用户将导致意外的结果。

我的理解是为什么上述技术如此糟糕是正确的，如果是的话，您会如何最好地重构此代码？

请注意，将无状态页面级服务（如随机）存储为页面类的成员变量是不正确的？

来源

2009-01-08 Ryan Michela

此代码没问题。每次页面加载时都会实例化成员变量，因为在每次页面加载时都会创建一个类的新实例。如果变量是静态的，那么只有在两个人同时加载页面时才会出现问题。

想想你放在页面上的控件。它们是成员变量，但它们在通过不同请求同时加载页面时没有问题。

来源

2009-01-08 02:16:35

谢谢。我完全误解了ASPX渲染周期。这很明显。 – 2009-01-08 04:08:48

在正常情况下，如果两个用户同时加载页面，他们将使用不同的页面实例。由于numberOne和numberTwo是实例变量，因此不会有共享状态，也不存在线程问题。

来源

2009-01-08 02:14:33 Brownie

虽然心不是真的anyhting错误的代码（如已经提到的），如果你想重构它，我可能会做沿着使用系统

线的东西; using System.Web;

namespace BadPassing 
{ 
    public partial class _Default : System.Web.UI.Page 
    { 
     private Random rnd = new Random(); //Is this bad? 

     protected int numberOne; //this is bad 
     protected int numberTwo; //this is bad 

     protected void Page_Load(object sender, EventArgs e) 
     { 
      numberOne = rnd.Next(100); 
      numberTwo = rnd.Next(100); 
      searchLink.NavigationUrl = String.Format("http://www.google.com/search?q={0}%2B{1}", numberOne, numberTwo); 
      searchLink.Text = String.Format("Add {0} and {1} using google", numberOne, numberTwo); 
     } 
    } 
}

，然后在你的aspx页面只需添加一个<asp:hyperlink>控制

来源

2009-01-08 03:06:29 lomaxx

你会如何重构页面成员的ASPX代码？

回答

相关问题