snsの新規登録で、おなじidとpasswordでは、アカウントを作れないようにしたいです。

Question

snsの新規登録画面で、userId、password、mailadressを登録して、
そのservletクラス内で、にすでに同姓同名で、password、mailadressが同じ人がいないかを
このコードの中に組み込み判断したいのですが、どのように書けばいいのでしょうか？
自分の書いた、新規登録のコードのnewRegistration.javaという、jspで入力したuserId、pass、mailadressを取得したリクエストパラメータに取得させ、servletクラスは、どのように書けばいいのでしょうか？
足りない情報がありましたら、指摘のほどどうぞよろしくお願いします。

----------(newRegistraction.java)----------

protected void doPost(HttpServletRequest request, HttpServletResponse response) throws ServletException, IOException {

  		//リクエストパラメータの取得
  		request.setCharacterEncoding("UTF-8");
  		String name=request.getParameter("userId");
  		String pass=request.getParameter("pass");
  		
  		//Userインスタンス(ユーザー情報)の生成
  		User user=new User(name,pass);
  	
  		//Accountにすでに同じuserがあった場合は・・・
  	//↓この行の（ ）のところに何を入れれば、リクエストパラメータから先ほどnewで作った、userがuserIdとpassが他の人とかぶらなくなるのかがわかりません			
boolean isLogin=（ ）.execute(user);

  				
  				//もしtrueが返された場合は、すでに登録済みだから、
  				if(isLogin){
  	   //can not set(登録済みのため登録できません) cannot.jspにフォワード
  					RequestDispatcher dispatcher=request.getRequestDispatcher("/WEB-INF/jsp/cannotset.jsp");
  					dispatcher.forward(request, response);
  				}
  		else{
  					//おなじ人は、いないということなので、ユーザー情報をセッションスコープに保存
  					HttpSession session=request.getSession();
  	            //sessionスコープに保存して、今後ようこそ〜さんという時に使う
  					session.setAttribute("userId",userId);
  		                      }
        	 //引数が２つのLoginも作成
  			  Login login=new Login(userId,pass);
  		
  		
  		//ログイン結果画面にフォワード
  		RequestDispatcher dispatcher=request.getRequestDispatcher("/WEB-INF/jsp/loginResult.jsp");
  		dispatcher.forward(request, response);
  		}

Answer 1

すみません、技術的な回答でなく設計面での意見なのですが

新規登録ではIDの重複チェックだけで十分ではないでしょうか？

IDは一意である必要があるでしょうから、それが重複したらPasswordをチェックするまでもないですね
また、Passwordの一致までチェックしていることがユーザーに知られたら、既存アカウントでのログイン方法が知られてしまいます

mailaddressの重複チェックもまた、エラーを返せば有効であることを知らせることになるし、普通に利用しようとする人なら他人のアドレスを入れてシステムからの情報を受け取らないようにする…なんて人もいないかと思います

同姓同名のチェックも、たとえば鈴木一郎さんは日本には結構いらっしゃいますので同じく不要…というか、そこはチェックしたらダメかなと思います

SNSのユーザー登録に関する設計ポリシーがそのようなものでしたら仕方ないですが…

Comments

[edoooooo]

　ありがとうございます。設計面での考えが浅かったです。
言われてみればなのですが、その通りでした。
　userIdをこちらで決めて、与えるという方法を取りたいと思います。
リクエストスコープなどに？数字（乱数の組み合わせ）を保存して、それをidとして発行し、初回だけ入力して、その後は、自動入力という設計でいきたいと思います。それらの登録番号と、パスワードの一致で判断したいと思います。
　ありがとうございました。