MSDN文档中的一个不那么和谐的示例-白红宇

MSDN文档中的一个不那么和谐的示例

阅读量：5119 次

发布时间：2019-06-13

本文共 6167 字，大约阅读时间需要 20 分钟。

最近写网站用到FileUpload控件，好长时间不用手生了，所以特地又仔细地看了一遍MSDN文档。在看到其FileBytes属性时，看到其中的示例代码，颇感郁闷。

// 后面有给dudu的附言，如果有幸dudu莅临，请阅

感觉其中有不少问题，虽然很细微，但出现在MSDN中，总感却不那么和谐。

下面是完整的示例。

@ Page Language

html

head

script runat

server

private

void

DisplayFileContents(HttpPostedFile file)

{

int fileLen;

string displayString = "";

// Get the length of the file.

fileLen = FileUpload1.PostedFile.ContentLength; /**///// <---- 1.1

// Display the length of the file in a label.

LengthLabel.Text = "The length of the file is "

+ fileLen.ToString() + " bytes.";

// Create a byte array to hold the contents of the file.

byte[] input = new byte[fileLen];

input = FileUpload1.FileBytes; /**///// <---- 2, 1.2

// Copy the byte array to a string.

for (int loop1 = 0; loop1 <= fileLen - 1; loop1++)

{

displayString = displayString + input[loop1].ToString(); /**///// <---- 3

}

// Display the contents of the file in a

// textbox on the page.

ContentsLabel.Text = "The contents of the file as bytes:";

TextBox ContentsTextBox = new TextBox(); /**///// <---- 4

ContentsTextBox.TextMode = TextBoxMode.MultiLine;

ContentsTextBox.Height = Unit.Pixel(300);

ContentsTextBox.Width = Unit.Pixel(400);

ContentsTextBox.Text = displayString;

// Add the textbox to the Controls collection

// of the Placeholder control.

PlaceHolder1.Controls.Add(ContentsTextBox);

}

protected void UploadButton_Click(object sender, EventArgs e)

