Androidでサーバーとのログイン処理をやりたいのですがすべてログイン完了となります。

Question

このコードでLoginCheckAnswerボタンが押された時のCheckUserNameとCheckPasswordがサーバーにあるかどうかを確認して、if(e!=null){}else{ログイン完了}としているのですが、サーバーにないCheckUserNameとCheckUserPassに入力してLoginCheckAnswerボタンを押しても全て、ログイン完了となっていまいます。
どうすれば良いのでしょうか？

package ncmbdataquick.mbaas.com.nifty.datastorequickstart;

import android.app.Activity;
import android.app.AlertDialog;
import android.os.Bundle;
//いろいろいらなそうなの消した。
import android.view.View;
import android.widget.EditText;

import com.nifty.cloud.mb.core.FindCallback;
import com.nifty.cloud.mb.core.NCMBException;
import com.nifty.cloud.mb.core.NCMBObject;
import com.nifty.cloud.mb.core.NCMBQuery;

import java.util.List;

public class ActivityThird extends Activity implements View.OnClickListener{
    //変数の宣言
    private EditText mUserName2;
    private EditText mPassword2;

    @Override
    protected void onCreate(Bundle savedInstanceState) {
        super.onCreate(savedInstanceState);
        setContentView(R.layout.activity_activity_third);

        //これはonClick()で  doCheckUser();

 //**************** APIキーの設定とSDKの初期化 **********************
        NCMB.initialize(this, "");


        findViewById(R.id.LoginCheckAnswer).setOnClickListener(this);
    }

    //なにがクリックされたかのリスナー
    //登録したいのは、LoginCheckAnswer



    @Override
    public void onClick(View v) {
        //ここではデータを送信し、そのデータ(2つ)がサーバーにあったらログインする中ったら画面にログイン失敗と出す。
        // MainActivityのデータを取得するやつを参考にする
        //このIDでこのパスワードの人は存在するかを確かめる(if elseで)
        //public void doGetDemo(View view) {


        doCheckUser();

    }

        public void doCheckUser () {
            //NewUserAddという新規登録完了者のぱすと、nameがあるサーバーを検索するためのNCMBQueryインスタンスを作成
            NCMBQuery<NCMBObject> query = new NCMBQuery<>("NewUserAdd");
            //keyというフィールドがvalueとなっているデータを検索する条件を設定
            //ここでボタンからデータを取得
            //ぼたんから引っ張るコードはまえやった。リスナーを作るやつ。
            //ボタンのデータの前に
            //ここで入力したデータを取得するためもう一つのbotannのなんかをつkる


            //xmlのidを変えた時にnoを押さないとsecondとthirdの間で同じデータになる。

            //この下のようにcontent_activity_third.xmlのUserNameとPasswordに2をつけた。
            mUserName2 = (EditText) findViewById(R.id.CheckUserName);//ユーザーが入力するUserName
            mPassword2 = (EditText) findViewById(R.id.CheckPassword);//ユーザーが入力するPassword
            String userName2 = mUserName2.getText().toString();
            String password2 = mPassword2.getText().toString();

            query.whereEqualTo("UserName", userName2);
            //このような下の条件の追加はAndroid入門なんとかっていうブックマークにある
            query.whereGreaterThan("Password", password2);
//サーバーからデータを検索
            query.findInBackground(new FindCallback<NCMBObject>() {
                @Override
                public void done(List<NCMBObject> results, NCMBException e) {
                    if (e != null) {

                        //検索失敗。

                    } else {

                        new AlertDialog.Builder(ActivityThird.this)
                                .setTitle("Notification from Nifty")
                                .setMessage("Get successfull!" + results)
                                .setPositiveButton("ログイン完了です", null)
                                .show();
                        //検索成功時の処理
                        //ここでログインする引数のからのメソッドを呼び出し、
                        //Activityforseに移行させたい。
                        //intentとか使う
                    }


                }
            });
        }


    }

content_activity_third.xmlです。

