Question

jsp 登录页面不管输入什么,点击登录后,都是跳转到成功页面，大虾们，帮忙看看~~谢谢！

jsp 登录页面不管输入什么,点击登录后,都是跳转到成功页面。数据库是mysql，下面是LoginAction的代码 ，可以提供别的代码，麻烦大家帮帮忙
package dao;import java.sql.*;import java.util.*;import java.sql.PreparedStatement;import java.sql.ResultSet;import java.sql.SQLException;import java.util.ArrayList;import java.util.List;import util.ConnOfSQLServer;
public class UserDao{ private Connection conn; private PreparedStatement pt; private ResultSet rs; public UserDao(){ ConnOfSQLServer sqlconn; sqlconn=new ConnOfSQLServer(); conn=sqlconn.getConn(); } public boolean Login(User userID) { boolean flag = false; try { PreparedStatement pt = conn.prepareStatement("select *from user where UserID=?and Password=?"); pt.setString(1,User.getUserID()); pt.setString(2,User.getPassword()); rs=pt.executeQuery(); if(rs.next()) { flag = true; } } catch(Exception e) { e.printStackTrace(); } return flag; } }

Answer 1

  public boolean Login(User userID)    {
   boolean flag = false;             
   try   {        
    PreparedStatement pt = conn.prepareStatement("select *from user where UserID=?and Password=?"); 
    pt.setString(1,User.getUserID());          
    pt.setString(2,User.getPassword());          
    rs=pt.executeQuery();    
    if(rs.next())    {      
     flag = true;     
     }            
    }   catch(Exception e)   {
     e.printStackTrace();     
    }  
    return flag;    } }

仔细看看你写的代码.

传进来的javabean名字叫User,实例对像叫userID.

PreparedStatement提交参数时却是用的名叫User的JAVABEAN对像实例.

把你JAVABEAN贴出来看看,如果你在JAVABEAN中给这两个参数设了初始值.连接数据库时却这么写,数据库中又存在这个默认值的话,不管你怎么查,肯定结果是TRUE.如果JAVABEAN中没有初始值.这么提交下去,会直接报空指针异常.返回结果也肯定会假,并且登陆不会成功.

刚刚没仔细看就给你回答了.

 

另外把JSP或者SERVLET或者STRUTS的提交ACTION贴出来看看.

不过最后这里最好改下.不推荐省略写法.这么写很容易自己绕晕了,出现意外造成flag没能被重新赋值.同时最好把数据连接给关掉.

最好写成这样.要么就用三元运算符.

if (rs.next()){
    flag = true;
} else{
   falg = false;
}
return flag;

flag = rs.next()? true:false;
return flag;

 

 

给你贴个我写的完整的方法:

 public boolean Login(AdminBean ab) throws Exception {
  String sql = "select * from login where name = ? and pwd = ?";
  PreparedStatement pstm = this.Conn().prepareStatement(sql);
  pstm.setString(1, ab.getName());
  pstm.setString(2, ab.getPwd());
  ResultSet rs = pstm.executeQuery();
  if (rs.next()) {
   ab.setLv(rs.getString("lv"));
   return true;
  }
  rs.close();
  pstm.close();
  this.Conn().close();
  return false;
 }

追问

我改成了这样：
if (rs.next()){ flag = true; } else{ flag = false; } return flag;
还是不行，按你的我不会改……追问，不能贴太多东西，可以加下扣扣么？我的417456435

追答

把你所有代码全贴出来.

追问

LoginAction

UserDao:

strurts:

Answer 2

检查下你的select语句
我一般先查询用户名是否存在，再校对密码，可以给客户提出那输错了。感觉这样用户体验好一些。
jsp中可以加一些控制台输出，好判断错误点。。

追问

查询语句？
这个吗？PreparedStatement pt = conn.prepareStatement("select *from user where UserID=?and Password=?"); 我重新检查了下，没什么错误，你说的方法，我不会……可以发下代码，或者说下思路吗。因为我是刚学这个，谢谢