我正试图为员工和经理创建一个单一的登录系统——我的数据库设置正确,java代码正确连接到数据库。我遇到的问题是,当用户以员工身份登录时,根据数据库中的用户名,将出现一个员工窗口,或者如果经理试图根据数据库中他们的用户名再次登录,则会出现一个经理窗口
例如,对于员工,数据库中的用户名看起来像staff1、staff1等,而对于经理,它类似于mng1、mngr2等。
我有点让代码工作,但出于某种原因,它只加载管理器视图。
final Statement s = conn.createStatement ();
ResultSet rs= s.executeQuery ("SELECT * FROM users where username='"+username1+"' && password='"+psw+"'");
String mng1="mng*";
String staff="staff*";
String pass1="";
while (rs.next ())
{
mng1 = rs.getString ("");
staff =rs.getString ("");
pass1 = rs.getString ("password");
if(username1.equals(mng1)&&psw.equals(pass1))
{
ManagerMainMenu mng= new ManagerMainMenu();
mng.main(null);
//System.out.println("its working");
}
else if(username1.equals(staff)&&psw.equals(pass1))
{
TakingOrder to = new TakingOrder();
to.main(null);
}
else
{
System.out.println("password or username is wrong");
}
}
事实上,这里是您的问题的答案
if(username1.equals(mng1)&&psw.equals(pass1)){
mng= new ManagerMainMenu()
在你的情况下,这种情况似乎总是正确的您的用户名1为null或为空或其他原因密码也许也是正确的。您没有在用户名不可见的情况下执行任何操作在代码中。rs.getString(")很可能会返回null
。所以你只是在取消它们。不要像select * from whatever
那样使用选择这是件坏事尤其是在大桌子上,而你选择的东西却不太显眼。
但你应该跟随建议和使用技术和/或一些好的ORM类似hibernate或使用好的旧springjdbc。:)
首先,我强烈建议使用事先准备好的发言http://docs.oracle.com/javase/tutorial/jdbc/basics/prepared.html它还可以帮助您避免sql注入,也应该利用数据库的资源以一种更好的方式,因为sql已经预编译到数据库中。
下面是一些实例。
static public void setParameters(PreparedStatement preparedStatement, Object[] params)
throws SQLException
{
if (params != null)
{
for (int i = 0; i < params.length; i++)
{
if (params[i] == null)
{
preparedStatement.setNull(i + 1, Types.INTEGER);
continue;
}
String className = params[i].getClass().getName();
if (className.compareTo("java.lang.String") == 0)
{
preparedStatement.setString(i + 1, (String) params[i]);
}
else if (className.compareTo("java.lang.Integer") == 0)
{
preparedStatement.setInt(i + 1, ((Integer) params[i]).intValue());
}
...............///your types go here
}
}
}
public static ResultSet sqlSelect(Connection con, String sql, Object[] params) throws SQLException
{
ResultSet rs = null;
PreparedStatement preparedStatement = con.prepareStatement(sql);
try
{
setParameters(preparedStatement, params);///your parameters
rs = preparedStatement.executeQuery();
logger.info(rs.toString());
}
catch (SQLException e)
{
//TODO log.error("select failed: " + preparedStatement.toString());
throw e;
}
return rs;
}
public static UserInfo getUser(String username,String pass)
{
UserInfo user=null;
Connection con=null;
//// logger.info("user: "+username+" pass: "+pass); if you want to
String query = "SELECT id,username,password,email,name,company,description,enabled" +
" FROM users WHERE username=? and password=? and enabled=true;";
try{
con = DBManager.getConnection();
ResultSet rs = DBManager.sqlSelect(con,query,new String[]{username,pass});
while(rs != null && rs.next())
{
user = new UserInfo();
user.setId(rs.getInt(1));
user.setUsername(rs.getString(2));
user.setPassword(rs.getString(3));
user.setEmail(rs.getString(4));
user.setFullName(rs.getString(5));
user.setCompany(rs.getString(6));
user.setDescription(rs.getString(7));
user.setEnabled(rs.getBoolean(8));
logger.info("user written");
}
}catch(SQLException sqle){logger.info(sqle.getMessage());}
finally
{
if(con != null)
DBManager.freeConnection(con);
}
return user; }
如果该方法将您的设置返回给您的用户,则该用户是有效的。您可能想要拥有用户和权限或组表,然后您如果您愿意,应该创建一个多对多表,如users_permissions或users_groups哪个将具有(id、user_id、permission_id)或group_id,然后您必须加入这两个表的用户,允许使用这个多对多表像这个
select u.user_id,p.permission from
users u,user_permissions up,permissions p
where u.user_id=up.user_id and up.permission_id=p.permission_id
and u.user=? and u.password=? and u.enabled=1
或使用联接:
select u.user_id,p.permission from
users u
inner join user_permissions up
on u.user_id=up.user_id
inner join permissions p
on up.permission_id=p.permission_id
where u.user=? and u.password=? and u.enabled=1
然后调整select和getUser()方法。
UserInfo pojo类似于:
public class UserInfo {
private int id;
private String username;
private String password;
private String email;
private String fullName;
private String company;
private String description;
private List<Integer or String> permissions;
//getters//setters
}
希望对abit有所帮助。