1
私はHTMLとJSPを使用してログインページを構築しています。しかし、私は、ユーザー名がテーブルと一致しないときに表示されるはずのエラー "username incorrect"がSQL Serverであるたびに取得します。ここでは、ログインフォームページのコードは次のとおりです。JSPを使用したユーザー検証
以下<%@ page language="java" contentType="text/html; charset=UTF-8"
pageEncoding="UTF-8"%>
<!DOCTYPE html>
<html>
<head>
<meta http-equiv="Content-Type" content="text/html; charset=UTF-8">
<title>Expense System</title>
<link rel="stylesheet" href="style.css" type="text/css">
</head>
<body>
<div class=form>
<form name = login method = post action = "login1.jsp">
Username : <input name = user type = text placeholder = username> <br><br>
Password : <input name = pass type = password placeholder = password><br><br>
<input type = submit value = "Submit">
<input type = button value = "Register">
</form>
</div>
</body>
</html>
はlogin1.jspのコードです：
<%@ page language="java" contentType="text/html; charse=UTF-8"
pageEncoding="UTF-8" import="java.sql.*"%>
<% Class.forName("com.microsoft.sqlserver.jdbc.SQLServerDriver"); %>
<!DOCTYPE html>
<html>
<head>
<meta http-equiv="Content-Type" content="text/html; charset=UTF-8">
<title>login check</title>
</head>
<body>
<% String connectionUrl = "jdbc:sqlserver://localhost:1433;" +
"databaseName=signin;integratedSecurity=true;";
Connection con = DriverManager.getConnection(connectionUrl);
String uname = new String("");
String upass = new String("");
ResultSet resultset;
Statement statement = con.createStatement();
statement.executeQuery("select username, password from signintable");
resultset = statement.getResultSet();
while(resultset.next()){
uname = resultset.getString("username");
upass = resultset.getString("password");
}
if(!request.getParameter("user").equals("")){
if(uname.equals(request.getParameter("user"))){
if(upass.equals(request.getParameter("pass"))) {%>
<jsp:forward page="welcome.html"></jsp:forward>
<% }
else {
out.println("pass incorrect");
}
}
else {
out.println("username incorrect");
}
}
else { out.println("user not found!");
}
%>
</body>
</html>
誰かがログインするたびに、テーブル全体を持ち帰り、アプリケーションでそれをループするのは良い考えではありません！ 'WHERE'条件でパラメータ化されたクエリを調べて、存在する特定の行を選択し、パスワードをハッシュするだけであれば、それを調べるべきです。 –
Javaで 'string uname = new String（" "）;'や 'String upass = new String（" "）;'のような文を使用するのは避けられます。悪い習慣。 'string uname =" "'と 'String upass =" "'を使うだけです。この場合、それらの文字列オブジェクトをプールしています。 – Lion
@MartinSmith私はWHERE条件を使用しましたが、プログラムを正常に実行できました。しかし、私がやりたいことは、最初にユーザー名を確認し、それが正しい場合はパスワードを確認することです。どのようにWHERE条件を使用してこれを行うことができます。ありがとう。 –