質問をすることでしか得られない、回答やアドバイスがある。

15分調べてもわからないことは、質問しよう!

ただいまの
回答率

90.32%

  • Java

    14437questions

    Javaは、1995年にサン・マイクロシステムズが開発したプログラミング言語です。表記法はC言語に似ていますが、既存のプログラミング言語の短所を踏まえていちから設計されており、最初からオブジェクト指向性を備えてデザインされています。セキュリティ面が強力であることや、ネットワーク環境での利用に向いていることが特徴です。Javaで作られたソフトウェアは基本的にいかなるプラットフォームでも作動します。

  • JSP

    960questions

    JSP(Java Server Pages)とは、ウェブアプリケーションの表示レイヤーに使われるサーバーサイドの技術のことです。

  • servlet

    486questions

    Servletとは、Webページの動的な生成やデータ処理などをサーバ上で実行するために、Javaで作成されたプログラムです。 ショッピングサイトやオンラインバンキングといった、動的なウェブサイトの構築に用いられています。

別ユーザーの同時アクセス時に対応について

解決済

回答 1

投稿 編集

  • 評価
  • クリップ 0
  • VIEW 1,592

n000n00

score 15

◆追記 1/24
UpdateUserCheck を作成しました。

ページを離れた事の判定ですが、
現在は、ListServletに戻った時点で、mapからremoveする処理を追加しました。

恐らく、これで対応できると思いますが、
一点、編集中の画面をブラウザの閉じるボタンを実施したときの対応について悩んでおります。

引き続き、知恵を貸していただけないでしょうか。
よろしくお願いします。

◆追記 1/23
Mapを利用した、ソースを書いてみました。UpdateUserCheck_N
(まだ、動かしてません)

引き続き、良い方法がないか知恵を貸していただきたいです。

◆追記 
UpdateUserCheckを編集しました。
最初にページに入ったログインユーザIDとページネームをtmpに格納して、比較しようと考えています。

途中の段階ですが、引き続き知恵を貸していただけないでしょうか。


現在、ServletとJSPで掲示板を作成しています。

◆実現したいこと
ログイン後に既存ページを更新中に、別のユーザーから更新ページにアクセスがあった場合に、更新ページに入らせない。
ユーザーIDはsessionで保持。
一覧ページから編集したい項目の名前を押して、編集を押すと、パラメータをUsercheckServletに渡し、同一ユーザーかチェックし、同一ユーザーであれば、更新ページ(update.jsp)に飛ぶという流れにしようと思っています。

同時アクセス時の編集したい項目の名前(name)とユーザーID(id)を比較するロジックに関して、知恵を拝借したいです.

よろしくお願いします。

※UsercheckServletは編集中のものです。

<h1>${wikiPage.name}</h1>
${wikiPage.formatedContent}
<hr>
<c:if test="${login == 'OK' && wikiPage.name != 'welcome'}">
    <a href="/wiki/UpdateUserCheck?id=${account.id}&name=${wikiPage.name}&content=${wikiPage.content}">このページを更新</a>
</c:if>
<h1>${param.name}の編集</h1>
<form action="/wiki/update">
    <input type="hidden" name="cmd" value="update">
    <input type="hidden" name="name" value="${param.name}">
    <textarea rows="15" cols="60" name="content">${param.content}</textarea>
    ※200文字以内
    <br>
    <input type="submit"  value="更新ˆ">
    <input type="submit"  value="削除" onclick="cmd.value='delete'">
    <input type="button"  value="キャンセル" onclick="location.href='list?name=${param.name}&id=${param.id}'">
</form>
package wiki;

import java.io.IOException;
import java.util.List;
import java.util.Map;

import javax.servlet.ServletContext;
import javax.servlet.ServletException;
import javax.servlet.http.HttpServlet;
import javax.servlet.http.HttpServletRequest;
import javax.servlet.http.HttpServletResponse;

public class ListServlet extends HttpServlet {

    @Override
    public void doGet(HttpServletRequest request, HttpServletResponse response)
            throws ServletException, IOException {

        ServletContext sc = getServletContext(); // アプリケーションスコープの取得
        String name = request.getParameter("name");

        /* リストページアクセス時に編集ページにアクセスの履歴があれば、処理を実施 */
        if (sc.getAttribute("tmpMap") != null) {
            int id = 0;
            if (request.getParameter("id") == null || request.getParameter("id").isEmpty()) {
                id = -1;
            } else {
                id = Integer.parseInt(request.getParameter("id"));
            }

            Map<String, Integer> map = autoCast(sc.getAttribute("tmpMap"));
            if (map.containsKey(name)) {
                if (map.get(name) == id) {
                    map.remove(name);
                    sc.setAttribute("tmpMap", map);
                }
            }
        }

        try {
            List<WikiBean> list = WikiPageDAO.getInstance().findAll();
            request.setAttribute("list", list);
        } catch (Exception e) {
            throw new ServletException(e);
        }

        request.getRequestDispatcher("/wikiView/list.jsp").forward(request, response);
    }

