即使条件不正确，登录活动也会进入下一个活动[关闭]

Question

关闭。 此问题不可重现或由拼写错误引起。 它目前不接受答案。

---

想改进这个问题？ 更新问题，使其成为 Stack Overflow的主题。

5 个月前关闭。

改进这个问题

这是我的代码，我在其中尝试移动到下一个活动，即用户通过身份验证后的仪表板，它运行良好，但即使凭据不正确或错误，它也会移动到下一个活动......

我已经写了 if else 条件用于移动到下一个活动而不是移动到下一个活动......但它仍然移动到下一个活动我不知道为什么会这样......

MainActivity.java

package com.example.demo;

import androidx.appcompat.app.AppCompatActivity;

import android.content.Intent;
import android.os.Bundle;
import android.util.Log;
import android.view.View;
import android.widget.EditText;
import android.widget.Toast;

import com.android.volley.AuthFailureError;
import com.android.volley.Request;
import com.android.volley.RequestQueue;
import com.android.volley.Response;
import com.android.volley.VolleyError;
import com.android.volley.toolbox.StringRequest;
import com.android.volley.toolbox.Volley;
import com.google.android.material.floatingactionbutton.FloatingActionButton;

import java.util.HashMap;
import java.util.Map;

public class MainActivity extends AppCompatActivity {

    EditText etusername, etpassword;
    String username, password;
    FloatingActionButton loginbtn;


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

        etusername = findViewById(R.id.etemail);
        etpassword = findViewById(R.id.etpw);
        loginbtn=findViewById(R.id.floatingloginButton);
        loginbtn.setOnClickListener(new View.OnClickListener() {
            @Override
            public void onClick(View v) {
                Log.d("Output 1","Button clicked");
                loginverify();
            }
        });

    }

    private void loginverify() {
        username = etusername.getText().toString().trim();
        password = etpassword.getText().toString().trim();

        Log.d("Output 2","login verify Function called");
        if (username.isEmpty()) {
            Toast.makeText(MainActivity.this, "Username cannnot be empty", Toast.LENGTH_SHORT).show();
            Log.d("Output 3","if con 1");

            return;
        } else if (password.isEmpty()) {
            Toast.makeText(MainActivity.this, "Password cannnot be empty", Toast.LENGTH_SHORT).show();
            Log.d("Output 3","if con 2");
            return;
        }else{
            Log.d("Output 5","inside else");
            StringRequest request= new StringRequest(Request.Method.POST, "https://**url**/login.php", new Response.Listener<String>() {

                @Override
                public void onResponse(String response) {
                   // Log.d("Output 4",response);
                    if(!response.equalsIgnoreCase("Login Failed.......Try Again..")){
                        Log.d("Output 6","login failed");
     

               Toast.makeText(getApplicationContext(), response, Toast.LENGTH_SHORT).show();
                    startActivity(new Intent(getApplicationContext(),Dashboard.class));
                 }else{
                    Log.d("Output 6","login passed");
                    Toast.makeText(getApplicationContext(), response, Toast.LENGTH_SHORT).show();
                   return;
                }

             //   Toast.makeText(MainActivity.this, response, Toast.LENGTH_SHORT).show();
              //  startActivity(new Intent(getApplicationContext(),Dashboard.class));
                //Toast.makeText(MainActivity.this, response, Toast.LENGTH_SHORT).show();

            }
        }, new Response.ErrorListener() {
            @Override
            public void onErrorResponse(VolleyError error) {
                Toast.makeText(MainActivity.this, error.getMessage(), Toast.LENGTH_SHORT).show();
                Log.d("Response error",error.getMessage().toString());
            }
        }
        ){
            @Override
            protected Map<String, String> getParams() throws AuthFailureError {
                Map<String,String> params=new HashMap<String, String>();
            params.put("login_name",username);
            params.put("login_pass",password);
                return params;
            }
        };
        RequestQueue requestQueue= Volley.newRequestQueue(MainActivity.this);
        requestQueue.add(request);
    }

}

}

下面是我的 php 代码......对不起，这么弱的代码......

<?php  
 require "../dbcn.php";  
 $user_name = $_POST["login_name"];  
 $user_pass =  $_POST["login_pass"];  
 $sql_query = "select Firstname,Lastname from users where emailid like '$user_name' and password like '$user_pass';";  
 $result = mysqli_query($db_conx,$sql_query);  
 if(mysqli_num_rows($result) >0 )  
 {  
 $row = mysqli_fetch_assoc($result);  
 $name =$row["Firstname"]." ".$row["Lastname"];  
 echo "Login Success..Welcome ".$name;  
 }  
 else  
 {   
 echo "Login Failed.......Try Again..";  
 }  
 ?>  

Answer 1

比较String结果是非常非常错误的方法......除了这个if也非常不合适

if(!response.equalsIgnoreCase("Login Failed.......Try Again..")){

}

所以除了String上面的任何东西都将打开下一个Activity ，例如，当你在服务器端更改甚至一个字符时，或者在中间断开连接时，你只会得到String的一半......（理论上）

为了让用户进入一些隐藏/受限区域，您应该始终使用积极、明确说明的条件，而不是与拒绝相反...尝试使用if(response.startsWith("Login Success")) ，但仍然比较String s 不好想法......（这就是你投票失败的原因）

考虑使用一些标准来格式化文本，例如 XML 或 JSON，或者只返回一些int代码，例如， 0表示肯定登录， -1密码不匹配， -2用户未找到等取决于您的目的。 当然，在登录条件下使用if(result==0) ，而不是! 所有其他可能的代码...

Answer 2

我建议您在 onResponse 方法中使用 JSON 格式并使用 equals() 而不是 equalsIgnoreCase.OnResponse 方法需要一些时间，具体取决于您的响应数据和互联网速度。 如果您使用 boolean 值作为响应，这是最好的条件。