标题,关键词,描述,调用对不对,

pengxiaoshuang0 2012-09-25 05:55:49
这都是从数据库里面调用出来的,
这样对搜索也没有什么影响


if (!IsPostBack) {

DataBase d = new DataBase();
SqlDataReader dr;
d.runSQL("select * from toux", out dr);
dr.Read();
Page.Title = dr["touTitle"].ToString();

l1.Text = "<meta name='description' content='" + dr["touMiaoshu"].ToString() + "' /><br /><meta name='keywords' content='" + dr["touGjianzi"].ToString() + "'/>";
Page.Header.Controls.Add(l1);


}
...全文
59 4 打赏 收藏 转发到动态 举报
写回复
用AI写文章
4 条回复
切换为时间正序
请发表友善的回复…
发表回复
baishaoge 2012-09-26
  • 打赏
  • 举报
回复
UI 不要出现SQL语句 把常用的操作封装成一个类吧。
我个人都喜欢把通用的代码功能写到一个类库里面,方便调用
  • 打赏
  • 举报
回复
既然每个画面的标题都一样干嘛还要存在数据库里,直接在模板写死了就是。
<meta 这种标签用label显示?你这是要做什么?
如果要添加meta标签下面这样做。但还是建议直接写在画面上

HtmlMeta meta = new HtmlMeta();
meta.Name = "www";
meta.Content = "wqeq";
this.Header.Controls.Add(meta);

没有必要动态生成的就不要去用动态
xiangaylian 2012-09-26
  • 打赏
  • 举报
回复
我其实没太明白楼主的意思,这样调用,我观察到,有几点值得考虑的
1.dr.Read(),一般应该要加逻辑判断,就是看是否取到结果,避免因取不到结果引起程序异常;
2.dr["touTitle"].ToString(),直接ToString不可取,要先判断dr[]是否为DBNull值,才能作ToString转换,否则也有可能引起程序异常;
3.select * from toux的查询方式不太好,当不需要所有的结果时,尽量不去取出无关的结果
4.你的数据库连接要在Read过后才能关闭吧,这点我不太确定对不对,我觉得,需要确认你的数据库连接有没有在用完后关闭;
----------------
可以看出,写这些代码的人,在编码规范方面,还要加强一点了。
pengxiaoshuang0 2012-09-25
  • 打赏
  • 举报
回复
这个调用放在模版里面,还是放在index页面

62,133

社区成员

发帖
与我相关
我的任务
社区描述
.NET技术交流专区
javascript云原生 企业社区
社区管理员
  • ASP.NET
  • .Net开发者社区
  • R小R
加入社区
  • 近7日
  • 近30日
  • 至今
社区公告

.NET 社区是一个围绕开源 .NET 的开放、热情、创新、包容的技术社区。社区致力于为广大 .NET 爱好者提供一个良好的知识共享、协同互助的 .NET 技术交流环境。我们尊重不同意见,支持健康理性的辩论和互动,反对歧视和攻击。

希望和大家一起共同营造一个活跃、友好的社区氛围。

试试用AI创作助手写篇文章吧