jspでservletのログインユーザー名を使用したいです。<%=Account.getUserId()%>で赤波線が出てしまいます。

どうぞよろしくお願いします。

- - - - loginOK.jsp ここで、赤波線が出てします。

<%@ page language="java" contentType="text/html; charset=UTF-8"
pageEncoding="UTF-8"%>
<%@ page import="model.Account" %>
<%
//セッションスコープからユーザー情報を取得
String userId=(String)session.getAttribute("userId");
%>

<!DOCTYPE html > <html> <head> <meta charset="UTF-8"> <title>すっきり商店</title> </head> <body> <p>ようこそ<%=Account.getUserId() %>さん、無事ログインできました。</p> <a href="newsns/Main">呟き投稿・閲覧へ</a> </body> </html> - - - - - このまでがloginOK.jsp

- - - - Account.java

package model;

public class Account {
//フィールド
private String userId;
private String pass;
private String mail;

//Accountメソッド引数全て、
public Account(String userId,String pass,String mail){
this.userId=userId;
this.pass=pass;
this.mail=mail;
}
//UserId.Passとか
public String getUserId(){return userId;}
public String getPass(){return pass;}
public String getMail(){return mail;}
}

- - - - ここまでがAccount.java
- - - - login.Servlet.java

package servlet;

import java.io.IOException;

import javax.servlet.RequestDispatcher;
import javax.servlet.ServletException;
import javax.servlet.annotation.WebServlet;
import javax.servlet.http.HttpServlet;
import javax.servlet.http.HttpServletRequest;
import javax.servlet.http.HttpServletResponse;
import javax.servlet.http.HttpSession;

import model.Login;
import model.LoginLogic;

@WebServlet("/LoginServlet")
public class LoginServlet extends HttpServlet{

private static final long serialVersionUID = 1L;


protected void doGet(HttpServletRequest request,HttpServletResponse response)throws ServletException,IOException{	
	//フォワード
	RequestDispatcher dispatcher=request.getRequestDispatcher("/WEB-INF/jsp/login.jsp");
	dispatcher.forward(request,response);
}

//taichiのMain.javvaの分のログインしているかのチェックは、ここでおk
protected void doPost(HttpServletRequest request,HttpServletResponse response)throws ServletException,IOException{
	
	//リクエストパラメータの取得
	request.setCharacterEncoding("UTF-8");
	String userId=request.getParameter("userId");
	String pass=request.getParameter("pass");
	
	//ログイン処理の実行
	Login login=new Login(userId,pass);
	LoginLogic bo=new LoginLogic();
	boolean result=bo.execute(login);
	
	//ログイン処理の成否によって処理を分岐
	if(result){//ログイン成功時
		//sessionスコープにuserIdを保存
		HttpSession session=request.getSession();
		session.setAttribute("userId",userId);
	
		//フォワード
		RequestDispatcher dispatcher=request.getRequestDispatcher("/WEB-INF/jsp/loginOK.jsp");
		dispatcher.forward(request, response);
	}else{//ログイン失敗時
		//フォワード
		RequestDispatcher dispatcher=request.getRequestDispatcher("/WEB-INF/jsp/login.jsp");
		dispatcher.forward(request, response);
	}
}

}

- - - ここまで
どうぞよろしくお願いします。

行動規範の内容に同意します

回答1件

ベストアンサー

getUserメソッドは、staticでないので、クラス名＋メソッド名のようにしてで呼び出すことはできません。

それよりも、

lang
1String userId=(String)session.getAttribute("userId");

でユーザーIDを取ってきているので、それを使えば良いのではないでしょうか。

投稿2015/11/28 09:43

argius

総合スコア9388

edoooooo

2015/11/28 11:19

そうなのですね、過去に書いた本のコードを見ながら、　String userId=(String)session.getAttribute("userId");も自分で書いてみたのですが、頭を柔らかく使うことができず、その発想に至りませんでした。　確かにuserIdを<%=userId%>のように使うだけですね、こういう柔らかい発想ができると、優秀なプログラマーさんになれるのでしょうか？ありがとうございました。

argius

2015/11/28 11:48

頭を柔らかくするというのは、ユニークな発想を得るには大切ですが、今回のケースはちょっと違うと思います。これは特別なことをしているわけではないからです。プログラミングでは大抵、こういうことをする場合はこういう風に書く、という一般的な形式があります。必ずしもそうしなければならないわけではありませんが、奇抜なことしても読みにくくなるだけなので、たいていの人は一般的な形式で書くのです。 JSPでは、セッションスコープから値を取得する処理を冒頭に書いて、埋め込みタグではその値だけを書くようにするのが一般的です。それに、ベテランさんであれば、この程度の規模であれば、書いてあることひとつひとつをすべて理解できますし、コードがどのように組み合わさって動くのかを理解しています。無駄なことは書いても仕方ないので、使わない者は消しますし、使えるものは使います。今表示しようとしているIDはどこから取ってこようとしているのか、と考え、冒頭で取ってきてあればそれを使うし、なければ改めてとってくる、それだけのことなのです。

行動規範の内容に同意します