    /* 戻り値の型に合わせてキャスト */
    @SuppressWarnings("unchecked")
    public static <T> T autoCast(Object obj) {
        T castedObject = (T) obj;
        return castedObject;
    }
}
package wiki;

import java.io.IOException;
import java.util.HashMap;

import javax.servlet.ServletContext;
import javax.servlet.ServletException;
import javax.servlet.http.HttpServlet;
import javax.servlet.http.HttpServletRequest;
import javax.servlet.http.HttpServletResponse;
import javax.servlet.http.HttpSession;

import Account.AccountBean;

public class UpdateUserCheck extends HttpServlet {

    @Override
    public synchronized void doGet(HttpServletRequest request, HttpServletResponse response)
            throws ServletException, IOException {

        HttpSession session = request.getSession(false);
        ServletContext sc = getServletContext(); // アプリケーションスコープの取得

        String name = request.getParameter("name"); // 編集中のページ名を取得
        AccountBean account = (AccountBean) session.getAttribute("account");

        /* アプリケーションスコープがnullであれば、新しくMapを取得する */
        HashMap<String, Integer> map = null;
        if (sc.getAttribute("tmpMap") == null) {
            map = new HashMap<String, Integer>();
        } else {
            map = autoCast(sc.getAttribute("tmpMap")); // メソッドでキャスト
        }

        String message = null;
        if (map.containsKey(name)) {
            if (map.get(name) != account.getId()) {
                message = "他のユーザーにて編集中です";
                request.setAttribute("message", message);
                request.getRequestDispatcher("list").forward(request, response);
            } else {
                map.put(name, account.getId());
                sc.setAttribute("tmpMap", map);
                request.getRequestDispatcher("/wikiView/update.jsp").forward(request, response);
            }
        } else {
            map.put(name, account.getId());
            sc.setAttribute("tmpMap", map);
            request.getRequestDispatcher("/wikiView/update.jsp").forward(request, response);
        }
    }

    /* 戻り値の型に合わせてキャスト */
    @SuppressWarnings("unchecked")
    public static <T> T autoCast(Object obj) {
        T castedObject = (T) obj;
        return castedObject;
    }
}
  • 気になる質問をクリップする

    クリップした質問は、後からいつでもマイページで確認できます。

    またクリップした質問に回答があった際、通知やメールを受け取ることができます。

    クリップを取り消します

  • 良い質問の評価を上げる

    以下のような質問は評価を上げましょう

    • 質問内容が明確
    • 自分も答えを知りたい
    • 質問者以外のユーザにも役立つ

    評価が高い質問は、TOPページの「注目」タブのフィードに表示されやすくなります。

    質問の評価を上げたことを取り消します

  • 評価を下げられる数の上限に達しました

    評価を下げることができません

    • 1日5回まで評価を下げられます
    • 1日に1ユーザに対して2回まで評価を下げられます

    質問の評価を下げる

    teratailでは下記のような質問を「具体的に困っていることがない質問」、「サイトポリシーに違反する質問」と定義し、推奨していません。

    • プログラミングに関係のない質問
    • やってほしいことだけを記載した丸投げの質問
    • 問題・課題が含まれていない質問
    • 意図的に内容が抹消された質問
    • 広告と受け取られるような投稿

    評価が下がると、TOPページの「アクティブ」「注目」タブのフィードに表示されにくくなります。

    質問の評価を下げたことを取り消します

    この機能は開放されていません

    評価を下げる条件を満たしてません

    評価を下げる理由を選択してください

    詳細な説明はこちら

    上記に当てはまらず、質問内容が明確になっていない質問には「情報の追加・修正依頼」機能からコメントをしてください。

    質問の評価を下げる機能の利用条件

    この機能を利用するためには、以下の事項を行う必要があります。

回答 1

checkベストアンサー

+1

現在編集中のユーザの情報(tmpId, tmpName)はリクエスト、セッションを越えて共有できる必要がありますので、アプリケーションスコープの変数として保持する必要があります。

Servlet スコープとは?のapplicationスコープのサンプルを参照してください。 ServletContext に編集中のユーザの ID と名前を setAttribute し、あとで getAttribute することで現在編集中のユーザの情報を管理することができると思います。

ただし、掲示板が複数ある場合は、掲示板毎に管理する必要があるので、ハッシュテーブルなどを使う必要があると思います。

また、複数のユーザがまったく同時に編集しようとすると、今、編集中のユーザがいないことを確認する if 文とユーザ情報を設定する setAttribute 呼び出しを同時に実行してしまう可能性があるため、 synchronized なメソッドで実装して排他制御する必要があると思います。


以下に排他制御について書いてみますが、手元に servlet をデバッグする環境がないので、すべて未検証(コンパイルエラーがでるかもしれない)であることをご了承ください。

いろいろ、ググッて調べたのですが、「もし、未登録であれば登録する」というロジックで満足に排他制御できているサンプルが見つかりませんでした。ServletContext は複数のスレッド(ユーザからのリクエスト)から同時にアクセスされるので、「もし、未登録であれば」という if 文を2つのスレッドが同時に実行すると、両方のスレッドが「登録する」処理を実行してしまうので、バグってしまいます。また、この手のバグはなかなか顕在化しないので、たちが悪いです。