<?xml version="1.0" encoding="utf-8"?>
<RelativeLayout xmlns:android="http://schemas.android.com/apk/res/android"
    xmlns:tools="http://schemas.android.com/tools"
    xmlns:app="http://schemas.android.com/apk/res-auto" android:layout_width="match_parent"
    android:layout_height="match_parent" android:paddingLeft="@dimen/activity_horizontal_margin"
    android:paddingRight="@dimen/activity_horizontal_margin"
    android:paddingTop="@dimen/activity_vertical_margin"
    android:paddingBottom="@dimen/activity_vertical_margin"
    app:layout_behavior="@string/appbar_scrolling_view_behavior"
    tools:showIn="@layout/activity_activity_third"
    tools:context="ncmbdataquick.mbaas.com.nifty.datastorequickstart.ActivityThird">




    <EditText
        android:layout_width="wrap_content"
        android:layout_height="wrap_content"
        android:inputType="textPassword"
        android:ems="10"
        android:id="@+id/CheckPassword"
        android:layout_centerVertical="true"
        android:layout_centerHorizontal="true" />

    <EditText
        android:layout_width="wrap_content"
        android:layout_height="wrap_content"
        android:id="@+id/CheckUserName"
        android:layout_marginTop="52dp"
        android:layout_below="@+id/textView3"
        android:layout_alignLeft="@+id/textView3"
        android:layout_alignStart="@+id/textView3"
        android:layout_alignRight="@+id/CheckPassword"
        android:layout_alignEnd="@+id/CheckPassword" />

    <TextView
        android:layout_width="wrap_content"
        android:layout_height="wrap_content"
        android:textAppearance="?android:attr/textAppearanceSmall"
        android:text="UserName"
        android:id="@+id/textView3"
        android:layout_alignParentTop="true"
        android:layout_toLeftOf="@+id/LoginCheckAnswer"
        android:layout_toStartOf="@+id/LoginCheckAnswer" />

    <TextView
        android:layout_width="wrap_content"
        android:layout_height="wrap_content"
        android:textAppearance="?android:attr/textAppearanceSmall"
        android:text="Password"
        android:id="@+id/textView4"
        android:layout_below="@+id/CheckUserName"
        android:layout_alignLeft="@+id/CheckPassword"
        android:layout_alignStart="@+id/CheckPassword"
        android:layout_marginTop="84dp" />

    <Button
        android:layout_width="wrap_content"
        android:layout_height="wrap_content"
        android:text="ログインする"
        android:id="@+id/LoginCheckAnswer"
        android:layout_alignParentBottom="true"
        android:layout_toRightOf="@+id/textView4"
        android:layout_toEndOf="@+id/textView4"
        android:layout_marginBottom="82dp" />

</RelativeLayout>

AndroidMoniterには、こんなエラーが出ていました。

このコードでできました。

11-02 19:12:01.879 20145-20145/ncmbdataquick.mbaas.com.nifty.datastorequickstart E/SpannableStringBuilder: SPAN_EXCLUSIVE_EXCLUSIVE spans cannot have a zero length
11-02 19:12:01.879 20145-20145/ncmbdataquick.mbaas.com.nifty.datastorequickstart E/SpannableStringBuilder: SPAN_EXCLUSIVE_EXCLUSIVE spans cannot have a zero length
1

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

 @Override
                public void done(List<NCMBObject> results, NCMBException e) {
                    if (e!= null||results.size()>=1) {
                        //e!=nullかつresult.sizeが1件
                        new AlertDialog.Builder(ActivityThird.this)
                                .setTitle("Notification from Nifty")
                                .setMessage("Get successfull!" + results)
                                .setPositiveButton("ログイン完了です", null)
                                .show();
                        //検索成功時の処理
                        //ここでログインする引数のからのメソッドを呼び出し、
                        //Activityforseに移行させたい。
                        //intentとか使う

                        //elseをけした。
                    }

Answer 1

NIFTYCloudに関するとあるwebページをみて思ったのですが、いわゆるログイン処理を行うならNCMBQuery#findInBackgroundではなくNCMBUser#signUpInBackgroundを使うのではないかと思いました。

次のようになっているような気がします。（きちんと調べてないので推測も含みます）

1. NIFTYCloudへのログイン

NCMBUser#signUpInBackgroundを使う。ログインに失敗した場合は例外が発生。

1. NIFTYCloudへログインした後のデータ検索

NCMBQuery#findInBackgroundを使う。おそらく他の方の回答のとおり、検索結果の件数如何にかかわらず(通信エラーや使用方法に誤りがない限り)例外は発生しない。

Comments

[edoooooo]

ありがとうございます。
試してみます。

Answer 2

e!=nullは検索自体の成功、失敗を判定しているものであり、条件にあったユーザーがいるかは判定でき無いと思います。

下記の修正をするといいと思いますよ。
・whereGreaterThan("Password",password2)をwhereEqualTo("Password", password2)に変更する。
・e!=nullかつresult.sizeが1件以上の場合はログイン成功、それ以外はログイン失敗にする。

Comments

[edoooooo]

ありがとうございました。
無事できるようになりました。
コードも記載しました
ありがとうございました。

Answer 3

実際に動かせてないので、予想ですが、
データが取得処理が正常に動作していれば、０件でも「NCMBException e」は常にnullだと思われます。
resultsの件数で判定するべきだと思います。

Comments

[edoooooo]

回答ありがとうございます
eがnullでないなら失敗
nullだと成功にしていて、成功だとログインが完了しましたと表示されるようにしているのですが、エラーがnullでないのにもかかわらず成功してログインが完了しましたと表示されていると思うのですが、
おっしゃられていることからずれていましたら申し訳ありません

[edoooooo]

おっしゃられていることがわかった気がします。
NCMBException e はnullだけど、エラーが出ているということであっていますでしょうか？
NCMBException eがnullか、エラーが出ていない場合は、ログイン成功というコードにすればいいのでしょうか?

[himakuma]

すいません、見間違えてました。resultsの中身を見て、ログイン判定をするのが正しいのではないでしょうか？例外のオブジェクトがNULLかどうかでは判定できないと思います。後、細かいですが「!=」を使用しているのにelseがあるのは可読性を落とします。

[edoooooo]

ありがとうございます。
resultの値でチェックして、if文もellseを無くしました。
ありがとうございました。