{

// Specify the path on the server to

// save the uploaded file to.

string savePath = @"c:\temp\uploads\";

// Before attempting to perform operations

// on the the file, verify that the FileUpload

// control contains a file.

if (FileUpload1.HasFile)

{

// Append the name of the file to upload to the path.

savePath += FileUpload1.FileName;

// Call the SaveAs method to save the

// uploaded file to the specified path.

// This example does not perform all

// the necessary error checking.

// If a file with the same name

// already exists in the specified path,

// the uploaded file overwrites it.

FileUpload1.SaveAs(savePath);

// Notify the user that the file was uploaded successfully.

UploadStatusLabel.Text = "Your file was uploaded successfully.";

// Call a helper routine to display the contents

// of the file to upload.

DisplayFileContents(FileUpload1.PostedFile); /**///// <---- 1.3

}

else

{

// Notify the user that a file was not uploaded.

UploadStatusLabel.Text = "You did not specify a file to upload.";

}

</script>

</head>

<body>

<h3>FileUpload.FileContent Property Example</h3>

<h4>Select a file to upload:</h4>

<asp:FileUpload id="FileUpload1"

runat="server">

</asp:FileUpload>

<asp:Button id="UploadButton"

Text="Upload file"

OnClick="UploadButton_Click"

100

runat="server">

101

</asp:Button>

102

103

104

105

<asp:Label id="UploadStatusLabel"

106

runat="server">

107

</asp:Label>

108

109

<hr />

110

111

<asp:Label id="LengthLabel"

112

runat="server">

113

</asp:Label>

114

115

116

117

<asp:Label id="ContentsLabel"

118

runat="server">

119

</asp:Label>

120

121

122

123

<asp:PlaceHolder id="PlaceHolder1"

124

runat="server">

125

</asp:PlaceHolder>

126

127

</form>

128

129

</body>

130

</html>

问题1

注释中的1.1、1.2、1.3

private

void

DisplayFileContents(HttpPostedFile file)

{

int fileLen;

string displayString = "";

// Get the length of the file.

fileLen = FileUpload1.PostedFile.ContentLength; /**///// <---- 1.1

input

FileUpload1.FileBytes;

/**/

//// <---- 2, 1.2

protected

void

UploadButton_Click(

object

sender, EventArgs e)

{

// Call a helper routine to display the contents

// of the file to upload.

DisplayFileContents(FileUpload1.PostedFile); /**///// <---- 1.3

在这里，DisplayFileContents方法接受一个HttpPostedFile类型的参数file，表示从FileUpload控件上传的文件。的确，在1.3处将FileUpload1.PostedFile属性传了进来。

先往复杂了说，从方法的名字和参数可以猜想，作者的目的是降低显示文件内容的操作和FileUpload控件的耦合度。比如，如果页面上有多个FileUpload控件时，就可以重用该方法；或者如果不使用FileUpload控件了，而是用了自己编写的上传控件，只要能提供HttpPostedFile类型的对象，也可以使用该方法。但1.1和1.2两个地方颠覆了这个初衷，这个方法还是紧紧地与FileUpload1控件紧紧地关联在了一起。如果添加了另外一个FileUpload控件，或者原控件改名了，这个方法必须同时修改。

再往简单了说，整个方法体都没有用到过这个file参数，又何必添加呢？

问题2

注释中的2

Create a byte array to hold the contents of the file.

byte

[] input

new

byte

[fileLen];

input

FileUpload1.FileBytes;

/**/

//// <---- 2, 1.2

这是一个比较严重的内存泄露问题。首先，之前的一行byte[] input = new byte[fileLen];在堆上分配了与文件大小相同的字节数组（注意数组都是引用类型），然后input = FileUpload1.FileBytes;直接修改了input变量所引用的对象，而原来new byte[fileLen]得到的数组成了孤立对象，遗留在堆中等待被回收。

试想如果这个代码出现在产品中，如果允许用户数MB甚至更大的文件，如果同时（或很短一段时间内）有数百甚至更多的人上传文件……

改进方法：1 直接使用FileUpload1.FileBytes属性，或者直接作这样的赋值：byte[] input = FileUpload1.FileBytes，然后使用input变量；2 如果有必要用一个数组的话，写一个循环将FileUpload1.FileBytes的内容复制到input中。

问题3

注释中的3

Copy the byte array to a string.

for

(

int

loop1

; loop1

fileLen

; loop1

)

{

displayString = displayString + input[loop1].ToString(); /**///// <---- 3

}

这也是一个比较严重的问题，但我就不用多说了，几乎提到了字符串操作的每篇技术文章或每本书都提到，慎用字符串的相加（连接）操作，尤其是在循环内部。

问题4

注释中的4

TextBox ContentsTextBox

new

TextBox();

/**/

//// <---- 4

ContentsTextBox.TextMode = TextBoxMode.MultiLine;

ContentsTextBox.Height

Unit.Pixel(

300

);

ContentsTextBox.Width

Unit.Pixel(

400

);

ContentsTextBox.Text

displayString;

Add the textbox to the Controls collection

of the Placeholder control.

PlaceHolder1.Controls.Add(ContentsTextBox);

这是一个小问题，主要在于用户体验。每上传一次就要创建一个文本框显示文件内容，并且文本框是在服务器端添加的，刷新页面也无法消除。这样的话，如果用户查看了几个文件的内容后，要想清空页面，只有关闭浏览器重新打开，这样的用户体验是非常不好的。实际上，硬编码一个文本框，每次修改其Text属性即可。

----------- 分割线华丽地降临 -------------------

实际上，对于示例代码来说，问题1和4无伤大雅，毕竟受众所关心的是如何通过FileBytes得到文件中的字节。但问题2和3些许有些“误人子弟”（如果有人直接像这样读取文件字节，那就得浪费fileLen个字节的内存，如果有n个人用这个功能，那就得浪费n*fileLen个字节，你付得起责任吗？——小学老师如是教育）。而如果是在产品中，这4个问题恐怕都会产生不良后果。

----------- 分割线再次华丽地降临 ---------------

P.S.

to dudu：

园子的回复下面有一个“刷新评论列表”，实在是个好东西。

但有个问题，就是如果一篇文章一条回复也没有的话，这个链接是不会出现的。

可如果我在浏览一个尚没有回复的文章时，希望刷一下看看有没有人在我浏览的时候发表了回复，那就只有刷新页面了。

所以，能否改进一下，让这个链接任何时候都能出现？

小问题，呵呵~

转载于:https://www.cnblogs.com/AndersLiu/archive/2007/05/20/752889.html

你可能感兴趣的文章

iOS--控制器加载自定义view的xib

查看>>