下面的方法怎么改写，使得更加简洁容易理解

[caicoko]

下面是web控件的部分重写代码，有些乱，用了很多if语句，如何改写使得代码更加简洁，更加容易理解。

protected override void OnPreRender(EventArgs e)
{
	if (this._controlType == "DateTime" && (BB.Get("action") == "insert" || BB.Get("action") == "copy"))
	{
		this.Text = DateTime.Now.ToString();
	}
	if (this._controlType == "UpdateDate" && BB.Get("action") == "update")
	{
		this.Text = DateTime.Now.ToString();
	}
	if (this._controlType == "InputUser" && BB.Get("action") == "insert")
	{
		this.Text = BB.LoginKey("Name") != null ? BB.LoginKey("Name") : String.Empty;
	}
	if (this._controlType == "InputUserName" && BB.Get("action") == "insert")
	{
		this.Text = BB.LoginKey("Name") != null ? BB.LoginKey("Name") : String.Empty;
	}
	if (this._controlType == "InputUserID" && (BB.Get("action") == "insert" || BB.Get("action") == "copy"))
	{
		this.Text = BB.LoginKey("UserID") != null ? BB.LoginKey("UserID") : String.Empty;
	}
	if (this._controlType == "Password")
	{
		this.TextMode = TextBoxMode.Password;
	}
	if (this._controlType == "MultiLine")
	{
		this.TextMode = TextBoxMode.MultiLine;
	}
	base.OnPreRender(e);
}

[xjfhnsd]

switch(this._controlType)吧，不过还是一样，只是好看点

[TimZhuFaith]

这么多if都要判断的。。。为什么不用elseif switch

[Fibona]

用switch好点

switch(this._controlType)
case ""

[fox_sky]

protected override void OnPreRender(EventArgs e)
{
if ((this._controlType == "DateTime" && (BB.Get("action") == "insert" || BB.Get("action") == "copy"))||
(this._controlType == "UpdateDate" && BB.Get("action") == "update"))
{
this.Text = DateTime.Now.ToString();
}

if (((this._controlType == "InputUser"||this._controlType == "InputUserName") && BB.Get("action") == "insert"))
{
this.Text = BB.LoginKey("Name") != null ? BB.LoginKey("Name") : String.Empty;
}

if (this._controlType == "InputUserID" && (BB.Get("action") == "insert" || BB.Get("action") == "copy"))
{
this.Text = BB.LoginKey("UserID") != null ? BB.LoginKey("UserID") : String.Empty;
}
if ((this._controlType == "Password")||(this._controlType == "MultiLine"))
{
this.TextMode = TextBoxMode.Password;
}

base.OnPreRender(e);
}

[baysos]

同意switch

[laowang134]

我咋感觉用switch会更复杂。。。

[lilianjie0426]

楼主这个可变量太多了，用什么都很复杂的。。。。如果非要简单的话，还是看看有没有别的思路吧

[刀枪blue]

还是if吧。switch没什么简洁

[ReyZhang]

首先，上面的代码是必定要被重构的。如果一个条件进来，现在的每一个if 语句都会被执行，这不是我们想要看到的。
可以用if ..else if 来判断，或用switch

[BATTLERxANGE]

感觉用switch更难看

[LQ_651119244]

唯有换个思想。。。

[caicoko]

比如换个什么思想~

[lee576]

设计模式之State(状态模式)

http://www.cnblogs.com/springyangwc/archive/2011/04/11/2013073.html