まず、ServletContext にマップを登録する処理ですが、初期化イベントのリスナーで実行しておくべきです。アプリケーションが動作する前に実行されるので、排他制御の必要がありません。

import javax.servlet.ServletContextEvent;
import javax.servlet.ServletContextListener;
import java.util.concurrent.ConcurrentHashMap;
@javax.servlet.annotation.WebListener
public class WebListener implements ServletContextListener {
    @Override
    public void contextDestroyed(ServletContextEvent paramServletContextEvent) {}
    @Override
    public void contextInitialized(ServletContextEvent paramServletContextEvent) {
        paramServletContextEvent.getServletContext().setAttribute("tmpMap",
          new ConcurrentHashMap<String, Integer>());
    }
}

こうしておいて、サーブレット側では ConcurrentHashMap の putIfAbsent を使って排他制御します。

package wiki;

import java.io.IOException;
import java.util.concurrent.ConcurrentHashMap;

import javax.servlet.ServletContext;
import javax.servlet.ServletException;
import javax.servlet.http.HttpServlet;
import javax.servlet.http.HttpServletRequest;
import javax.servlet.http.HttpServletResponse;
import javax.servlet.http.HttpSession;

import Account.AccountBean;

public class UpdateUserCheck extends HttpServlet {

    @Override
    public synchronized void doGet(HttpServletRequest request, HttpServletResponse response)
            throws ServletException, IOException {

        HttpSession session = request.getSession(false);
        ServletContext sc = getServletContext(); // アプリケーションスコープの取得

        String name = request.getParameter("name"); // 編集中のページ名を取得
        AccountBean account = (AccountBean) session.getAttribute("account");

        /* 初期化イベントリスナーで登録したMapを取得する */
        ConcurrentHashMap<String, Integer> map = autoCast(sc.getAttribute("tmpMap")); // メソッドでキャスト

        String message = null;
        final Integer currentUid = autoCast(map.putIfAbsent(name, account.getId()));
        if (currentUid != null) {
            if (! currentUid.equals(account.getId())) {
                message = "他のユーザーにて編集中です";
                request.setAttribute("message", message);
                request.getRequestDispatcher("list").forward(request, response);
            } else {
                request.getRequestDispatcher("/wikiView/update.jsp").forward(request, response);
            }
        } else {
            request.getRequestDispatcher("/wikiView/update.jsp").forward(request, response);
        }
    }

    /* 戻り値の型に合わせてキャスト */
    @SuppressWarnings("unchecked")
    public static <T> T autoCast(Object obj) {
        T castedObject = (T) obj;
        return castedObject;
    }
}

また、ユーザが編集を終了するときに map から削除するようにしておかないと、上記コードは動作しません。さらに編集状態のまま、ログアウト時や放置されてセッションがタイムアウトした時も編集状態を開放する処理が必要であると思われます。

投稿

編集

  • 回答の評価を上げる

    以下のような回答は評価を上げましょう

    • 正しい回答
    • わかりやすい回答
    • ためになる回答

    評価が高い回答ほどページの上位に表示されます。

  • 回答の評価を下げる

    下記のような回答は推奨されていません。

    • 間違っている回答
    • 質問の回答になっていない投稿
    • スパムや攻撃的な表現を用いた投稿

    評価を下げる際はその理由を明確に伝え、適切な回答に修正してもらいましょう。

  • 2017/01/22 20:53

    回答ありがとうございます。

    ご指導頂いたように、applicationスコープを利用して、ifでユーザーを特定するソースは書くことができました。ありがとうございます。

    ご指摘頂いたように、掲示板にはページが複数あります。

    教えて頂いたようにHashMapを利用して、複数ページに対応しようと考えていますが、あまり利用したことがないため、考えがまとまらない状況です。

    何か、このような流れで等の例を、よろしければご教授いただけないでしょうか。

    キャンセル

  • 2017/01/27 14:09

    回答ありがとうございます。

    リスナーについては、私が読んでるServletのテキストには載っていなくて、
    ググって勉強をしている段階です。

    実装についてまだ理解ができていないですが、参考にさせていただきます。

    ご丁寧に対応していただき、ありがとうございます。

    キャンセル

同じタグがついた質問を見る

  • Java

    14437questions

    Javaは、1995年にサン・マイクロシステムズが開発したプログラミング言語です。表記法はC言語に似ていますが、既存のプログラミング言語の短所を踏まえていちから設計されており、最初からオブジェクト指向性を備えてデザインされています。セキュリティ面が強力であることや、ネットワーク環境での利用に向いていることが特徴です。Javaで作られたソフトウェアは基本的にいかなるプラットフォームでも作動します。

  • JSP

    960questions

    JSP(Java Server Pages)とは、ウェブアプリケーションの表示レイヤーに使われるサーバーサイドの技術のことです。

  • servlet

    486questions

    Servletとは、Webページの動的な生成やデータ処理などをサーバ上で実行するために、Javaで作成されたプログラムです。 ショッピングサイトやオンラインバンキングといった、動的なウェブサイトの構築に用いられています。