c#吧 关注：188,324贴子：822,415
- 19回复贴，共1页
DBHelper呀！！！勾起了我满满的回忆
。既然你的问题解决了，那我就说下你的代码不足之处和注意的地方 1.你的DBHelper类，超过两行重复的代码，可以只写一句放在外面 private static SqlConnection coon = null;；2.你的sql语句在公司代码检查中肯定是不合格的,“ * ” 取出的是全部的信息，登录无非就是名字和密码，其它根据业务再取，不应该全部取出，为什么呢？你目前可能是名学生，数据库不大，表不多，列不多，表关系少，数据量更少，取一个表或多个表的数据感觉不到什么，可是，在一些实际软件开发过程中，一张用户表可能有几十列上百列，上百万用户量，那么取出全部信息就影响效率（慢）；3.还是你的sql语句，这样写出来不怕恶意注入吗！你这种拼接语句，一眼就看出来可以进行注入的，怎么写？方法一：调用存储过程，方法二：string.Fmt(忘了，是Format？还是什么的，有这个，用这个去拼接sql语句)，还有就是建议sql语句关键字大写。最后再说一句，在学校学的DBHelper没用，没用，没用(唯一一点就是，有个基础)，在外面C#操作数据库有很多方法，唯独这个就了解了解吧，出来工作的话，就把它忘了，建议你从现在开始了解并掌握EF框架的三种模式。额，说的有点多了，都是回忆惹的祸呀
。既然你的问题解决了，那我就说下你的代码不足之处和注意的地方 1.你的DBHelper类，超过两行重复的代码，可以只写一句放在外面 private static SqlConnection coon = null;；2.你的sql语句在公司代码检查中肯定是不合格的,“ * ” 取出的是全部的信息，登录无非就是名字和密码，其它根据业务再取，不应该全部取出，为什么呢？你目前可能是名学生，数据库不大，表不多，列不多，表关系少，数据量更少，取一个表或多个表的数据感觉不到什么，可是，在一些实际软件开发过程中，一张用户表可能有几十列上百列，上百万用户量，那么取出全部信息就影响效率（慢）；3.还是你的sql语句，这样写出来不怕恶意注入吗！你这种拼接语句，一眼就看出来可以进行注入的，怎么写？方法一：调用存储过程，方法二：string.Fmt(忘了，是Format？还是什么的，有这个，用这个去拼接sql语句)，还有就是建议sql语句关键字大写。最后再说一句，在学校学的DBHelper没用，没用，没用(唯一一点就是，有个基础)，在外面C#操作数据库有很多方法，唯独这个就了解了解吧，出来工作的话，就把它忘了，建议你从现在开始了解并掌握EF框架的三种模式。额，说的有点多了，都是回忆惹的祸呀
Alan
